仕事の話。
今週から今まで書いていたプログラムのコードレビューをすることになった。
頼まれたんだ。
リーダーがボロボロのコードを読んだようで、これじゃイカンと思ったらしい。
取りあえず、全員のコードを見る予定だけど、それとは別に朝からみんなで討論会みたいなこともすることになった。
しかし、今までコードレビューをしなかったというのが問題であるｗ
俺は先陣を切って1発目の発表だった。
1日1人が発表する。
何を思って実装したかを説明した。
何か勉強会のような感じだった。
俺のコードは基本きれいだから、何ら問題は無かった。
みんなはコードレビューってどうしてるんだろう？
例えばある基準書みたいなものがあったとして、それを基準としてレビューをしたとしても、
長年培ってきた、より効率が良いと思われる実装方法は書いてないわけで、
すなわちレビューする人の技量によるところが大きい。
色んなデザインパターンを知っていたり、オープンソースを読んでいる人は少なからず周りに居る人達よりも効率の良い実装方法を知っているはずだ。
今のところ超絶レベルの人にコードレビューをしてもらったことがないので、逆に他人のコードレビューする側である。
だから、模範解答知らないままコードレビューしてたりするから、何か不味いかもと思ってしまうｗ
偉そうなこと言っているが結局俺も大したこと無い。
話は少し変わって、他人のコードに口出しするかどうかという問題。
俺は気に入らない書き方をしていたら、言うようにしている。
でも、最近は言わない。
もうどうでも良くなった。
ある時どうでもよくなるってないだろうか・・・
これを自分の担当分だけと割り切るのか、他人の分も指摘するのか。
大きく意見がわかれそうな気がする。