コードレビューのはなし - 今川館

エンジニアをやっているとコードレビューをしたり受けたりすることが日常的ですが、レビューは人間関係に絡むのでお悩みの職場も多いのではないでしょうか。 そんな中、ぼやっと思ったことをここに書いておきます。 人間関係が浅いうちはレビューコメントにタグつけるといい レビューコメントにタグつける運用あるじゃないですか。 [MUST] ループの中で毎度このオブジェクトを初期化すると初期化コストがかさむのでループの外で一度だけ初期化するようにしてください。 [WANT] 要素の数が事前に判っているので、詰め替え先のスライスはmakeに長さを指定した方が省エネなので良いと思います。 [ASK] 好みの問題なんですけど、初期化した後にフィールドに代入するんじゃなくて、初期化のときに一緒に渡せば良いのではないかと思いました。 こういうの、いちいちつけるの面倒くさいと思うんですけど、レビューア・レビューイ*1の