Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[BackEnd] 한 service 클래스에 여러가지 repository가 있는 것이 부자연스럽습니다 #4

Open
soulchicken opened this issue Aug 28, 2022 · 2 comments

Comments

@soulchicken
Copy link
Member

  • 다른 서비스로 매소드를 통해 결과를 받아와서 사용하는 것이 좋을것 같아요.

지금 작성한 StayLikeService 클래스의 경우

@Service
public class StayLikeService {
    @Autowired
    private StayLikeRepository stayLikeRepository;

    @Autowired
    private StayRepository stayRepository;

    @Autowired
    private UserRepository userRepository;

한 클래스에 다른 여러 repository 객체를 가져오게 됩니다.
차라리 다른 service에 요청을 해서 결과를 가져오면 어떨까 합니다.

@95Glory
Copy link
Contributor

95Glory commented Sep 1, 2022

차주 예정되어있던 Code Refactoring 시간에 해당 안건에 대해서 적용해보면 좋을것 같습니다.

@soulchicken
Copy link
Member Author

객체의 순환참조가 있어서 일단은 이대로 가야할 것 같습니다.
우선순위에 밀리는 일이기 때문에 추후에 여유있을 때 작업 들어가면 될 것 같습니다.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants