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

Refactor: 1차 기능 리팩토링 및 테스트코드 추가 #72

Merged
merged 11 commits into from
Nov 3, 2024

Conversation

yeonjookang
Copy link
Contributor

Resolves #없음

Issue Define

  • 1차 기능 리팩토링 및 테스트코드 추가

Summary of resolutions or improvements

  • 다음과 같은 기준을 가지고 리팩토링을 진행합니다.
    • 중복된 코드 제거
    • Ctrl + Alt + L 로 코드를 모두 정렬
    • Ctrl + Alt + O 로 import 안 쓰는 거 삭제
    • 메소드명, 변수명이 일관성이 있는가
    • 메소드 정렬(public 위로 private 아래로)
    • 불필요한 주석 제거
    • 하나의 함수는 하나의 목적을 가지고 한 가지 일만 하도록
    • 발생할 수 있는 모든 예외를 테스트하는지
    • Service단에서 Domain의 역할을 수행하고 있지는 않은지
    • ResponseDto, RequestDto에 record 도입
    • dtoConverter 도입
    • 수정이 되지 않는 인자값은 final로 선언
    • 표현계층, 응용계층 클래스 크기 줄이기

Note

@github-actions github-actions bot added the refactor 리팩토링 label Nov 3, 2024
Copy link
Contributor

@pkl0912 pkl0912 left a comment

Choose a reason for hiding this comment

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

LGTM~

@pkl0912 pkl0912 merged commit 0ad3913 into dev Nov 3, 2024
2 checks passed
@pkl0912 pkl0912 deleted the refactor/mvp branch November 3, 2024 09:38
@auto-assign auto-assign bot requested a review from pkl0912 November 3, 2024 09:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
refactor 리팩토링
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants