Violation | Line |
---|---|
Avoid if (x != y) ..; else ..; | 66 |
Avoid if (x != y) ..; else ..; | 70 |
Avoid if (x != y) ..; else ..; | 72 |
Violation | Line |
---|---|
Avoid empty catch blocks | 177 |
Avoid empty catch blocks | 203 |
Violation | Line |
---|---|
The class 'GenericFrameworkPortlet' has a Cyclomatic Complexity of 3 (Highest = 11). | 59 |
Avoid empty catch blocks | 231 |
Avoid empty catch blocks | 257 |
The method 'determineLogicalView' has a Cyclomatic Complexity of 11. | 295 |
Switch statements should have a default label | 479 |
Avoid unused formal parameters such as 'view' | 502 |
Avoid if (x != y) ..; else ..; | 512 |
Avoid using if statements without curly braces | 515 |
Avoid using if statements without curly braces | 556 |
Violation | Line |
---|---|
No need to import a type that's in the same package | 41 |
Avoid empty catch blocks | 170 |
Avoid if (x != y) ..; else ..; | 223 |
Violation | Line |
---|---|
Avoid empty catch blocks | 70 |
Avoid empty catch blocks | 74 |
Avoid empty catch blocks | 78 |
Avoid unused local variables such as 'action' | 362 |
Avoid unused local variables such as 'f1' | 376 |
Avoid unused local variables such as 'f2' | 377 |
Avoid unused local variables such as 'f0' | 378 |
All methods are static. Consider using Singleton instead. Alternatively, you could add a private constructor to silence this warning. | 404 |