search

almondiz / almondiz-server

0 stars 1 forks source link

[feature] #49 post 디테일 페이지 api 구현 #51

Closed coderclcl closed 1 year ago

coderclcl commented 1 year ago

개요 ✋

post 디테일 페이지 api 구현

작업사항 🗒️

기존 PostResponseDto의 데이터들을 프론트엔드 쪽에서 보기 편하게 정돈하였습니다!

변경로직 🛠️

내용을 적어주세요.

yejipractice commented 1 year ago

@coderclcl LGTM 👍 💯
한 가지 제안 드리면 user를 그냥 엔티티 자체로 dto에 넣는 것보단 user에서도 필요한 항목만 넣으면 더 좋을 것 같긴 해요! 예를 들어서 this.user = user 에서 this.userId = user.getId(); this.nickName = user.getNickName(); 이런 식으로?

coderclcl commented 1 year ago

@yejipractice 아하 사실 말씀해주신 방식이 기존 방식이고 dto 넣는 방식이 새로운 방식인데 프론트에서는 관련된 것끼리 묶어서 전달해주길 원하는 것 같더라구요! (Post 안의 user, store가 각각의 객체처럼 묶여서) 그래서 Post에서 필요한 user 정보만 넣은 UserAsWriterResponseDto 를 만들어 봤습니다..! 객체 안의 객체로 묶기 vs 객체 안에 정보 나열하기 중에 뭐가 나을까요?? 저는 성능상 이슈만 없다면 전자가 조금 깔끔한 것 같더라구요! (성능 비교는 아직 잘 모르겠네요..)

yejipractice commented 1 year ago

@coderclcl 아하 프론트에서 원하는 방식으로 주는 게 나을 것 같아요! 그건 몰랐군요! 필요한 정보만 받길 원할 줄 알았거든여.. 좋아요 그렇다면 머지하셔도 될 것 같습니다! 👍

coderclcl commented 1 year ago

@yejipractice 네네 ㅎㅎ 항상 꼼꼼한 리뷰 감사합니다 :smiley::+1: