内部クラスをいっぱい持つのよくない？

Question

クラスの下に内部クラスをいっぱい作って…

public class MainClass {
   private String aaa;
   private String bbb;
   private SubClsA clsa;
   private SubClsB clsb;

   public void setAaa(String aaa) {
      this.aaa = aaa;
   }
   public String getAaa() {
      return this.aaa;
   }
   public void setBbb(String bbb) {
      this.bbb= bbb;
   }
   public String getBbb() {
      return this.bbb;
   }
   public void setClsA(String subaa, String subab) {
      clsa = new SubClsA();
      clsa.setSubaa(subaa);
      clsa.setSubab(subab);
   }
   public SubClsA getClsA() {
      return this.clsa;
   }

  class SubClsA {
    private String subaa;
    private String subab;
    ・・・
  }

  class SubClsB {
    private String subba;
    private String subbb;
    ・・・
  }
}

test = new MainClass ();
test.setAaa("test");
test.setClsA("testA","testB");
・・・

setClsAを呼ぶときに、内部でnewしてインスタンス保持して～
という作り方って、メモリ的にどうなんでしょう？

keyに紐づくデータ群で1つのJSONを作成して、それがkeyの数分出力するバッチです。
1つのJSON出力後にtest=nullとか突っ込んどけば問題なさそうでしょうか？

補足：
なんでこんなつくりするかというと
jacksonライブラリでJSON出力するのですが階層構造にしたかったので…

Answer 1

以下、クラスが自由に作れる状況を前提にお話します。
（課題や規約で、自由にクラスが作れない場合は別の話）

内部クラスを用いるのは、その外部（エンクロージング）クラスでしか使わないとき、
というシンプルなルールを私は考えています。要するに専用クラスです。

---

しかし、実際には個人的に、内部クラスを、あまり多用しません。
なぜかというと、クラスを細かく分割して、直交性を上げていくのが、
Java流のオブジェクト指向（小クラス主義）だと思うからです。

このクラスは専用でいいだろう、と当初思っていても、
後でふり返ると、共通化できたりする場合が、意外とよくあります。
（もちろん、共通化がつねに正しいとは限りません）

どんな処理が共通化できるかは、コード全体から見えてくるので、
多くの場合で、最初から内部クラスで作るより、
後から内部化する方が無難だと感じます。

---

内部クラスを避ける理由をもう少し述べてみます。

たとえば、何でもStaticにすると、気分はStaticになるかもしれませんが、
状態を持ったインスタンスを生成する、というOOPの基本から外れますよね。

同様に、何でも内部クラスにすると、クラスを組み合わせて重複を排除していく、
というOOPの基本から外れていってしまうように思います。

さらに言うと、内部クラスは継承に似ていて、基本的に密結合なので、
使い所がないわけでもないが、そこそこ難しいように感じています。

サンプルコードも今はスッキリしてますが、後で本格的な処理をしたときに、
それぞれの内部クラスのコード量が10倍などとなると、肥大化してしまいます。

もしかして、「後で必要になってから、バラせばいいでしょ」と思うかもしれませんが、
じゃあ継承だって、後でバラせばいいかというと、やはり大変なので、
だから「継承より委譲」みたいに言われるわけです。それにどこか似ていると思います。

ただし、Static不要だとも、継承不要だとも思わないので、
内部クラス不要だとも思いません。使うべきときは使って良いと思います。

Answer 2

階層構造であれば、Compositeパターン等で代替できないかは検討できるかと思います。
内部クラスは、クラスの定義内容が増えるものと考えられるので、本当に内部的に使用するクラスにのみ用いた方がよいかと思います。