[step1] refactor: Controller 로직 분리 및 서비스 계층 적용 #4
+119
−30
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
Collaborator
seeeeeeong
commented
Jan 24, 2025
•
seeeeeeong
commented
- Controller 로직을 Service와 Repository로 분리하였습니다.
- UrlShortenService에 대한 테스트 코드를 작성했습니다.
Hyune-c
requested changes
Jan 24, 2025
seeeeeeong
commented
Jan 25, 2025
Hyune-c
requested changes
Jan 25, 2025
build.gradle.kts
Outdated
|
|
||
| // Lombok | ||
| compileOnly("org.projectlombok:lombok") | ||
| annotationProcessor("org.projectlombok:lombok") |
There was a problem hiding this comment.
Controller와 Service에서 편의성을 위해 @requiredargsconstructor를 사용하기 위해 Lombok을 활용해왔습니다. 이에 대해 찾아보니 생성자 주입의 경우 Lombok을 사용해도 괜찮다는 의견과, 지양해야 한다는 의견이 있어 이에 대해 멘토님의 의견이 궁금합니다.
There was a problem hiding this comment.
저는 lombok 사용을 선호하지 않습니다. 하지만 어쩔수 없이 사용하긴 할 것 같습니다.
- dto 생성은 record 로 대체할 수 있지만, 생성자 주입과 로그는 대체가 힘들어서 입니다.
그리고 코멘트에 @ 를 다실 때는 블럭으로 감싸주세요.
Comment on lines
17
to
18
| @DisplayName("shorten-url을 생성하고 조회한다.") | ||
| void shortenUrlCreateAndSearch() { |
There was a problem hiding this comment.
Suggested change
| @DisplayName("shorten-url을 생성하고 조회한다.") | |
| void shortenUrlCreateAndSearch() { | |
| void shorten_url을_생성하고_조회한다() { |
nit 저는 이런 형태를 더 선호 합니다.
- DisplayName 을 따로 써야되는 것은 둘째치고, 그것에 해당하는 메서드를 영작해야되는데 꽤 귀찮습니다.
- 꼭 이렇게 써야된다면 저는 메서드 이름을 그냥 test1 로 했던 것 같습니다.
- 사실 이제는 kotlin 으로 넘어와서 큰 신경 안쓰고 있습니다.
| String key = urlShortenService.shortenUrlCreate(expectedOriginUrl); | ||
| String originUrl = urlShortenService.shortenUrlSearch(key); | ||
|
|
||
| assertTrue(originUrl.equals(expectedOriginUrl)); |
There was a problem hiding this comment.
Suggested change
| assertTrue(originUrl.equals(expectedOriginUrl)); | |
| assertThat(originUrl).isEqualTo(expectedOriginUrl); |
nit
| } | ||
|
|
||
| public String shortenUrlCreate(String originUrl) { | ||
| String randomKey = String.valueOf(new Random().nextInt(10000)); |
There was a problem hiding this comment.
new Random() 이 메서드 안에 있는 것이 의도된 것일까요?
|
|
||
| private final UrlShortenRepository urlShortenRepository; | ||
|
|
||
| public String shortenUrlSearch(String key) { |
There was a problem hiding this comment.
shortenUrl 이 반복되는 것이 의도된 것일까요?
|
|
||
| public String shortenUrlSearch(String key) { | ||
| return urlShortenRepository.findByKey(key) | ||
| .orElseThrow(() -> new IllegalArgumentException("Invalid key")); |
There was a problem hiding this comment.
로직상 존재하지 않는 key 인 경우 예외가 발생됩니다.
- Invalid key 는 존재하지 않는 key 를 뜻하는 것일까요?
| } | ||
|
|
||
| public String shortenUrlCreate(String originUrl) { | ||
| String randomKey = String.valueOf(new Random().nextInt(10000)); |
There was a problem hiding this comment.
randomKey 채번의 위치가 여기가 맞을까요?
Hyune-c
approved these changes
Jan 25, 2025
Hyune-c
changed the title
Hyune-c
changed the title
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.