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

[ohii] Chpater08_API 응답 통일 & 에러 핸들러 #35

Merged
merged 4 commits into from
Jun 6, 2024
Merged

Conversation

KkomSang
Copy link

@KkomSang KkomSang commented Jun 5, 2024

#️⃣ 연관된 이슈


#30

✅ 워크북 체크리스트

  • 모든 핵심 키워드 정리를 마쳤나요?
  • 핵심 키워드에 대해 완벽히 이해하셨나요?
  • 이론 학습 이후 직접 실습을 해보는 시간을 가졌나요?
  • 미션을 수행하셨나요?
  • 미션을 기록하셨나요?

✅ 컨벤션 체크리스트

  • 디렉토리 구조 컨벤션을 잘 지켰나요?
  • pr 제목을 컨벤션에 맞게 작성하였나요?
  • pr에 해당되는 이슈를 연결하였나요?
  • 적절한 라벨을 설정하였나요?
  • 스터디원들에게 code review를 요청하기 위해 reviewer를 등록하였나요?
  • 닉네임/main 브랜치의 최신 상태를 반영하고 있는지 확인했나요?

🥸 미션


image

image

image

📌 주안점

@KkomSang KkomSang requested review from ls-rain and Jangwon0319 June 5, 2024 14:50
@KkomSang KkomSang self-assigned this Jun 5, 2024
Copy link

@Jangwon0319 Jangwon0319 left a comment

Choose a reason for hiding this comment

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

이번 주도 수고많으셨습니다~!

-> @ControllerAdvice와 동일한 예외 처리 역할을 수행하면서 @ResponseBody를 통해 응답으로 Json 객체를 리턴할 수 있음
-> @Controller 또는 @RestController이 붙은 컨트롤러에서 발생하는 예외를 전역적으로 처리할 수 있음

@ExceptionHandler : 특정 controller에서만 발생하는 예외만 처리하므로 여러 controller에서 발생하는 에러 처리 코드가 중복될 가능성이 있음

Choose a reason for hiding this comment

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

추가 정보까지 좋네요!

Copy link

@ls-rain ls-rain left a comment

Choose a reason for hiding this comment

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

수고하셨습니다!

2) 누락 방지 -> 주입 데이터가 누락되면 컴파일 오류 발생
3) fianl 키워드 -> 생성자 주입만 사용가능! 값이 설정되지 않는 오류를 컴파일 시점에서 막아줌

<@Autowired>
Copy link

Choose a reason for hiding this comment

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

김영한님과의 콜라보,,,, 유익한정보 감사합니다!

@KkomSang KkomSang merged commit 624c442 into ohii/main Jun 6, 2024
@KkomSang KkomSang deleted the ohii/#8 branch June 6, 2024 02:26
@chaeyeon0130 chaeyeon0130 added the ✨ Chapter08 챕터 8 label Jun 17, 2024
@chaeyeon0130
Copy link
Member

최고!! 너무 수고하셨습니다💓💓
챕터 라벨도 슬쩍 연결해두고 갈게용

ohii/#8 브랜치 -> 여기서 #8은 이슈/PR 번호를 의미해요! 그래서 8번으로 생성된 pr, [ohii] Chpater02_AWS (VPC & Internet Gateway & EC2) #8 해당 pr에 연결됩니다🫠

8주차 이슈는 #30이니 ohii/#30 요렇게 브랜치를 생성하는 것이 맞습니다!!

다음 9주차도 파이팅!!!🔥🔥

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

Successfully merging this pull request may close these issues.

4 participants