Skip to content

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