ソースコードレビューで発見されたバグの数でエンジニアを評価してはいけない

ソースコードを書く時、スキルの低い人ほどバグが入ったコードを書きやすい。 しかし、ソースコードレビューで発見されたバグの数を元に、ヤバそうなモジュール(やエンジニア)を知ろうなどと、エンジニアのスキルを評価しようとするのは、止めた方が良い。 なぜなら、バグの多さは、エンジニアのスキルだけに依存するわけではないバグを隠そうとし、適切なレビューが行われなくなる可能性がある ■バグの多さとエンジニアのスキル 一般的には、バグが多い→エンジニアのスキルが低い、と見られがちだが、レビュー時にバグの発見数が多い原因は他にも考えられる。エンジニアのスキルが本当に低い。そのモジュールはもともと複雑度が高く、バグになりやすい。プレゼンがうまいので、バグが見つかりやすい。(「このへんがあやしいので、重点的に見てください」など)レビュアーのスキルが高い。 このように様々な原因があり、バグが多く発見される事とエン

[iga_k]

SAKURAIさんはこういうのをちゃんと文章に落とすのがすごく上手い。

[nirvash]

じゃあどうやって回避しようか。

[suVene]

件数だけで絶対評価にはしないのは賛成だけど、参考にはなるでしょ。難易度と鑑みて。/『バグを隠そうとし、適切なレビューが行われなくなる可能性がある』隠匿するから、(評価する為の)調査はするなと。