Ignore Properties

Accessor functions for properties are not required.

Getter And Setter

Do not use JavaScript getters/setters as they cause unexpected side effects and are harder to test and maintain. Instead, if accessor functions are used, implement custom methods with get and set prefixes.

Examples

⇣ Incorrect code for this rule:

class Winter {
  get snowflakes() {
    // ...
  }

  set snowflakes(snowflakes) {
    // ...
  }
}

⇡ Correct code for this rule:

class Winter {
  getSnowflakes() {
    // ...
  }

  setSnowflakes(snowflakes) {
    // ...
  }
}

Make sure to be consistent:

class Winter {
  constructor(elements = {}) {
    const snowflakes = elements.snowflakes || 0;
    this.set("snowflakes", snowflakes);
  }

  set(key, val) {
    this[key] = val;
  }

  get(key) {
    return this[key];
  }
}

Boolean Prefix

If the property/method is a boolean, use a is or has prefix.

Examples

⇣ Incorrect code for this rule:

if (!winter.snowflakes()) {
  return false;
}

⇡ Correct code for this rule:

if (!winter.hasSnowflakes()) {
  return false;
}

results matching ""

    No results matching ""