Skip to content
This repository has been archived by the owner on Aug 13, 2022. It is now read-only.

자바코드 코틀린 전환 #14

Closed
kmmin78 opened this issue May 2, 2021 · 0 comments · Fixed by #16
Closed

자바코드 코틀린 전환 #14

kmmin78 opened this issue May 2, 2021 · 0 comments · Fixed by #16

Comments

@kmmin78
Copy link
Collaborator

kmmin78 commented May 2, 2021 •
edited
Loading

개요
기존 자바코드로 작성했던 로그인 인터페이스/구현 코드 코틀린으로 재작성

AS-IS
로그인 인터페이스/구현 코드 자바로 구현.

TO-BE
로그인 인터페이스/구현 코드 코틀린으로 구현.
소스코드 전체 코틀린으로 통일하기 때문에 코드의 가독성 증가 및 커뮤니케이션 비용 감소가 예상됨.
kmmin78 pushed a commit that referenced this issue May 2, 2021
#14 자바코드 코틀린 전환
69c275a 
1. 자바코드 삭제
2. 코틀린으로 재작성
3. 기존 자바코드와 동일하게 동작하는 것 까지 확인
@kmmin78 kmmin78 mentioned this issue May 2, 2021
Merged
@FrancescoJo FrancescoJo linked a pull request May 12, 2021 that will close this issue
#14 자바코드 코틀린 전환 #15 코드리뷰 피드백 반영 #16
Merged
FrancescoJo pushed a commit that referenced this issue May 12, 2021
@kmmin78
#14 자바코드 코틀린 전환 #15 코드리뷰 피드백 반영 (#16)
93ea306 
* #14 자바코드 코틀린 전환

1. 자바코드 삭제
2. 코틀린으로 재작성
3. 기존 자바코드와 동일하게 동작하는 것 까지 확인

* #15 코드리뷰 피드백 반영

주요 내용
1. 싱글턴(object)로 작성
2. User Validation을 UserEntity로 delegate
    -> UserLoginServiceImpl에서 로직 제거
    -> UserEntity에 custom setter 추가
    -> UserEntity setter 테스트는 어디서 해야 될까?

Co-authored-by: kms <kmmin78@gmail.com>
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

#14 자바코드 코틀린 전환 #15 코드리뷰 피드백 반영 kmmin78/awesome-wiki
1 participant
@kmmin78