# smarthr/best-practice-for-optional-chaining

optional chaining（`xxx?.yyy`記法）を使うことを促すルールです。

## なぜoptional chainingを使うべきなのか

関数の存在確認をif文で行ってから実行する、というパターンは optional chaining を使うことで簡潔に書けます。ただし、else句がある場合や複数の条件がある場合など、optional chainingでは表現できないパターンもあるため、それらは許容されます。

## チェックする内容

以下のパターンで、optional chainingが使えるのに使っていない場合にエラーになります:

```jsx
// NG: 単純な関数の存在確認 → 実行
if (action) action()

// NG: オブジェクトのメソッドでも同様
if (obj.action) {
  obj.action(hoge, fuga)
}
```

ただし、以下の場合は許容されます:

### 他の処理が含まれる場合

```jsx
// OK: if内でreturnなど他の処理がある
if (action) {
  return action()
}
```

### 複数の条件がある場合

```jsx
// OK: 関数の存在チェック以外の条件がある
if (action && any) {
  action()
}
```

### else句がある場合

```jsx
// OK: else句があるため、optional chainingでは表現できない
if (action) {
  action()
} else {
  // ...
}
```

### else if句の場合

```jsx
// OK: else if内なのでoptional chainingでは表現できない
if (any) {
  // ...
} else if (action) {
  action()
}
```

### 無関係な条件の場合

```jsx
// OK: 実行する関数と無関係の条件
if (any) {
  action()
}
```

## rules

```js
{
  rules: {
    'smarthr/best-practice-for-optional-chaining': 'error', // 'warn', 'off'
  },
}
```

## ❌ Incorrect

```jsx
if (action) action()
```

```jsx
if (obj.action) {
  obj.action(hoge, fuga)
}
```


## ✅ Correct

```jsx
action?.()
```

```jsx
obj.action?.(hoge, fuga)
```

```jsx
// 実行する関数と無関係の条件の場合は許容
if (any) {
  action()
}
```

```jsx
// 実行する関数の存在チェック以外が条件に含まれている場合は許容
if (action && any) {
  action()
}
```

```jsx
// if - else などifに他条件が連なる場合は許容
if (action) {
  action()
} else {
  ...
}
```

```jsx
// if内で関数呼び出し以外の処理が存在すれば許容
if (action) {
  return action()
}
```

```jsx
// else if内の場合は許容
if (any) {
  ...
} else if (action) {
  action()
}
```