[5기 박유진] Shorten-URL 과제 제출합니다. by eugene225 · Pull Request #70 · prgrms-be-devcourse/springboot-url-shortener

eugene225 · 2023-12-17T11:19:33Z

📌 과제 설명

domain

Url : id, shortenKey, originUrl, shortenUrl, count
UrlRepository

service

UrlService
- save : 저장
- shortenUrlByMurMur / shortenUrlByBase62 : 인코딩 방식
- addLink : encodingType에 따른 shortenUrl 저장
- getUrlByKey : key 이용해서 Url 찾고 redirect count 증가
EncodingType : 인코딩 방식 저장 Enum
EnumConverter : String -> Enum 변환
RadixConverter : 2진수, 16진수 변환
Base62Encoding : Base62Encoding 구현

controller

UrlRestController
UrlWebController
dto : UrlRequestDto

👩‍💻 요구 사항과 구현 내용

URL 입력폼 제공 및 결과 출력
URL Shortening Key는 8 Character 이내로 생성
단축된 URL 요청시 원래 URL로 리다이렉트
단축된 URL에 대한 요청 수 정보저장 (optional)
Shortening Key를 생성하는 알고리즘 2개 이상 제공하며 애플리케이션 실행중 동적으로 변경 가능 (optional)

✅ 피드백 반영사항

✅ PR 포인트 & 궁금한 점

라이브러리를 사용한 인코딩 방식에 성능적인 문제가 있을지 궁금합니다.

SangHyunGil

작성해주시느라 너무 고생많으셨어요.
간단한 코멘트 남겼으니 확인부탁해요.

테스트 코드가 없는 것 같은데요. 적어도 도메인 로직과 서비스 로직과 같은 중요한 비즈니스 로직을 다루는 영역에 대한 테스트는 작성하는걸 권장해요.
레이아웃이 조금 틀어진 부분이 있었는데 수정해주시면 좋을 것 같아요.
라이브러리를 다룰때 단순 인코딩이라면 성능적인 부분을 고려하기 위해서는 부하테스트 같은걸 통해서 검증을 해야할 것 같아 추후에 한 번 관련하여 진행해보시면 좋을 것 같아요.

SangHyunGil · 2023-12-17T15:34:33Z

pom.xml

+        <dependency>
+            <groupId>org.springframework.boot</groupId>
+            <artifactId>spring-boot-starter-data-redis</artifactId>
+        </dependency>


~~레디스가 왜 필요한 걸까요?~~
제거해주신 것 봤습니다!

SangHyunGil · 2023-12-17T15:36:24Z

src/main/resources/application.yml

+  servlet:
+    encoding:
+      charset: UTF-8
+      enabled: true
+      force: true


왜 적용하신건가요?

SangHyunGil · 2023-12-17T15:37:34Z

src/main/java/com/prgrms/shortenurl/url/domain/Url.java

+
+    @NonNull
+    @Column(name = "orgin_url")
+    private String originUrl;


originUrl은 유니크하지 않아도 되는건가요?
만약, 유니크해야한다면 유니크 제약 조건을 걸어보면 좋을 것 같네요.

SangHyunGil · 2023-12-17T15:38:08Z

src/main/java/com/prgrms/shortenurl/url/domain/Url.java

