inflearn logo
강의

Course

Instructor

Spring Core Principles - Advanced

Proxy Pattern and Decorator Pattern Summary

Decorator 에서 추상메서드로 뺄때 질문 있습니다.

56

dev dev

1 asked

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() 실행"); 이부분이 어쩔수 없이 중복으로 호출되는데, 코드를 줄이기 위해서 사용하기 때문이 중복으로 호출되는건 상관이 없는 건가요?

 

spring 디자인-패턴

Answer 1

0

communityai8509

안녕하세요, 인프런 AI 인턴입니다.

데코레이터 패턴을 다루면서 추상 클래스를 사용해 중복 코드를 줄이는 방안에 대해 고민하신 점 잘 이해했습니다. 실제로 현재 코드 구조에서는 AbstractorDecoratoroperation() 메서드가 항상 호출되면서 그 안의 로그 메시지가 중복으로 출력되는 상황입니다.

이 부분에 대해 다음과 같은 두 가지 관점에서 고려해볼 수 있습니다:

  1. 중복된 로깅의 의미:
    AbstractorDecoratoroperation() 메서드에서 로그 메시지가 출력되는 것은 그 메서드가 호출되었다는 것을 알리는 것입니다. 이를 통해 각 데코레이터가 어떤 순서로 호출되고 있는지 확인할 수 있으며, 데코레이터 패턴의 특성상, 중첩된 호출 경로를 이해하는데 도움이 될 수 있습니다. 그렇기 때문에 이 중복된 로깅이 꼭 나쁘다고만 볼 수는 없습니다. 그러나 이로 인해 로그가 불필요하게 많아지는 경우 오히려 디버깅을 방해할 수도 있습니다.

  2. 로깅 위치 조정:
    만약 이러한 중복 로그가 불필요하거나 너무 많다고 생각되신다면, 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