インターフェイスを実装し、他のクラスを実装しないクラスでのみ使用可能なメソッドを呼び出すより良い方法はどれですか？

基本的に、特定の条件が与えられたときにさまざまなアクションを実行する必要があります。既存のコードはこのように書かれています

基本インターフェース

// DoSomething.java
interface DoSomething {

   void letDoIt(String info);
}

最初のワーカークラスの実装

class DoItThisWay implements DoSomething {
  ...
}

2番目のワーカークラスの実装

class DoItThatWay implements DoSomething {
   ...
}

メインクラス

   class Main {
     public doingIt(String info) {
        DoSomething worker;
        if (info == 'this') {
          worker = new DoItThisWay();
        } else {
          worker = new DoItThatWay();
        }
        worker.letDoIt(info)
     }

このコードは問題なく動作し、理解しやすいです。

ここで、新しい要件のために、にのみ意味のある新しい情報を渡す必要がありDoItThisWayます。

私の質問は、次のコーディングスタイルがこの要件を処理するのに適しているかどうかです。

新しいクラス変数とメソッドを使用する

// Use new class variable and method

class DoItThisWay implements DoSomething {
  private int quality;
  DoSomething() {
    quality = 0;
  }

  public void setQuality(int quality) {
    this.quality = quality;
  };

 public void letDoIt(String info) {
   if (quality > 50) { // make use of the new information
     ...
   } else {
     ...
   }
 } ;

}

このようにすると、呼び出し元に対応する変更を加える必要があります。

   class Main {
     public doingIt(String info) {
        DoSomething worker;
        if (info == 'this') {
          int quality = obtainQualityInfo();
          DoItThisWay tmp = new DoItThisWay();
          tmp.setQuality(quality)
          worker = tmp;

        } else {
          worker = new DoItThatWay();
        }
        worker.letDoIt(info)
     }

それは良いコーディングスタイルですか？それとも私はそれをキャストすることができます

   class Main {
     public doingIt(String info) {
        DoSomething worker;
        if (info == 'this') {
          int quality = obtainQualityInfo();
          worker = new DoItThisWay();
          ((DoItThisWay) worker).setQuality(quality)
        } else {
          worker = new DoItThatWay();
        }
        worker.letDoIt(info)
     }

のquality各letDoIt()呼び出しと一緒に設定する必要があると仮定していますDoItThisWay()。

ここで発生している問題は次のとおりです。一時的なカップリングを導入しています（つまり、？を呼び出すsetQuality()前に呼び出しを忘れるとどうなりますか？）。そして、の実装とは、持っている（1つのニーズを発散していますletDoIt()DoItThisWayDoItThisWayDoItThatWaysetQuality()と呼ばれ、他のではありません）。

これは今のところ問題を引き起こさないかもしれませんが、最終的にあなたを悩ませるように戻ってくるかもしれません。letDoIt()品質情報がinfoあなたが通過するものの一部である必要があるかどうかをもう一度見て検討する価値があるかもしれません。しかし、これは詳細に依存します。

それは良いコーディングスタイルですか？

私の観点からは、どちらのバージョンもそうではありません。最初の呼び出しすることsetQualityの前にletDoIt呼び出すことができることで、時間のカップリング。あなたはDoItThisWay、の派生物として見ることにこだわっていますDoSomethingが、それは（少なくとも機能的には）そうではなく、むしろ

interface DoSomethingWithQuality {
   void letDoIt(String info, int quality);
}

これはMainむしろ次のようなものになります

class Main {
    // omitting the creation
    private DoSomething doSomething;
    private DoSomethingWithQuality doSomethingWithQuality;

    public doingIt(String info) {
        DoSomething worker;
        if (info == 'this') {
            int quality = obtainQualityInfo();
            doSomethingWithQuality.letDoIt(info, quality);
        } else {
            doSomething.letDoIt(info);
        }
    }
}

一方、パラメータをクラスに直接渡し（これが可能であると仮定）、どちらを使用するかを決定することをファクトリに委任できます（これにより、インスタンスが再び交換可能になり、両方を派生させることができますDoSomething）。これは次のようにMainなります

class Main {
    private DoSomethingFactory doSomethingFactory;

     public doingIt(String info) {
         int quality = obtainQualityInfo();
         DoSomething doSomethingWorker = doSomethingFactory.Create(info, quality);
         doSomethingWorker.letDoIt();
     }
}

あなたがそれを書いたことを知っています

パフォーマンス上の理由から、DoItThisWayとDoItThatWayの構築はMainのコンストラクターで1回行われるため

ただし、ファクトリで作成するのにコストがかかる部分をキャッシュし、コンストラクタに渡すこともできます。

qualityコンストラクタに渡すことができないと仮定し、への呼び出しsetQualityが必要。

現在、次のようなコードスニペット

    int quality = obtainQualityInfo();
    worker = new DoItThisWay();
    ((DoItThisWay) worker).setQuality(quality);

あまりにも多くの考えを投資するには小さすぎます。私見それは少しugいように見えますが、理解するのは本当に難しくありません。

このようなコードスニペットが大きくなり、リファクタリングのために次のような結果になると問題が発生します

    int quality = obtainQualityInfo();
    worker = CreateAWorkerForThisInfo();
    ((DoItThisWay) worker).setQuality(quality);

現在、そのコードがまだ正しいかどうかはすぐにはわかりません。コンパイラーはそれを知らせません。したがって、正しい型の一時変数を導入し、キャストを回避することは、実際の追加作業なしで、もう少し型安全です。

ただし、実際にtmpはより良い名前を付けます。

      int quality = obtainQualityInfo();
      DoItThisWay workerThisWay = new DoItThisWay();
      workerThisWay.setQuality(quality)
      worker = workerThisWay;

このような命名は、間違ったコードを間違って見えるようにするのに役立ちます。

ランタイムデータに基づいて初期化が異なる一般的なインターフェイスは、通常、ファクトリパターンに適しています。

DoThingsFactory factory = new DoThingsFactory(thingThatProvidesQuality);
DoSomething doSomething = factory.getDoer(info);

doSomething.letDoIt();

DoThingsFactoryはgetDoer(info)、DoSomethingインターフェイスにキャストされた具体的なDoThingsWithQualityオブジェクトを返す前に、メソッド内で品質情報を取得および設定することを心配できます。