feat: GraduationUser의 도메인, 엔티티 모델링#279
Conversation
|
Important Review skippedAuto incremental reviews are disabled on this repository. Please check the settings in the CodeRabbit UI or the You can disable this status message by setting the Walkthrough졸업생 관리 도메인 모델링 작업으로, GraduationType 열거형, GraduationUser 도메인 엔티티, 그리고 GraduationUserJpaEntity JPA 영속성 엔티티를 추가합니다. 각각 필요한 필드와 Lombok 기반의 접근자/빌더, 그리고 엔티티-도메인 간 매핑 헬퍼 메서드를 포함합니다. Changes
Estimated code review effort🎯 2 (Simple) | ⏱️ ~10 minutes
Possibly related issues
Pre-merge checks and finishing touches✅ Passed checks (3 passed)
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. Comment |
Test Coverage Report
Files
|
Codecov Report❌ Patch coverage is
❌ Your patch status has failed because the patch coverage (0.00%) is below the target coverage (10.00%). You can increase the patch coverage or adjust the target coverage. @@ Coverage Diff @@
## develop #279 +/- ##
=============================================
- Coverage 93.38% 92.81% -0.58%
Complexity 164 164
=============================================
Files 49 50 +1
Lines 484 487 +3
Branches 4 4
=============================================
Hits 452 452
- Misses 26 29 +3
Partials 6 6
Continue to review full report in Codecov by Sentry.
🚀 New features to boost your workflow:
|
![coderabbitai[bot]](https://avatars.githubusercontent.com/in/347564?s=60&v=4){kind=small}
There was a problem hiding this comment.
Actionable comments posted: 1
🧹 Nitpick comments (2)
aics-domain/src/main/java/kgu/developers/domain/graduationUser/domain/GraduationUser.java (2)
23-25: 타임스탬프 필드의 가시성을private로 변경하는 것을 권장합니다.도메인 모델에서
protected가시성은 일반적으로 상속 구조가 있을 때 사용됩니다. 현재GraduationUser는 상속을 위한 클래스가 아니므로, 더 나은 캡슐화를 위해private으로 변경하는 것이 좋습니다.- protected LocalDateTime createdAt; - protected LocalDateTime updatedAt; - protected LocalDateTime deletedAt; + private LocalDateTime createdAt; + private LocalDateTime updatedAt; + private LocalDateTime deletedAt;
9-26: 빌더 패턴에 필수 필드 검증을 추가하는 것을 고려하세요.현재 빌더는 검증 없이 모든 필드를 null로 생성할 수 있습니다. 비즈니스 규칙상 필수인 필드(예: name, email, advisorProfessor, graduationType)에 대해 커스텀 빌더를 통한 검증을 추가하면 도메인 무결성을 향상시킬 수 있습니다.
예시:
@Getter @Builder public class GraduationUser { // ... fields ... public static class GraduationUserBuilder { public GraduationUser build() { if (name == null || name.isBlank()) { throw new IllegalArgumentException("이름은 필수입니다."); } if (email == null || email.isBlank()) { throw new IllegalArgumentException("이메일은 필수입니다."); } // 추가 검증... return new GraduationUser(/* ... */); } } }
📜 Review details
Configuration used: CodeRabbit UI
Review profile: CHILL
Plan: Pro
📒 Files selected for processing (3)
aics-domain/src/main/java/kgu/developers/domain/graduationUser/domain/GraduationType.java(1 hunks)aics-domain/src/main/java/kgu/developers/domain/graduationUser/domain/GraduationUser.java(1 hunks)aics-domain/src/main/java/kgu/developers/domain/graduationUser/infrastructure/entity/GraduationUserJpaEntity.java(1 hunks)
🧰 Additional context used
🧬 Code graph analysis (2)
aics-domain/src/main/java/kgu/developers/domain/graduationUser/infrastructure/entity/GraduationUserJpaEntity.java (1)
aics-domain/src/main/java/kgu/developers/domain/graduationUser/domain/GraduationUser.java (1)
Getter(9-26)
aics-domain/src/main/java/kgu/developers/domain/graduationUser/domain/GraduationType.java (1)
aics-domain/src/main/java/kgu/developers/domain/graduationUser/domain/GraduationUser.java (1)
Getter(9-26)
🔇 Additional comments (3)
aics-domain/src/main/java/kgu/developers/domain/graduationUser/domain/GraduationType.java (1)
6-14: 잘 구현된 열거형입니다.졸업 유형을 나타내는 열거형이 명확하게 정의되어 있으며, Lombok을 활용한 구현도 적절합니다.
aics-domain/src/main/java/kgu/developers/domain/graduationUser/infrastructure/entity/GraduationUserJpaEntity.java (2)
36-57: 필드 제약 조건과 관계 명세를 검증하세요.다음 사항들을 확인해 주세요:
userId(라인 57): 다른 String 필드와 달리 길이 제약이 없습니다. 어떤 사용자를 참조하는지, 외래키 관계가 필요한지 확인이 필요합니다.midThesisId,finalThesisId,certificateId(라인 51-55): 다른 엔티티와의 관계가 명확하지 않습니다. 외래키 제약이나@ManyToOne관계 매핑이 필요한지 검토해 주세요.- 각 String 필드의 길이 제약(10, 50, 20)이 실제 비즈니스 요구사항에 부합하는지 확인해 주세요.
59-76: 타임스탬프 필드 매핑의 일관성을 확인하세요.
toEntity()메서드는createdAt,updatedAt,deletedAt필드를 매핑하지 않는 반면,toDomain()메서드(라인 94-96)는 이 필드들을 명시적으로 매핑합니다.
BaseTimeEntity가 JPA 라이프사이클 콜백으로 자동 관리한다면 현재 구현이 맞지만, 만약 도메인 객체에서 엔티티로 변환 시 타임스탬프 보존이 필요한 경우(예: 업데이트 작업)에는 이 필드들도 매핑해야 합니다.업데이트 시나리오에서 타임스탬프 보존이 필요하다면 다음과 같이 수정을 고려하세요:
public static GraduationUserJpaEntity toEntity(GraduationUser graduationUser) { if(graduationUser == null) { return null; } GraduationUserJpaEntity entity = GraduationUserJpaEntity.builder() .id(graduationUser.getId()) .name(graduationUser.getName()) .email(graduationUser.getEmail()) .advisorProfessor(graduationUser.getAdvisorProfessor()) .graduationType(graduationUser.getGraduationType()) .graduationDate(graduationUser.getGraduationDate()) .midThesisId(graduationUser.getMidThesisId()) .finalThesisId(graduationUser.getFinalThesisId()) .certificateId(graduationUser.getCertificateId()) .userId(graduationUser.getUserId()) .build(); // BaseTimeEntity의 protected 필드에 직접 설정이 필요한 경우 // 또는 별도의 setter 메서드 제공 필요 return entity; }
3 participants
Summary
해당 PR에 대한 요약을 작성해주세요.
Tasks
To Reviewer
Screenshot