+@Getter
+@Entity
+@NoArgsConstructor(access = AccessLevel.PROTECTED)
+public class Url {


생성일시 / 업데이트일시도 있으면 좋을 것 같네요.

SangHyunGil · 2023-12-17T15:38:52Z

src/main/java/com/prgrms/shortenurl/url/domain/UrlRepository.java

+
+import java.util.Optional;
+
+public interface UrlRepository extends JpaRepository<Url, String> {


왜 ID 값에 String이 들어간걸까요?

SangHyunGil · 2023-12-18T14:43:40Z

src/main/java/com/prgrms/shortenurl/service/UrlService.java

+                .originUrl(originUrl).build());
+    }
+
+    public static String shortenUrlByMurMur(@NonNull Url url) {


인코딩 관련 책임들이 여러곳에 분산되어있는 것 같아요.

실제 인코딩하는 로직은 서비스에 존재해요.

인코딩 관련 관리는 EncodingType을 통해서 관리해요.

이넘을 사용해 변환하는건 EnumConverter을 통해 처리해요.

코드를 응집도 있게 관리하는 부분을 고민해보시면 좋을 것 같아요.
인코딩 수행하는 로직은 Url 서비스가 가져야할 책임이 아닌 것 같아요. 별도의 인코딩 로직만 수행하는 객체가 존재(별도 클래스 혹은 이넘)하고 UrlService는 이를 호출만하면 될 것 같아요.

서비스 레이어에서 하는 역할이 무엇일지 고민해봐주시면 더욱 좋을 것 같아요.

SangHyunGil · 2023-12-18T14:52:11Z

src/main/java/com/prgrms/shortenurl/service/UrlService.java

+    }
+
+    public static String shortenUrlByMurMur(@NonNull Url url) {
+        return Hashing.murmur3_32_fixed().hashString(url.getOriginUrl(), Charset.defaultCharset()).toString();


사용하신 이유가 궁금해요!

SangHyunGil · 2023-12-18T14:54:46Z

src/main/java/com/prgrms/shortenurl/domain/Url.java

+        if (isValidUrl(originUrl)) {
+            this.originUrl = originUrl;
+        } else {
+            throw new IllegalArgumentException("Invalid URL: " + originUrl);


간단의견) 나중에는 CustomException을 적용해봐도 좋을 것 같아요.

SangHyunGil · 2023-12-18T14:55:09Z

src/main/java/com/prgrms/shortenurl/service/UrlService.java

+    public String getUrlByKey(@NonNull String key) {
+        Url url = urlRepository.findByShortenKey(key)
+                .orElseThrow(UrlNotFoundException::new);
+        url.updateCount();


동시에 요청이 들어오면 동시성 이슈가 생길 수 있을 것 같아요.
어떻게 처리할지 고민하고 생각을 공유해주세요.

SangHyunGil · 2023-12-18T14:58:16Z

src/main/java/com/prgrms/shortenurl/service/Base62Encoding.java

@@ -0,0 +1,27 @@
+package com.prgrms.shortenurl.service;
+
+public class Base62Encoding {


왜 base64가 아닌 base62를 사용하셨나요?

…ortener

eugene225 added 29 commits December 7, 2023 14:34

init commit

f21a7cc

docs: h2 database, jpa, server 환경설정

3b879ef

docs: h2 database, guava 의존성 추가

7836615

feat: Url 엔티티 구성 및 repository 생성

9f89a4f

feat: UrlService 기본 기능 구현 (인코딩 방식 추가 필요)

4926d6a

feat: UrlController 기본구현

1be563c

feat: shortenURL 화면 구성

4ebb782

Chore: 패키지 수정

43a530d

chore: 패키지 구조 변경

3e78f05

refactor: 기본 service 구성

74b0d62

refactor: 기본 rest api 구성

38b98c0

feat: web controller 구성

3029ced

refactor: 기본 api 주소 변경

f77ca25

chore: 패키지 구조 변경

ff3d166

feat: GlobalExceptionHandler 구성 및 적용

10d658b

feat: base62 인코딩 방식 추가

d11a9ea

feat: 이진수, 16진수 converter 추가

80b6440

refactor: Base62 Encoding 클래스명 변경 (이넘명 중복)

dcf2fb1

feat: encodingType 이넘 생성 및 함수형 인터페이스 적용

0702ff3

feat: string 값 이넘 변환 converter 생성

585d81a

refactor: 인코딩 함수 로직 변경

5165c18

refactor: encodingType 파라미터 추가

8eb7746

feat: html encodingType 라디오 버튼 추가

651e09e

feat: url 요청 수 count 필드 추가 및 서비스 코드 수정

42f7595

refactor: 진수 Converter 클래스명 수정

9315f8e

refactor: HttpStatus 활용 수정

c1642c9

feat: UrlRequestDto 생성 및 적용

59686bf

feat: base62 encoding 길이 추가

34df475

feat: originUrl 유효성 검사 코드 추가

0fcdb18

eugene225 requested a review from moosongsong December 17, 2023 11:20

eugene225 requested a review from SangHyunGil December 17, 2023 11:20

eugene225 self-assigned this Dec 17, 2023

eugene225 added 🔥 매운맛 Team-Dozzi labels Dec 17, 2023

SangHyunGil reviewed Dec 18, 2023

View reviewed changes

eugene225 added 6 commits December 19, 2023 02:02

docs: servlet 설정 제거

d1f77c0

feat: createdAt, updatedAt 컬럼 추가

7e5f03d

feat: ID 값 타입수정

5dc7ac6

Create maven.yml

48df906

refactor: 이미 존재하는 url 처리

f2808c5

Merge branch 'main' of https://github.com/eugene225/springboot-url-sh…

786e478

…ortener

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

[5기 박유진] Shorten-URL 과제 제출합니다. #70

[5기 박유진] Shorten-URL 과제 제출합니다. #70
eugene225 wants to merge 35 commits intoprgrms-be-devcourse:eugenefrom
eugene225:main

eugene225 commented Dec 17, 2023 •

edited

Loading

Uh oh!

SangHyunGil left a comment

Uh oh!

SangHyunGil Dec 17, 2023 •

edited

Loading

Uh oh!

SangHyunGil Dec 17, 2023

Uh oh!

SangHyunGil Dec 17, 2023

Uh oh!

SangHyunGil Dec 17, 2023

Uh oh!

SangHyunGil Dec 17, 2023

Uh oh!

SangHyunGil Dec 18, 2023

Uh oh!

SangHyunGil Dec 18, 2023

Uh oh!

SangHyunGil Dec 18, 2023

Uh oh!

SangHyunGil Dec 18, 2023

Uh oh!

SangHyunGil Dec 18, 2023

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

2 participants


		import java.util.Optional;

		public interface UrlRepository extends JpaRepository<Url, String> {

		@@ -0,0 +1,27 @@
		package com.prgrms.shortenurl.service;

		public class Base62Encoding {

Conversation

eugene225 commented Dec 17, 2023 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

📌 과제 설명

domain

service

controller

👩‍💻 요구 사항과 구현 내용

✅ 피드백 반영사항

✅ PR 포인트 & 궁금한 점

Uh oh!

SangHyunGil left a comment

Choose a reason for hiding this comment

Uh oh!

SangHyunGil Dec 17, 2023 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

2 participants

eugene225 commented Dec 17, 2023 •

edited

Loading

SangHyunGil Dec 17, 2023 •

edited

Loading