[♻️ refactor/#237] 도메인 기반 폴더링 구조로 변경#240
Merged
Merged
Conversation
2 tasks
Member
|
수고하셨습니다!! 회의에서 논의된대로 잘 분리해주신 것 같아요! 한 가지 논의하고 싶은게 있는데요! 저번 회의 때 잠깐 얘기하고 넘어간 것으로 기억하는데, 구체적으로 저희가 정하진 않았던 것 같아서요!
그때 회의에서 도메인별로 에러 및 성공 메시지를 분리하자는 이야기가 나왔던 것 같았는데요! 요거 확실히 어떻게 할 건지 정하면 좋을 것 같아요! 요거에 대해서 저도 생각을 해봤는데, 도메인 별로 분리하는 것 자체는 좋은 취지인 것 같아요! 지금 하나의 클래스에서 관리하고 있다보니.. 개발하면서 충돌나기도 쉬웠던 기억이 있네요..! 물론 도메인별로 분리하려면 제가 생각한 방안은 3개인데 다들 각자 어떤 방안이 좋으신지 궁금해요! 👀
저는 개인적으로 3번 방안도 괜찮다고 생각합니다! 도메인으로 분리하고, 전역적으로 파일을 모아두는 게 관리 측면에서 편할 것 같아서요! |
This file contains hidden or 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
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.

📄 Work Description
기존 폴더링 구조는 아래와 같습니다. (계층형)
변경한 폴더링 구조는 아래와 같습니다. (도메인)
설명
⚙️ ISSUE
💬 To Reviewers