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

230 common throw custom exception #240

Open
wants to merge 36 commits into
base: dev
Choose a base branch
from

Conversation

somarok
Copy link
Contributor

@somarok somarok commented Jun 7, 2024

:: 최근 작업 주제 (하나 이상의 주제를 선택해주세요.)

  • 기능 추가
  • 리뷰 반영
  • 리팩토링
  • 버그 수정
  • 컨벤션 수정

:: 구현 목표 (해당 브랜치에서 구현하고자 하는 하나의 목표를 설정합니다.)

  • 데이터 소스 단에서 exception 처리를 합니다.

:: 구현 사항 설명 (작업한 내용을 상세하게 기록합니다.)

  1. �데이터 소스 쪽 exception 처리
  2. 데이터 소스에서 데이터 없이 성공 실패 여부를 확인하기 위해 bool로 반환되는 부분 수정
  3. 데이터소스단 변경에 따른 레포지토리 수정
  4. 데이터소스단 변경에 따른 테스트 코드 수정

:: 성장 포인트 (해당 기능을 구현하며 고민했던 사항이나 새로 알게된 부분, 어려웠던 점 등을 작성합니다.)

  • 데이터 소스 단에서 발생 가능한 다양한 예외 케이스에 대해 고민하게 되었다.
  • exception 핸들링을 어떤 방식으로 할지 많이 고민해 볼 수 있었던 계기가 되었다.
  • 메서드의 동작이 명확해 진 것 같아서 코드의 가독성이 좋아졌다.

:: 기타 질문 및 특이 사항

hoogom88 and others added 28 commits June 6, 2024 10:44
@somarok somarok linked an issue Jun 7, 2024 that may be closed by this pull request
1 task
Copy link
Collaborator

@KimDonghyeok KimDonghyeok left a comment

Choose a reason for hiding this comment

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

LGTM 👍

Copy link
Collaborator

@SeongJongHo SeongJongHo left a comment

Choose a reason for hiding this comment

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

LGTM 👍

Copy link
Collaborator

@NalaJang NalaJang left a comment

Choose a reason for hiding this comment

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

LGTM 👍

@hoogom88 hoogom88 added the [PR] 추가 기능 구현 중 추가 기능 구현 중이니 잠시 리뷰 대기 바랍니다. label Jun 8, 2024
Copy link
Collaborator

@gyubit gyubit left a comment

Choose a reason for hiding this comment

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

LGTM 👍

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
[PR] 추가 기능 구현 중 추가 기능 구현 중이니 잠시 리뷰 대기 바랍니다.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[공통] 예외처리 통일
6 participants