外部クラス内に複数のリストを保持する静的ネストされたCass

Question

以下のような擬似コードを実行しようとしています。これはネストされたクラスを使用するための規則やベストプラクティスに反しますか？

public class Foo{ 
    private list1, list2; 

    Bar bar = new Bar(list1, list2); 
    // use bar.listA, bar.listB, bar.listC, 
    //bar.listD, bar.listE 

    private static class Bar{ 
    private list1, list2; 
    private listA, listB, listC, listD, listE; 

    private Bar(list1, list2) { 
     this.list1 = list1; 
     this.list2 = list2; 
     //logic to split list1 and list2 into 5 lists: listA, listB, listC, listD, listE. 
     //initialize the 5 lists 
    } 
    } 
} 

私はそれが論理的にのみそれがカプセル化 を高める

1つの 代わりに使用されるクラスをグループ化する方法であるOracleのor the reasons mentioned by Oracle:

• で述べた理由のために、ネストされたクラスを使用しています
• それはより読みやすくなり、維持可能なコード
• そして理由mentioned here

外部クラスメンバーに直接アクセスする必要がないため、ネストされたクラスを静的にしています。

：より具体的には、ネストされたクラスの代わりにENUMを使用する必要がありますか？ネストされたクラスの5つのリストに対してgetters/setterを使うべきですか？ネストされたクラスのコンストラクタは分割ロジックを配置する正しい場所ですか？ネストされたクラスのプライベートメンバーを外部クラスで直接参照することはできますか？

Answer 1

list1とlist2には 'events'が含まれています。そして、イベントの「タイプ」に基づいて、私は5つのリストにそれらを分割

さて、あなたはその副作用のためのコンストラクタを使用すべきではない一般的な経験則として。

したがって、私はおそらくの範囲内でgroupEventsの機能を選択します。グループ化ロジックがかなり複雑な場合、GroupedEventsインスタンスまたはMap<EventType, List<Event>>などを返すEventGrouperステートレスサービスを持つことができます。

など。

public class Foo { 
    private GroupedEvents groupedEvents; 

    public Foo(List<Event> list1, List<Event> list2) { 
     this.groupedEvents = groupEvents(list1, list2); 
    } 

    private GroupedEvents groupEvents(List<Event> list1, List<Event> list2) { 
     Map<EventType, List<Event>> eventsByType = //perform grouping 
     return new GroupedEvents(eventsByType); 
    } 
} 

ロジックはあなたがネストされたかどうかのいずれかであるステートレスEventGrouperクラスにgroupEvents機能を移動することができ、十分に複雑である場合。デザインの重要な側面は、EventGrouperがそのコンストラクタ内でグループ化を実行してはならないということです。

今、私はあなたが言っていることを聞いています。いいえ、Fooは今ではコンストラクタでグループ化していますか？私たちはチェーンの問題をより強く押し込んでいませんか？さて、Fooの構築中にグループ化が行われるのは間違いですが、Fooオブジェクトのクライアントは、グループ化の副作用についてのみFooインスタンスをインスタンス化していません（できればそうではありません）。グループ化は、Fooの初期化プロセスの一部として実装の詳細になり、クライアントはそのようなグループ化が発生したことさえ知りません。