Q1.

<aside> 💡

아래 코드와 설명을 보고, [섹션 3. 논리, 사고의 흐름]에서 이야기하는 내용을 중심으로 읽기 좋은 코드로 리팩토링해 봅시다.

✔️ 사용자가 생성한 '주문'이 유효한지를 검증하는 메서드. ✔️ Order는 주문 객체이고, 필요하다면 Order에 추가적인 메서드를 만들어도 된다. (Order 내부의 구현을 구체적으로 할 필요는 없다.) ✔️ 필요하다면 메서드를 추출할 수 있다.

</aside>

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;
}

A1.

최종 코드

public class OrderException extends RuntimeException {
		public OrderException(String message) {
				super(message);
		}
}

public class EmptyItemException extends OrderException {
		public EmptyItemException(String message) {
				super(message);
		}
}

public class TotalPriceNotValidException extends OrderException {
		public TotalPriceNotValidException(String message) {
				super(message);
		}
}

public class EmptyCustomerInfoException extends OrderException {
		public EmptyCustomerInfoException(String message) {
				super(message);
		}
}

public class Order {

      private List<Item> items;
      private Customer customerInfo;
      
      private Order(List<Item> items, Customer customerInfo) {
            this.items = items;
            this.customerInfo = customerInfo;
      }
      
      public static Order createOrder(List<Item> items, Customer customerInfo) {
            return new Order(items, customerInfo);
      }

      public void validateOrder(Order order) {
          if (order.isEmptyItem()) {
                 throw new EmptyItemException("주문 항목이 없습니다.");
           }
           
           if (order.doesNotHaveCustomerInfo()) {
                 throw new EmptyCustomerInfoException("사용자 정보가 없습니다.");
            }
      }
      
      public long calculateTotalPrice() {
		      return this.items.stream()
				      .mapToLong(Item::getPrice)
					    .sum();
      }
      
}

리팩토링은 강의의 [Section 3. 논리, 사고의 흐름]에서 학습했던 내용들 중 적용이 필요하다고 생각되는 주제들을 뽑아서 학습했던 순서대로 하나씩 적용시키는 방식으로 진행하였다.

아래에 각 과정 별로 간단하게 정리해보았다.

1. Early Return 적용하기(else 사용을 지양하기)

2. 사고의 depth 줄이기

3. 공백 라인으로 의미 단위 나누기

4. 가독성을 떨어뜨리는 부정 연산자 제거하기

5. 의도한 예외 생성하기

여기까지가 [Section 3. 논리, 사고의 흐름]에서 학습한 순서로 리팩토링을 진행한 결과이다.

아래는 같은 날 학습한 [Section 4. 객체 지향패러다임 - 객체 설계하기]까지의 내용을 추가로 적용해 보았다.

Q1.