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.

Sign up for GitHub

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

Jump to bottom

refactor - 정적 코드 분석을 이용한 리팩토링 #103

Merged
merged 13 commits into from
Feb 29, 2024
Merged

refactor - 정적 코드 분석을 이용한 리팩토링 #103

merged 13 commits into from
Feb 29, 2024

Conversation

kseysh
Copy link
Member

@kseysh kseysh commented Feb 26, 2024

Related issue 🚀

Work Description 💚

  • 네이밍 컨벤션에 맞지 않는 폴더 네이밍을 변경했습니다.
  • 상수 클래스의 생성자를 사용하지 못하도록 매개변수가 사용되지 않는 생성자를 private로 생성하였습니다.
  • string 값이 공백인지 확인하는 로직을 동일성 비교에서 동등성 비교로 수정하였습니다.
  • 클래스의 이름과 필드 이름이 겹치는 것을 수정하였습니다.
  • 세미콜론이 두 번 들어간 것을 삭제했습니다.
  • 네이밍 컨벤션에 위반되는 함수 변수의 첫 글자를 수정했습니다.
  • 사용하지 않는 대괄호 삭제 및 람다를 메서드 참조로 변경하였습니다.
  • 리턴된 변수를 메모리 효율을 위해 인라인으로 변경하였습니다.
  • StringUtil.isEmpty 함수가 deprecated 되어 hasText로 변경하였습니다.
  • 중복되어 있던 UserError 파일을 삭제하였습니다.
  • 제네릭 와일드 카드 반환 타입을 삭제하였습니다.
  • 사용하지 않는 메서드를 삭제하였습니다.

PR 참고 사항

  • 폴더 네이밍으로 인해 전체 변경된 파일을 확인하시는 것보다 커밋을 확인하는 것이 나을 것으로 판단됩니다!
  • exceptionHandler는 아직 제네릭 와일드 카드 반환타입을 삭제하지 않았습니다. exceptionHandler에 대해 더 학습하고 변경할 예정입니다.
  • JwtGenerator에서 @value로 들어오는 변수들이 상수로 사용되므로 final을 붙여서 사용하고 싶은데 아직 수정하지 못하였습니다. 빠른 시일내에 수정하겠습니다.

kseysh added 13 commits February 22, 2024 22:59
@kseysh
refactor - #102 HMH 폴더 이름 소문자로 변경
915a416
@kseysh
refactor - #102 상수 클래스 생성자 private로 변경
f4427ad
@kseysh
refactor - #102 string 빈 값 체크 함수 변경
b602dcb
@kseysh
refactor - #102 클래스의 이름과 필드 이름이 겹치지 않도록 수정
2e7d11b
@kseysh
refactor - #102 빈 statement 삭제
5061414
@kseysh
refactor - #102 첫 글자가 대문자인 함수, 변수 첫 글자 소문자로 변경
af31e32
@kseysh
refactor - #102 사용하지 않는 대괄호 삭제 및 람다를 메서드 참조로 변경
0bbb14f
@kseysh
refactor - #102 리턴된 변수 인라인으로 변경
622be02
@kseysh
refactor - #102 deprecated인 isEmpty hasText로 변경
9163eb0
@kseysh
refactor - #102 리턴 value 인라인으로 변경
2374adb
@kseysh
refactor - #102 중복된 UserError 파일 삭제
5b8646b
@kseysh
refactor - #102 제네릭 와일드 카드 반환 타입 삭제
8c27a37
@kseysh
refactor - #102 사용하지 않는 메서드 삭제
cd3f809
@kseysh kseysh added 🧹 Chore 동작에 영향 없는 코드 or 파일, 폴더 관련 수정사항 👨🏻‍💻 승환 승환이가 작성한 Label 🔥 Pull Request 🔨 Refactor 코드 리펙토링 (기능 변경 없이 코드만 수정할 때) labels Feb 26, 2024
@kseysh kseysh self-assigned this Feb 26, 2024
jumining
jumining approved these changes Feb 29, 2024
View reviewed changes
Copy link
Collaborator

@jumining jumining left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

수정된 곳이 많네요 잘 반영해서 작업하겠습니다 얼른 머지해주시요 :)

src/main/java/sopt/org/hmh/domain/challenge/dto/response/ChallengeResponse.java
.map(dailyChallenge -> { return dailyChallenge.getStatus(); })
.map(DailyChallenge::getStatus)
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

정적분석이 이것도 해주다니 아주 좋군요

@kseysh kseysh merged commit c294ac6 into develop Feb 29, 2024
1 check passed
@jumining jumining deleted the refactor/#102-refactor-code-by-sonarlint branch November 19, 2024 04:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@jumining jumining jumining approved these changes

Assignees

@kseysh kseysh

Labels
👨🏻‍💻 승환 승환이가 작성한 Label 🧹 Chore 동작에 영향 없는 코드 or 파일, 폴더 관련 수정사항 🔨 Refactor 코드 리펙토링 (기능 변경 없이 코드만 수정할 때)
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

refactor - 정적 분석 툴(sonarlint)을 사용한 리팩토링
2 participants
@kseysh @jumining