Skip to content

useGetterReturn (since v1.0.0)

Diagnostic Category: lint/suspicious/useGetterReturn

Sources:

Enforce get methods to always return a value.

class Person {
get firstName() {}
}
suspicious/useGetterReturn.js:2:5 lint/suspicious/useGetterReturn ━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━

   This getter should return a value.
  
    1 │ class Person {
  > 2 │     get firstName() {}
       ^^^^^^^^^^^^^^^^^^
    3 │ }
    4 │ 
  
const obj = {
get firstName() {
return;
}
}
suspicious/useGetterReturn.js:3:9 lint/suspicious/useGetterReturn ━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━

   This return should return a value because it is located in a getter.
  
    1 │ const obj = {
    2 │     get firstName() {
  > 3 │         return;
           ^^^^^^^
    4 │     }
    5 │ }
  
class Option {
get value() {
if (this.hasValue) {
log();
} else {
return null;
}
}
}
suspicious/useGetterReturn.js:2:5 lint/suspicious/useGetterReturn ━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━

   This getter should return a value.
  
     1 │ class Option {
   > 2 │     get value() {
        ^^^^^^^^^^^^^
   > 3 │         if (this.hasValue) {
   > 4 │             log();
   > 5 │         } else {
   > 6 │             return null;
   > 7 │         }
   > 8 │     }
        ^
     9 │ }
    10 │ 
  
class Person {
get firstName() {
return this.fullname.split(" ")[0];
}
}
const obj = {
get firstName() {
return this.fullname.split(" ")[0];
}
}