해당 글은 인프런 박우빈 강사님의 『Readable Code: 읽기 좋은 코드를 작성하는 사고법』을 바탕으로 작성하였습니다.

미션 2-1.

public boolean validateOrder(Order order){
    if(order.getItems().size() == 0) {
        log.info("주문 항목이 없습니다.");
        return false;
    } else{
        if(order.getTotalPrice() > 0){
            if(!order.hasCustomerInfo()){
                log.info("사용자 정보가 없습니다.");
                return false;
            } else{
                return true;
            }
        }else if(!(order.getTotalPrice() > 0)) {
                log.info("올바르지 않은 총 가격입니다.");
                return false;
        }
    }
    return true;
}

문제 파악

validateOrder(Order order) - 사용자가 생성한 주문이 유효한지 검증하는 메서드

로직 작동 순서를 간단히 파악해보자.

• 주문서의 아이템들의 개수가 0일 시 log 발생
• 0이 아닐 시,
  1. 주문서의 가격이 0보다 크면
     1. 주문서의 손님 정보가 존재하지 않을 시 log 발생
     2. 손님 정보가 존재 할 시 true(주문이 유효하다.) 반환
  2. 주문서의 가격이 0보다 크지 않으면
     1. log 발생

1. Early return 체크 + 부정 연산자 제거 + 중첩 줄이기

• 빨리 리턴 가능하고 else문을 사용하지 않아도 되는 파트를 찾아보자. 위 로직 순서를 자세히 살펴보면, 의외로 간단하게 압축이 가능하다.
  • 주문서의 가격이 0이 아니면 false 반환
  • 총 가격이 0 이하여도 false 반환
  • 손님 정보가 없어도 false 반환
• 위 세 조건을 제외한 경우에만 true를 반환하면 되기 때문에 바로 아래와 같이 리팩토링 하였다.

public boolean validateOrder(Order order){
    if(order.getItems().size() == 0) {
        log.info("주문 항목이 없습니다.");
        return false;
    }
    if(order.getTotalPrice() <= 0) {
        log.info("올바르지 않은 총 가격입니다.");
        return false;
    }
    if(!order.hasCustomerInfo()){
        log.info("사용자 정보가 없습니다.");
        return false;
    }
    return true;
}

2. 공백 라인에 의미 가지기

• 각 if 문마다 판단하는 것이 다르다고 생각해서 각 if문마다 공백을 주는 것으로 변경

public boolean validateOrder(Order order){
    if(order.getItems().size() == 0) {
        log.info("주문 항목이 없습니다.");
        return false;
    }

    if(order.getTotalPrice() <= 0) {
        log.info("올바르지 않은 총 가격입니다.");
        return false;
    }

    if(!order.hasCustomerInfo()){
        log.info("사용자 정보가 없습니다.");
        return false;
    }

    return true;
}

3. 메서드 추출

• !order.hasCustomerInfo() 역시 불필요한 부정 연산자, 별도의 doesNotHaveCustomerInfo() 메서드를 Order 객체 내부에 생성했다고 가정 후 리팩토링