コドレビューの観点 コドレビューの観点をまとめてみました。チェックリスト的なものになりますた。 機能性 コードが設計通りの機能を有するか? データの流れ、取得方法は、設計と一致するか? データのチェックの漏れはないか? データの生成、修正、加工は、設計と矛盾がないか? ループの中で更にSQLなど重い処理を発行するなど、パフォーマンスの懸念はないか? 必要な場合、国際化の対応しているか? コネクションやリソースは適切な方法で閉じられているか? NULLとなる場合や値が取れない場合を考慮しているか? NULLをある程度許容したコードになっているか? 読みやすさ 名前の付け方は、共通認識の範囲か? スタイルガイドにそっているか? ループ、分岐等の処理記述方法は同じか? コード設計 パラメータ、プロパティなど正しく機能区別されて実装されているか? インスタンス等の上書き、重複など考慮されているか?
![コードレビューするときの観点 - Qiita](https://cdn-ak-scissors.b.st-hatena.com/image/square/8db37a0bf428db1454bcd7efd4bd7a82f5132224/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-9f5428127621718a910c8b63951390ad.png%3Fixlib%3Drb-4.0.0%26w%3D1200%26mark64%3DaHR0cHM6Ly9xaWl0YS11c2VyLWNvbnRlbnRzLmltZ2l4Lm5ldC9-dGV4dD9peGxpYj1yYi00LjAuMCZ3PTkxNiZoPTMzNiZ0eHQ9JUUzJTgyJUIzJUUzJTgzJUJDJUUzJTgzJTg5JUUzJTgzJUFDJUUzJTgzJTkzJUUzJTgzJUE1JUUzJTgzJUJDJUUzJTgxJTk5JUUzJTgyJThCJUUzJTgxJUE4JUUzJTgxJThEJUUzJTgxJUFFJUU4JUE2JUIzJUU3JTgyJUI5JnR4dC1jb2xvcj0lMjMyMTIxMjEmdHh0LWZvbnQ9SGlyYWdpbm8lMjBTYW5zJTIwVzYmdHh0LXNpemU9NTYmdHh0LWNsaXA9ZWxsaXBzaXMmdHh0LWFsaWduPWxlZnQlMkN0b3Amcz1mZjUyZTMxZTQ3NWMzOWIyNjNhNmRlNDg4OTI4NjUyZQ%26mark-x%3D142%26mark-y%3D112%26blend64%3DaHR0cHM6Ly9xaWl0YS11c2VyLWNvbnRlbnRzLmltZ2l4Lm5ldC9-dGV4dD9peGxpYj1yYi00LjAuMCZ3PTcxNiZ0eHQ9JTQwc2hpZ2VzaGlnZXNoaWdlJTIwaW4lMjAlRUYlQkMlQTElRUYlQkMlQjIlRTMlODIlQTIlRTMlODMlODklRTMlODMlOTAlRTMlODMlQjMlRTMlODIlQjklRTMlODMlODglRTMlODMlODYlRTMlODIlQUYlRTMlODMlOEUlRTMlODMlQUQlRTMlODIlQjglRTYlQTAlQUElRTUlQkMlOEYlRTQlQkMlOUElRTclQTQlQkUlRUYlQkMlODhBUkklRUYlQkMlODkmdHh0LWNvbG9yPSUyMzIxMjEyMSZ0eHQtZm9udD1IaXJhZ2lubyUyMFNhbnMlMjBXNiZ0eHQtc2l6ZT0zMiZ0eHQtYWxpZ249bGVmdCUyQ3RvcCZzPWI0ZmQ1YTkyZjUyODMwMTYxNzhkZDY1MzgyMjVhOTI2%26blend-x%3D142%26blend-y%3D491%26blend-mode%3Dnormal%26s%3Dac250054116f348f4a1300b4fa033042)