2 種類が混ざって使われていないか?という気づきをメモしておく。 正常系パス 例えば、boolean を返す問い合わせメソッドのように 分岐のコードパスが 2 つしかない場合 これは何も問題ない 例外を避けて安全側に倒したデフォルト これが正常系パスのように扱われているケースがないか? ほんとに後続処理をして問題ないか? 例外を出せない理由があるなら、ログを追加して運用で対処したりすべきではないか?(と考えているが、そのように判断されることがないように思うのでこの記事をメモした次第。) 備忘録。 ダメなルーティングの臭い 名詞ではなく動詞が使われている resources/resourceよりもget/post/put/patch/deleteが使われている コントローラ内にscaffoldで生成される7つのアクション以外のアクション定義が多い resources/resourceのonl