🎉

コードレビューは通らない

2023/11/19に公開

こんにちは、さとうです！

先日、こんな記事を書きました。
コードレビューしてみよう！

レビューをした結果、すべての行に指摘が入り、更には設計ごと変更する、という結末を迎えてしまいました。
この例は極端に思えるかもしれませんが、似たようなことが実はよく発生します。

コードレビューを通る人は高度人材

よく指摘が入る部分を並べてみました。

(これらすべてを人間がレビューする必要はありません。LinterなどのツールをCIに組み込めば、自動で検査してくれるものもあります。)

さて、これらすべてをクリアできる基準のコードを書ける人はどれくらいいるでしょうか？

このレビューを素通りできる人は、相当レベルが高いと考えられます。
このような、コードレビューの要求値が高い、という問題をどうすれば解決できるでしょうか。

もしあなたが初心者で、ひとりでコードを書いた場合、すべての行に指摘が入る可能性があります。

すると、あなたはこのように思うはずです。

これではモチベーションが下がりますし、何より学びを得られていません。
学びの最適化をするためには、この状況は改善しなければなりません。

これを達成するために、複数人でコードを書くようにしましょう。
また、モブプロにはシニアエンジニアを呼ぶようにします。

より素早くフィードバックを得ながら、コードを書くことによって、先ほどの問題を軽減することができます。

スピードを確保するために、コードレビューの要求値を下げることで対応しようとしてしまうことがありますが、これは間違いです。
なぜなら、「品質を捨てたらスピードが確保できないから」です。

ソフトウェアは現代において欠かせない存在になっています。
ソフトウェアエンジニアに求められるレベルは非常に高いですが、それをクリアした上で、私たちは「規律、基準、倫理」を守らなければいけません。