about

プルリク時のチェックシート - 2017版

プルリク時のチェックシート - 2017版

コードレビューの指摘で「イージーミス」を抽出、列挙
自分が「完全にマスターした!」と思うものは削除していく
絶えず変化させていくが1年ごとにスナップショット

前提

出来る限り沿う、程度のもの
- 可読性を落としてまで原則に従うかは、十分に検討する

チェック項目

使ってないクラス/メソッドが無い
- ただし hashcode() は除く
自身オブジェクトのフィールドCollectionを外に出すときには必ず unmodifiable に
- ArrayList$SubList() は注意
再代入はしない
TODOつけるのは修正対象のほうに付ける
コレクションを生成するものでも「新しく産んで自身の状態を変えるものではない」ならunmodifiableしない
「不要なアノテーション」は付けてないか

SQL関係

INNER JOIN でイケるとこを OUTER JOIN にしてないか

blog comments powered by Disqus