[Refactor] MEMBER 도메인 리팩토링 by na311ng · Pull Request #215 · Team-Finly/Finly-backend

na311ng · 2026-02-10T22:17:30Z

🔗 관련 이슈

관련된 이슈 번호를 적어주세요.

closes #214

📌 작업 내용

이번 PR에서 작업한 내용을 간략히 설명해주세요.

주석 달기
에러코드 통일

🧪 테스트 결과

Postman 스크린샷, 테스트 통과 여부 등을 첨부해주세요.

📸 스크린샷 (선택)

필요시 스크린샷을 첨부해주세요.

📎 참고 사항 (선택)

리뷰어에게 전달할 내용이 있다면 작성해주세요.

Summary by CodeRabbit

릴리스 노트

New Features
- 비밀번호 변경 기능 추가
- 마이페이지 상단에 마인드 피스 개수 정보 포함
Documentation
- 마이페이지 및 페르소나 테스트 관련 메서드에 설명 주석 추가
Refactor
- 중복된 메서드 제거로 코드 간결화
- 예외 처리 모듈 구조 개선

coderabbitai · 2026-02-10T22:17:46Z

Warning

Rate limit exceeded

@na311ng has exceeded the limit for the number of commits that can be reviewed per hour. Please wait 26 minutes and 21 seconds before requesting another review.

⌛ How to resolve this issue?

After the wait time has elapsed, a review can be triggered using the @coderabbitai review command as a PR comment. Alternatively, push new commits to this PR.

We recommend that you space out your commits to avoid hitting the rate limit.

🚦 How do rate limits work?

CodeRabbit enforces hourly rate limits for each developer per organization.

Our paid plans have higher rate limits than the trial, open-source and free plans. In all cases, we re-allow further reviews after a brief timeout.

Please see our FAQ for further information.

개요

MEMBER 도메인의 종합적인 리팩토링 작업으로, 패스워드 변경 기능을 정리하고 MemberErrorCode를 새로운 패키지로 이동하며, 여러 클래스와 메서드에 문서화 주석을 추가합니다.

변경사항

Cohort / File(s)	요약
패스워드 변경 기능 정리 `src/main/java/com/umc/finly/domain/member/controller/MyPageController.java`, `src/main/java/com/umc/finly/domain/member/service/MyPageService.java`, `src/main/java/com/umc/finly/domain/member/service/MyPageServiceImpl.java`	패스워드 변경 엔드포인트 및 구현을 정리하고, 중복된 메서드 정의를 제거하여 단일 구현으로 통합합니다.
Converter 문서화 `src/main/java/com/umc/finly/domain/member/converter/MyPageConverter.java`, `src/main/java/com/umc/finly/domain/member/converter/PersonaTestConverter.java`	공개 메서드들에 JavaDoc 주석을 추가하고, MyPageConverter의 `toMyPageResDTO` 메서드 시그니처를 `mindPieceCount` 파라미터 추가로 확장합니다.
DTO 클래스 문서화 `src/main/java/com/umc/finly/domain/member/dto/request/PasswordChangeReqDTO.java`, `src/main/java/com/umc/finly/domain/member/dto/request/UpdateNicknameReqDTO.java`, `src/main/java/com/umc/finly/domain/member/dto/response/*`	여러 요청/응답 DTO 클래스에 설명 주석을 추가하여 가독성을 개선합니다.
엔티티 및 Enum 문서화 `src/main/java/com/umc/finly/domain/member/entity/mapping/MemberPersonaOptions.java`, `src/main/java/com/umc/finly/domain/member/entity/mapping/MemberPersonaResults.java`, `src/main/java/com/umc/finly/domain/member/enums/PersonaMode.java`, `src/main/java/com/umc/finly/domain/member/enums/PersonaType.java`	클래스 및 Enum에 주석을 이동 또는 추가하여 코드 의도를 명확히 합니다.
Exception 패키지 리팩토링 `src/main/java/com/umc/finly/domain/member/exception/code/MemberErrorCode.java`	`MemberErrorCode`를 새 패키지 경로로 이동하고, `IMAGE_FILE_TOO_LARGE` 에러 코드를 `"MEMBER413"`에서 `"MEMBER413_1"`로 업데이트합니다.
Import 경로 업데이트 `src/main/java/com/umc/finly/domain/member/service/PersonaTestSubmitServiceImpl.java`, `src/main/java/com/umc/finly/domain/member/service/PersonaScoringService.java`, `src/main/java/com/umc/finly/global/apiPayload/exception/ExceptionAdvice.java`, `src/main/java/com/umc/finly/global/infra/image/LocalImageStorageService.java`	`MemberErrorCode` 임포트 경로를 `exception.code` 패키지로 업데이트하고 관련 JavaDoc을 추가합니다.

