noUnnecessaryConditions

このコンテンツはまだ日本語訳がありません。

JavaScript (and super languages)

Summary

Rule available since: vv2.1.4
Diagnostic Category: lint/nursery/noUnnecessaryConditions
This rule doesn’t have a fix.
The default severity of this rule is warning.
This rule belongs to the following domains:
- project
Sources:
- Same as @typescript-eslint/no-unnecessary-condition

Description

Disallow unnecessary type-based conditions that can be statically determined as redundant.

This rule detects if expressions inside conditions are statically inferrable and yield falsy or truthy values that don’t change during the life cycle of the program.

Examples

Invalid

1
function head<T>(items: T[]) {
2
  if (items) {  // This check is unnecessary
3
    return items[0].toUpperCase();
4
  }
5
}

1
function foo(arg: 'bar' | 'baz') {
2
  if (arg) {  // This check is unnecessary
3
  }
4
}

1
function bar(arg: string) {
2
  return arg?.length;  // ?. is unnecessary
3
}

Valid

1
function head<T>(items: T[] | null) {
2
  if (items) {  // This check is necessary
3
    return items[0].toUpperCase();
4
  }
5
}

1
function foo(arg: 'bar' | 'baz' | null) {
2
  if (arg) {  // This check is necessary
3
  }
4
}

1
function bar(arg: string | undefined) {
2
  return arg?.length;  // ?. is necessary
3
}

How to configure

1
{
2
  "linter": {
3
    "rules": {
4
      "nursery": {
5
        "noUnnecessaryConditions": "error"
6
      }
7
    }
8
  }
9
}