싱글톤 방식의 주의점

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

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

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

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

• 가급적 읽기만 가능해야 한다.

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

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

시나리오

ThreadA가 사용자A 코드를 호출하고 ThreadB가 사용자B 코드를 호출한다. StatefulService의 price 필드는 공유되는 필드인데, 특정 클라이언트가 값을 변경한한다. 사용자 A의 주문금액은 1000원이 되어야 하는데, 2000원이라는 결과가 나오는 것이다!

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

수정 전 : 하나의 같은 인스턴스기 때문에 스레드 A,B 모두 price도 접근해서 값을 덮어 쓰게 된다!

package hello.core.singleton;

public class StatefulService {//어떤 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;
    }
}

수정 후 StatefulSertvice.java 상태 유지하는 private int price를 지우고 int타입으로 price를 리턴한다! 테스트코드에서도 스레드(사용자)마다 지역변수를 사용해서 price를 저장한다! 그러면 지역변수는 공유되지 않으므로 스레드(사용자)마다 값이 덮어쓰지 않고 고유의 값을 가질 수 있다!

package hello.core.singleton;

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