[2단계 - 웹 자동차 경주] 제이(이재윤) 미션 제출합니다. #103
Merged
Conversation
This file contains 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
begaonnuri
requested changes
Apr 17, 2023
src/main/java/racingcar/controller/RacingGameWebController.java
Outdated
Show resolved
Hide resolved
src/main/java/racingcar/controller/RacingGameConsoleController.java
Outdated
Show resolved
Hide resolved
src/test/java/racingcar/controller/RacingGameWebControllerTest.java
Outdated
Show resolved
Hide resolved
src/main/java/racingcar/controller/RacingGameWebController.java
Outdated
Show resolved
Hide resolved
src/main/java/racingcar/controller/RacingGameWebController.java
Outdated
Show resolved
Hide resolved
begaonnuri
reviewed
Apr 19, 2023
src/main/java/racingcar/controller/RacingGameWebController.java
Outdated
Show resolved
Hide resolved
begaonnuri
requested changes
Apr 19, 2023
Comment on lines
+10
to
+15
| @NotBlank(message = "이름을 입력해주세요.") | ||
| private String names; | ||
|
|
||
| @NotNull(message = "시도 회수를 입력해주세요.") | ||
| @Min(value = 2, message = "최소 횟수는 2입니다.") | ||
| @Max(value = 10, message = "최대 횟수는 10입니다.") |
There was a problem hiding this comment.
@Valid를 사용해주셨네요! 어떤 장단점이 있었나요?
There was a problem hiding this comment.
- 제가 생각한
@Valid의 장점은 다음과 같습니다.
- 사용자의 입력에 대한 데이터 유효성을 검사할 수 있습니다. (DB에 잘못된 데이터가 들어갈 확률이 낮아집니다!)
- 사용하다보니 규모가 커질 경우 예외를 검증하는 코드를 따로 작성하지 않아도 돼서 생산성에 있어서 좋다고 생각합니다.
- 제가 생각한 단점은 다음과 같습니다.
- 어노테이션 사용법을 알아야하기 때문에, 모르는 팀원이 있으면 오히려 시간이 더 걸릴 수 있습니다.
- 또한 직접 사용하다 보니 어떤 예외를 던지는지에 대해 공부를 따로 했습니다. 제가 아직은 자세히 모르는 것일 수도 있겠지만, 커스텀 예외를 따로 만들어서 발생시키기 힘든 것 같습니다.
There was a problem hiding this comment.
좋네요 👍🏻 여기서 조금 더 보완해드리자면,
사용하다보니 규모가 커질 경우 예외를 검증하는 코드를 따로 작성하지 않아도 돼서 생산성에 있어서 좋다고 생각합니다.
편리한 점이 있어도 웹 요청에 한정해서 비교적 간단한 검증만 이루어져야하고, 핵심 도메인에 대한 검증까지 이를 대체하려고 하면 안됩니다. 복잡한 도메인 로직을 타기 전에 간단한 검증을 도와주는 장치이지 비즈니스 로직을 대체하는 장치가 아니라는 점이 중요합니다 🙂
커스텀 예외를 따로 만들어서 발생시키기 힘든 것 같습니다.
이는 좋게 작용할 여지도 있다고 생각해요. 잘못된 요청에 대해선 일괄로 가벼운 처리와 4xx 응답 코드를 던져주는처리를 할 수 있고, 또 커스텀 예외가 필요하다면 MethodArgumentNotValidException을 상속받는 커스텀 예외를 만들어서 처리할 수도 있을 것 같네요!
begaonnuri
approved these changes
Apr 20, 2023
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.
터틀, 안녕하세요!
이번 미션을 구현하면서, Service와 DAO 레이어의 역할을 분리할 수 있도록 노력했습니다.
Service 레이어는 비즈니스 로직을 처리할 수 있는 것의 초점을 맞췄고,
DAO 레이어는 조회, 저장 같은 데이터베이스에 접근할 수 있는 것의 초점을 맞췄습니다.
그리고 웹 컨트롤러에서 중복된 URI를 없애기 위해서
@RequestMapping()을 사용했습니다.또한 테스트 같은 경우 Controller는 요청에 초점을 맞추고, Service는 비즈니스 로직을 제대로 수행하는지에 초점을 맞추고 진행하였습니다.
이번에도 잘 부탁 드립니다!
감사합니다 :)