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

[JPA2] 이혜원 #22

Open
wants to merge 10 commits into
base: main
Choose a base branch
from

Conversation

icegosimperson
Copy link
Contributor

  1. 내가 개발한 기능

    • User CRUD, 비밀번호 재설정
    • Post CRUD, 조회수 증가, 게시글 검색 기능(제목, 내용, 작성자)
    • Comment CRUD, 댓글 좋아요, 댓글 취소 기능
  2. 내가 개발할 때 유의깊게 개발한 부분

    • 비밀번호 재설정 기능 : 사용자 인증X여서, PwdDto에 newPassword를 만들어 단순히 비밀번호를 새롭게 변경할 수 있게 구현 했습니다.
    • Post 게시글 검색 시 Like와 = 사용을 고민했었는데, =를 사용하여 완전 일치해야 검색 되도록 구현 했습니다.
    • Comment 좋아요 기능 : 좋아요를 누른 User의 순서가 중요하지 않은거 같아서 HashSet으로 구현 했습니다.
  3. 내가 개발하면서 들었던 의문 사항

    • 비밀번호 재설정 기능을 구현하는 다른 방법에 대해
    • Post 검색 범위를 Like와 = 중 무엇을 사용해야 좋을지
    • Comment 댓글 좋아요 기능을 구현하면서, 유저 1명이 게시글 당 좋아요 1개를 누를 수 있는데, this.likes--로 구현한게 맞는지
    • URL을 적절하게 사용 했는지
    • 테스트 코드를 어떤 위치에 작성해야 하는지 궁금합니다. 서치해보니 comment, post, user 바로 하단에 넣던데 다른 분들은 어떻게 구현하셨는지 궁금합니다.
  4. 리뷰어가 중점적으로 확인해야하는 부분

    • 비밀번호 재설정 기능을 이렇게 구현하는 것이 맞는지 확인 받고 싶습니다.
    • Comment 좋아요 기능에 대한 피드백
    • 불필요한 코드가 있는지 확인 받고 싶습니다.
  5. 테스트 코드X, 기능 구현 Postman 구현 기능 첨부

User 비밀번호 재설정 (rest-pwd)한 뒤, 유저 조회

스크린샷 2024-05-30 오전 12 16 10

Post 검색 기능 : 제목

스크린샷 2024-05-30 오전 12 14 31

Post 검색 기능 : 내용

스크린샷 2024-05-30 오전 12 14 39

Post 검색 기능 : 작성자

스크린샷 2024-05-30 오전 12 14 49

Comment 좋아요

스크린샷 2024-05-30 오전 12 14 58

Comment 좋아요 취소

스크린샷 2024-05-30 오전 12 15 06

Copy link

@Tentennball Tentennball left a comment

Choose a reason for hiding this comment

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

드디어 코드를 보게 되네요..ㅎㅎ 코드 MVC 잘지켜서 잘짜신것 같아요!! 코드 잘보고 갑니다!!

User user = userRepository.findById(userId)
.orElseThrow(() -> new IllegalArgumentException("해당 유저 없음"));

user.update(user.getUsername(), pwdDto.newPassword());

Choose a reason for hiding this comment

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

해당 메서드의 기능은 유저이름, 비밀번호를 업데이트 하는 메서드이니까,, user엔티티에 비밀번호 변경만 해주는 메서드가 따로 있어야 할것 같긴 해요!

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