既存の設計に変更を加えないことを考慮したOOP設計

Aug 17 2020

質問は:

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クラス自体を変更する権限がありません。

回答

25 TulainsCórdova Aug 17 2020 at 15:07

デコレータパターンを使用して、サブクラス化せずに動物に追加の責任を追加できます。


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());


    }

}

22 RobertBräutigam Aug 17 2020 at 14:39

作曲に最適な時期です。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();
1 candied_orange Aug 17 2020 at 14:01

新しい動作で新しいクラスを作成します。次にmainTestクラスを更新します。

class Test {
    public static void main(String[] args) {
        Animal lion = new AnimalThatKeepsAMealLog();
        lion.eat();
        lion.eat();
        lion.eat();
    }
}

または、テストファイルを読んで、呼び出した回数を数えますeat()。答えは3つになると思います。

Shadov Aug 18 2020 at 23:35

別のアプローチの一種で、タスクの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());
    }
}