제안 리뷰어

yjhss
jangsh7
ye-zin

🚥 Pre-merge checks | ✅ 4 | ❌ 1

❌ Failed checks (1 inconclusive)

Check name	Status	Explanation	Resolution
Linked Issues check	❓ Inconclusive	변경사항(주석 추가, 에러코드 통일, 클래스 구조 개선)이 `#214의` '도메인 리팩토링' 목표와 부분적으로 일치하나, 구체적 요구사항 부재로 완전한 검증 불가.	이슈 `#214에` 구체적인 리팩토링 요구사항(주석 표준, 에러코드 통일 기준 등)을 명시하여 요구사항 명확성을 강화하세요.

✅ Passed checks (4 passed)

Check name	Status	Explanation
Title check	✅ Passed	PR 제목이 주석 추가 및 에러코드 통일 작업을 포괄적으로 설명하지만, 구체적인 변경사항을 명확히 드러내지 않음.
Description check	✅ Passed	PR 설명이 템플릿의 필수 섹션을 포함하고 있으나, 작업 내용이 매우 간략하고 테스트 결과가 비어있음.
Out of Scope Changes check	✅ Passed	모든 변경사항(주석 추가, 에러코드 경로 변경, 메서드 정렬)이 도메인 리팩토링 범위 내에 있고, 새로운 기능이나 비관련 기능 변경 없음.
Docstring Coverage	✅ Passed	Docstring coverage is 100.00% which is sufficient. The required threshold is 80.00%.

_{✏️ Tip: You can configure your own custom pre-merge checks in the settings.}

✨ Finishing touches

🧪 Generate unit tests (beta)

Create PR with unit tests
Post copyable unit tests in a comment
Commit unit tests in branch refactor/member-214-리팩토링

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

_{Comment @coderabbitai help to get the list of available commands and usage tips.}

coderabbitai

Actionable comments posted: 1

🧹 Nitpick comments (3)

src/main/java/com/umc/finly/domain/member/enums/PersonaMode.java (1)
3-3: 문서화 주석 추가 좋습니다!

Enum의 목적을 명확히 설명하는 주석이 추가되어 코드 가독성이 향상되었습니다.

