はじめてのにき

_ ロジックを追わないコードレビュー

人的リソースが無限に近くある会社から、普通に有限というか普通に小さい会社に移ったことで、スタイルを変えてみたトピックがいくつかあって、そのひとつがコードレビューかなあと思う

コードレビューで撃墜ケース探すのは、楽しくはあるんだけど、時間の無駄であるということがある。昔は topcoder の撃墜フェーズは良いものだと思ってたけど、今思えば良ろしくない。何が良くないかって、コードの問題を探す一番良い方法はテストを書くことであって、目を皿のようにして眺めることではない。

というわけで、とりあえずテストを見る、てのはなんか割ともともとしていた気がする。が、最近はもっとテスト偏重にしている気がする。CI通ってんならええよ！みたいな。PRで変更したと主張してることについて、その主張が満たされるならこういうテストが通るはずだよね、て想像をして、そういうテストがあるかな、って見て、あるならまあたぶんコードはええやろ、くらいの感じでやっている。テストが無いなら、暇があるなら撃墜ケース考えるけど、無いなら深く考えず、こういうテスト書いてみてーというだけですます。テストだけ書いてもらえば CI で次から情報が増えるので……

人的リソースが無限にあるなら、前職のような丁寧なレビューの方が良いことは多いのだろうけど、なんかああいうのは、余裕のある会社だからやれることという感じがするんだよな。あるものでなんとかするって感じだと、コードレビューは真っ先にサボりたいところである。ただそういう余裕が無いようではいいものは作れないという気もしないでもないが……

あと、スタイルは……なんか個人的に int を auto で受けるのとかだけはホント勘弁してくれという気分なので、そういう自分の宗教の表明は、初回は言うようにしてる。関数名が小文字スタートvs大文字であるとかは、(昔の？)LLVMみたいにファイル内で大混乱してない限りは、正直そんなに問題にならない、のでまあなんでもいいやという気持ち

(00:32)