45:ちょっとした修正のつもりでコードを際限なく書き換えてしまう¶

プログラミング迷子: ちょっと修正、のついでに

後輩W：昨日レビューしてもらったPRなんですが、問題があったのでちょっと修正しました。修正した問題はちょっとなのでレビューなしで大丈夫です。
先輩T：お、了解（どれどれ、チラッと見ておこうかな……ちょっとじゃない、がっつり書き換わってる！！）がっつり書き直されてるんだけど、どのへんが「ちょっと修正」なの？
後輩W：特定の組合せのときだけエラーになる、ちょっとした問題を修正しました。
先輩T：いやそうじゃなくて、コードレビューしたところがあらかた書き換わってるじゃない。
後輩W：直しながら、どうせなら設計変えたほうがいいと気づいたので、ついでに修正しました。

「修正によって直した動作はちょっとしたもの」だとしても、コードを大幅に書き換えているのであれば再度レビューするべきです。変更した挙動の大小でレビューするかどうかを決めてしまうと、動作が変わらないリファクタリングはレビュー不要、ということになってしまいます。

ベストプラクティス¶

挙動が変わるなら、レビューしましょう。挙動が変わらなくても、変更範囲が大きいならレビューしましょう。

（中略）詳細は書籍自走プログラマーをご参照ください