Common Utilities and Interfaces
| Violation | Line |
|---|---|
| Ensure you override both equals() and hashCode() | 55 |
| Violation | Line |
|---|---|
| Avoid instantiating Boolean objects; you can usually invoke Boolean.valueOf() instead. | 167 |
| Avoid unnecessary comparisons in boolean expressions | 200 |
| Avoid empty 'if' statements | 211 |
| Avoid unnecessary return statements | 218 |
| Avoid unnecessary comparisons in boolean expressions | 232 |
| Avoid unnecessary return statements | 250 |
| Avoid unnecessary comparisons in boolean expressions | 272 |
| Avoid unnecessary return statements | 288 |
| Avoid unnecessary comparisons in boolean expressions | 310 |
| Avoid unnecessary return statements | 327 |
| Avoid unnecessary comparisons in boolean expressions | 349 |
| Avoid unnecessary return statements | 366 |
| Violation | Line |
|---|---|
| Avoid if (x != y) ..; else ..; | 51 |
| The same String literal appears 5 times in this file; the first occurrence is on line 80 | 80 |
| Avoid really long methods. | 98 |
| Avoid using 'if...else' statements without curly braces | 140 |
| Avoid using if statements without curly braces | 141 |
| Avoid unnecessary comparisons in boolean expressions | 163 |
| Avoid unnecessary comparisons in boolean expressions | 167 |
| Violation | Line |
|---|---|
| Avoid calls to overridable methods during construction | 38 |
| Deeply nested if..then statements are hard to read | 50 |
| Violation | Line |
|---|---|
| All methods are static. Consider using Singleton instead. Alternatively, you could add a private constructor to silence this warning. | 88 |
| Violation | Line |
|---|---|
| All methods are static. Consider using Singleton instead. Alternatively, you could add a private constructor to silence this warning. | 70 |
| Violation | Line |
|---|---|
| All methods are static. Consider using Singleton instead. Alternatively, you could add a private constructor to silence this warning. | 108 |