선택적으로, 클래스 레벨 문서화의 경우 JavaDoc 스타일(/** ... */)을 사용하면 IDE의 문서 생성 및 툴팁 지원이 더 잘 되지만, 현재의 단순한 주석도 충분히 명확합니다.
📝 JavaDoc 스타일 제안 (선택사항)
-// 페르소나 테스트 제출 분기
+/**
+ * 페르소나 테스트 제출 분기
+ */
 public enum PersonaMode {
src/main/java/com/umc/finly/domain/member/service/MyPageService.java (1)

15-16: newPasswordConfirm 파라미터의 위치에 대한 설계 의견.

비밀번호 확인(newPasswordConfirm) 일치 검증은 프레젠테이션 레이어(DTO validation 또는 Controller)에서 처리하는 것이 일반적입니다. 서비스 인터페이스에 확인용 파라미터를 전달하면 서비스 계층이 입력 검증 책임까지 갖게 됩니다.

현재 구조로도 동작에는 문제가 없으나, 향후 서비스 메서드 시그니처를 changePassword(Long memberId, String newPassword)로 단순화하고, 비밀번호 일치 검증은 DTO의 커스텀 validator나 컨트롤러에서 처리하는 방안을 고려해 볼 수 있습니다.
src/main/java/com/umc/finly/domain/member/service/MyPageServiceImpl.java (1)
82-98: 현재 비밀번호 확인 없이 비밀번호를 변경하고 있습니다.

인증된 사용자의 세션/토큰이 탈취된 경우, 기존 비밀번호 확인 없이 비밀번호를 변경할 수 있습니다. 일반적으로 비밀번호 변경 시에는 현재 비밀번호를 함께 받아 검증하는 것이 보안 모범 사례입니다.

의도된 설계라면 무시해도 좋지만, 그렇지 않다면 PasswordChangeReqDTO에 currentPassword 필드를 추가하고 passwordEncoder.matches()로 검증하는 것을 권장합니다.
🔒 현재 비밀번호 검증 추가 제안
     public void changePassword(Long memberId, String newPassword, String newPasswordConfirm) {
+    // 참고: currentPassword 파라미터 추가 및 아래 검증 로직 추가 권장
+    // if (!passwordEncoder.matches(currentPassword, member.getPassword())) {
+    //     throw new CustomException(AuthErrorCode.INVALID_PASSWORD);
+    // }
+
         if (newPassword == null || !newPassword.equals(newPasswordConfirm)) {
             throw new CustomException(AuthErrorCode.PASSWORD_CONFIRM_MISMATCH);
         }

ye-zin

API도 추가된 거 확인했습니다!

coderabbitai

Actionable comments posted: 1

na311ng added 3 commits February 11, 2026 07:14

Feat: Member 도메인 주석 추가 (#214)

1574631

Refactor: 멤버 에러 코드 방식 통일 (#214)

6486f40

Feat: Member 도메인 주석 추가 (#214)

cb7118a

na311ng requested review from dosp74, jangsh7, wonee1, ye-zin and yjhss February 10, 2026 22:17

na311ng self-assigned this Feb 10, 2026

na311ng added the ♻️Refactor 리팩토링 label Feb 10, 2026

na311ng linked an issue Feb 10, 2026 that may be closed by this pull request

[Refactor] MEMBER 도메인 리팩토링 #214

Closed

2 tasks

coderabbitai Bot reviewed Feb 10, 2026

View reviewed changes

Comment thread src/main/java/com/umc/finly/domain/member/controller/MyPageController.java

ye-zin approved these changes Feb 11, 2026

View reviewed changes

dosp74 approved these changes Feb 11, 2026

View reviewed changes

Refactor: 에러코드 파일 위치 변경 (#214)

5ced24a

coderabbitai Bot reviewed Feb 11, 2026

View reviewed changes

Comment thread src/main/java/com/umc/finly/domain/member/exception/code/MemberErrorCode.java

Merge branch 'develop' into refactor/member-214-리팩토링

76f0f54

na311ng merged commit 208f1f9 into develop Feb 11, 2026
2 checks passed

Navigation Menu

[Refactor] MEMBER 도메인 리팩토링 - #215

Hidden character warning

[Refactor] MEMBER 도메인 리팩토링#215
na311ng merged 5 commits into
developfrom
refactor/member-214-리팩토링

na311ng commented Feb 10, 2026 •

edited by coderabbitai Bot

Loading

Uh oh!

coderabbitai Bot commented Feb 10, 2026 •

edited

Loading

Rate limit exceeded

Uh oh!

coderabbitai Bot left a comment

Uh oh!

Uh oh!

ye-zin left a comment

Uh oh!

coderabbitai Bot left a comment

Uh oh!

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

3 participants

Hidden character warning

Conversation

na311ng commented Feb 10, 2026 • edited by coderabbitai Bot Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

🔗 관련 이슈

📌 작업 내용

🧪 테스트 결과

📸 스크린샷 (선택)

📎 참고 사항 (선택)

Summary by CodeRabbit

릴리스 노트

Uh oh!

coderabbitai Bot commented Feb 10, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Rate limit exceeded

개요

변경사항

관련 가능 PR

제안 리뷰어

Uh oh!

coderabbitai Bot left a comment

Choose a reason for hiding this comment

Uh oh!

Uh oh!

ye-zin left a comment

Choose a reason for hiding this comment

Uh oh!

coderabbitai Bot left a comment

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

3 participants

na311ng commented Feb 10, 2026 •

edited by coderabbitai Bot

Loading

coderabbitai Bot commented Feb 10, 2026 •

edited

Loading