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

[자판기] 드디어 완료했다!!!! 햄볶~!! #188

Open
wants to merge 10 commits into
base: main
Choose a base branch
from

Conversation

Suxxxxhyun
Copy link

No description provided.

@JHyun0302
Copy link

JHyun0302 commented Nov 28, 2023

전체적으로 패키지를 잘 나누신거 같아요! 👍
여러 방법으로 객체지향 코드를 짤려는 노력이 보입니다!!

ProductsRepository의 findByProductName() 메서드와 같이 예외처리를 따로 해준 부분을 볼 수 있었어요.
하지만 문제는 예외를 처리하고 프로그램이 비정상 종료해버린다는 점이예요!!
"사용자가 잘못된 값을 입력할 경우 IllegalArgumentException를 발생시키고, "[ERROR]"로 시작하는 에러 메시지를 출력 후 해당 부분부터 다시 입력을 받는다." 라는 요구사항 때문에 저는 validation 패키지를 따로 만들어서 관리하는게 좋았어요!! 👍

@Suxxxxhyun
Copy link
Author

우앙!!! 꼼꼼하게 봐주셔서 감사합니다!!

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.

2 participants