あ、これめっちゃ経験ある…。この前コードレビューで同期の変数命名について「ここもっと明確にしたほうが読みやすくない?」って指摘してて、その直後に自分の別のブランチ見返したら全く同じパターンがあるんですよ。何か恥ずかしくなった(笑)
なんでこんなことが起きるのか考えてみたんですけど、多分自分は時間がない時に書いたコードは頭に入ってるから平気だと思ってて、他人のコードを見る時は「これ何してるんだ?」って外部視点で見るんですよね。だから明らかに見えるんです。でも自分のコードはその時の脳内コンテキストで解釈してるから、実は読みづらくなってることに気づかない。
その後、レビューされる側になった時に「あ、自分が指摘した内容だ…」って思い当たることが増えたんです。自分のコードも他人に見てもらうと、さっき指摘した同じ問題が平気であることに気づかされる。悔しいけど、多分これのおかげで指摘する時の説得力が上がった気がします。「これ読みにくい」じゃなくて「実は自分もやったことあるんだけど〜」みたいに伝えられるようになったから。
コードレビューって教育とかじゃなくて、みんなが同じ落とし穴に何度も嵌めてくれる仕組みなんだなって思いました。
著作権は著者に帰属します
いいね 2・コメント 0・保存 0
この観点から問いを立てる
コメント