[BE] 사용자 몸무게, 성별 디폴트 값 관련

[minSsan]

원문: #127 (comment)

그렇다면 무게 / 성별 필드가 null인 경우 (= 사용자가 온보딩에서 입력하지 않은 경우) 에, 무게 또는 성별 필드가 필요한 로직에서 어떻게 처리되는 건가요?

1. 디폴트 값을 토대로 로직을 정상적으로 처리 하나요?
2. 혹은 예외를 발생시켜야 하나요? (ex. "ERROR - 권장 음수량을 계산하려면 몸무게를 입력해주세요")

현재 PhysicalAttributes 코드가 아래처럼 작성되어 있는데, 코드상으로 보면 1번을 의도하신 것으로 보여요.
그런데 개인적으로 아래에서 설명드릴 문제(?) (= 로직 상에서 처리되는 값과 DB에 저장된 값 사이의 불일치) 를 의도하신건지 의문이 들었습니다.

@Getter
@NoArgsConstructor
@AllArgsConstructor
@Embeddable
public class PhysicalAttributes {

    @Enumerated(EnumType.STRING)
    private Gender gender;

    private Double weight;

    public Gender getGender() {
        if (gender == null) {
            return Gender.MALE;
        }
        return gender;
    }

    public Double getWeight() {
        if (weight == null) {
            return 60.0;
        }
        return weight;
    }
}

실제 DB에 저장된 값이 NULL일 때, 로직 상에서는 성별과 무게가 각각 MALE과 60.0으로 처리되는 것으로 보이는데요,
그렇다면 실제 DB에 저장된 값과 로직에서 수행되는 값 사이에 정합성이 어긋나지 않나요??

만약 사용자가 값을 입력하지 않은 경우, 몸무게와 성별을 각각 60, MALE로 저장하고자 한다면 아래와 같이 @PrePersist 어노테이션을 활용하는 것이 어떨까 싶습니다.

@Getter
@NoArgsConstructor
@AllArgsConstructor
@Embeddable
public class PhysicalAttributes {

    @Enumerated(EnumType.STRING)
    private Gender gender;

    private Double weight;

    @PrePersist
    public void applyDefault() {
        if (gender == null) {
            this.gender = Gender.MALE;
        }
        if (weight == null) {
            this.weight = 0.0;
        }
    }
}

그리고 맨 처음에 질문 드린 것처럼
사용자가 무게 혹은 성별을 입력하지 않았을 때, 해당 값들을 필요로 하는 로직에서 예외가 발생하지 않도록 하는 것이 기획에서도 합의되어 의도된 것인지도 다시 한번 확인해보고 싶습니다

디폴트 값이 남성, 60kg 인 것도 기획에서 미리 확정된건가요?

[2Jin1031]

밍곰! 좋은 포인트 짚어주셔서 감사합니다~

저는 온보딩 단계에서 사용자가 신체 정보를 입력하지 않아도 회원 가입이 완료되고, 홈 화면까지 진입할 수 있도록 하는 방향을 고려했습니다.
그 이유는 다음과 같습니다:

1. 홈 화면 진입의 간편화
   우리 서비스를 처음 사용하는 유저는 기능을 빠르게 확인하고자 할 수 있습니다.
   따라서 신체 정보는 필수가 아닌 선택으로 두어, 추후에도 입력할 수 있도록 구현했습니다.

2. 신체 정보의 비확실성 고려
   사용자가 자신의 정확한 몸무게나 성별 정보를 즉시 알지 못할 가능성도 존재한다고 생각했습니다.
   음용량 추천은 편의 기능일 뿐이므로, 이 정보들은 선택적으로 받아야 한다고 판단했습니다.

3. 기본값 기반 추천의 유도 효과
   기본값으로 음용량이 추천된 후, 그 값을 통해 사용햘 경우 유저는 자신의 데이터를 기반으로 "정확한 추천을 받고 싶다"는 동기를 느끼기에 훨씬 쉬울 것이라 생각하였습니다.

이러한 이유로, 로직 구현 중 의도에 대해 논의를 진행하였고 결론적으로 신체 정보는 선택 필드로 처리하기로 결정되었습니다!

로직 상으로는PhysicalAttributes에서 null일 경우 기본값을 반환하도록 설계했습니다.

public Gender getGender() {
    return gender != null ? gender : Gender.MALE;
}
public Double getWeight() {
    return weight != null ? weight : 60.0;
}

성별의 경우, 남성 기준이 비교적 많은 음용량을 추천하게 되므로,
목표치를 초과 달성하는 방향이 건강 상 안전하다고 판단해 MALE을 기본값으로 설정했습니다.
몸무게에 대해서는 별도 기획 의도는 없었으며, 의견 있으시면 적극 반영하겠습니다!

또한 말씀하신 것처럼, 이 방식은 로직에서 사용되는 값과 DB에 저장되는 값 사이의 불일치가 존재합니다.
이 점에 대해 고민했던 부분은, 기본값을 DB에 저장하게 될 경우 사용자가 입력한 값인지, 시스템이 임의로 채운 값인지 구분할 수 없다는 점이었는데요!

현재는 음용량 추천이라는 특정 목적에서만 해당 값들을 사용하고 있으며,
이 때만 get()으로 가공해 사용하고, DB에는 사용자가 입력한 값 그대로(null 포함) 저장되도록 설계하였습니다!

따라서 말씀 주신 1번이 의도된 동작입니다~

기획 의도나 방향에 대해 추가적인 의견 있으시면 언제든 편하게 말씀해주세용!!