📌 [MarketBridge] 06. 개발일지 - 절대 싱글톤을 간과하지마..!

📖 절대 싱글톤을 간과하지마..!

05 개발일지를 보고 올 것을 추천드린당.

앞에서 말했다시피 커맨드와 쿼리를 분리했기 때문에 mock 객체를 따로만들었다. 자연스레 같은 데이터 저장소를 참조 할 수 없게 되었다.
어떻게 해결해야 할까?

✍️ 첫 번째 생각

public class BaseFakeOrderDetailRepository {

    protected Long autoGeneratedId = 0L;
    protected List<OrderDetail> data = new ArrayList<>();

    public void clear() {
        data.clear();
    }
}

public class FakeOrderDetailCommendRepository extends BaseFakeOrderDetailRepository implements OrderDetailCommendRepository {

    @Override
    public void save(OrderDetail orderDetail) {
        if (orderDetail.getId() == null || orderDetail.getId() == 0) {
            ReflectionTestUtils.setField(orderDetail, "id", ++super.autoGeneratedId, Long.class);
            super.data.add(orderDetail);
        } else {
            super.data.removeIf(item -> Objects.equals(item.getId(), orderDetail.getId()));
            super.data.add(orderDetail);
        }
    }
}

public class FakeOrderDetailQueryRepository extends BaseFakeOrderDetailRepository implements OrderDetailQueryRepository {

    @Override
    public List<OrderDetail> findByOrderNoAndProduct_IdIn(String orderNo, List<Long> productIds) {
        return super.data.stream()
                .filter(od -> od.getOrder().getOrderNo().equals(orderNo)
                        && productIds.contains(od.getProduct().getId()))
                .toList();
    }
}

이런식으로 부모의 필드에 공유값을 넣고 super로 접근해서 사용하면 되지 않을까 싶었다.

그런데 생각과 달리 save를 하더라도 절~대 queryRepository 쪽에서 찾아 올 수 없었다. 어디선가 동기화가 안되는 기분이 들었다.

그렇게 왜지왜지? 고민을 하던 도중 이건 너무나.. 간단한 자바 기본이었다는 것을 깨달았다.

바로 생성자 호출시 부모의 생성자(super())를 먼저 호출한다는 점이다.
이게 왜? 라고 생각한다면 커맨드에서 super()를 호출하면 인스턴스가 하나 만들어 질 것이다. 이어서 쿼리 쪽에서 super()를 호출하게 되면 하나의 인스턴스가 또 만들어질 것이다. 즉 서로 다른 인스턴스를 각 Repository에서 가지고 있으므로 당연히 동기화가 되지 않는 것이 당연하다...(삽질 한시간은 한듯,,이래서 기초가 중요하구나 ㅠㅠ)

그렇다면 이걸 어떻게 해결해야 할까?
바로 인스턴스를 하나만 만들면 해결되는 것이다. 바로 싱글톤 패턴을 사용하는 것이다..! (진짜 기초인데 너무너무 간과해버렸다..)

✍️ 두 번째 생각

@Getter
public class BaseFakeOrderDetailRepository {

    @Getter
    private static final BaseFakeOrderDetailRepository instance = new BaseFakeOrderDetailRepository();

    private Long autoGeneratedId = 0L;
    private List<OrderDetail> data = new ArrayList<>();

    protected Long increaseId() {
        return ++autoGeneratedId;
    }

    public void clear() {
        data.clear();
    }
}

public class FakeOrderDetailCommendRepository extends BaseFakeOrderDetailRepository implements OrderDetailCommendRepository {

    @Override
    public void save(OrderDetail orderDetail) {
        if (orderDetail.getId() == null || orderDetail.getId() == 0) {
            ReflectionTestUtils.setField(orderDetail, "id", getInstance().increaseId(), Long.class);
            getInstance().getData().add(orderDetail);
        } else {
            getInstance().getData().removeIf(item -> Objects.equals(item.getId(), orderDetail.getId()));
            getInstance().getData().add(orderDetail);
        }
    }
}

public class FakeOrderDetailQueryRepository extends BaseFakeOrderDetailRepository implements OrderDetailQueryRepository {

    @Override
    public List<OrderDetail> findByOrderNoAndProduct_IdIn(String orderNo, List<Long> productIds) {
        return getInstance().getData().stream()
                .filter(od -> od.getOrder().getOrderNo().equals(orderNo)
                        && productIds.contains(od.getProduct().getId()))
                .toList();
    }
}

앞에서 말했다시피 부모를 싱글톤으로 생성하면 된다. 이러면 자식 인스턴스에서 모두 같은 인스턴스를 바라보고 있기 때문에 저장하고 찾을 수 있게된다!!!

싱글톤에 주의할 점은 뭐늬뭐뉘해도 동시성 문제이다.
싱글톤에서 절~대 필드를 가지고 있어서는 안된다. 그런데 지금 코드에는 가지고 있지 않는가?
이는 테스트 코드이기 때문에 가능하다고 말하고 싶다. 동시에 작업이 되지 않을 것이기 때문에 동시성 문제를 고려할 필요가 없다.

실제 코드를 작성할 땐 매우매우 조심해야한다..

프로젝트가 끝나고 디자인 패턴에 대해 정리하려고 하는데 궁금하다면 본인 블로그의 싱글톤 패턴을 참고하면 좋을 것 같다..!