[STEP 3-1-1] 만료시간 추가 #95
Open
+93
−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
jiheon2234
commented
Mar 6, 2025
jiheon2234
commented
- api 사용자가 단축 url을 만들 때, ttlMinutes를 추가할 수 있도록 변경
- 원래는 단축된 URL만 저장했으나, 요구사항의 변경에 따라 ID, 원본URL, 단축URL, expiredAt 을 한번에 저장하도록 변경
- 만료시간이 지난 URL은 조회할 수 없도록 변경 (추후에 스케줄링을 통한 논리적 삭제로 변경 예정)
- api 사용자가 단축 url을 만들 때, ttlMinutes를 추가할 수 있도록 변경 - 원래는 단축된 URL만 저장했으나, 요구사항의 변경에 따라 ID, 원본URL, 단축URL, expiredAt 을 한번에 저장하도록 변경 - 만료시간이 지난 URL은 조회할 수 없도록 변경 (추후에 스케줄링을 통한 논리적 삭제로 변경 예정)
jiheon2234
commented
Mar 6, 2025
There was a problem hiding this comment.
기존에 단축된 url string값만 저장하는대신, 연관된 정보들을 함께 저장하도록 변경하였습니다.
jiheon2234
commented
Mar 6, 2025
| @@ -29,28 +33,35 @@ public class ShortenURLService { | |||
| * @throws BusinessLogicException 원본 URL을 찾을 수 없는 경우 | |||
| */ | |||
| public String getOriginURL(String shortenedURL) { | |||
| String originURL = repository.findByShortenedURL(shortenedURL) | |||
| ShortenedURLEntity entity = repository.findByShortenedURL(shortenedURL) | |||
| .filter((se) -> se.getExpiredAt().isAfter(LocalDateTime.now())) | |||
| .orElseThrow(() -> BusinessLogicException.from(BusinessExceptionCode.ORIGIN_URL_NOT_FOUND)); | |||
There was a problem hiding this comment.
만료시간이 지난 URL들은 조회시 응답하지 않도록 설정했습니다.
추후 스케줄러를 통한 논리적 삭제 방식으로 변경할 생각입니다.
Hyune-c
approved these changes
Mar 6, 2025
|
|
||
| @Min(value = 0, message = "만료 기간은 음수일 수 없습니다.") | ||
| @Max(value = 1440, message = "만료 기간은 최대 하루(1440분) 입니다.") | ||
| int ttlMinutes |
There was a problem hiding this comment.
nit wrapper 자료형 사용을 권장 합니다.
| this.originURL = originURL; | ||
| this.shortenedURL = shortenedURL; | ||
| this.expiredAt = expiredAt; | ||
| } |
There was a problem hiding this comment.
빌더는 위험 합니다. 불가피하다면 생성자나 팩토리 메서드 안에서만 사용을 권장 합니다.
- 위험한 이유는 직접 검색해보세요.
|
|
||
| public void markDelete() { | ||
| this.isDeleted = true; | ||
| } |
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.