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

[java-baseball] ivy.lee(이다예) 과제 제출합니다. #4

Open
wants to merge 2 commits into
base: ivy/java-baseball
Choose a base branch
from

Conversation

dayaelee
Copy link

@dayaelee dayaelee commented Oct 1, 2024

늦어서 죄송합니다..!!

지난주에 기능 구현은 끝났지만 에러처리에서 애를 많이 먹어서 오늘에서야 제출합니다.

입력값을 받은 후, 유효한지 체크하여 IllegalArgumentException만 발생시키면 되는거였는데,
제가 그걸 자체적으로 try-catch문으로 잡는 코드를 작성한 채로 계속 예외 처리 테스트를 돌리고 있었더라고요..

그리고 종료도 계속 system.exit을 사용하고..
이번 기회에 문서를 꼼꼼하게 읽고 정확하게 요구사항 파악후 코드를 작성하는 것이 얼마나 중요한 것인지 알게됐습니다..

기능 별 커밋을 해야하는데 그 부분도 하지못하고 일괄 커밋을 해버렸습니다.
다음에는 유의해서 제출하겠습니다.

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.

1 participant