[BCSD Lab] 박종범 4차시 미션 제출합니다. #76
Merged
+364
−0
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
ImTotem
reviewed
Aug 12, 2025
| throw new IllegalArgumentException("실행 결과는 비어있을 수 없습니다."); | ||
| } | ||
| } | ||
| } |
| .limit(length) | ||
| .toList(); | ||
| } | ||
| } |
|
|
||
| package util; | ||
|
|
||
| public class LadderVaildator { |
There was a problem hiding this comment.
검증 로직의 위치는 사람마다 다른거 같아요.
지금까지 spring 사용해오던 방식대로는 validate 로직은 검증 대상(여기서는 Ladder)의 외부에 존재하는게 익숙한 것 같아요. 그런데 객체지향적으로 보면 검증 대상의 내부에 있는게 맞는 것 같아요.
개인적으로는 여러곳에서 범용적으로 사용되는 공통적인 validate 로직만 Validator로 분리하고, 나머지는 클래스 내부에 선언하는 방식을 선호해요.
validate의 위치에 대해 어떻게 생각하시나요? 의견이 궁금합니다!
There was a problem hiding this comment.
저는 가능하면 분리하는 방향을 선호하는거같아요.
저는 validate로직을 쓸때 if 문-> throw 형식을 자주 사용해서 코드가 길어지는 편인데,
클래스 내부에 위치하게 되면 클래스 코드가 너무 길어보여서 읽기가 힘든느낌을 받은거같아요.
저도 생각해보니 짦게 검증로직이 나올때는 객체 안에 짧게 정의해두는거같은데, 여러 객체들이 있을때는
길어지는 검증로직이 나오는 객체가 있을때를 대비해서 일관성을 위해 검증로직이 있으면 객체마다 validate클래스를 각각만들어두는거같아요
Co-authored-by: 박성빈 <[email protected]>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
4차시 미션 제출입니다.