ゴッドオブジェクト とは？

一つのクラスに、別々の理由で変更する処理が集まっていないか見よう。たとえばUserManagerが認証、メール送信、課金まで抱えている場合だよ。行数やメソッド数は手掛かりになるけれど、「1000行なら該当」のような共通の境界ではないんだ。

名前や大きさだけでは決まらないよ。処理どうしの関連が薄い、ほかのクラスのデータに過度に依存する、変更が無関係な機能に及ぶ、といった点を調べるんだ。機能を追加するたびに同じクラスへ詰め込むと、こうした状態になりやすいね。

まず今の振る舞いをテストで確認できるようにしよう。関連するメソッドとフィールドを新しいクラスへ取り出し、元のクラスから呼び出す。Extract Classというリファクタリングだよ。小さく変更して、外から見た動作が変わらないことを確かめるんだ。

単一責任の原則を意識し、何が変わったときにこのクラスを直すのかをレビューしよう。同じ理由で変わる処理はまとめ、異なる理由で変わる処理は分ける。静的解析の警告も、設計を見直すきっかけとして使えるよ。