[기능추가] 공통 에러 처리 로직 및 리스폰스 로직 작성 완료 - [merged] #9

Closed seoyoung059 closed 2 months ago

In GitLab by @sjhlko on Jul 18, 2024, 17:08

Merges back/init/S11P12A702-129 -> back/master

:link: 해당 MR과 연결된 JIRA의 태스크는 무엇인가요?

S11P12A702-129

🔍️ 해당 MR을 통해 해결하려는 문제가 무엇인가요?

어떤 기능을 구현한 건지, 이슈 대응이라면 어떤 이슈인지 MR이 열리게 된 계기와 목적을 Reviewer들이 쉽게 이해할 수 있도록 적어 주세요 다이어그램, 피그마를 첨부해도 좋아요

공통적인 응답 로직 처리 및 서비스 내 에러 처리 로직 구현

AppException을 통해 에러 코드를 커스터마이징 하고, 이를 정해진 Response 형식에 맞게 리턴할 수 있다.

✨ 해당 MR에서 핵심적으로 변경된 사항은 무엇일까요?

문제를 해결하면서 주요하게 변경된 사항들을 적어 주세요

Global 패키지 내의 프로젝트 공통 에러/리스폰스 처리 로직

🔖 핵심 변경 사항 외에 추가적으로 변경된 부분이 있나요?

없으면 "없음" 이라고 기재해 주세요

없음

📝 작업중 고찰하신게 있나요?

이번에 특정 기술 도입 또는 방향성 선택의 이유가 있다면, 그 부분에 대해서 설명해주세요!

정상 응답 및 에러 응답 시에 따른 에러 코드를 enum으로 처리하는데, 같은 패키지 내의 ResponseCode 인터페이스를 상속받은 ResponseEnum 에 각각 관리합니다. 이 부분에 대해 더 좋은 의견이 있다면 말씀해주세요

🙏 Reviewer 분들이 이런 부분을 신경써서 봐 주시면 좋겠어요

개발 과정에서 다른 분들의 의견은 어떠한지 궁금했거나 크로스 체크가 필요하다고 느껴진 코드가 있다면 남겨주세요

없으면 "없음" 이라고 기재해 주세요

더 나은 처리 방법이 있다면 공유해주세요.

✅ 제출 전 필수 확인 사항

[X] 빌드가 되는 코드인가요?
[X] 버그가 발생하지 않는지 충분히 테스트 해봤나요?

📌 MR 진행 시 이러한 점들을 참고해 주세요

Reviewer 분들은 코드 리뷰 시 좋은 코드의 방향을 제시하되, 코드 수정을 강제하지 말아 주세요.
Reviewer 분들은 좋은 코드를 발견한 경우, 칭찬과 격려를 아끼지 말아 주세요.
Comment 작성 시 Prefix로 P1, P2, P3 를 적어 주시면 Assignee가 보다 명확하게 Comment에 대해 대응할 수 있어요
- P1 : 꼭 반영해 주세요 (Request Changes) - 이슈가 발생하거나 취약점이 발견되는 케이스 등
- P2 : 반영을 적극적으로 고려해 주시면 좋을 것 같아요 (Comment)
- P3 : 이런 방법도 있을 것 같아요~ 등의 사소한 의견입니다 (Chore)

In GitLab by @sjhlko on Jul 18, 2024, 17:08

requested review from @wnso-kim

In GitLab by @sjhlko on Jul 18, 2024, 17:10

changed target branch from master to back/master

In GitLab by @wnso-kim on Jul 18, 2024, 17:26

Commented on BACKEND/ownBang/src/main/java/com/bangguddle/ownbang/global/response/Response.java line 9

[P2]
API 명세서상 성공에 대한 상태 코드는 200과 201만 사용합니다.
오버로딩을 통한 이점도 좋지만, 200과 201에 대한 메소드명을 구분하는 것이 좋을 것 같습니다.
ex) success / createSuccess

In GitLab by @wnso-kim on Jul 18, 2024, 17:26

200과 201을 구분하는 코드로 변경 요청 드립니다.

In GitLab by @wnso-kim on Jul 18, 2024, 17:26

requested changes

In GitLab by @sjhlko on Jul 18, 2024, 17:29

added 1 commit

c2c08d1b - 🐛 Fix: 공통 리스폰스 로직 중 상태코드 201,200 의 구분을 오버라이딩으로 두던것을 메소드 명을 바꾸는 것으로 수정

Compare with previous version

In GitLab by @wnso-kim on Jul 18, 2024, 17:34

resolved all threads

In GitLab by @wnso-kim on Jul 18, 2024, 17:34

Commented on BACKEND/ownBang/src/main/java/com/bangguddle/ownbang/global/response/Response.java line 9

반영해 주셔서 감사합니다.

In GitLab by @wnso-kim on Jul 18, 2024, 17:35

approved this merge request

In GitLab by @wnso-kim on Jul 18, 2024, 17:35

mentioned in commit eeec98ae77b6ea5ed2ff99012bd7de7d6c2cdbe4

seoyoung059 / Ownbang