Found unused variables in TypeScript code JS-0356
Performance
Major
3 months ago
—
3 months old
'password' is defined but never used
18 throw new Error('Not implemented');
19 }
20
21 public authenticate(identity: Identity, password: string): boolean {22 // Authenticate using the identity
23 throw new Error('Not implemented');
24 }'identity' is defined but never used
18 throw new Error('Not implemented');
19 }
20
21 public authenticate(identity: Identity, password: string): boolean {22 // Authenticate using the identity
23 throw new Error('Not implemented');
24 }'identity' is defined but never used
13 throw new Error('Not implemented');
14 }
15
16 public storeIdentity(identity: Identity): void {17 // Store the identity on the blockchain
18 throw new Error('Not implemented');
19 }'DIDManager' is defined but never used
1import { Identity } from './identity';
2import { Blockchain } from './blockchain';
3
4class DIDManager { 5 private blockchain: Blockchain;
6
7 constructor() {
Description
Unused variables are generally considered a code smell and should be avoided.
Removing unused references - It prevents unused modules from being loaded at runtime, improving performance, and preventing the compiler from loading metadata that will never be used. - It prevents conflicts that may occur when trying to reference another variable.
NOTE: If you have intentionally left a variable unused, we suggest you to prefix the variable name with a _ to prevent them from being flagged by DeepSource.
Bad Practice
import fs from 'fs' // <- unused
import { readFileSync } from 'fs'
const text = readFileSync('declaration_of_independence.txt', 'utf-8')
console.log(text)
Recommended
import { readFileSync } from 'fs'
const text = readFileSync('declaration_of_independence.txt', 'utf-8')
console.log(text)