iBATIS JPetstore Demo Portlet
| Violation | Line |
|---|---|
| A high number of public methods and attributes in an object can indicate the class may need to be broken up for exhaustive testing may prove difficult. | 330 |
| Violation | Line |
|---|---|
| All methods are static. Consider using Singleton instead. Alternatively, you could add a private constructor to silence this warning. | 50 |
| Violation | Line |
|---|---|
| Avoid if (x != y) ..; else ..; | 129 |
| Avoid if (x != y) ..; else ..; | 146 |
| Avoid if (x != y) ..; else ..; | 148 |
| Violation | Line |
|---|---|
| The same String literal appears 4 times in this file; the first occurrence is on line 63 | 63 |
| Violation | Line |
|---|---|
| Deeply nested if..then statements are hard to read | 148 |
| Violation | Line |
|---|---|
| No need to import a type that's in the same package | 19 |
| Violation | Line |
|---|---|
| No need to import a type that's in the same package | 19 |
| Violation | Line |
|---|---|
| No need to import a type that's in the same package | 19 |
| Violation | Line |
|---|---|
| No need to import a type that's in the same package | 19 |