search

Jaejun-Sagong / memoTeamProject

MIT License
0 stars 4 forks source link

Entity 설계 #8

Open Code-Angler opened 2 years ago

Code-Angler commented 2 years ago

commit history에 달린 코드리뷰 참조 마이페이지 전 최종

Code-Angler commented 2 years ago

댓글과 대댓글을 하나의 Entity 로 설계하지 않고, 대댓글은 댓글의 하위로 들어가게 되는 또다른 entity로 독립적으로 설계하여, 대댓글은 댓글과의 연관 관계만 가지게 된다면, 대댓글을 작성하고 수정하는 부분에 있어서, 메모를 매번 불러 와서 대댓글을 생성해야하는 불필요한 과정을 하지 않아도 될것 같습니다. 댓글의 주체는 메모가 맞지만, 대댓글은 댓글의 달리는 하위라고 생각했을때 , 고려해볼만한 사항이라고 생각합니다.

https://github.com/Jaejun-Sagong/memoTeamProject/blob/d15822c87ba9b2ea38a4a2578727181713a06fe8/src/main/java/com/sparta/memoproject/model/Comment.java#L32

Code-Angler commented 2 years ago

다른것들과 다르게 해당 joinColumn 의 name 을 MEMO_ID로 쓰게 된것에 대한 이유를 설명할수있을까요? 다른 것들과 동일하게 소문자로 쓰려면 어떻게 하면 좋을지 생각해보기

https://github.com/Jaejun-Sagong/memoTeamProject/blob/d15822c87ba9b2ea38a4a2578727181713a06fe8/src/main/java/com/sparta/memoproject/model/Comment.java#L52

Code-Angler commented 2 years ago

count 값인 heartCnt 가 primitive type 인 long 이 아닌 Long 을 사용한 이유가 있을까요?

https://github.com/Jaejun-Sagong/memoTeamProject/blob/d15822c87ba9b2ea38a4a2578727181713a06fe8/src/main/java/com/sparta/memoproject/model/Comment.java#L81

Code-Angler commented 2 years ago

위와 같은 설계를 함으로서 발생할수 있는 상황에 대한 처리는 잘되어있는것으로 보입니다. 다만 현재 프로젝트에서 해당 entity를 양쪽에서 참조 함으로 얻을 수 있는 장점이 있는지 고려해보고 결정하는것이 좋을듯합니다. 또한 한쪽 entity 에서만 참조하도록 설계했을 때의 상황도 한번 생각해 봐도 좋을것 같습니다. 순환참조 문제는 해결이 되므로

https://github.com/Jaejun-Sagong/memoTeamProject/blob/d15822c87ba9b2ea38a4a2578727181713a06fe8/src/main/java/com/sparta/memoproject/model/Memo.java#L43