たいてい、プロジェクトの計画をする時に、責任者にコードレビューの重要性を説くのだが、真っ先に、その工程を省いてくれるので自分の作業時間を削って、抜き打ちで勝手コードレビューをする事が多い。

抜き打ちでルール違反を発見したら、修正をお願いし、ルールを遵守する重要性を説く。

しかし、保守フェーズに入って、抜き打ちでレビューできなかったコードを見ると、ルール違反している箇所が案外ある。

もちろん、守ってくれる人もいるし、割合的にはこちらの方が多い。

しかし、守ってくれない人も少なからずいるのだ。

そういう人は、ルールを守ることも仕事の一環であるという意識が薄いため、

「自分がいる間に、見つからなければいい。」

という動機で何度諭してもルール違反をする。

以上のことからも、ちゃんとコードレビューを行う事が大切であることがわかる。

いなくなってから、違反している箇所を見つけても遅いのである。

この話は、コードレビューだけではなく、設計レビューやその他のレビューにおいても言える事である。

レビューという一見何も生産していないように見える工程を省くことが多いが、「何も生産していない」というのは大きな間違いである。

#まぁ、昔から言われている当たり前のことを、こうやってエントリする必要性があるかと言われると微妙なのだけれども。