PMD Results

The following document contains the results of PMD 3.9.

Files

org/limmen/crs/init/ApplicationListener.java

ViolationLine
Each class should declare at least one constructor25 - 41
Found 'DU'-anomaly for variable 'sce' (lines '28'-'31').28 - 31
Parameter 'sce' is not assigned and could be declared final28
Found 'DU'-anomaly for variable 'sce' (lines '34'-'40').34 - 40
Parameter 'sce' is not assigned and could be declared final34

org/limmen/crs/servlet/ConfigurationServlet.java

ViolationLine
Each class should declare at least one constructor42 - 163
Parameter 'config' is not assigned and could be declared final63
Local variable 'port' could be declared final67
Local variable 'contextPath' could be declared final68
Local variable 'addressesEnum' could be declared final72 - 73
Local variable 'properties' could be declared final87
Local variable 'serviceInfo' could be declared final90 - 96
Parameter 'response' is not assigned and could be declared final111
Parameter 'request' is not assigned and could be declared final111
Local variable 'configurationService' could be declared final113
Local variable 'configurationId' could be declared final115
Local variable 'section' could be declared final116
Local variable 'setting' could be declared final117
Found 'DD'-anomaly for variable 'configurationSetting' (lines '118'-'130').118 - 130
Found 'DU'-anomaly for variable 'configurationSetting' (lines '130'-'144').130 - 144
Parameter 'event' is not assigned and could be declared final147
Parameter 'event' is not assigned and could be declared final153
Parameter 'event' is not assigned and could be declared final159

org/limmen/crs/web/bean/ConfigurationManagementBean.java

ViolationLine
Perhaps 'service' could be replaced by a local variable.13
Avoid unnecessary constructors - the compiler will generate these for you15 - 19
Local variable 'configurations' could be declared final23
Local variable 'configurationInfo' could be declared final27
Avoid instantiating new objects inside loops27

org/limmen/crs/web/model/ConfigurationInfo.java

ViolationLine
Each class should declare at least one constructor3 - 24
Parameter 'name' is not assigned and could be declared final13
Parameter 'baseName' is not assigned and could be declared final21

org/limmen/crs/web/util/SimpleNavigationHandler.java

ViolationLine
Each class should declare at least one constructor12 - 70
Found 'DU'-anomaly for variable 'fromAction' (lines '21'-'69').21 - 69
Parameter 'fromAction' is not assigned and could be declared final21
Parameter 'facesContext' is not assigned and could be declared final21
Avoid reassigning parameters such as 'outcome'22
Avoid reassigning parameters such as 'outcome'22
Avoid reassigning parameters such as 'outcome'22
Found 'DD'-anomaly for variable 'history' (lines '29'-'32').29 - 32
Local variable 'externalContext' could be declared final31
Substitute calls to size() == 0 (or size() != 0) with calls to isEmpty()43
Local variable 'viewHandler' could be declared final58
This call to String.startsWith can be rewritten using String.charAt(0)60
Local variable 'viewRoot' could be declared final66