ある程度経験を積んだレビュワーがやりがちな失敗は、 指摘しやすいコーディング規約違反だけ指摘している というもの。 コードレビューで指摘するべき欠陥とは、必ずしも規約違反だけではなく、 仕様考慮もれや機能的なバグ、非機能的なセキュリティやパフォーマンス上の問題点も含まれる。 一つ関数に対して複数の視点でソースチェックをしないといけないが、 人間は同時に複数のことは考えられない。 そこでどうすればいいかと情報をあさっていたところ、 われらがIPAがセキュアプログラミング講座というWEBページで、 四回に分けてレビューすることを提唱していた。 1回目はどこに何があるか、 2回目は可読性が確保されているか、規約にのっとっているか 3回目は機能性 4回目はセキュリティ といった具合である。 IPAの講座では4回目はセキュリティに限定しているが、 担当していたプロダクトは、非機能面はセキュリティはも
![メンバーに恨まれそうな3つのコードレビュー施策を徹底したら、逆にメンバーが爆速で成長した話 - Qiita](https://cdn-ak-scissors.b.st-hatena.com/image/square/05f5c89e16aaec8ecf83c07866431008677e128f/height=288;version=1;width=512/https%3A%2F%2Fqiita-user-contents.imgix.net%2Fhttps%253A%252F%252Fcdn.qiita.com%252Fassets%252Fpublic%252Farticle-ogp-background-412672c5f0600ab9a64263b751f1bc81.png%3Fixlib%3Drb-4.0.0%26w%3D1200%26mark64%3DaHR0cHM6Ly9xaWl0YS11c2VyLWNvbnRlbnRzLmltZ2l4Lm5ldC9-dGV4dD9peGxpYj1yYi00LjAuMCZ3PTk3MiZoPTM3OCZ0eHQ9JUUzJTgzJUExJUUzJTgzJUIzJUUzJTgzJTkwJUUzJTgzJUJDJUUzJTgxJUFCJUU2JTgxJUE4JUUzJTgxJUJFJUUzJTgyJThDJUUzJTgxJTlEJUUzJTgxJTg2JUUzJTgxJUFBMyVFMyU4MSVBNCVFMyU4MSVBRSVFMyU4MiVCMyVFMyU4MyVCQyVFMyU4MyU4OSVFMyU4MyVBQyVFMyU4MyU5MyVFMyU4MyVBNSVFMyU4MyVCQyVFNiU5NiVCRCVFNyVBRCU5NiVFMyU4MiU5MiVFNSVCRSVCOSVFNSVCQSU5NSVFMyU4MSU5NyVFMyU4MSU5RiVFMyU4MiU4OSVFMyU4MCU4MSVFOSU4MCU4NiVFMyU4MSVBQiVFMyU4MyVBMSVFMyU4MyVCMyVFMyU4MyU5MCVFMyU4MyVCQyVFMyU4MSU4QyVFNyU4OCU4NiVFOSU4MCU5RiVFMyU4MSVBNyVFNiU4OCU5MCVFOSU5NSVCNyVFMyU4MSU5NyVFMyU4MSU5RiVFOCVBOSVCMSZ0eHQtYWxpZ249bGVmdCUyQ3RvcCZ0eHQtY29sb3I9JTIzMjEyMTIxJnR4dC1mb250PUhpcmFnaW5vJTIwU2FucyUyMFc2JnR4dC1zaXplPTU2JnM9YjkwOTA3NjZiYTFmZDRmMmQ2YTE4ZWI4MTcyOGQzMzc%26mark-x%3D142%26mark-y%3D57%26blend64%3DaHR0cHM6Ly9xaWl0YS11c2VyLWNvbnRlbnRzLmltZ2l4Lm5ldC9-dGV4dD9peGxpYj1yYi00LjAuMCZoPTc2Jnc9NzcwJnR4dD0lNDBnYWt1cmkmdHh0LWNvbG9yPSUyMzIxMjEyMSZ0eHQtZm9udD1IaXJhZ2lubyUyMFNhbnMlMjBXNiZ0eHQtc2l6ZT0zNiZ0eHQtYWxpZ249bGVmdCUyQ3RvcCZzPTQ2Y2I2MjU0ZDBiOWI4YmM1MzU0OTM0YmEwMDBkMTIy%26blend-x%3D142%26blend-y%3D486%26blend-mode%3Dnormal%26s%3Dffd227081a3768cdbe9e84acf23d7395)