17장 - 냄새와 휴리스틱
리팩터링이 필요한 코드에서 나쁜 냄새가 난다고한다. 코드를 작성하고 해당되는지 확인해야할 것을 정리해놓았다.
주석 - 부적절한 정보, 쓸모 없는 주석, 중복된 주석, 성의 없는 주석, 주석 처리된 코드
환경 - 여러 단계로 빌드 및 테스트
함수 - 너무 많은 인수, 출력 인수, 플래그 인수, 죽은 함수
일반 - 한 소스 파일에 여러 언어를 사용한다, 당연한 동작을 구현하지 않는다, 경계를 올바로 처리하지 않는다, 안전 절차 무시, 중복, 추상화 수준이 올바르지 못하다, 기초 클래스가 파생 클래스에 의존한다, 과도한 정보, 죽은 코드, 수직 분리, 일관성 부족, 잡동사니, 인위적 결합, 기능 욕심, 선택자 인수, 모호한 의도, 잘못 지운 책임, 부적절한 static 함수, 서술적 변수, 이름과 기능이 일치하는 함수, 알고리즘을 이해하라, 논리적 의존성은 물리적으로 드러내라, 조건문보다 다형성을 사요용하라, 표준표기법, 매직 숫자는 명명된 상수로 교체하라, 정확하라, 관례보다 구조를 사용하라, 조건을 캡슐화하라, 부정 조건은 피하라, 함수는 한 가지만 해야한다, 숨겨진 시간적인 결합, 일관성을 유지하라, 경계 조건을 캡슐화하라, 설정 정보는 최상위 단계에 둬라, 추이적 탐색을 피하라
자바 - 긴 import 목록을 피하고 와일드 카드(*)를 사용하라, 상수는 상속하지 않는다, enum 활용하기
이름 - 서술적인 이름을 사용하라, 적절한 추상화 수준에서 이름을 선택하라, 가능하다면 표준 명명법을 사용하라, 명확한 이름, 긴 범위는 긴 이름을 사용하라, 인코딩을 피하라, 이름으로 부수 효과를 설명하라
테스트 - 불충분한 테스트, 커버리지 도구를 사용하라, 사소한 테스트를 건너뛰지 마라, 무시한 테스트는 모호성을 뜻한다, 경계 조건을 테스트하라, 버그 주변은 철저히 테스트해라, 실패 패턴을 살펴라, 테스트 커버리지 패턴을 살펴라, 테스트는 빨라야 한다.