Private members should be marked as
readonly JS-0368
Anti-pattern
Major
2 months ago
—
7 months old
Member 'data' is never reassigned; mark it as
readonly110 * Convert placeholders symbols from the configuration settings.
111 */
112export class PlaceholdersConverter {
113 private data: DataModel;114 public statement: string;
115
116 /**Member 'config' is never reassigned; mark it as
readonly11}
12
13export class Config {
14 private config: IConfiguration;15
16 constructor(configurations: IConfiguration) {
17 this.config = configurations;
Description
Private members are marked as readonly if they're never modified outside of the constructor.
Bad Practice
class Container {
// These member variables could be marked as readonly
private neverModifiedMember = true;
private onlyModifiedInConstructor: number;
public constructor(
onlyModifiedInConstructor: number,
// Private parameter properties can also be marked as readonly
private neverModifiedParameter: string,
) {
this.onlyModifiedInConstructor = onlyModifiedInConstructor;
}
}
Recommended
class Container {
// Public members might be modified externally
public publicMember: boolean;
// Protected members might be modified by child classes
protected protectedMember: number;
// This is modified later on by the class
private modifiedLater = 'unchanged';
public mutate() {
this.modifiedLater = 'mutated';
}
}