인프런 커뮤니티 질문&답변
작성한 질문수
Decorator 에서 추상메서드로 뺄때 질문 있습니다.
작성
·
17
0
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() 실행"); 이부분이 어쩔수 없이 중복으로 호출되는데, 코드를 줄이기 위해서 사용하기 때문이 중복으로 호출되는건 상관이 없는 건가요?
답변 1
0
안녕하세요, 인프런 AI 인턴입니다.
데코레이터 패턴을 다루면서 추상 클래스를 사용해 중복 코드를 줄이는 방안에 대해 고민하신 점 잘 이해했습니다. 실제로 현재 코드 구조에서는 AbstractorDecorator 의 operation() 메서드가 항상 호출되면서 그 안의 로그 메시지가 중복으로 출력되는 상황입니다.
이 부분에 대해 다음과 같은 두 가지 관점에서 고려해볼 수 있습니다:
중복된 로깅의 의미:
AbstractorDecorator의operation()메서드에서 로그 메시지가 출력되는 것은 그 메서드가 호출되었다는 것을 알리는 것입니다. 이를 통해 각 데코레이터가 어떤 순서로 호출되고 있는지 확인할 수 있으며, 데코레이터 패턴의 특성상, 중첩된 호출 경로를 이해하는데 도움이 될 수 있습니다. 그렇기 때문에 이 중복된 로깅이 꼭 나쁘다고만 볼 수는 없습니다. 그러나 이로 인해 로그가 불필요하게 많아지는 경우 오히려 디버깅을 방해할 수도 있습니다.로깅 위치 조정:
만약 이러한 중복 로그가 불필요하거나 너무 많다고 생각되신다면,AbstractorDecorator클래스 안의 로깅을 제거하고, 각 개별 구현 클래스(DecoratorTest1,DecoratorTest2) 안에서 더 구체적인 로깅을 추가하는 것도 방법 중 하나입니다. 이렇게 하면 각 구현 클래스의 흐름에 맞춰 로그를 조정할 수 있으며, 원하는 구문에서만 로깅이 출력되도록 할 수 있습니다.
결국 중복된 로깅을 유지할지 여부는 상황에 따라 다르게 판단할 수 있으며, 로깅의 양을 적절히 조정하여 전체 코드의 가독성 및 디버깅 효율성을 높이시는 것이 좋습니다.
아래 유사한 주제를 다룬 기존 답변 링크를 참조하시면 더 도움이 될 수 있습니다: “component 중복초기화 부분 추상클래스로 변경”.
저는 질문자님의 이해를 돕고자 지식공유자님의 기존 답변을 참고하여 유사한 답변 링크를 추천해드리고 있습니다. 현재 베타 기능으로 답변이 만족스럽지 않을 수 있는 점 양해 부탁드립니다. 🙏 추가적으로 궁금한 점이 있으시면, 이어서 질문해 주세요. 곧 지식공유자께서 답변해 주실 것입니다.