指摘・レビュー○×難易度 標準

「変数名 tmp では中身が分からない」という指摘は、動作に影響しないので出すべきではない。

正答

× 誤り

解説

コードは書く回数より読まれる回数のほうが多く、名前が中身を表していないと次に読む人が誤解しやすい。レビューでは正しさに加えて読みやすさ・保守しやすさも見るのが一般的。ただし必須の修正なのか提案なのかは区別して伝える。「動作しているなら指摘しない」とすると、読みにくいコードが少しずつ積み上がる。

参考: google.github.io

reviewnamingreadability