Bad Smells in Code

• 코드가 여기저기 중복되어 있다.
• 메소드가 너무 길다.
• 클래스의 파일이나 메소드가 너무 많다.
• 메소드에 전달하는 인수의 수가 너무 많다.
• 사양변경이 발생할 경우 수정할 곳이 여기저기 흩어져 있다.
• 어떤 클래스를 수정하면 다른 클래스도 수정하지 않으면 안된다.
• 언제나 다른 클래스의 속성을 건드리고 있다.
• 정리해서 다룰 수밖에 없는 여러개의 데이터가 하나의 클래스에 정리되어 있지 않다.
• 클래스를 만들지 않고 기본 데이터형만을 사용한다.
• switch 문이나 if 문을 사용하여 동작을 분할하고 있다.
• 서브 클래스를 만들면서 클래스 계층에 따로 서브클래스를 만들어야 한다.
• 클래스가 별로 하는 일이 없다.
• 언젠가 이렇게 확장 하겠지 하고 기대하는 지나친 일반화
• 일시적으로만 사용할 필드가 있다.
• 메소드가 호출하는 연쇄가 너무 많다.
• 위양(권리를 위임하고) 자신이 하는 일은 없는 클래스가 있다.
• 필요없는 쌍방향 링크가 걸려 있거나 IS-A 관계가 아니면서 상속을 사용한다.
• API 가 부적절하다.
• 기존의 클래스 라이브러리를 사용하기 힘들다.
• 필드, getter 메소드, setter 메소드만 가지고 있는 클래스가 있다.
• 상속하고 있는 메소드면서 그것을 호출하면 문제가 발생한다.
• 코드의 부족을 보충하기 위해 상세한 코멘트가 있다.