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

충남대 BE_강지훈 6주차 과제 (1단계) #407

Open
wants to merge 8 commits into
base: kangwlgns
Choose a base branch
from

Conversation

kangwlgns
Copy link

코드 리뷰 시, 멘토님이 중점적으로 리뷰해줬으면 하는 부분

  • dto에서 형식 검사를 하니까 빨리 응답할 수 있다는 장점이 있는데, 유효성 검사에 대한 코드 중복이 생기게 되는 것 같습니다. 이에 대한 방안으로 entity에서 형식 검사를 하는 방법이나, 유효성 검사만 하는 util 클래스를 만들어서 검증하는 방법을 생각해보았습니다. 잘못된 방법인지 궁금합니다.
  • optionService에서는 ProductId가 존재하는지 검증하기 위해 productService가 필요하고, ProductService에서는 option을 최소 하나 넣은 상태를 유지하기 위해 OptionService가 필요합니다. 꽤 오랜 시간을 고민했지만 답을 찾을 수 없어서 논리적 유효성 검사를 Entity에서 하지 않고 service로 옮기는 식으로 처리하였습니다. 논리적 유효성 검사를 Service로 옮기는 것은 잘못됐다는 생각이 들어 검색해보니 중재자 패턴이라는 디자인 패턴을 적용하면 상호 참조를 해결할 수 있다는 것을 알았습니다. 하지만 애초에 이런 상호참조의 가능성이 생기게 설계한 잘못인 것인지에 대한 고민을 하게 되었습니다.

늦어서 죄송합니다.
항상 감사합니다.

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

Successfully merging this pull request may close these issues.

1 participant