Making Software ―エビデンスが変えるソフトウェア開発

315

集団力学

ドはどれか？

そのコードは元の動作に依存していないのか？

副作用があるのなら、それによって影響

を受ける他のコードはどれか？

コードに付属しているユニットテストはあるか？

この変更のために、

全く別の関数のためのユニットテストに変更の必要が生じていないか？

いるとすればそれは受け入れ

られるか？

このため、すぐ目の前の変更を検討するだけではなく、さらに別のファイル、クラス、ドキュメンテー

ション、その他のコンテキストについてのコードレビューを行うことが、どれくらい重要でしょうか？

実験では、レビュー担当者は「コンテキスト」として、現在見ている変更に関係するファイル一覧を、さ

まざまな範囲で、与えられました［

Hunsmore 2000

］。呼ばれるメソッド（問題のコードが呼ぶメソッド）

に基づいて「関係するファイル」の一覧を自動生成する簡単な方法を用いた場合、見つかった欠陥数は

15%

増加しました。自動生成の代わりに、人間が本当に関係のあるファイルはどれかを判断する呼び出

しを利用するように置き換えた場合、最大

30%

増の欠陥が見つかりました。

このことは、コードの切れはしをそれ単独でレビューしてはいけない、ということを示しています。

最低限でも、あなたは全体のコードベースがすぐ手に入る

IDE

の前にいる必要があるのです。コード

の書き手が関連のあるコードをハイライトできれば、かなり時間の使い方も良くなるでしょう。

集団力学

ここまで、開発者

1

人が単独でコードをレビューすることを考えてきましたが、それはほぼ確実に、 ...