어떤 프로그램이건 가장 기본이 되는 것이 함수다.

1. 작게 만들어라!

(1) 블록과 들여쓰기 : if문 / else문 / while 문 등에 들어가는 블록은 한줄이 되도록 하라

1. 한 가지만 해라

• 함수가 한 가지만 하는지 확인하는 법
• ㄴ 함수 내 섹션 : 의미 있는 다른 함수로 추출할 수 있다면 한 가지만 하는 게 아니다.

1. 함수당 추상화 수준은 하나라

• 함수가 한 가지 작업만 하려면 함수 내 모든 문장의 추상화 수준이 동일해야 한다.
• 위에서 아래로 코드 읽기 : 내려가기 규칙 (이야기가 흘러가듯 내려 읽혀지게!)

1. Switch문

• 아래 코드의 문제 :
  • (1) 함수가 길다
  • (2) 여러가지 작업을 수행한다.
  • (3) SRP를 위반한다. (코드 변경 이유가 여럿)
  • (4) OCP를 위반한다. (새 직원 유형 변경 시 코드를 바꿔야한다.)

public Money calculatePay(Employee e) throws InvalidEmployeeType{
	switch(e.type){
		case COMMISIONED : return calculateCommissionedPay(e);
		case HOURLY      : return calculateHourlyPay(e);
		case SALARIED    : return calculateSalariedPay(e);
		default          : throw new InvalidEmployeeType(e.type);
	}
}

• 수정된 버전 (팩토리 패턴을 사용했다.)
  • Employee를 추상 클래스로 만들었다.
  • Employee를 만드는 팩토리 인터페이스를 만들었다.
  • Employee를 만드는 팩토리 인터페이스 구현체를 만들었다.

public abstract class Employee {
	public abstract boolean isPayday();
	public abstract Money calculatePay();
	public abstract void deliveryPay(Money pay);
}

-----------------------------

public interface EmployeeFactory {
	public Employee makeEmployee(EmployeeRecord r) throws InvalidEmployeeType;
}

-----------------------------

public class EmployeeFactoryImpl implements EmployeeFactory {
	public Employee makeEmployee(EmployeeRecord r) throws InvalidEmployeeType{
		switch(e.type){
			case COMMISIONED : return new CommissionedEmployee(r);
			case HOURLY      : return new HourlyEmployee(r);
			case SALARIED    : return new SalariedEmployee(r);
			default          : throw new InvalidEmployeeType(r.type);
		}
	} 
}