변수 인라인하기 Inline Variable

0

리팩터링 카탈로그

목록 보기
4/10

배경


변수는 함수 안에서 표현식을 가리키는 이름으로 쓰이며, 대체로 긍정적인 효과를 준다. 하지만, 그 이름이 원래 표현식과 다를 바 없을 때도 있다. 또 변수가 주변 코드를 리펙터링 하는 데 방해가 되기도 한다. 이럴 때는 그 변수를 인라인 하는 것이 좋다.

절차


  1. 대입문의 우변(표현식)에서 부작용이 생기지는 않는지 확인한다.
  2. 변수가 불변으로 선언되지 않았다면 불변으로 만든 후 테스트한다.
  • 이렇게 하면 변수에 값이 단 한 번만 대입되는지 확인할 수 있다.
  1. 이 변수를 가장 처음 사용하는 코드를 찾아서 대입문 우변의 코드로 바꾼다.
  2. 테스트한다.
  3. 변수를 사용하는 부분을 모두 교체할 때까지 이 과정을 반복한다.
  4. 변수 선언문과 대입문을 지운다.
  5. 테스트한다.

예시


before

let basePrice = anOrder.basePrice;
return (basePrice > 1000);

after

return anOrder.basePrice > 1000;

참조


마틴 파울러 저 리팩터링 2판

0개의 댓글