コンポジションについて

Question

先日コンポジションについて学び、以下のようにAaaにfuncBを加えたBbbというクラスを作成しました。
BbbのコンストラクタでAaaのインスタンスを作成することが、無駄なオブジェクトの作成に当たるのではないかと考えているのですが、この実装は正しいのでしょうか。それとも、コンポジションは、Aaaインスタンスが各Bbbクラスで共通なものになる場合のみ利用するべきなのでしょうか。他にも不適切な点があればご指摘ください。(自身の認識しているコンポジションが正しいのかどうかも不安です。)

class IAaa {
  void funcA();
}

class Aaa implements IAaa {

  private Aaa(Xxx x, Yyy y) {
    // do something
  }

  public static Aaa of(Xxx x, Yyy y) {
    return new Aaa(x, y);
  }

  @Override
  public void funcA() {
    // do something
  }

}

class ForwardingAaa implements IAaa {

  private final IAaa a;

  protected ForwardingAaa(IAaa a) {
    this.a = a;
  }

  @Override
  public void funcA() {
    this.a.funcA();
  }

}

class Bbb extends ForwardingAaa {

  private Bbb(Xxx x, Yyy y, Zzz z) {
    super(Aaa.of(x, y));
    // do something
  }

  public static Bbb of(Xxx x, Yyy y, Zzz z) {
    return new Bbb(x, y, z);
  }

  void funcB() {
    // do something
  }

}

※スマホからの投稿なので、構文的なミスがあるかもしれませんが、ご容赦願います。

Answer 1

BbbのコンストラクタでAaaのインスタンスを作成する

コンポジションの性質上、普通はそうなるはずです。

Aaaインスタンスが各Bbbクラスで共通なものになる場合

それはコンポジションとは呼べないはずです。一応、『「参照」という概念を部品とするコンポジションだ』と強引ながら解釈可能ですが・・・まあ無理があるでしょう。

以上が本文を読んでの回答です。が、ソースを読んだらなにか不安なものを感じました。実装、継承、集約、コンポジションについて言葉の意味から学びなおすことをお勧めします。

Comments

[ponkotate]

ご回答ありがとうございます。
やはり、インスタンスの生成については欠点として見て良いようですね。
ソースに関してですが、私のやろうとしていることはもしかしたら、継承のほうが正しいのかもしれませんね。もう少しそれぞれについて学び直してから、プログラムを書いていこうと思います。

[HogeAnimalLover]

クラスBがフィールドとしてクラスAをもっているとき、（つまりクラスAがクラスBの部品であるとき）これを集約の関係（has a関係）といいます。コンポジションはこれの特別な場合です。コンポジションの場合は、両クラスが分離不可能な場合（同時に作られ、同時に壊される場合）のみの集約関係です。一方、クラスBがクラスAを継承する場合はis a関係といいます。

Answer 2

簡単に回答します。

結論から言うと無駄になります。必要なければ継承のほうが簡単で良いでしょう。

ただし、それはデメリットの一つで、拡張性がないクラスだったり２つのクラスを纏めて扱ったりしてメリットがデメリットを超える場合もあるのです。（あと言語による部分も大きいです。）

個別的に判断しなければならないし、正解がない場合も多いので場合によるというしかありません。