[CleanCode]함수-는 작게+내려가기 규칙

 

프로그램의 기본단위 :
함수

 

 프로그래밍 초창기에는 시스템을 루틴과 하위루틴으로 나눴고

포트란과 PL/1 시절에는 시스템을 프로그램, 하위 프로그램, 함수로 나눴다.

지금은 함수만 남아있다.

 

---

 

작게 만들어라.
가로 150자, 세로 100줄을 넘어서는 안된다.

 

 책에서는 20줄도 길다한다.

이렇게 만드는 방법을 제시하자면

2단/2중 제어문까지 허용

1. 제어문 is/else문 while문 등에 들어가는 블록은 한줄이어야 한다.
2. 대개 거기서 함수를 호출한다.
3. 따라서 if문 안에 다시 if문을 쓰는 식의
   중첩 제어문 2단을 초과해서는 안된다.

---

함수는 한가지를 해야한다

 

• 그 한가지를 잘해야 한다.
• 함수의 의미를 유지하는 수준까지 줄인다.
• 함수 안 코드를 다른 함수로 추출할 수 있다면
  →그 함수는 여러가지를 하는 셈이다.

---

 

내려가기 규칙

 

참고로 책 저자가 만든 규칙이름인데 적절한 이름이라 동의한다.

method0 에서 method1과 2를 호출하고 있다.

 

1.  method0은 1과 2보다 많은 일을한다.
2. 따라서 method0의 여러가지 일을 더 단순하게 압축했으므로
   추상화수준이 더 높으므로 이를 고차원이라 하겠다.
3. method1과 method2는 반대로 0에 비해 저차원이라 할 수 있다.

고차원→저차원 함수를 따라 이야기처럼 읽어가는 것을 내려가기라고 한다.

---

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

 

만약 한 함수 내에 고차원과 저차원 함수가 섞여있다면

함수를 모두 따라가보지 않는 한 해당 함수가 무슨 일을 하는 지 알 수 없다.

비슷하게 고차원→저차원 →고차원 식의 잘못된 흐름을 피해야한다.

---

Switch문

Switch문은 본질적으로 N개의 작업을 수행하므로 작게 만들기 어렵다.

안 좋은 예

위 함수에는 네가지 문제가 있다.

1. 함수가 길다.
   새 직원 유형을 추가하면 더 길어질 것이다.
2. 한가지 작업만 수행하지 않는다.
3. SRP 위반
   Single Responsibility Principle : 클래스는 하나의 책임만을 가져야한다.
4. OCP 위반
   Open Closed Principle : 기존의 코드를 변경하지 않으면서 기능을 추가할 수 있도록 설계가 되어야 한다.

가장 위험한 문제로, 급여 정산을 위한 더 많은 함수들이 존재할 수 있다는 점이다.

이를 개선한 함수는 아래와 같다.

1. switch문을 추상 팩토리에 숨긴 후 아무에게도 보여주지 않는다.
2. 팩토리는 switch문을 통해 적절한 Employee 파생 클래스의 인스턴스를 생성new한다.
3. 이렇게 다형성을 통해 실제 파생 클래스의 함수가 실행된다.

switch문을 다형성 객체를 생성하는 코드로 한번 정도만 허용한다.
상속 관계에 숨긴 후에는 다른 코드에 노출하지 않는다.
물론 불가피한 상황도 있다ㅎ