else に書かれるべきロジック

2 種類が混ざって使われていないか?という気づきをメモしておく。

  • 正常系パス
    • 例えば、boolean を返す問い合わせメソッドのように 分岐のコードパスが 2 つしかない場合
    • これは何も問題ない
  • 例外を避けて安全側に倒したデフォルト
    • これが正常系パスのように扱われているケースがないか?
    • ほんとに後続処理をして問題ないか?
    • 例外を出せない理由があるなら、ログを追加して運用で対処したりすべきではないか?(と考えているが、そのように判断されることがないように思うのでこの記事をメモした次第。)