Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feature : Post 객체 구현 및 관련 기능 구현 완료 #188

Merged
merged 9 commits into from
May 6, 2024

Conversation

kf3907
Copy link
Member

@kf3907 kf3907 commented May 5, 2024

제목

Post 객체 구현 완료

PR 체크리스트

  • 기능 구현 사항 작성
  • 테스트 코드 구현
  • 생각되는 코드 개선점 (부족한점)
  • 코드 리뷰
  • 기타(아래에 자세한 내용 기입해주세요)

관련 이슈

  • 관련된 이슈 넘버가 있으면 이곳에 기입해주세요

기능 구현 사항

  • 장소 리뷰 등록 기능 ( 파사드 패턴 적용)
  • 장소 리뷰 단일 조회 기능
  • 북마크 개수 조회 기능
  • 유저의 모든 계시글 조회 기능
  • 계시글(장소 리뷰) 개수 조회 기능
  • 테스트 완료
  • PostReqDto, PostRespDto로 나눠서 각 요청, 응답 Dto 설계 완료
  • ReqDto에 Lombok 라이브러리를 이용한 유효성 검사 코드 완료

차후 개선 사항 및 나아갈 점

현재 S3 연동이 되어있지 않아 등록 부분에 관련 부분이 빠져있습니다.
ReviewRespDto와 관련된 부분의 삭제가 필요합니다.
Tag관련 부분을 5월6일 회의 때 논의가 필요합니다.
Visit관련 부분에 대한 논의가 필요합니다.

Review와 Post객체의 교체가 필요 합니다. 이 부분이 생각보다 방대해서 차후에 더 변경하는 것으로 하는게 좋을 것 같습니다.
현재 Review 코드 관련해서는 다 살려두었고 PinService에 핀생성 부분의 코드를 두 개로 나누어 두었습니다.

기타

Review와 Post 객체가 서로 흡사하여 Review 등록 시 Controller에서 여러 서비스를 불러오는 방식 보다
Facade 패턴 방식을 채용해 Controller에서 facade만 의존하여 좀 더 깔끔한 코드로 변경했습니다.

요청Dto와 응답Dto로 나누어서 관리하도록 설계했습니다.

#@! 주석을 통해 필요하다고 생각되는 코드를 파일안에 적어두었습니다! ex) //#@!좋아요 계시글 조회 기능 필요
Ctrl + Shift + F 를 통해서 검색해서 개발하시면 유용하실 겁니다!

Copy link
Collaborator

@dltjdgh0428 dltjdgh0428 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

고생하셨습니다

@dltjdgh0428 dltjdgh0428 merged commit b130588 into SWYP-3-Reading-everywhere:dev May 6, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants