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

3주차 심화과제 #5

Open
wants to merge 2 commits into
base: #3
Choose a base branch
from
Open

3주차 심화과제 #5

wants to merge 2 commits into from

Conversation

onpyeong
Copy link
Collaborator

@onpyeong onpyeong commented May 3, 2023

✅ 과제 구현 명세

  • [심화과제]
    • user와 post를 1:N관계로 매핑했습니다.
    • API
      • 유저 생성
      • 게시물 생성, 유저아이디로 게시물 조회, 제목으로 게시물 조회
    • 예외 처리
      • [유저아이디로 게시물 조회] 시 유저 아이디가 없는 경우 UserNotFoundException을 service에서 throw하고
        controller에서 catch하여 처리하도록 구현했습니다.

😊 이런 점이 새로웠어요 / 어려웠어요

  • 매핑할 때 사용하는 어노테이션이 새로웠습니다
  • user와 post를 양방향 매핑으로 구현했는데, 단방향으로 해도 되는지 궁금합니다 (post만 user를 알면 되는건지?)
  • 예외 처리를 제대로 구현한건지 잘 모르겠어서 피드백 부탁드립니다!

@onpyeong onpyeong self-assigned this May 3, 2023
Copy link

@dong2ast dong2ast left a comment

Choose a reason for hiding this comment

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

잘 봤습니다!

try {
return ApiResponseDto.success(SuccessStatus.CREATE_POST_SUCCESS, postService.createPost(request));
} catch (UserNotFoundException e) {
return ApiResponseDto.error(ErrorStatus.USER_NOT_FOUND);
Copy link

Choose a reason for hiding this comment

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

혹시 이 에러 상황일 때 http 응답코드가 어떻게 반환되나요? @ResponseStatus(HttpStatus.CREATED)가 있어도 응답코드가 404로 나올지 궁금합니다!

@Builder
public User(String nickname, String email, String password) {
this.nickname = nickname;
this.email = email;
this.password = password;
}

//mappedBy 양방향 연관관계 주인 user x -> post(외래키를 가진 쪽)
@OneToMany(mappedBy = "user") //일대다(1:N)
private final List<Post> posts = new ArrayList<>();
Copy link

Choose a reason for hiding this comment

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

user를 가지고 post에 접근할 일이 없다면 양방향으로 설정하지 않아도 될 것 같아요!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants