[Refactoring] 중복 코드

중복 코드

• 중복 코드의 단점
  • 비슷하진, 완전히 동일한 코드인지 주의 깊게 봐야한다.
  • 코드를 변경할 때, 동일한 모든 곳의 코드를 변경해야 한다.
• 이에 해당하는 리팩토링 기술은 3종류 가 있다.
  • 동일한 코드를 여러 메소드에서 사용하는 경우, 함수 추출하기 (Extract Function)
  • 코드가 비슷하게 생겼지만 완전히 같지는 않은 경우, 코드 분리하기 (Slide Statements)
  • 여러 하위 클래스에 동일한 코드가 있다면, 메소드 올리기 (Pull Up Method)

함수 추출하기 (Extract Function)

---

• 의도와 구현 분리하기
• 무슨 일을 하는 코드인지 알아내려고 노력해야 하는 코드라면 해당 코드를 함수로 분리하고 함수 이름으로 무슨 일을 하는지 표현할 수 있다.
• 한줄 짜리 메소드도 괜찮다.
• 거대한 함수 안에 들어있는 주석은 추출한 함수를 찾는데 있어서 좋은 단서가 될 수 있다.
  • 함수가 거대하다면 그 함수는 이미 너무 많은 일을 하고 있는 것.

코드 정리하기 (Slide Statements)

---

• 관련있는 코드끼리 묶여있어야 코드를 더 쉽게 이해할 수 있다.
• 함수에서 사용할 변수를 상단에 미리 정의하기 보다는, 해당 변수를 사용하는 코드 바로 위에 선언하자.
• 관련있는 코드끼리 묶은 다음, 함수 추출하기를 사용해서 더 깔끔하게 분리할 수도 있다.

메소드 올리기 (Pull Up Method)

---

• 중복 코드는 당장은 잘 동작하더라도 미래에 버그를 만들어 낼 빌미를 제공한다.
  • ex) A에서 코드를 고치고, B에는 반영하지 않은 경우
• 여러 하위 클래스에 동일한 코드가 있다면, 손쉽게 이 방법을 적용할 수 있다.
• 비슷하지만 일부 값만 다른 경우라면 함수 매개변수화하기 리팩토링을 적용한 이후에 이 방법을 사용할 수있다.
• 하위 클래스에 있는 코드가 상위 클래스가 아닌 하위 클래스 기능에 의존하고 있다면, 필드 올리기를 적용한 이후에 이 방법을 적용할 수 있다.
• 두 메소드가 비슷한 절차를 따르고 있다면, 템플릿 메소드 패턴 적용을 고려할 수도 있다.

Reference

---

백기선. 코딩으로 학습하는 리팩토링. 인프런. https://www.inflearn.com/course/%EB%A6%AC%ED%8C%A9%ED%86%A0%EB%A7%81/dashboard