엔티티 구조 변경 제안

[CheChe903]

이번에 히로의 PR인 목표 달성률을 구하는 것에서 개인적으로 어색함을 느꼈는데요! 그 이유에 대해서 고민해보고, 제안하고자 합니다.

기존에는 한 멤버의 하루의 IntakeHistory가 여러 개 생성됩니다.

따라서 달성률을 구하기 위해서는 마지막 레코드를 확인하여 구해야하는데요!

이를 IntakeDetail로 구분한다면 좀 더 객체지향적이 될 것 같아요.

예시,

public class IntakeHistory {

    @Id
    @GeneratedValue(strategy = GenerationType.IDENTITY)
    private Long id;

    @ManyToOne(fetch = FetchType.LAZY)
    @JoinColumn(nullable = false)
    private Member member;

    @Column(nullable = false)
    private LocalDate date;

    @Embedded
    @AttributeOverride(name = "value", column = @Column(name = "targetAmount", nullable = false))
    private Amount targetAmount;

    @OneToMany(mappedBy = "intakeHistory", cascade = CascadeType.ALL, orphanRemoval = true)
    private List<IntakeDetail> details = new ArrayList<>();

    public IntakeHistory(
            Member member,
            LocalDate date,
            Amount targetAmount
    ) {
        this.member = member;
        this.date = date;
        this.targetAmount = targetAmount;
    }
}

public class IntakeDetail {
    
    @Id
    @GeneratedValue(strategy = GenerationType.IDENTITY)
    private Long id;

    @Embedded
    @AttributeOverride(name = "value", column = @Column(name = "intakeAmount", nullable = false))
    private Amount intakeAmount;

    @Column(nullable = false)
    private LocalTime intakeTime;

    @ManyToOne(fetch = FetchType.LAZY)
    @JoinColumn(name = "intake_history_id", nullable = false)
    private IntakeHistory intakeHistory;

    public IntakeDetail(Amount intakeAmount, LocalTime intakeTime, IntakeHistory intakeHistory) {
        this.intakeAmount = intakeAmount;
        this.intakeTime = intakeTime;
        this.intakeHistory = intakeHistory;
    }
} 

만약 이렇게 된다면 서비스 로직 상 예측 가능성이 높아질 것으로 예측됩니다.

하지만 이렇게 한다면 @OneToMany 를 피할 수는 없을 것 같아요.

혹시나 제가 모르는 부분이 있을 수도 있으니 편하게 코멘트 주시면 감사하겠습니다!

[minSsan]

반영 완료