190일차(모험 99일차) - 싱글톤 방식의 주의점⭐

[add] 싱글톤 방식의 주의점 [#21] #25

- 싱글톤 패턴이든, 스프링 같은 싱글톤 컨테이너를 사용하든,

객체 인스턴스를 하나만 생성해서 공유하는 싱글톤 방식은

여러 클라이언트가 하나의 같은 객체 인스턴스를 공유하기 때문에

싱글톤 객체는 상태를 유지(stateful)하게 설계하면 안된다.

 

- 무상태(stateless)로 설계해야 한다!

  • 특정 클라이언트에 의존적인 필드가 있으면 안된다.

  • 특정 클라이언트가 값을 변경할 수 있는 필드가 있으면 안된다!

  • 가급적 읽기만 가능해야 한다.  (가급적 값을 수정하면 안된다.)

  • 필드 대신에 자바에서 공유되지 않는, 지역변수, 파라미터, ThreadLocal 등을 사용해야 한다.

 

- 스프링 빈의 필드에 공유 값을 설정하면 정말 큰 장애가 발생할 수 있다.

 

 

 

 

StatefulService와 StatefulServiceTest 코드 작성

 

 

최대한 단순히 설명하기 위해, 실제 쓰레드는 사용하지 않았다.

ThreadA가 사용자A 코드를 호출하고 ThreadB가 사용자B 코드를 호출한다 가정하자.

StatefulService 의 price 필드는 공유되는 필드인데, 특정 클라이언트가 값을 변경한다.

 

// StatefulServiceTest
statefulService1에서 사용자 A는 10000원을 주문 

// StatefulService
this.price = price 에 10000원이 들어가서 private int price에 10000원이 할당

statefulService2에서 사용자 B는 20000원을 주문 → private int price가 20000원으로 세팅

* statefulService는 같은 객체이다. 

private int price가 원래는 10000원 이었는데 20000원으로 변경되었다.

 

사용자A의 주문금액은 10000원이 되어야 하는데, 20000원이라는 결과가 나왔다.

실무에서 이런 경우를 종종 보는데, 이로인해 정말 해결하기 어려운 큰 문제들이 터진다.

(몇년에 한번씩 꼭 만난다.) 

공유필드는 조심해야 한다! 스프링 빈은 항상 무상태(stateless)로 설계하자

 

 

 

어떻게? 👉🏻 필드 대신에 자바에서 공유되지 않는, 지역변수, 파라미터, ThreadLocal 등을 사용한다.

public class StatefulService {
    public int order(String name, int price) {
        System.out.println("name = " + name + " price = " + price);
        return price;
    }
}