既存の設計に変更を加えないことを考慮したOOP設計
質問は:
interface Animal {
void eat();
}
class Lion implements Animal{
public void eat(){
//do somethng
}
}
class Test {
public static void main(String[] args) {
Animal lion = new Lion();
lion.eat();
lion.eat();
lion.eat();
}
}
要件は、インターフェースとクラス自体を変更せずに、eatメソッドが呼び出される回数を計算することです。
1つの方法は、lionクラスを拡張して結果を取得することですが、クラスを拡張するすべてのオブジェクトについて、そのようなクラスを作成する必要があります。
これを行うための最適化された方法はありますか?
パブリッシュサブスクライブは1つの方法ですが、インターフェイスまたはLionクラス自体を変更する権限がありません。
回答
デコレータパターンを使用して、サブクラス化せずに動物に追加の責任を追加できます。

public interface Animal {
void eat();
}
public class Lion implements Animal {
public void eat() {
// do something
}
}
/* In the original Decorator pattern,
the decorator is an abstract class,
but for the sake of brevity,
in this example it's a concrete class. */
public class AnimalWithEatCountDecorator implements Animal {
private Animal animalWeWantToCountEats;
private int eatCount=0;
public AnimalWithEatCountDecorator(Animal animal) {
this.animalWeWantToCountEats= animal;
}
public void eat(){
this.animalWeWantToCountEats.eat();
this.eatCount++;
}
public int getEatCount() {
return this.eatCount;
}
}
public class Test {
public static void main(String[] args) {
AnimalWithEatCountDecorator lion = new AnimalWithEatCountDecorator(new Lion());
lion.eat();
lion.eat();
lion.eat();
System.out.println(lion.getEatCount());
}
}
更新
デコレータパターンにもっと忠実になりたい場合は、getEatCount()
ゲッターをまったく使用できず、代わりにコンストラクターにCounterオブジェクトを挿入します。
public interface Counter {
public void increment();
public int getCount();
}
/* I will omit the trivial implementation of Counter */
public class AnimalWithEatCountDecorator implements Animal {
private Animal animalWeWantToCountEats;
private Counter counterThingy;
public AnimalWithEatCountDecorator(Animal animal, Counter counterThingy) {
this.animalWeWantToCountEats= animal;
this.counterThingy=counterThingy;
}
public void eat(){
this.animalWeWantToCountEats.eat();
this.counterThingy.increment();;
}
}
public class Test {
public static void main(String[] args) {
Counter counterThingy = new CounterThingy();
AnimalWithEatCountDecorator lion =
new AnimalWithEatCountDecorator(new Lion(), counterThingy);
lion.eat();
lion.eat();
lion.eat();
System.out.println(counterThingy.getCount());
}
}

作曲に最適な時期です。Animal
カウントを行うだけでなく、「実際の」関数を委任する新しい実装を作成します。このような:
public final class LoggingAnimal implements Animal {
private final Animal delegate;
private int eatCount = 0;
public LoggingAnimal(Animal delegate) {
this.delegate = delegate;
}
@Override
public void eat() {
eatCount++;
delegate.eat();
log("Animal ate {} times", eatCount); // Pseudo-functionality
}
}
既存のクラスを変更する必要はなく、必要な実装と一緒にこれをプラグインできAnimal
ます。このように使用します:
Animal lion = new LoggingAnimal(new Lion());
lion.eat();
新しい動作で新しいクラスを作成します。次にmain
、Test
クラスを更新します。
class Test {
public static void main(String[] args) {
Animal lion = new AnimalThatKeepsAMealLog();
lion.eat();
lion.eat();
lion.eat();
}
}
または、テストファイルを読んで、呼び出した回数を数えますeat()
。答えは3つになると思います。
別のアプローチの一種で、タスクの50クラスが数回しか実行されないようにすることができます。これはより一般的なものであり、技術的な側面では非常に有用ですが、ビジネスの側面ではそれほど有用ではありませんが、それでも非常に有用です。
ところでビルダーは単に説明するためのものです、もちろんあなたはそれをそのように使う必要はありません、preEat / postEatはここで重要です
class PointcutAnimal implements Animal {
private Runnable preEat;
private Runnable postEat;
@NonNull
private Animal downstream;
@Override
public void eat() {
if(preEat != null)
preEat.run();
downstream.eat();
if(postEat != null)
postEat.run();
}
}
class Test {
public static void main(String[] args) {
AtomicInteger eatCount = new AtomicInteger(0);
Animal lion = PointcutAnimal.builder(new Lion())
.postEat(eatCount::getAndIncrement)
.build();
lion.eat();
lion.eat();
lion.eat();
System.out.println(eatCount.get());
}
}