(22.11.20) 9. 싱글톤 컨테이너(2) - 싱글톤 패턴의 주의점

싱글톤 방식의 주의점

싱글톤 패턴이든, 스프링 같은 싱글톤 컨테이너를 사용하든, 객체 인스턴스를 하나만 생성해서 공유하는 싱글톤 방식은 여러 클라이언트가 하나의 같은 객체 인스턴스를 공유하기 때문에 싱글톤 객체는 상태를 유지(stateful)하게 설계하면 안된다.

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

 

// 싱글톤으로 구현할 클래스
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;
    }
    
    
}


// Test
public class StatefulServiceTest {
	
    @Test
    void statefulServiceSingleton() {
    	ApplicationContext ac = 
        	new AnnotationConfigApplicationContext(TestConfig.class);	
        StatefulService statefulService1 = ac.getBean(StatefulService.class);
        StatefulService statefulService2 = ac.getBean(StatefulService.class);
        
        //ThreadA: A사용자 10000원 주문
        statefulService1.order("userA", 10000);
        
        //ThreadB: B사용자 20000원 주문
        statefulService2.order("userB", 20000);
        
        //ThreadA: 사용자 A 주문 금액 조회
        int price = statefulService1.getPrice();
        System.out.printlnf("price = " + price);
        
        Assertions.assertThat(statefulService1.getPrice()).isEqualTo(20000);
        
    }
    
    static class TestConfig {
    
    	@Bean
        public StatefulService statefulService() {
        	return new StatefulService();
        }
    }
}

• 'statefulServiceSingleton' Test에서, ThreadA에서 userA가 10000원을 주문한 뒤, ThreadB에서 userB가 20000원을 주문한다.
  • 문제는, userA가 ThreadA 주문 금액을 조회하면 userB의 주문 금액이 조회된다.
  • 싱글톤 패턴이기 때문에 특정 필드를 공유하기 때문에 발생하는 문제
  • *단순한 설명을 위해서 Thread는 실제로 사용하지 않았음
• 실무에서 이런 경우를 종종 보는데, 이로 인해 정말 해결하기 어려운 큰 문제들이 터진다.(몇년에 한 번씩 꼭 만난다.)
• 진짜 공유필드는 조심해야 한다! 스프링 빈은 항상 무상태(stateless)로 설계하자.

 

 

출처: 스프링 핵심 원리 - 기본편 (김영한)