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

코드리뷰 피드백 반영 #15

Closed
kmmin78 opened this issue May 2, 2021 · 1 comment
Closed

코드리뷰 피드백 반영 #15

kmmin78 opened this issue May 2, 2021 · 1 comment

Comments

@kmmin78
Copy link
Collaborator

kmmin78 commented May 2, 2021 •
edited
Loading

개요
코드리뷰 피드백 반영하여 코드 품질 향상

AS-IS
수정 전 코드로 code smell이 많이 나는 상태.

TO-BE
피드백 반영해서 코드 품질 향상.
앞으로 코드 작성할 때 참고하여 더 좋은 코드를 생산하기 위함.
kmmin78 pushed a commit that referenced this issue May 2, 2021
#15 코드리뷰 피드백 반영
e0549a2 
주요 내용
1. 싱글턴(object)로 작성
2. User Validation을 UserEntity로 delegate
    -> UserLoginServiceImpl에서 로직 제거
    -> UserEntity에 custom setter 추가
    -> UserEntity setter 테스트는 어디서 해야 될까?
@kmmin78 kmmin78 mentioned this issue May 2, 2021
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>
@FrancescoJo
Copy link
Collaborator

관련 Pull request 가 완료 상태이므로 이슈 종결 처리

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

No branches or pull requests
2 participants
@FrancescoJo @kmmin78