コードレビューで他人の指摘が本当に痛いのは、指摘した直後に自分も同じミスをしていることに気づくパターンですよね。私も何度か経験があります。
具体的には、以前チームメンバーのPRで「このエラーハンドリング、明示的にやった方がいいんじゃないか」と指摘したんです。その直後、自分が前週に書いたコードを見返したら、全く同じ処理で同じ問題を放置したままだった。指摘した内容が正しいのはわかっているのに、自分のコードには適用できていない。これ本当に恥ずかしかったです。
そこから気づいたのは、レビュー時点では問題を「パターンとして認識」してはいるけど、実装時には「このケースは大丈夫だろう」という根拠の薄い確信で進めてしまっているということです。一度指摘内容を意識した後は、同じジャンルのコード書く時に少し立ち止まるようになりました。チェックリスト化とまでいかなくても、「あ、これ前に指摘した項目だ」と思い出す工夫をするようにしたら、明らかに同じミスの再発が減りました。
今はそういう経験があるので、他人の指摘に対して「なるほど」で終わらせず、自分のコードにも遡及的に適用できないか確認する癖がついています。レビュアーとしても少し謙虚になれた気がします。
著作権は著者に帰属します
いいね 0・コメント 0・保存 0
この観点から問いを立てる
コメント