オブジェクト参照がコンストラクタを経由して渡される

Question

比較的大きなオブジェクト指向プログラムを構築しています。私はAerodynamicCalculatorというクラスを持っています。このクラスは数多くの計算を行い、結果をシステムに分配します。私の主な関心事は、私はそれにmorパラメータを追加すると、私のコンストラクタシグネチャがますます大きくなるということです。

以下に示すように、私は既にこのコンストラクタに9つのオブジェクト参照が渡されていますが、さらに7つ必要です。このオブジェクトを正しく作成していますか？私の理解は、コンストラクタに関連付けられたオブジェクト参照を渡し、オブジェクトの参照にclass'esローカル変数を割り当てることです。この場合、必要なすべてのオブジェクトでクラスを適切に初期化する唯一の方法は、それらをコンストラクタに渡すことです（非常に長い署名につながります）。

public AreodynamicCalculator(AircraftConfiguration config, AileronOne aOne, 
     AileronTwo aTwo, ElevatorOne eOne, ElevatorTwo eTwo, Rudder r, 
     Rudder rr, RateGyros rG) { 
    // ... 
} 

このアプローチに関するアドバイスは、事前に感謝の意を表します。

Answer 1

前述のように、これはあなたのクラスがあまりにも多くのことをしているサインかもしれませんが、この問題によく使用される「解決策」があります。

ビルダーパターンはこの状況でよく使用されますが、引数が異なるコンストラクタが多数ある場合は非常に便利です。特にブールリテラルが使用されている場合は、引数の意味が明確になるためです。ここで

は、Builderパターンで、これが機能する方法は、このようなものです：

AreodynamicCalculator calc = AreodynamicCalculator.builder() 
    .config(theAircraftConfiguration) 
    .addAileron(aileronOne) 
    .addAileron(aileronTwo) 
    .addElevator(elevatorOne) 
    .addElevator(elevatorTwo) 
    .addRudder(rudderOne) 
    .addRudder(rudderTwo) 
    .build() 

内部的には、ビルダーは、これらすべてのフィールドを格納する、とbuild()が呼び出されたときには（今プライベート）コンストラクタを呼び出します。そのこれらのフィールドを取る：Builderパターンを使用する同様

class AreodynamicCalculator { 
    public static class Builder { 
     AircraftConfiguration config; 
     Aileron aileronOne; 
     Aileron aileronTwo; 
     Elevator elevatorOne; 
     Elevator elevatorTwo; 
     ... 

     public Builder config(AircraftConfiguration config) { 
      this.config = config; 
      return this; 
     } 

     public Builder addAileron(Aileron aileron) { 
      if (this.aileronOne == null) { 
       this.aileronOne = aileron; 
      } else { 
       this.aileronTwo = aileron; 
      } 
      return this; 
     } 

     // adders/setters for other fields. 

     public AreodynamicCalculator build() { 
      return new AreodynamicCalculator(config, aileronOne, aileronTwo ...); 
     } 
    } 

    // this is the AircraftConfiguration constructor, it's now private because 
    // the way to create AircraftConfiguration objects is via the builder 
    // 
    private AircraftConfiguration config, AileronOne aOne, AileronTwo aTwo, ElevatorOne eOne, ElevatorTwo eTwo, Rudder r, Rudder rr, RateGyros rG) { 
     /// assign fields 
    } 
} 

Answer 2

、あなたは春のような依存性注入フレームワークを使用することができ、davebの応答で提案されています。