Decorator 에서 추상메서드로 뺄때 질문 있습니다.
56
1 asked
1) 실제 로직
@Slf4j
public class TrueComponent implements Component {
@Override
public String operation() {
log.info("TrueComponent 가 실행되었습니다.");
return "data";
}
}2) 추상클래스
@Slf4j
public abstract class AbstractorDecorator implements Component {
private Component component;
public AbstractorDecorator(Component component) {
this.component = component;
}
@Override
public String operation() {
log.info("AbstractorDecorator.operation() 실행");
String result = component.operation();
log.info("AbstractorDecorator.operation() 종료");
return result;
}
}2) 자식
@Slf4j
public class DecoratorTest1 extends AbstractorDecorator {
public DecoratorTest1(Component component) {
super(component);
}
@Override
public String operation() {
log.info("DecoratorTest1.operation() 실행");
super.operation();
log.info("DecoratorTest1.operation() 실행");
return "data";
}
}3) 자식
@Slf4j
public class DecoratorTest2 extends AbstractorDecorator {
public DecoratorTest2(Component component) {
super(component);
}
@Override
public String operation() {
log.info("DecoratorTest2.operation() 실행");
super.operation();
log.info("DecoratorTest2.operation() 실행");
return "data";
}
}4) 테스트 코드
@Slf4j
public class DecoratorPatternTest2 {
@Test
void abstractDecorator() {
Component trueComponent = new TrueComponent();
DecoratorTest2 decoratorTest2 = new DecoratorTest2(trueComponent);
DecoratorTest1 decoratorTest1 = new DecoratorTest1(decoratorTest2);
DecoratorPatternClinet decoratorPatternClinet = new DecoratorPatternClinet(decoratorTest1);
decoratorPatternClinet.execute();
}
}질문))) MessageDecorator, TimeDecorator 의 중복부분인
private Component component;
public AbstractorDecorator(Component component) {
this.component = component;
}해당 부분을 줄이기 위해서 추상메서드로 올렸는데
추상메서드로 올리고 코드를 실행하면
추상메서드의 log.info("AbstractorDecorator.operation() 실행"); 이부분이 어쩔수 없이 중복으로 호출되는데, 코드를 줄이기 위해서 사용하기 때문이 중복으로 호출되는건 상관이 없는 건가요?
Answer 1
0
안녕하세요, 인프런 AI 인턴입니다.
데코레이터 패턴을 다루면서 추상 클래스를 사용해 중복 코드를 줄이는 방안에 대해 고민하신 점 잘 이해했습니다. 실제로 현재 코드 구조에서는 AbstractorDecorator 의 operation() 메서드가 항상 호출되면서 그 안의 로그 메시지가 중복으로 출력되는 상황입니다.
이 부분에 대해 다음과 같은 두 가지 관점에서 고려해볼 수 있습니다:
중복된 로깅의 의미:
AbstractorDecorator의operation()메서드에서 로그 메시지가 출력되는 것은 그 메서드가 호출되었다는 것을 알리는 것입니다. 이를 통해 각 데코레이터가 어떤 순서로 호출되고 있는지 확인할 수 있으며, 데코레이터 패턴의 특성상, 중첩된 호출 경로를 이해하는데 도움이 될 수 있습니다. 그렇기 때문에 이 중복된 로깅이 꼭 나쁘다고만 볼 수는 없습니다. 그러나 이로 인해 로그가 불필요하게 많아지는 경우 오히려 디버깅을 방해할 수도 있습니다.로깅 위치 조정:
만약 이러한 중복 로그가 불필요하거나 너무 많다고 생각되신다면,AbstractorDecorator클래스 안의 로깅을 제거하고, 각 개별 구현 클래스(DecoratorTest1,DecoratorTest2) 안에서 더 구체적인 로깅을 추가하는 것도 방법 중 하나입니다. 이렇게 하면 각 구현 클래스의 흐름에 맞춰 로그를 조정할 수 있으며, 원하는 구문에서만 로깅이 출력되도록 할 수 있습니다.
결국 중복된 로깅을 유지할지 여부는 상황에 따라 다르게 판단할 수 있으며, 로깅의 양을 적절히 조정하여 전체 코드의 가독성 및 디버깅 효율성을 높이시는 것이 좋습니다.
아래 유사한 주제를 다룬 기존 답변 링크를 참조하시면 더 도움이 될 수 있습니다: “component 중복초기화 부분 추상클래스로 변경”.
저는 질문자님의 이해를 돕고자 지식공유자님의 기존 답변을 참고하여 유사한 답변 링크를 추천해드리고 있습니다. 현재 베타 기능으로 답변이 만족스럽지 않을 수 있는 점 양해 부탁드립니다. 🙏 추가적으로 궁금한 점이 있으시면, 이어서 질문해 주세요. 곧 지식공유자께서 답변해 주실 것입니다.
어드바이스 순서 디폴트 기준이 궁금합니다.
0
51
1
AspectV1 예제를 @Configuration 수동 등록으로도 가능한가요?
0
78
2
구체 클래스를 상속받아 확장한 형태도 클래스패턴/데코레이터 패턴이라고 칭하나요?
0
52
1
TraceTemplate 을 미리 빈으로 등록해서 사용할때 이렇게 설정하는게 맞는건가요?
0
53
1
대상 클래스에 기본 생성자가 없을 때
0
74
1
스프링 부트 버전 4.0 aop 의존성 명칭 변경
1
346
2
final 키워드 사용 관련 질문
0
75
1
안녕하세요
0
64
1
ThradLocal 실무 사례
0
97
2
실무에서의 동시성 문제
0
62
1
로그 등 부가 기능에 대한 테스트코드
0
50
1
ProceedingJoinPoint와 MethodInvocation에 대하여
0
67
1
실무 멀티스레딩에 대한 궁금증
0
69
1
안녕하세요 질문있습니다
0
43
1
AspectV3 aop 실행 순서 문의 드립니다
0
37
1
MDC vs AOP 중에 고민중입니다.
0
74
1
순환참조
0
158
2
this를 단독으로 사용할때랑 파라미터 바인딩 할때 차이
0
77
1
빈 후처리에 등록 질문 있습니다.
0
75
1
AppConfig 설정하는 설명중에 이해가 안되는게 있습니다.
0
88
1
ThreadLocal을 지역변수로 선언하면 remove가 필요할까요?
0
117
2
@Aspect 어노테이션으로 생성된 Advisor의 Bean 저장 여부
0
67
1
ThreadLocal 사용시 부작용이 있을수 있을까요? (ThreadLocal.remove()를 잘한다고 해도..)
0
73
2
빈으로 주입하지 않고 new로 객체 생성해도 되는데, 왜 ThreadLocal을 사용하나요?
0
72
1

