Service layer 순환참조

Question

전에도 글을 올렸는데예제 코드를 이상하게 올린 것 같아 다시 올립니다.우선, Service 단에서는 다른 Service나 자신의 Mapper만 의존하도록 정하고 개발하고 있습니다.AService 에서BService 의 getBList()를 사용하고BService 에서 AService 의 getAList(), checkA()를 사용하는 상황입니다.@Service @RequiredArgsConstructor public class AService { private final AMapper aMapper; private final BService bService; public void saveA() { ... if (checkA(getAList("2024")[0])) { ... } bService.getBList("2022"); ... } public List getAList(String code) { ... } public boolean checkA(A a) { ... } }@Service @RequiredArgsConstructor public class BService { private final BMapper bMapper; private final AService aService; public void saveB() { ... if (aService.checkA(aService.getAList("2024")[0])) { ... } ... } public List getBList(String code) { ... } }이전 글에서 A, B를 같이 의존하는 상위 클래스(CService)를 만들라는 답변을 받았습니다.그래서 제가 생각한 것은 아래와 같이 설계를 해보았습니다.@Service @RequiredArgsConstructor public class CService { private final AService aService; private final BService bService; // BService에서 사용하는 AService 메서드들 public List getAList(String code) { return aService.getAList(code); } public boolean checkA(A a) { return aService.checkA(a); } // AService에서 사용하는 BService 메서드들 public List getBList(String code) { return bService.getBList(code); } }하지만 이것들을 결국 또 사용하려면 A,BService에서 각각 CService를 의존해야될텐데 역시 순환참조가 발생하게 됩니다.상위클래스를 만들라는게 이런식이 아닌걸까요? 도움주시면 정말 감사하겠습니다.

Clickin · Answer

잘 이해가 안가시나봅니다...일단 AService에서 B의 메소드를 호출한다고만 명시되어있고 왜 쓰는지, 예를 들어 파라미터로 들어가는지 아니면 검증용으로 쓰는지 등등은 하나도 알 수가 없으니까 제 마음대로 예시를 써보겠습니다.@Service @RequiredArgsConstructor public class CService { private final AService aService; private final BService bService; public void saveA() { ... if (aService.checkA(aService.getAList("2024")[0])) { ... } bService.getBList("2022"); SaveADto saveA = new SaveADto(); .... // saveA에 필요한 필드를 가진 DTO 생성 및 값 바인딩. aService.saveA(saveA); } }

코딩하는 두루미 · Answer

순환 참조가 일어나는 이유를 먼저 파악해보시는게 좋을 듯 합니다.왜 A, B 서비스에서 각각 서로를 주입받았을 때 문제가 되는지요

감독님테스트가하고싶어요 · Answer

위에 알려주신 방법대로 상위 서비스로 올리는 방법도 있고,근데 보통은 서비스레이어에서 모든 것을 해결하려고하다보니 이런 문제가 생깁니다.보통은 entity에 비지니스로직을 담으면 굳이 service끼리 의존성을 지니는 상황을 벗어날 수 있습니다.예를 들면 userSevice.validate(user), teamService.validate(team) 이게 아니라user.validate(), team.validate() 로 사용할 수 있게 하는거죠

Service layer 순환참조

3개의 답변

Q&A 목록