Skip to content

Conversation

@nowijnah
Copy link

@nowijnah nowijnah commented Oct 9, 2024

로그인 성공

Screenshot 2024-10-09 at 1 25 16 PM

비번 바꾸기 실패

response
Screenshot 2024-10-09 at 1 24 19 PM
log
Screenshot 2024-10-09 at 1 24 43 PM

Comment on lines 45 to 56
// TODO : 회원 이름으로 회원 정보를 조회한다.
public List<MemberResponse> searchMembersWithName(String memberName) {
// DB에서 memberName에 해당하는 회원 정보를 조회한다.
List<Member> members = memberRepository.findByMemberName(memberName);
if(members==null)
throw new IllegalArgumentException("해당 회원이 없어여");

// 조회된 회원 정보를 List<MemberResponse>로 변환하여 반환한다.
return null;
return members.stream()
.map(MemberResponse::from)
.toList();
}
Copy link
Member

Choose a reason for hiding this comment

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

요 부분 꼼꼼하게 체크하는 것도 좋습니다!

여기서 한 가지 생각해보아야할 것이 있습니다.
지금은 회원 이름으로 회원 정보를 조회 요청하였는데 이에 대한 결과가 없는 상황입니다.

  1. 예외를 발생시키는 것이 프론트가 처리하기 편할 지
  2. 빈 리스트를 내려주는 것이 프론트가 처리하기 편할 지
    고민해보시면 좋을 것 같습니다.

저는 2번쪽으로 해주는 것이 편하다고 생각하고, 프론트 분들도 편하다고 해서 아래처럼 빈 것인지 체크하지 않고 바로 내려주는 쪽으로 진행했습니다.
참고하시면 될 것 같아요!

Suggested change
// TODO : 회원 이름으로 회원 정보를 조회한다.
public List<MemberResponse> searchMembersWithName(String memberName) {
// DB에서 memberName에 해당하는 회원 정보를 조회한다.
List<Member> members = memberRepository.findByMemberName(memberName);
if(members==null)
throw new IllegalArgumentException("해당 회원이 없어여");
// 조회된 회원 정보를 List<MemberResponse>로 변환하여 반환한다.
return null;
return members.stream()
.map(MemberResponse::from)
.toList();
}
// TODO : 회원 이름으로 회원 정보를 조회한다.
public List<MemberResponse> searchMembersWithName(String memberName) {
// DB에서 memberName에 해당하는 회원 정보를 조회한다.
List<Member> members = memberRepository.findByMemberName(memberName);
// 조회된 회원 정보를 List<MemberResponse>로 변환하여 반환한다.
return members.stream()
.map(MemberResponse::from)
.toList();
}

Comment on lines +73 to +74
if(member.isEmpty())
throw new IllegalArgumentException("넌 무슨 이름을 바꾼다는 말을");
Copy link
Member

Choose a reason for hiding this comment

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

ㅋㅋ 꿀잼이네요

Optional<Member> member = memberRepository.findById(memberId);
if(member.isEmpty())
throw new IllegalArgumentException("넌 무슨 이름을 바꾼다는 말을");
member.get().changeName(request.getNewname());
Copy link
Member

Choose a reason for hiding this comment

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

요 부분도 아까 보내드린 블로그 링크 참고하셔서 orElseThrow() 쓰는 쪽으로 수정해보시면 좋을 것 같네요!

Copy link
Member

@jjunhub jjunhub left a comment

Choose a reason for hiding this comment

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

고생하셨습니다 :)))

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.

3 participants