MP-166 ♻️ Refactor: 경고 제거 및 코드 정제#21
Merged
Reality-out merged 10 commits intodevelopfrom May 12, 2025
Merged
Conversation
Collaborator
Reality-out
commented
May 12, 2025
Reality-out
commented
- 불필요한 코드 삭제
- deprecated된 요소들 대체
- 문자열 상수화 및 boolean 값 명확화
* domain 내부 패키지는 common을 제외하고는 통일된 기능을 제공하려고 함(현재는 CRUD를 위한 ApplicationService 및 검증을 위한 ValidationService를 주 기능으로 제공) * 나머지 기능 일체는 전역으로 사용되는 경우 global에, 그렇지 않은 경우 modules에 두고 있음 * 따라서 전역이라기보다는 모듈화된 특정 기능을 포함하고 있다고 여길 수 있는 example 패키지를 modules로 보냄
* commons와 common 모두 좋기는 하나 패키지 뒤에 s를 붙이는 경우는 반드시 그래야만 할 때(enums, supers 등)로 제한하여 패키지 네이밍의 통일성 확보
* TokenDataNotFoundException을 TokenNotFoundException으로 재명명(Data라는 단어가 그리 중요하지 않다고 판단) * refresh_token 문자열은 상수화 * 로깅에는 private final 변수 대신 @slf4j 도입 * 기타 문법에 맞도록 코드 개선
* Jwts.builder()와 Jwts.parser()에 대한 연쇄 메소드를 현대적으로 변경 * 사용되지 않는 payloadEmail 변수 제거 * SameSite 속성 값을 Lax로 변경 * 응답을 정적 팩토리 메소드를 사용하여 더욱 깔끔하게 변경
* 예외 클래스 및 오류 메시지 변경 * 기타 자잘한 코드 변경 존재
* 매퍼 클래스 재명명 * @Mapper의 속성 값 정리 * 주석 처리되어 있던 변수 제거 * 변수명에서의 오기 수정
* SiteMemberAuthDomainInfraMapper로 변경
* 불필요한 null 값 처리 로직 제거 * true 및 false 값 명확화
* 매직 넘버 가능성이 있는 숫자 값 변수화 * 불필요한 null 처리 로직 제거
* TokenKeyCreationException의 상위 클래스를 AuthTokenException으로 설정
This file contains hidden or 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
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
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.