Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[5기 소승수] Shorten-URL 과제 제출합니다. #65

Open
wants to merge 13 commits into
base: voidmelody
Choose a base branch
from

Conversation

voidmelody
Copy link

📌 과제 설명

url을 입력받아서 단축 url을 생성해주는 서비스입니다.

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

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

✅ PR 포인트 & 궁금한 점

  1. base62를 사용하는 알고리즘들을 보니 DB의 id를 많이 활용해서 진행하는데, auto increment를 활용하기 때문에 어쩔 수 없이 setter를 사용할 수 밖에 없었는데 이게 과연 좋은 선택이었을지 의문입니다.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant