Code Smells  Is there any magic number or unnamed constant?  Is there any unnecessary global or class variable?  Is there duplicate code?  Are there long methods?  Does any method have a long parameter list?  Is there any over-complex expression?  Is there any switch or if-then-else statement that should be replaced with polymorphism?  Is there any variable or method name whose intent is unclear?  Are there similar methods in multiple classes?