コードレビュー - ntuf/Tips GitHub Wiki

『良いコード、悪いコードで学ぶ設計入門』

・意味不明な命名
　・技術駆動命名　：技術ベースでの命名
　　　void changeIntValue01(int changeValue)
　・連番命名　：　
class Class001{}
・条件分岐のネスト

・データクラスが引き起こす弊害
データしか持たない金額データクラスと
別で計算ロジックを持つクラスを作る
　・数十か所で消費税の計算ロジックを入れてしまう
低凝集（：凝集度が低い、凝集度：ソースコードが特定の機能に協調している度合い）
↓　以下を引き起こす
・重複コード　：　関連するコードが離れているから
・修正漏れ　：　重複コードから発生
・可読性低下　：　処理の流れをどれだけ素早く読めるかの指標
・未初期化状態（生焼けオブジェクト）：　クラスを初期化しても中のメンバが初期化されていないと使えない、それを知らないと使えない
・不正値の混入　：　注文数がマイナスになるなど、バリデーションをたくさん実装してしまう可能性がある

・省略せずに意図が伝わる名前
・再代入はしない：　読み手が混乱する