리팩토링에 관한 다양한 방법들중 쓸만한 규칙

리팩토링 관련해서는 clean code 책에서 잘 설명되어있기때문에 전체를 설명할필요는 없지만  

그 책에 따른 파생된? 책인 five lines of code 책에서 설명하고있는 몇가지가 개인적으로 와닿아서 포스팅함!

 

 

1. 호출 또는 전달, 한가지만 할것

아래 함수를 보면 첫번째꺼는 전달을 하고있지만 동시에 Length 내부함수를 호출하고있다.  

이것을 완전히 호출역할만 하던지 전달 역할만 하던지 목적을 최소화시켜주는것이 신박한 방법이여서 뽑았다!

// bad
function average(arr: number[]) {
  return sum(arr) / arr.length;
}

// good
function average(arr: number[]) {
  return sum(arr) / size(arr);
}

 

2. 변경될 여지가 절대 없으면 굳이 리팩토링하지마라

굳이 잘되는부분을 변경될 요소가 앞으로도 절대 없는것들을 괜히 클래스, 인터페이스 구조화한다고 복잡하게 만드는것보다 막말로 하드코딩이 되어있는거라면 홀딩하고 다른 변경될 요인이 높은 부분을 리팩토링하는것이 더 안정적이다. 

 

3. if 문은 함수시작에만 두자

if 문을 함수 앞에만 두는것을 고수하고있는데 사실 어딜 둬도 똑같은거 아닌가? 싶어서 코드를 다 따라해보니

읽기 더 수월해진것도 있고 그 이후 코드를 읽을때는 그대로 진행될 가능성이 백퍼센트라서 길을 잃지않을수있었다.