[Refactoring] 냄새 14. 성의없는 요소

[leeyuunsung]

냄새 14. 성의없는 요소 (Lazy Element)

• 여러 프로그래밍적인 요소(변수, 메소드, 클래스 등)를 만드는 이유
  • 나중에 발생할 변화를 대비해서...
  • 해당 함수 또는 클래스를 재사용하려고...
  • 의미있는 이름을 지어주려고...
• 가끔 그렇게 예상하고 만들어 놓은 요소들이 기대에 부응하지 못하는 경우가 있는데 그런 경우에 해당 요소들을 제거해야 한다.
• 관련 리팩토링 기술
  • “함수 인라인 (Inline Function)”
  • “클래스 인라인 (Inline Class)”
  • 불필요한 상속 구조는 “계층 합치기 (Collapse Hierarchy)”를 사용할 수 있다.

---

리팩토링 34. 계층 합치기 (Collapse Hierarchy)

• 상속 구조를 리팩토링하는 중에 기능을 올리고 내리다 보면 하위클래스와 상위클래스 코드 에 차이가 없는 경우가 발생할 수 있다. 그런 경우에 그 둘을 합칠 수 있다.
• 하위클래스와 상위클래스 중에 어떤 것을 없애야 하는가? (둘 중에 보다 이름이 적절한 쪽 을 선택하지만, 애매하다면 어느쪽을 선택해도 문제없다.)

Before

상위클래스

public class Reservation {
    private LocalDateTime startDateTime;
    private LocalDateTime endDateTime;
    private List<String> members;
    private String owner;
    private boolean paid;
}

하위클래스

public class CourtReservation extends Reservation {
    private String courtNumber;
}

• 위 예제상 하위클래스의 필드를 상위클래스로 올리거나,
• 상위클래스의 필드를 하위클래스로 내리는 경우가 발생할 수 있음
• 이 때, 인텔리제이의 우클릭 -> Refactor -> Push Members Down 기능을 이용하면 됨
• 반대로 하위클래스의 멤버에서는 하위클래스에서 Pull Members Up 리팩토링을 사용하면 된다