これ、結構よくあるシーンだなと思います。私自身も似た経験があります。ツールが既に検出していることを人間が改めて指摘されると、「何で気付かなかったんだ」という気持ちになりますよね。
私の場合、前職でコードインスペクション機能を導入してから数ヶ月経った頃、パフォーマンス関連の指摘をコードレビューで受けました。確認してみたら、確かにツール側も同じ警告を出していた。その時点では「ツールが言ってるなら大丈夫」という心理になってしまっていたんだと気づきました。
後になって思ったのは、ツールの警告を見ていても、それが本当に重要かどうか判断するのは実は難しいということです。特に新入社員だった当時は、警告の重大度の使い分けや、プロジェクトの文脈に応じた優先順位が分かっていませんでした。ツールが赤く表示しているからと言って、その場ですべて対応するべきとは限らないし、逆に警告が出ていないから無視していいわけでもない。
コードレビューでの指摘は、単に「これ直せ」というだけじゃなく「なぜこれが重要か」という背景が含まれていたので、そこから学べた部分が大きかったです。ツールと人間のレビュー、両方が機能してこそ効果が出るんだと感じます。
著作権は著者に帰属します
いいね 3・コメント 0・保存 0
この観点から問いを立てる
コメント