いずれにしても,「間違い探し」「犯人探し」や「誰が優れたプログラマーかコンテスト」ではなく,「参加者全員から何かを学ぶチャンスがあるフォーラム」といった雰囲気作りが大切なのだということ.
まず基本的に,チームのメンバーを全員集めて対象ソースコードをファイルの頭から順番に舐めていくのは,あまり効率的ではないと思う.当然プログラムはその順番で動くわけではないし,ソフトウェアは時間概念を持って動くので,静的にコードを眺めていても絶対に抜けが発生する.いわゆる「文法チェック」に毛が生えた程度のチェックしかできず,それはそれで必要だが何人も集めて行うのは少しもったいない.
ただ、ペアレビューの進め方がかなりポイントになりそうですね。コンパイラが見つけてくれない問題というのは、コーディングルールが徹底されていてかつその該当モジュールのアルゴリズムが分かっていることが前提にしないと、有意義なものにならない可能性があります。
僕のチームの場合は、チームレビューを教育のため、ペアレビューをコード品質を保つため、と明確に分けています。
そして時間の都合上、「教育」のためのチームレビューがなかなかできないのですが。