Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
ScottSung7
changed the title
heeve1
approved these changes
Oct 31, 2024
|
|
||
| @Service | ||
| @RequiredArgsConstructor | ||
| public class ProgressBusinessWriteService { |
There was a problem hiding this comment.
어플리케이션 레이어에 존재하는군요. 요것 지난 멘토링때 서비스의 위치를 헷깔리셨던 것 같은데 이 구조로 계속 가게 될까요?
There was a problem hiding this comment.
안그래도 도메인 서비스와 어플리케이션 서비스를 리팩토링하려고 하고 있습니다. 아마 대부분의 어플리케이션 서비스는 많이 단순화 될것 같고 비즈니스 언어만 남을 것같은데, 도메인 서비스를 만들때 여러파일로 해야할지 하나의 서비스 파일에 담아야 할지 고민이 됩니다. (예를 들면 푸시 알림과 이메일 서비스를 따로 만들고 도메인에 넣어 주어야 할지 하나의 도메인 서비스 파일에 있어야 할지 고민 됩니다.)
| public class ProgressBusinessWriteService { | ||
|
|
||
| private final HelpDBAdapter helpDBAdapter; | ||
| private final PhotoSaver photoSaver; |
There was a problem hiding this comment.
인프라 계층에 PhotoSaver가 있긴한데 이렇게 하는 경우 인프라 계층을 나눈 이점을 얻을 수 있으까요?
There was a problem hiding this comment.
음.. 이 부분은 인터페이스를 두어야 할 것 같습니다. 생각해보면 응용계층이 인프라에 의존하는 상황인것 같긴합니다. 로직이 단순해서 편의를 위해서 아직 인터페이스까지 두기는 과하다고 생각했습니다.
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
변경사항