inflearn logo
강의

강의

N
챌린지

챌린지

멘토링

멘토링

N
클립

클립

로드맵

로드맵

지식공유

스프링 핵심 원리 - 고급편

쓰레드 로컬 동기화 - 개발

쓰레드 로컬 관련 질문 있습니다.

106

황설탕

작성한 질문수 3

0


private void syncTraceId() {
    TraceId traceId = traceIdHolder.get();
    if (traceId == null) {
        traceIdHolder.set(new TraceId());
    } else {
        traceIdHolder.set(traceId.createNextId());
    }
}
private void syncTraceId() {
    if (traceIdHolder.get() == null) {
        traceIdHolder.set(new TraceId());
    } else {
        traceIdHolder.set(traceIdHolder.get().createNextId());
    }
}

강의 중에는 위 쪽처럼 코드를 작성해주셨는데
아래에 작성한 코드 처럼 TraceId 객체를 바로 get()으로 가져오지 않고,

별도의 traceId를 만들어 사용하신 이유가 궁금합니다

.
traceIdHolderThreadLoacl<TraceId> 타입인데
traceIdHolder.set(new TraceId()); 로 값 변경은 바로 하시는데 값을 가져오는 건 다른 방식인 것 같아서 궁금합니다~

spring 디자인-패턴

답변 1

0

인프런 AI 인턴

안녕하세요, 인프런 AI 인턴입니다. 황설탕님!

질문 주신 내용에 대해 설명드리겠습니다. ThreadLocal을 사용하여 동시성 문제를 해결할 때 코드의 작성 방식에 따라 조금의 차이가 있을 수 있습니다. 두 코드 모두 ThreadLocal<TraceId> 타입의 traceIdHolder를 사용하여 각 스레드에 대해 TraceId 객체를 관리합니다.

  1. 첫 번째 코드 스타일:
private void syncTraceId() {
    TraceId traceId = traceIdHolder.get();
    if (traceId == null) {
        traceIdHolder.set(new TraceId());
    } else {
        traceIdHolder.set(traceId.createNextId());
    }
}

첫 번째 방식은 traceId를 지역 변수로 가져와서 사용하는 방식입니다. 이 방식은 코드의 가독성을 높일 수 있습니다. traceId라는 명확한 이름을 사용하여 코드 읽기가 쉽고, 중복 호출을 피할 수 있습니다.

  1. 두 번째 코드 스타일:
private void syncTraceId() {
    if (traceIdHolder.get() == null) {
        traceIdHolder.set(new TraceId());
    } else {
        traceIdHolder.set(traceIdHolder.get().createNextId());
    }
}

두 번째 방식은 traceIdHolder.get() 메서드를 두 번 호출합니다. 이 방식은 코드가 약간 덜 명확할 수 있지만, 한 번의 코드 라인을 줄일 수 있습니다. 그러나 ThreadLocal에 접근하는 것이 비용이 큰 작업은 아니지만, 같은 결과를 두 번 호출하는 것은 비효율적일 수 있습니다.

두 코드는 결과적으로 같은 동작을 하지만, 첫 번째 방식이 조금 더 주석 없이도 의도를 파악하기 쉬운 장점을 가진다고 볼 수 있습니다.

관련 질문에 대한 답변을 참고하시길 바랍니다:
- TraceId 관련하여 질문 드립니다

이 링크에서는 TraceId와 관련된 다른 질문과 답변도 확인하실 수 있습니다. 참고하시어 더욱 깊이 있는 이해를 하시길 바랍니다.

저는 질문자님의 이해를 돕고자 지식공유자님의 기존 답변을 참고하여 유사한 답변 링크를 추천해드리고 있습니다.
현재 베타 기능으로 답변이 만족스럽지 않을 수 있는 점 양해 부탁드립니다. 🙏
추가적으로 궁금한 점이 있으시면, 이어서 질문해 주세요. 곧 지식공유자께서 답변해 주실 것입니다.

어드바이스 순서 디폴트 기준이 궁금합니다.

0

52

1

AspectV1 예제를 @Configuration 수동 등록으로도 가능한가요?

0

78

2

구체 클래스를 상속받아 확장한 형태도 클래스패턴/데코레이터 패턴이라고 칭하나요?

0

52

1

TraceTemplate 을 미리 빈으로 등록해서 사용할때 이렇게 설정하는게 맞는건가요?

0

53

1

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

0

56

1

대상 클래스에 기본 생성자가 없을 때

0

74

1

스프링 부트 버전 4.0 aop 의존성 명칭 변경

1

350

2

final 키워드 사용 관련 질문

0

76

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