워밍업 클럽 2기(클린코드, 테스트코드) 과제 (Day 4)

Readable Code: 읽기 좋은 코드를 작성하는 사고법(링크)

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

• 리팩토링 코드

public boolean validateOrder(Order order) {
    if (order == null) {
        log.info("주문 정보가 없습니다.");
        return false;
    }
    if (order.hasNoItems()) {
        log.info("주문 항목이 없습니다.");
        return false;
    }
    if (order.isInvalidTotalPrice()) {
        log.info("올바르지 않은 총 가격입니다.");
        return false;
    }
    if (order.hasNoCustomerInfo()) {
        log.info("사용자 정보가 없습니다.");
        return false;
    }
    return true;
}

public class Order {
    
    private List<Item> items;
    private double totalPrice;
    private CustomrInfo customerInfo;

    public boolean hasNoItems() {
        return items == null || items.isEmpty();
    }

    public boolean isInvalidTotalPrice() {
        return totalPrice <= 0;
    }

    public boolean hasNoCustomerInfo() {
        return customerInfo == null;
    }
}

2. SOLID 정리

SOLID

SRP

• Single Responsibility Principle (단일 책임 원칙)

• 하나의 클래스는 단 한 가지의 변경 이유만을 가져야 한다.

• 객체가 가진 공개 메서드, 필드, 상수 등은 해당 객체의 단일 책임에 의해서만 변경 되어야 함.

• 관심사의 분리

• 높은 응집도, 낮은 결합도

OCP

• Open-Closed Principle (개방-폐쇄 원칙)

• 확장에는 열려 있고, 수정에는 닫혀 있어야 한다.

  • 기존 코드의 변경 없이, 시스템의 기능을 확장할 수 있어야 한다.

• 추상화와 다형성을 활용해서 OCP를 지킬 수 있다.

LSP

• Liskov Substitution Principle (리스코프 치환 원칙)

• 상속 구조에서, 부모 클래스의 인스턴스를 자식 클래스의 인스턴스로 치환할 수 있어야 한다.

  • 자식 클래스는 부모 클래스의 책임을 준수하며, 부모 클래스의 행동을 변경하지 않아야 한다.

• LSP를 위반하면, 상속 클래스를 사용할 때 오동작, 예상 밖의 예외가 발생하거나, 이를 방지하기 위한 불필요한 타입 체크가 동반될 수 있다.

ISP

• Interface Segregation Principle (인터페이스 분리 원칙)

• 클라이언트는 자신이 사용하지 않는 인터페이스에 의존하면 안 된다.

  • 인터페이스를 잘게 쪼개야 함

• ISP를 위반하면, 불필요한 의존성으로 인해 결합도가 높아지고, 특정 기능의 변경이 여러 클래스에 영향을 미칠 수 있다.

DIP

• Dependency Inversion Principle (의존성 역전 원칙)

• 상위 수준의 모듈은 하위 수준의 모듈에 의존해서는 안 된다. 둘 모두 추상화에 의존해야 한다.

• 의존성의 순방향 : 고수준 모듈이 저수준 모듈을 참조하는 것
  의존성의 역방향 : 고수준, 저수준 모듈이 모두 추상화에 의존하는 것

  • 저수준 모듈이 변경되어도, 고수준 모듈에는 영향이 가지 않는다.