Violation | Line |
---|---|
Avoid if (x != y) ..; else ..; | 65 |
Avoid if (x != y) ..; else ..; | 69 |
Avoid if (x != y) ..; else ..; | 71 |
Violation | Line |
---|---|
Avoid empty catch blocks | 176 |
Avoid empty catch blocks | 202 |
Violation | Line |
---|---|
The class 'GenericFrameworkPortlet' has a Cyclomatic Complexity of 3 (Highest = 11). | 58 |
Avoid empty catch blocks | 230 |
Avoid empty catch blocks | 256 |
The method 'determineLogicalView' has a Cyclomatic Complexity of 11. | 294 |
Switch statements should have a default label | 478 |
Avoid unused formal parameters such as 'view' | 501 |
Avoid if (x != y) ..; else ..; | 511 |
Avoid using if statements without curly braces | 514 |
Avoid using if statements without curly braces | 555 |
Violation | Line |
---|---|
No need to import a type that's in the same package | 39 |
Avoid empty catch blocks | 170 |
Avoid if (x != y) ..; else ..; | 223 |
Violation | Line |
---|---|
Avoid empty catch blocks | 69 |
Avoid empty catch blocks | 73 |
Avoid empty catch blocks | 77 |
Avoid unused local variables such as 'action' | 361 |
Avoid unused local variables such as 'f1' | 375 |
Avoid unused local variables such as 'f2' | 376 |
Avoid unused local variables such as 'f0' | 377 |
All methods are static. Consider using Singleton instead. Alternatively, you could add a private constructor to silence this warning. | 403 |