コードレビュー
2種類のレビュー
正しさのレビュー
要件を満たす動作をするコードになっているのか？
より良い書き方へのレビュー
正しさのレビューをするためには、正しい要件を知っていなければいけない
→品質を上げるには正しい要件が透明性を持ってレビュー者に示されていなければいけない
やり方
IntellijのCompare withでmasterと比較する
gitlab,github上よりも差分が見やすいので
/icons/hr.icon
機能
仕様と実装があっているか
不明瞭な部分について勝手な仕様で実装をしていないか
違和感のある実装になっていないか
論理が破綻していないか
画面表示の違和感はないか
エラー処理
コード
一貫性があるか？
不要なコードがないか
コメントが適切か？
第三者が理解できる
関数のInput、Output
命名が適切か？
共通部分の修正によってデグレってないか？
メソッド名は動詞
他部分への影響範囲
影響があるのであれば、影響が及んでいるのか？
デグレになる
フォルダパス
実行時じゃないと確認できないため→実際にそのフォルダがあるのかアクセスして確認するしかない
可読性の高さ
少し長くなっても可読性の高さを優先する
副作用
クラス設計が適切か