[테크니컬 라이팅] 테드(김규태) 미션 완성본 제출합니다. #588
Open
+377
−1
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
donghoony
reviewed
Nov 5, 2024
| ```java | ||
| public class ReviewGroup { | ||
|
|
||
| private List<Review> reviews; |
| JPA를 사용하기 위해서 도메인 설계를 하기 전에 ERD를 먼저 작성했습니다. | ||
|
|
||
|
|
||
|  |
There was a problem hiding this comment.
도메인 용어가 나오기 이전에 충분한 설명이 필요합니다 !
| } | ||
| } | ||
| ``` | ||
| 이는 `Review` 가 `Review` 자체에 대한 책임뿐만 아니라, `ReviewGroup`의 책임도 지니게 하면서 **단일 책임 원칙을 위반**하게 됩니다. |
Comment on lines
+178
to
+180
| - 로직을 작성하기 쉬운 설계 | ||
| - 성능을 고려한 설계 | ||
| - 변경에 유연하고 유지 보수에 용이한 설계 |
| ```java | ||
| public class Review { | ||
|
|
||
| // 다른 필드 |
|
|
||
| 논리적으로 연관 관계를 재설정하면서 느꼈던 장점은 | ||
|
|
||
| 1. 객체 관계를 명확하게 정의함으로써, 설계 의도를 더 명확하게 드러낼 수 있었습니다. |
There was a problem hiding this comment.
있습니다 가 읽을 때 더 착 붙어요, 느낀 장점이라고 소개한 뒤에 현재형으로 시제를 바꿔도 좋겠습니다
| } | ||
| ``` | ||
|
|
||
| 각 템플릿에 속한 섹션마다, 섹션에 있는 질문 마다, 질문에 있는 질문 항목 마다 계속해서 데이터를 찾기 위해 DB에 접근하게 됩니다. |
nayonsoso
reviewed
Nov 5, 2024
Comment on lines
+16
to
+18
| 예를 들어, 특정 멤버가 자신이 참여한 프로젝트에 대한 리뷰를 받기 위해 프로젝트명(`group_name`)을 설정하고, 리뷰 그룹을 생성합니다. | ||
| 이후 리뷰를 받고 싶은 사람들에게 리뷰 요청을 보냅니다. 리뷰 요청을 받은 사람들이 해당 리뷰 그룹에 리뷰를 작성하면, | ||
| 멤버는 자신의 리뷰 그룹을 통해 받은 리뷰를 확인할 수 있어야 합니다. |
There was a problem hiding this comment.
이건 프로젝트 설명으로 옮기는건 어떨까요?😊
이 문단의 핵심 내용은 'JPA 를 사용한 엔티티 연관관계와 객체지향의 연관관계 방향이 다르다' 라는 것인데, 거기에 집중하게 하는게 더 좋을 것 같다 생각합니다.
Comment on lines
+326
to
+328
| ## 결론 | ||
| 불필요한 연관관계를 제거함에 있어서 오는 단점은 DB 접근 횟수 증가 외에도 더 많은 구현을 해보면 직접적으로 느끼는 것이 많을 거라 생각됩니다. | ||
|
|
There was a problem hiding this comment.
Suggested change
| ## 결론 | |
| 불필요한 연관관계를 제거함에 있어서 오는 단점은 DB 접근 횟수 증가 외에도 더 많은 구현을 해보면 직접적으로 느끼는 것이 많을 거라 생각됩니다. | |
| ## 결론 | |
| 불필요한 연관관계로부터 오는 단점은 DB 접근 횟수 증가 외에도 더 많은 구현을 해보면 직접적으로 느끼는 것이 많을 거라 생각됩니다. | |
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.
No description provided.