리팩터링은 언제 해야 하나?

기능을 추가하기 전에 

 

리팩터링의 기준은 무엇인가?

개발자가 기능을 추가했을 때 코드 베이스가 더 이해하기 힘들어지는 상황이라면 리팩터링한다.

 

항상 코드를 수정하기 전보다 깔끔하게 정리(리팩터링)하고 떠난다(커밋-푸시).

 

리팩터링 구체적인 기준

기능을 추가했을 때 기존 코드의 역할이 늘어나면.

내가 추가하려는 기능이 기존 if-else, switch를 더 복잡하게 만들면

기능을 추가하고 싶은데 기존 코드가 복잡해 기능 추가가 어려우면

추가할 기능에 대해 테스트 케이스를 만들고 싶은데, 기존 코드에서 새로운 기능을 테스트 케이스를 넣기 어려우면

 

리팩터링의 단계

잘게 쪼개서 단 한순간도 코드를 깨뜨리지 않아야 한다. (기존 동작 유지, 컴파일 에러X)

한번에 두 세 단계를 진행 후 커밋하지 않느다. (Push는 여러 커밋을 모아서 해도 되지만, 커밋은 되도록 작은 단계 마다 진행)