[CleanCode]3章関数

小さくしろ!

一つだけやろう!

各関数の抽象レベルは1つです!

スイッチドア

switch (e.type) {
	case COMMITSSIONED:
		return calculateCommissionedPay(e);
	case HOURLY:
		return calculateHourlyPay(e);
	case SALARIED:
		return calculateSalariedPay(e);
	default:
		throw new InvalidEmployeeType(e.type);
}

public interface EmployeeFactory {
	public Employee makeEmployee(EmployeeRecorder r) throws InvalidEmployType;
}
---------------------------------------
public class EmployeeFactoryImpl implements EmployeeFactory {
	public Employee makeEmployee(EmployeeRecorder r) throws InvalidEmployType{
		// switch문
	}
}

叙述的な名前を使う!

関数パラメータ

付随効果は発生しないでください。

コマンドとクエリーを分けて!

// ASIS - 의도가 모호하다. set하는 함수인가? username이 unclebob인지 확인하는 함수인가?
if(set("username", "unclebob")... 

// TOBE
if(attributeExists("username")) {
	setAttribute("username", "unclebob");
}

エラーコードではなく例外を使用します。

public void delete(Page page){
	try{
		deletePageAndAllReferences(page);
	}
	catch (Exception e) {
		logError(e);
	}
}

// try블록 별도 함수로 추출
private void deletePageAndAllReferences(Page page) throws Exception {
	deletePage(page);
	registry.deleteReference(page.name); // 실제동작함수
	configKeys.deleteKey(page.name.makeKey());
}
// catch블록 별도 함수로 추출
private void logError(Exception e){
	logger.log(e.getMessage());
}