Replies: 8 comments
-
|
문서 잘 정리해주셔서 쉽게 읽혔습니다ㅎㅎ 그런데, Readme에 있는 제목이랑 페이지에 들어가서 보는 제목과 달라서 좀 헷갈렸습니다. 예를 들어 어노테이션 규칙 페이지를 들어가면 주석 규칙이 나오고 디자인 패턴 들어가면 설계 정책이 나와서 잘 들어왔는지 헷갈렸습니다!! 고생하셨습니다!!!! |
Beta Was this translation helpful? Give feedback.
-
|
디자인 가이드 부분에 논의해야할 부분이 많아보입니다. 먼저 제 의견 몇 개 작성하겠습니다. CAUSW_backend_docs/docs/design-guide.md Lines 20 to 26 in 4f9ad82 Transactional은 개인적으로 메서드레벨에 달아 명확성을 확보하는 걸 선호합니다. 또 이번 아키텍처에서 Service 계층과 Implementation 계층으로 나뉠텐데 두 계층 모두에 각각 다는 것이 좋다고 생각합니다. 단 기본 전파동작이 Required로 동작함을 명확히 인지해야 할 것 같습니다. CAUSW_backend_docs/docs/design-guide.md Lines 36 to 38 in 4f9ad82 Record + Builder를 사용하는게 좋다고 생각합니다. CAUSW_backend_docs/docs/design-guide.md Lines 51 to 53 in 4f9ad82 저는 개별 인자로 넘기는 방식을 사용하긴 했는데 이번 아키텍처가 Service -> Implementation이니 Service까지는 DTO를 넘겨도 괜찮을 수 있겠다는 생각을 합니다. CAUSW_backend_docs/docs/design-guide.md Lines 57 to 59 in 4f9ad82 코드 포매팅 기능에 import 순서도 직접 적용해주는 것 같습니다. naver 코딩 컨벤션 사용하였으니 그대로 사용해도 될 것 같습니다. |
Beta Was this translation helpful? Give feedback.
-
|
완료 |
Beta Was this translation helpful? Give feedback.
-
|
완료 |
Beta Was this translation helpful? Give feedback.
-
|
완료 |
Beta Was this translation helpful? Give feedback.
-
|
완료 |
Beta Was this translation helpful? Give feedback.
-
|
완료 |
Beta Was this translation helpful? Give feedback.
-
|
완료 |
Beta Was this translation helpful? Give feedback.
Uh oh!
There was an error while loading. Please reload this page.
-
컨벤션에 대해서 자유롭게 피드백해주세요.
모두 읽으신 분들은 '완료' 라고 댓글 달아주세요.
Beta Was this translation helpful? Give feedback.
All reactions