인터페이스를 구현하지만 다른 클래스는 구현하지 않는 한 클래스에서만 사용 가능한 메소드를 호출하는 더 좋은 방법은 무엇입니까?

기본적으로 특정 조건에 따라 다른 작업을 실행해야합니다. 기존 코드는 이런 식으로 작성됩니다

기본 인터페이스

// DoSomething.java
interface DoSomething {

   void letDoIt(String info);
}

첫 번째 노동자 계급의 이행

class DoItThisWay implements DoSomething {
  ...
}

두 번째 노동자 계급의 이행

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()하기 전에 전화 를 잊어 버린 경우에 발생합니다 ). 그리고 구현 이 분기 되고 있습니다 (하나는 호출 해야 하고 다른 하나는 호출하지 않아야 함).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의 생성자에서 한 번 수행되므로

그러나 팩토리에서 생성하는 데 비용이 많이 드는 부품을 캐시하여 생성자에게 전달할 수도 있습니다.

quality생성자에 전달할 수 없으며 호출 setQuality이 필요 하다고 가정 해 봅시다 .

현재 코드 스 니펫은

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

너무 많은 생각을 투자하기에는 너무 작습니다. IMHO 조금 추한 것처럼 보이지만 실제로 이해하기는 어렵지 않습니다.

이러한 코드 스 니펫이 커지고 리팩토링으로 인해 문제가 발생합니다.

    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)콘크리트 DoThingsWithQuality 오브젝트 캐스트를 DoSomething 인터페이스로 리턴하기 전에 메소드 내부의 품질 정보를 가져오고 설정하는 것에 대해 걱정할 수 있습니다 .