DAY 25

🔖 오늘 읽은 범위 : 12. 창발성(216~223p)

🤓 책에서 기억하고 싶은 내용

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

단순한 설계 규칙 1: 모든 테스트를 실행하라

• 설계는 의도한 대로 돌아가는 시스템을 내놓아야 한다
• SRP를 준수하는 클래스는 테스트가 훨씬 더 쉽다
• DIP, DI, 인터페이스, 추상화 를 사용해 결합도를 낮춰야 테스트가 쉽다
• 테스트 케이스가 있으니까 코드를 정리

단순한 설계 규칙 2: 중복을 없애라

• 비슷한 코드는 더 비슷하게 고쳐준다
• 구현중복을 줄여라

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

• 단 몇줄이라도 중복 제거

  public void scaleToOneDimension(
      float desiredDimension, float imageDimension) {
      if (Math.abs(desiredDimension - imageDimension) < errorThreshold) return;
      float scalingFactor = desiredDimension / imageDimension;
      scalingFactor = (float)(Math.floor(scalingFactor * 100) * 0.01 f);
      RenderedOp newImage = ImageUtilities.getScaledImage(image, scalingFactor, scalingFactor);
      image.dispose();
      System.gc();
      image = newImage;
  }
  
  public synchronized void rotate(int degrees) {
      RenderedOp newImage = ImageUtilities.getRotatedImage(image, degrees);
      image.dispose();
      System.gc();
      image = newImage;
  }

  • scaleToOneDimension 메서드와 rotate 메서드가 일부 중복

    public void scaleToOneDimension(
        float desiredDimension, float imageDimension) {
        if (Math.abs(desiredDimension - imageDimension) < errorThreshold) return;
        float scalingFactor = desiredDimension / imageDimension;
        scalingFactor = (float)(Math.floor(scalingFactor * 100) * 0.01 f);
        replaceImage(ImageUtilities.getScaledImage(image, scalingFactor, scalingFactor));
    }
    public synchronized void rotate(int degrees) {
        replaceImage(ImageUtilities.getRotatedImage(image, degrees));
    }
    private void replaceImage(RenderedOp newImage) {
        image.dispose();
        System.gc();
        image = newImage;
    }

Template method 패턴

• 고차원 중복을 제거할 목적으로 자주 사용

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

• 최소 법정 일수 계산하는 코드 제외하면 거의 동일

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() {
            // 유럽연합 최소 법정 일수를 사용한다. }
        }
    }
}

• Template method 로 중복 제거하고 하위 클래스는 중복되지 않는 정보를 제공해 accrueVacation 알고리듬에서 빠진 구멍을 메움

단순한 설계 규칙 3: 표현하라

1. 좋은 이름을 선택
2. 함수와 클래스 크기를 가능한 줄임
3. 표준 명칭을 사용
4. 단위 테스트 케이스를 꼼꼼히 작성
5. 가장 중요한 방법은 꾸준한 노력, 나중에 코드를 읽을 사람은 바로 자신이다

클래스와 메서드 수를 최소로 줄여라

• 중복제거, 의도표현, SRP 준수를 너무 과하게 하면 클래스, 메서드 수가 늘어나 버린다
• 시스템 크기를 작게 유지하는게 최종 목표이므로 가능한 한 클래스, 메서드 수를 줄인다
• 하지만 앞의 1,2,3 이 더 중요하고 가능하면 네번째로 수행

🤔 떠오르는 생각

• 창발성이 뭔가 새로운 발견을 뜻하는 단어인것 같은데 원서에서 단어는 emergence 이다. 고난 역경에서의 탈출 혹은 곤충의 우화 가 좀 더 와닿는 뜻인것 같다. 내 더러운 코드를 ‘우화’ 시키기 위해 테스트 코드를 잘 작성하고 중복 제거하고 표현을 잘 하자.

🔎 질문

📝 소감 3줄 요약

• 설계가 의도한 대로 수행 되도록 모든 테스트를 수행
• 비슷한 코드는 더 비슷하게 고쳐 중복 제거
• 표현을 잘 하도록 꾸준히 도록 하자. 나중에 코드를 읽을 사람은 바로 자신이다