[클린 코드] 12장 - 창발성 (Emergence)

이번 포스팅에선 클린 코드 12장 - 창발성 (Emergence)에 대해 알아보겠습니다.

 

창발성이란?

 

이전에 보이지 않았던 것이 어느 순간 갑자기 나타나는 것

 

창발적 설계로 깔끔한 코드를 구현하자!

 

착실하게 따르기만 하면 우수한 설계가 나오는 간단한 규칙 4가지가 있다면? << 이게 창발성

 

켄트 벡이 제시한 4가지 규칙을 따르면 우수한 설계가 나온다고 한다.

 

1) 모든 테스트를 실행한다.

2) 중복을 없앤다.

3) 프로그래머 의도를 표현한다.

4) 클래스와 메서드 수를 최소로 줄인다.

 

위 목록은 중요도 순이다.

 

규칙 1 : 모든 테스트를 실행한다.

 

9장 단위 테스트에서도 했던 말이지만 테스트는 중요하다.

 

테스트가 있어야 검증이 가능하고 변경이 쉬워진다.

 

왜 테스트를 실행하면 우수한 설계가 나온다고 할까?

 

테스트를 실행하려면 테스트가 가능한 시스템을 만들어야 한다.

 

테스트를 가능한 시스템을 만들려고 노력하다 보면 자연스럽게 코드의 품질이 높아진다.

 

결합도가 높은 시스템은 테스트하기가 어려운데 이를 해결하려고 SRP, DIP, DI, 추상화 등 다양한 도구를 계속 적용하다보면

 

자연스럽게 품질이 높은 코드가 나오게 된다.

 

'테스트 케이스를 만들고 계속 돌려라' 라는 단순한 규칙을 따르면 낮은 결합도와 높은 응집력을 갖는 객체 지향 코드가 나온다.

 

규칙 2 : 중복을 없앤다.

 

테스트를 모두 작성했다면 이제 코드와 클래스를 정리해도 된다.

 

한번에 리팩토링해서 끝내버리는 게 가능하다면 그렇게 해도 좋다.

 

하지만 그렇게 잘 안 되기 때문에 점진적으로 리팩토링을 해나가자.

 

코드를 정리하면서 시스템이 깨질까봐 걱정할 필요가 없다. 테스트 케이스가 있으니까!!!!

 

리팩터링 방법엔 다양한 방법이 있지만 여기서 말하고자 하는 건

 

바로 중복을 없애는 것!!!

 

예1)

 

size(), isEmpty() 메소드를 구현한다고 하면 각각 구현할 수도 있지만

 

int size() {}
boolean isEmpty() {}

 

size를 비교하는 로직이 중복으로 들어가는 거보다 위에 정의한 size()를 이용하면 좋다.

 

boolean isEmpty() {
    return 0 == size();
}

 

예2) 

 

메소드에 중복이 있는 경우 메소드 추출을 이용해 중복을 줄이자.

 

public void deleteStudent(Long id) {
    Student student = studentRepository.findByid(id);

    if (student == null) throw new StudentNotFoundException();
    
    // 학생 삭제 로직
}

public void updateStudent(Long id) {
    Student student = studentRepository.findByid(id);
    
    if (student == null) throw new StudentNotFoundException();
    
    // 학생 업데이트 로직
}

 

student를 가지고 오는데 없으면 에러를 발생하는 로직을 getStudentOrElseThrow 메소드로 추출해서 코드 중복을 없앴다.

 

public void deleteStudent(Long id) {
    getStudentOrElseThrow(id);

    // 학생 삭제 로직
}

public void updateStudent(Long id) {
    getStudentOrElseThrow(id);

    // 학생 업데이트 로직
}

private void getStudentOrElseThrow(Long id) throws StudentNotFoundException {
    Student student = studentRepository.findByid(id);

    if (student == null) throw new StudentNotFoundException();
}

 

예3)

 

각 나라의 최소 법정 일수를 계산하는 코드가 중복되어 있는 아래 메소드를 보자

 

public class VacationPolicy {
    public void accrueUSDivisionVacation() {
        // 지금까지 근무한 시간을 바탕으로 휴가 일수를 계산하는 코드
        // ...
        // 휴가 일수가 미국 최소 법정 일수를 만족하는지 확인하는 코드
        // ...
        // 휴가 일수를 급여 대장에 적용하는 코드
        // ...
    }
    
    public void accrueEUDivisionVacation() {
        // 지금까지 근무한 시간을 바탕으로 휴가 일수를 계산하는 코드
        // ...
        // 휴가 일수가 유럽연합 최소 법정 일수를 만족하는지 확인하는 코드
        // ...
        // 휴가 일수를 급여 대장에 적용하는 코드
        // ...
    }
}

 

위와 같은 경우 템플릿 메소드 (Template Method) 패턴을 적용해 중복을 없앨 수 있다.

 

abstract public class VacationPolicy {
    public void accrueVacation() {
        calculateBaseVacationHours();
        alterForLegalMinimums();
        applyToPayroll();
    }
    
    private void calculateBaseVacationHours() { /*...*/ }
    abstract protected void alterForLegalMinimums();
    private void applyToPayroll() { /*...*/ }
}

public class USVacationPolicy extends VacationPolicy {

    @Override
    protected void alterForLegalMinimums() {
        // 미국 최소 법정 일수 사용
    }
}

public class EUVacationPolicy extends VacationPolicy {

    @Override
    protected void alterForLegalMinimums() {
        // 유럽연합 최소 법정 일수 사용
    }
}

 

깔끔해졌다.

 

규칙 3 : 프로그래머 의도를 표현한다.

 

개발자는 코드에 자신의 의도를 표현해야 한다.

 

그래야 이해하기 쉽고 수정하기 쉬운 우아한 코드가 나온다.

 

1) 좋은 이름을 선택한다. : 그래야 이해가 빠르다.

 

2) 함수와 클래스 크기를 가능한 줄인다. : 함수는 세로는 200줄 가로는 120자, 클래스는 하나의 책임 기준으로 줄인다.

 

3) 표준 명칭을 사용한다. : 해법 영역, 도메인 영역에서 가지고 온 명칭을 사용하자. 그래야 이해하기가 쉬워진다.

 

4) 단위 테스트 케이스를 꼼꼼히 알아볼 수 있게 작성하자. : 이해하지 못 하게 짜놓은 테스트 코드는 오히려 수정을 복잡하게만 만든다. 테스트는 예쩨로 보여주는 문서다. 깔끔하고 알아볼 수 있게 작성하자.

 

 

규칙 4 : 클래스와 메서드 수를 최소로 줄인다.

 

이 규칙은 4개의 규칙 중 가장 우선 순위가 낮다.

 

쓸때없이 어떤 규칙을 만들어서 클래스와 메소드를 불필요하게 늘리는 경우에 최소로 줄이자는 말이다.

 

함수와 클래스 크기를 줄이고 동시에 시스템 크기도 작게 유지하자.

 

 

이번 12장은 1~11장에서 해왔던 얘기를 간단하게 압축한 내용입니다.

 

당연한 내용인 거 같지만 몸에 베이지 않으면 나올 수 없는 그런 내용들입니다.

 

열심히 코드를 짜보면서 익혀봐야겠습니다.

 

13장부터는 좀 어려운 기술적인 얘기들이 나오니 마음의 준비하세요!