PMD Results
The following document contains the results of PMD 6.4.0.
Files
top/infra/core/ExpectedException.java
| Violation |
Priority |
Line |
| Parameter 'enableSuppression' is not assigned and could be declared final |
3 |
75 |
| Parameter 'writableStackTrace' is not assigned and could be declared final |
3 |
76 |
top/infra/core/ExpectedExceptionWithCode.java
| Violation |
Priority |
Line |
| Avoid unused imports such as 'lombok.Getter' |
4 |
3 |
| Parameter 'enableSuppression' is not assigned and could be declared final |
3 |
60 |
| Parameter 'writableStackTrace' is not assigned and could be declared final |
3 |
61 |
top/infra/localization/LocalizedMessages.java
| Violation |
Priority |
Line |
| Variables that are final and static should be all capitals, 'messageCache' is not all capitals. |
1 |
19 |
| Avoid using redundant field initializer for 'initialized' |
3 |
20 |
| Local variable 'resultMsg' could be declared final |
3 |
36 |
| Avoid throwing raw exception types. |
1 |
40 |
| The method 'init()' has a NCSS line count of 15. |
3 |
46–63 |
| Consider using varargs for methods or constructors which take an array the last parameter. |
4 |
65 |
| Parameter 'messageLocale' is not assigned and could be declared final |
3 |
79 |
| Parameter 'message' is not assigned and could be declared final |
3 |
79 |
top/infra/localization/MessageLocale.java
| Violation |
Priority |
Line |
| Avoid using short method names |
3 |
26 |
| A method should have only one exit point, and that should be the last statement in the method |
3 |
28 |
| Avoid empty catch blocks |
3 |
29–30 |
top/infra/web/servlet/handler/OrderedHandlerInterceptor.java
| Violation |
Priority |
Line |
| Classes implementing Serializable should set a serialVersionUID |
3 |
18–29 |
top/infra/web/servlet/handler/OrderedHandlerInterceptorAdapter.java
| Violation |
Priority |
Line |
| No abstract method which means that the keyword is most likely used to prevent instantiation. Use a private or protected constructor instead. |
1 |
11–13 |
