🦾3장 함수

작게만들어라

저자는 함수를 만들 때 가능하면 짧게 만들라고 조언한다. 함수를 이해하기 쉽게 하고 의도를 분명히 하고 프로그램 내부를 직관적으로 파악하려면 어떻게 해야 하는가? 저자는 매우 단순한 해결책을 제시한다. 함수 코드를 가능한 한 작게 만드는 것이다. 2개의 함수를 예로 들어보겠다.

아래 함수는 페이지를 렌더링하고 설정 및 청소하는 함수이다.

함수 1

public static String renderPageWithSetupAndTeardowns(
	PageData pageData, boolean isSuite
) throws Exception {
  boolean isTestPage = pageData.hasAttribute("Test");
	if (isTestPage) {
		WikiPage testPage = pageData.getWikiPage();
		StringBuffer newPageContent = new StringBuffer();
		includeSetupPages(testPage, newPageContent, isSuite);
		newPageContent.append(pageData.getContent());
		includeTeardownPages(testPage, newPageContent, isSuite);
		pageData.setContent(newPageContent.toString());
	}
	return pageData.getHtml();
}

함수2

public static String renderPageWithSetupsAndTeardowns(
	PageData pageData, boolean isSuite) throws Exception {
	if (isTestPage(pageData))
		includeSetupAndTeardownPages(pageData, isSuite);
	return pageData.getHtml();
}

어떤 함수가 직관적이고 이해하기 쉬운가? 함수 1의 경우 각각의 객체가 무엇을 의미하는지 이해 해야되지만 함수 2의 경우 조건문 하나와 내부 함수 하나 그리고 리턴문에서 html 리턴하는 것만 이해하면 된다.

한가지만 해라

함수는 한 가지만 해야된다. 지정된 함수 이름 아래에서 추상화 수준이 하나인 단계만 수행한다면 그 함수는 한가지 작업을 한다.

함수 당 추상화 수준은 하나로

함수의 추상화 수준이 섞이면 이해하기 어려워진다. 예를 들어 getHtml()은 추상화 수준이 아주 높다.
String pagePathName = PathParser.render(pagepath); 추상화 수준이 중간이다. 그리고 .append(”\n”)와 같은 코드는 추상화 수준이 아주 낮다.
한 함수 내에 추상화 수준을 섞으면 코드를 읽는 사람이 헷갈린다. 특정 표현이 근본 개념인지 아니면 세부사항인지 구분하기 어려운 탓이다. 하지만 문제는 이정도로 그치지 않는다. 근본 개념과 세부사항을 뒤섞기 시작하면, 깨어진 창문처럼 사람들이 함수에 세부사항을 점점 더 추가한다.

switch 문은 추상 팩토리로 해결

switch문은 본질적으로 n가지 역할을 수행한다. 본질적으로 switch문을 피할 수 있는 방법은 없다. 하지만 각 switch문을 저차원 클래스에 숨기고 절대로 반복하지 않기 위해서 다형성을 사용할 수 있다.

함수 인수는 가능한 적게

함수 인수는 대부분의 케이스에서 늘어날 수록 이해하기 어려워진다.

만약에 아래와 같이 두 가지 함수가 주어졌다고 하자

• includeSetupPageInto(new PageContent)
• includeSetupPageInto()

함수 인수가 없으면 함수 이름으로 추측하면 되지만 인수가 추가되면 함수를 한번에 이해하기 힘들어진다.