인프런 커뮤니티 질문&답변

안녕하세요. 도토리님, 공식 서포터즈 OMG입니다.
.

강의코드(ValidationItemControllerV1.addItem())

if (item.getPrice() == null || item.getPrice() < 1000 || item.getPrice() > 1000000) {
    errors.put("price", "가격은 1,000 ~ 1,000,000 까지 허용합니다.");
}
if (item.getQuantity() == null || item.getQuantity() >= 9999) {
    errors.put("quantity", "수량은 최대 9,999 까지 허용합니다.");
}

//특정 필드가 아닌 복합 룰 검증
if (item.getPrice() != null && item.getQuantity() != null) {
    int resultPrice = item.getPrice() * item.getQuantity();
    if (resultPrice < 10000) {
        errors.put("globalError", "가격 * 수량의 합은 10,000원 이상이어야 합니다. 현재 값 = " + resultPrice);
    }
}

질문코드

//검증 로직
if (item.getPrice() == null) {...}
if (item.getQuantity() == null) {...}


//특정 필드가 아닌 복합 룰 검증
if (item.getPrice() != null && item.getQuantity() != null) {
    if(resultPrice < 10000) {
        errors.put();
    }

}

검증 로직 파트에서 item.getPrice()==null, item.getQuantity()==null을 미리 검증해서
price, quantity가 null인 경우는 걸러지니까,

강의코드와 질문코드가 다릅니다.

강의코드는 if문 조건이 || 로 if문을 만족하는 조건이 추가로 붙습니다. 단순 null체크만 하지 않으며 다른 조건을 만족하는지에 따라 달라질 수 있기 때문에 가정이 잘못되었습니다.

그리고 걸러진다라고 표현하셨는데, if문 조건을 만족했을 때, return으로 종료하거나 무언가를 제거하지 않습니다.

아래 코드는 If문을 만족하면 errors에 벨리데이션이 추가됩니다.

(null일 때 추가되거나, null이 아니더라도 가격이 1000원미만이면 추가됩니다. )

if (item.getPrice() == null || item.getPrice() < 1000 || item.getPrice() > 1000000) {
    errors.put("price", "가격은 1,000 ~ 1,000,000 까지 허용합니다.");
}

아래도 마찬가지입니다.

if (item.getQuantity() == null || item.getQuantity() >= 9999) {
    errors.put("quantity", "수량은 최대 9,999 까지 허용합니다.");
}

바깥 if문을 제거해도 괜찮을까요?

아래의 강의 코드의 바깥 If문을 얘기하시는거 같은데요.

"(화면단에서 가격과 수량이 반드시 입력되도록 강제가 되어 있다고 한다면)

애초에 애플리케이션 화면단에서 가격과 수량이 반드시 입력되는 형태니까 price와 quantity가 null이 아니니 제거해도 되지 않나?" 라고 하면 맞다고 할 수 있겠지만 이전 If로직은 단순 errors에 벨리데이션 추가이기 때문에 Null을 필터링하여 price와 quantity가 Null이 아님을 보장하지 않습니다.

//특정 필드가 아닌 복합 룰 검증
if (item.getPrice() != null && item.getQuantity() != null) {
    int resultPrice = item.getPrice() * item.getQuantity();
    if (resultPrice < 10000) {
        errors.put("globalError", "가격 * 수량의 합은 10,000원 이상이어야 합니다. 현재 값 = " + resultPrice);
    }
}

추가로 한 가지 더 여쭤보고 싶은 것이 있는데,
검증 요구사항에서 '수량은 최대 9,999'라고 한다면 최소 수량에 대해서는 굳이 신경 쓰지 않아도 되는 것인가요?

애플리케이션 정책이기 때문에 개발자 혹은 회사 정책에 따라 정하면 됩니다. 영한님은 최대수량만을 고려하여 강의 예제를 구성하셨습니다.

.
감사합니다.