패턴을 사용한 리팩터링

Question

패턴 및 애플리케이션 아키텍처에 대해 많은 연습을하지 못했습니다. 간단히 말해, 필자는 어떤 피처를 대상으로하는 특정 속성을 찾아야합니다. 일부 코드는 작업을 더 잘 설명합니다.

IAttribute { 
    IAttribute analyze(IFunction func); 
} 

//up to 10 different attributes 
ArgumentsAttribute implements Attribute { 
    Map<String, ArgType> args = new HashMap<>(); 
    IAttribute analyze(IFunction func) { 
    for (Argument arg : func.getArgs()) { 
     args.put(arg.getName(), arg.getType()); 
    } 

    if (!args.isEmpty()) return this; 
    return null; 
    } 
} 

ReturnAttribute implements Attribute { 
    IAttribute analyze(IFunction func) { 
    if (func.hasReturn) return this; 
    return null; 
    } 
} 


AttributeAnalyzer { 
    List<Attributes> analyzeAttributes(IFunction func) { 


    List<IAttribute> attributes = new ArrayList<IAttribute>(); 
    attributes.add(new ArgumentAttribute()); 
    attributes.add(new ReturnAttribute()); 
    ... 

    for (IAttribute attr : attributes) { 
     attr = attr.analyze(func); 
     if (null == attr) attributes.remove(attr); 
    } 

    return attributes; 
    } 
} 

그러나이 구현은 약간 이상하게 보입니다. 나는 Attribute가 홀더의 일종이라는 사실을 싫어하지만 그 자체를 찾기 위해 메소드를 구현해야한다. 필자의 의견으로는 정적 방법에 과부하를주는 것이 가장 좋지만 분명히 불가능합니다. 이런 식으로, 우리는 새로운 추상화를 추가하지 않고 (아마도 내가 옳지 않은) 홀더를 논리를 분석하는 것과 분리 할 것입니다.

IAttribute { 
    static IAttribute analyze(); 
} 

ConcreteAttribute1 { 
    int x = 0; 
    static IAttribute analyze() { 
    ... 
    if (x != 0) return new ConcreteAttribute1(); 
    return null; 
    } 
} 

ConcreteAttribute2 { 
    String s = ""; 
    static IAttribute analyze() { 
    ... 
    if (!s.equals("")) return new ConcreteAttribute2(); 
    return null; 
    } 
} 


AttributeAnalyzer { 
    List<Attributes> analyzeAttributes() { 


    List<IAttribute> attributes = new ArrayList<IAttribute>(); 
    attributes.add(ConcreteAttribute1.analyze()); 
    attributes.add(ConcreteAttribute2.analyze()); 
    ... 

    for (IAttribute attr : attributes) { 
     if (null == attr) attributes.remove(attr); 
    } 

    return attributes; 
} 

}

또한, I 버릇 특성을 필터링한다. 그렇다면이 코드를 더보기 좋게 만들기위한 리팩터링 방법이 있습니까?

Answer 1

각 콘크리트 속성에 대해 중복이 거의 없거나 전혀없는 뚜렷한 analyze 함수가있는 경우 초기 코드 샘플이 그다지 좋지 않을 수 있습니다. 그런 다음 메서드의 서명을 boolean analyze()으로 변경합니다.

속성이 분석되는 방식에 더 많은 겹침이 있다면 AttributeAnalyzer 클래스 (또는 전용 클래스) 내에 단일 메서드 boolean analyze(IAttribute)을 고려할 수 있습니다.