Name | Checked | Synopsis |
---|
Controversial1
|
 |
Avoid unnecessary constructors - the compiler will generate these for you |
Controversial2
|
 |
Assigning an Object to null is a code smell. Consider refactoring |
Controversial3
|
 |
A method should have only one exit point, and that should be the last statement in the method |
Controversial4
|
 |
Avoid modifiers which are implied by the context |
Controversial5
|
 |
Avoid assignments in operands |
Controversial6
|
 |
Each class should declare at least one constructor |
Controversial7
|
 |
Avoid importing anything from the 'sun.*' packages |
Controversial8
|
 |
Suspicious decimal characters following octal escape in string literal |
Controversial9
|
 |
It is a good practice to call super() in a constructor |
Controversial10
|
 |
This statement may have some unnecessary parentheses |
Controversial11
|
 |
If a field is only used by one method, consider making it a local variable |
Controversial12
|
 |
Use explicit scoping instead of the default package private level |
Controversial12
|
 |
Use explicit scoping instead of the default package private level |
Controversial13
|
 |
Use bitwise inversion to invert boolean values |
Controversial14
|
 |
Make sure variable assignment has been performed correctly |