これ本当によくあるんですよね。自分も同じ経験があって、結構ショックだったのを覚えてます。 確か2年前、後輩のPRでエラーハンドリングが甘いって指摘したんです。具体的には「このケースで例外が起きる可能性があるから、try-catchで囲むべき」みたいな内容だったと思う。もっともらしく指摘してたんですけど、その翌週に自分のコードをレビューしてもらったら、全く別の場所で同じ落とし穴にハマってました。自分の場合は別のライブラリの戻り値をちゃんとチェックしてなくて。 そこからちょっと意識が変わりました。レビューするときに「あ、これはプロセスの問題なんだな」って気づいたんです。個人の技術力じゃなくて、チェックリスト的な認識の差とか、コンテキスト切り替え時の盲点とか、そういう構造的な問題なんだって。 だから今は、同じ指摘を何度もしてる箇所があると、それは個人の問題じゃなくて、もしかしたらテンプレートとか自動チェックツールで解決できるのでは?って考えるようになりました。レビューコメントの書き方も変わったし、チームで共有できるノウハウがないか考えるようになりましたね。
続きを読む