PMD Results

The following document contains the results of PMD 5.0.5.

Files

org/andromda/maven/plugin/cartridge/site/AbstractSiteMojo.java

Violation Line
Found non-transient, non-static member. Please mark as transient or provide accessors. 30
Parameter 'sourceFile' is not assigned and could be declared final 40
Parameter 'destFile' is not assigned and could be declared final 40
Avoid catching generic exceptions such as NullPointerException, RuntimeException, Exception in try-catch block 48
Parameter 'file' is not assigned and could be declared final 61
Parameter 'location' is not assigned and could be declared final 61
Avoid printStackTrace(); use a logger call instead. 80
Avoid catching generic exceptions such as NullPointerException, RuntimeException, Exception in try-catch block 84
Avoid printStackTrace(); use a logger call instead. 86

org/andromda/maven/plugin/cartridge/site/GenerateCartridgeHowtoArtifactsMojo.java

Violation Line
Avoid excessively long variable names like howtoCartridgePicturesSourcePath 33
Found non-transient, non-static member. Please mark as transient or provide accessors. 33
Found non-transient, non-static member. Please mark as transient or provide accessors. 40
Avoid excessively long variable names like howtoCartridgePicturesOutputDirectory 40
Found non-transient, non-static member. Please mark as transient or provide accessors. 48
Found non-transient, non-static member. Please mark as transient or provide accessors. 53
Found non-transient, non-static member. Please mark as transient or provide accessors. 63
Found non-transient, non-static member. Please mark as transient or provide accessors. 78
Found non-transient, non-static member. Please mark as transient or provide accessors. 86
A catch statement should never catch throwable since it includes errors. 124
An instanceof check is being performed on the caught exception. Create a separate catch clause for this exception type. 126
A catch statement should never catch throwable since it includes errors. 156
An instanceof check is being performed on the caught exception. Create a separate catch clause for this exception type. 158

org/andromda/maven/plugin/cartridge/site/HelpMojo.java

Violation Line
Comment is too large: Line too long 11
Found non-transient, non-static member. Please mark as transient or provide accessors. 27
Found non-transient, non-static member. Please mark as transient or provide accessors. 34
Found non-transient, non-static member. Please mark as transient or provide accessors. 41
Found non-transient, non-static member. Please mark as transient or provide accessors. 48
The method execute() has an NPath complexity of 600 52–159
Avoid really long methods. 52–159
Local variable 'sb' could be declared final 66
Avoid variables with short names like sb 66
The String literal "" appears 17 times in this file; the first occurrence is on line 69 69
Parameter 'repeat' is not assigned and could be declared final 170
Parameter 'str' is not assigned and could be declared final 170
Local variable 'buffer' could be declared final 172
Parameter 'sb' is not assigned and could be declared final 190
Parameter 'indent' is not assigned and could be declared final 190
Parameter 'description' is not assigned and could be declared final 190
Avoid variables with short names like sb 190
Local variable 'it' could be declared final 192
Parameter 'indentSize' is not assigned and could be declared final 208
Parameter 'lineLength' is not assigned and could be declared final 208
Parameter 'text' is not assigned and could be declared final 208
Parameter 'indent' is not assigned and could be declared final 208
Local variable 'lines' could be declared final 210
Local variable 'ind' could be declared final 212
Local variable 'plainLines' could be declared final 213
Parameter 'indentSize' is not assigned and could be declared final 230
Parameter 'lines' is not assigned and could be declared final 230
Parameter 'lineLength' is not assigned and could be declared final 230
Parameter 'line' is not assigned and could be declared final 230
Local variable 'lineIndent' could be declared final 232
Local variable 'buf' could be declared final 233
Local variable 'tokens' could be declared final 234
Local variable 'token' could be declared final 237
Local variable 'c' could be declared final 253
Avoid variables with short names like c 253
Parameter 'line' is not assigned and could be declared final 277