코드 리팩토링 시 참고할 것

- 한 메서드에 오직 한 단계의 들여쓰기(indent)만 허용했는가?

- else 예약어를 쓰지 않았는가?

- 모든 원시값과 문자열을 포장했는가?

- 콜렉션에 대해 일급 콜렉션을 적용했는가?

- 3개 이상의 인스턴스 변수를 가진 클래스를 구현하지 않았는가?

• 쉽지 않은 연습일 수 있다. 가능하면 인스턴스 변수의 수를 줄이기 위해 노력한다.

- getter/setter 없이 구현했는가?

• 핵심 로직을 구현하는 도메인 객체에 getter/setter를 쓰지 않고 구현했는가?
• 단, DTO는 허용한다.

- 메소드의 인자 수를 제한했는가?

• 4개 이상의 인자는 허용하지 않는다.
• 3개도 가능하면 줄이기 위해 노력해 본다.

- 코드 한 줄에 점(.)을 하나만 허용했는가?

• 디미터(Demeter)의 법칙(“친구하고만 대화하라”)을 지켰는가?
• 예를 들어 location.current.representation.substring(0, 1)와 같이 여러 개의 점(.)이 등장하면 리팩토링할 부분을 찾아본다.

- 메소드가 한가지 일만 담당하도록 구현했는가?

- 클래스를 작게 유지하기 위해 노력했는가?

 

 

 

---

https://github.com/woowacourse/woowacourse-docs/blob/main/cleancode/pr_checklist.md#else-%EC%98%88%EC%95%BD%EC%96%B4%EB%A5%BC-%EC%93%B0%EC%A7%80-%EC%95%8A%EC%95%98%EB%8A%94%EA%B0%80