[Spring] 싱글톤 컨테이너 - 주의점

싱글톤 방식의 주의점

• 싱글톤 방식은 여러 클라이언트가 하나의 같은 객체를 공유하기 때문에 싱글톤 객체는 상태를 유지(Stateful)하게 설계하면 안된다.
• 무상태(stateless)로 설계해야 한다.
  • 특정 클라이언트에 의존적인 필드가 있으면 안된다.
  • 특정 클라이언트가 값을 변경할 수 있는 필드가 있으면 안된다.
  • 가급적 읽기만 가능해야 한다.
  • 필드 대신 자바에서 공유되지 않는 지역변수, 파라미터, ThreadLocal등을 사용해야 한다.
• 스프링 빈의 필드에 공유 값을 설정하면 정말 큰 장애가 발생할 수 있다 !!!

상태를 유지(Stateful)할 경우 발생하는 문제점 예시

package hello.core.singleton;

public class StatefulService {
 private int price; //상태를 유지하는 필드
 
 public void order(String name, int price) {
 System.out.println("name = " + name + " price = " + price);
 this.price = price; //여기가 문제!
 }
 
 public int getPrice() {
 return price;
 }
}

상태를 유지(Stateful)할 경우 발생하는 문제점 예시

public class StatefulServiceTest {

   @Test
   void statefulServiceSingleton() {
   ApplicationContext ac = new AnnotationConfigApplicationContext(TestConfig.class);
 
 	StatefulService statefulService1 = ac.getBean("statefulService", StatefulService.class);
	StatefulService statefulService2 = ac.getBean("statefulService", StatefulService.class);

   //ThreadA: A사용자 10000원 주문
   statefulService1.order("userA", 10000);
   //ThreadB: B사용자 20000원 주문
   statefulService2.order("userB", 20000);

   //ThreadA: 사용자A 주문 금액 조회
   int price = statefulService1.getPrice();
   //ThreadA: 사용자A는 10000원을 기대했지만, 기대와 다르게 20000원 출력
   System.out.println("price = " + price);

   Assertions.assertThat(statefulService1.getPrice()).isEqualTo(20000);
 }
 
 static class TestConfig {
   @Bean
   public StatefulService statefulService() {
   	return new StatefulService();
   }
 }

• ThreadA가 사용자A코드를 호출하고 ThreadB가 사용자B코드를 호출한다 가정하자.
• StatefulService의 price필드는 공유되는 필드인데, 특정 클라이언트가 값을 변경한다.
• 사용자A의 주문금액은 10,000원이 되어야 하는데, 20,000원이라는 결과가 나왔다
• 실무에서 이런 경우를 종종 만나는데, 이로 인해 정말 해결하기 어려운 큰 문제가 터진다.
• 진짜 공유필드는 조심해야 한다 ! 스프링 빈은 항상 무상태(Stateless)로 설계하자.