팩토리 메서드 디자인 패턴을 구현하는 올바른 방법이

Question

특정 기준에 따라 다양한 프랜차이즈에 대한 수수료 구조를 구현해야합니다. 나는 등 다양한 구조에 대한

public class ST1 extends FranchiseFeeStructure{ 
    @Override 
    void getMonthlyFee(){ 
     monthlyFee=890; 
    } 

} 

과 ST2, ST3과 같은 FranchiseFeeStructure를 확장 클래스를 만들어 지금은 다른 구조에 대한 FranchiseFeeStructure

public abstract class FranchiseFeeStructure { 
     private BigDecimal registrationFee=1500; 
     protected BigDecimal monthlyFee; 

    public BigDecimal getMonthlyFees(){ 
      return monthlyFee; 
     } 

    public BigDecimal calculateFee{ 
     //calculate fee here 
     return fee; 
     } 
} 

추상 클래스를 만든다. 정적 팩토리 메소드를 가지는 클래스가 있습니다.

public class GetFeeStructureFactory { 

    public static FranchiseFeeStructure getFranchiseFeeStructure(String franchiseFeeStructureType){ 

     if(franchiseFeeStructureType == null){ 
      return null; 
     } 
     if(franchiseFeeStructureType.equalsIgnoreCase("ST1")) { 
       return new ST1(); 
      } 
     else if(franchiseFeeStructureType.equalsIgnoreCase("ST2")){ 
    /// so on... 

    } 
} 

지금

FranchiseFeeStructure franchiseFeeStructure = GetFeeStructureFactory.getFranchiseFeeStructure("ST1"); 
     franchiseFeeStructure.getMonthlyFee(); 
      franchiseFeeStructure.calculateFee(); 

이 방법 공장 패턴을 구현하는 올바른 방법인가로 사용합니다. 내가 잘못했거나 더 나아질 제안이 있다면 말해주십시오.

Answer 1

문제는 공장에서 구현되지 않습니다. 그것은 당신의 추상 클래스가 구현되는 방식에 있습니다. 모든 호출자가 (부적절하게 명명 된) 메소드 getMonthlyFee()을 호출하여 월간 요금을 초기화하도록합니다. 발신자가 그렇게해서는 안됩니다. 그리고 당신의 추상 클래스는 그렇게 추상적 인 방법에 ... 의존해야

public abstract class FranchiseFeeStructure { 
    private static final BigDecimal REGISTRATION_FEE = new BigDecimal(1500); 

    public final BigDecimal calculateFee { 
     BigDecimal monthlyFee = getMonthlyFee(); 
     // TODO compute 
     return fee; 
    } 

    /** 
    * Subclasses must implement this method to tell what their monthly fee is 
    */ 
    protected abstract BigDecimal getMonthlyFee(); 
} 

public class ST1 extends FranchiseFeeStructure { 
    @Override 
    protected BigDecimal getMonthlyFee(){ 
     return new BigDecimal(890); 
    } 
} 

그런 식으로, 호출 측이 월 사용료를 초기화 할 필요가 없습니다. 그것을 할 필요가 모든는 월 사용료가 항상 일정한 값이면, 당신도 추상 클래스와 여러 개의 서브 클래스를 필요로하지 않는 것이

BigDecimal fee = FeeStructureFactory.getFranchiseFeeStructure("ST1").calculateFee(); 

참고입니다. 필요한 것은 월간 요금을 생성자 인수로 사용하는 단일 클래스입니다.

Answer 2

그래, 생각은 똑같지 만 코드를 개선 할 수 있습니다. 나는 나에 따라 개선을 제안 할 것이다. 추상 클래스가 아닌 인터페이스와 구현으로 제시 하겠지만 아이디어는 같습니다. 당신이 무엇을해야하는 경우

interface FranchiseFee{ 
    BigDecimal getMonthlyFee(); 
    BigDecimal calculateFee(): 
} 

public class CocaColaFee implements FranchiseFee{ 

    public BigDecimal getMonthlyFee(){ 
    return new BigDecimal(".."); 
    } 

    public BigDeicaml calculateFee(){ 
    // do the calculation 
    } 

} 

public class PepsiFee implements FranchiseFee{ 

    public BigDecimal getMonthlyFee(){ 
    return new BigDecimal(".."); 
    } 

    public BigDeicaml calculateFee(){ 
    // do the calculation 
    } 

} 

public FranchiseFeeFactoty { 


// this is the easiest way to avoid if/else which makes the code awful. Also to use this you need to have default constructor, because as you can see you need additional code and it get's complicated. 

//the other way is to use map but it's just a more convenient if/else approach 

    public static FranchiseFee create(Class<? extends FranchiseFee> franchiseFeeType){ 

    try { 
      return franchiseFeeType.newInstance(); 
     } catch (InstantiationException e) { 
      e.printStackTrace(); 
     } catch (IllegalAccessException e) { 
      e.printStackTrace(); 
     } 

    } 
} 

그냥 물어 : 난 당신이