PMD Results

The following document contains the results of PMD 5.0.5.

Files

org/andromda/maven/plugin/bootstrap/AndroMDAMojo.java

Violation Line
Found non-transient, non-static member. Please mark as transient or provide accessors. 42
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. 71
Found non-transient, non-static member. Please mark as transient or provide accessors. 79
Found non-transient, non-static member. Please mark as transient or provide accessors. 96
Found non-transient, non-static member. Please mark as transient or provide accessors. 105
Found non-transient, non-static member. Please mark as transient or provide accessors. 112
Comment is too large: Line too long 137
Local variable 'repositoryCount' could be declared final 139
Deeply nested if..then statements are hard to read 143–158
A catch statement should never catch throwable since it includes errors. 178
An instanceof check is being performed on the caught exception. Create a separate catch clause for this exception type. 182
An instanceof check is being performed on the caught exception. Create a separate catch clause for this exception type. 182
Parameter 'configurationUri' is not assigned and could be declared final 193
Parameter 'project' is not assigned and could be declared final 201
Parameter 'settings' is not assigned and could be declared final 211
Parameter 'propertyFiles' is not assigned and could be declared final 221

org/andromda/maven/plugin/bootstrap/TranslationLibraryTestFormatter.java

Violation Line
Found non-transient, non-static member. Please mark as transient or provide accessors. 25
Found non-transient, non-static member. Please mark as transient or provide accessors. 30
Private field 'report' could be made final; it is only initialized in the declaration or constructor. 30
Found non-transient, non-static member. Please mark as transient or provide accessors. 35
Private field 'reportWriter' could be made final; it is only initialized in the declaration or constructor. 35
Avoid using redundant field initializer for 'numberOfErrors' 60
Found non-transient, non-static member. Please mark as transient or provide accessors. 60
Parameter 'test' is not assigned and could be declared final 66
Parameter 'throwable' is not assigned and could be declared final 67
Found non-transient, non-static member. Please mark as transient or provide accessors. 78
Avoid using redundant field initializer for 'numberOfFailures' 78
Parameter 'test' is not assigned and could be declared final 84
Parameter 'failure' is not assigned and could be declared final 85
Parameter 'test' is not assigned and could be declared final 96
Document empty method 97–98
Found non-transient, non-static member. Please mark as transient or provide accessors. 103
Avoid using redundant field initializer for 'numberOfTests' 103
Parameter 'test' is not assigned and could be declared final 108
Avoid using redundant field initializer for 'startTime' 116
Found non-transient, non-static member. Please mark as transient or provide accessors. 116
Variables that are final and static should be all capitals, 'newLine' is not all capitals. 121
Parameter 'test' is not assigned and could be declared final 145
Parameter 'throwable' is not assigned and could be declared final 146
Private field 'failures' could be made final; it is only initialized in the declaration or constructor. 153
Found non-transient, non-static member. Please mark as transient or provide accessors. 153
Use explicit scoping instead of the default package private level 161–207
Useless parentheses. 163
StringBuffer constructor is initialized with size 16, but has at least 40 characters appended. 164
No need to call String.valueOf to append to a string. 164
StringBuffer (or StringBuilder).append is called 2 consecutive times with literal Strings. Use a single append with a single combined String. 165
StringBuffer (or StringBuilder).append is called 2 consecutive times with literal Strings. Use a single append with a single combined String. 166
Avoid throwing raw exception types. 203
Use explicit scoping instead of the default package private level 214–215
Found non-transient, non-static member. Please mark as transient or provide accessors. 215
Use explicit scoping instead of the default package private level 216
Found non-transient, non-static member. Please mark as transient or provide accessors. 216

org/andromda/maven/plugin/bootstrap/TranslationLibraryTestMojo.java

Violation Line
Found non-transient, non-static member. Please mark as transient or provide accessors. 36
Comment is too large: Line too long 39
Found non-transient, non-static member. Please mark as transient or provide accessors. 44
Found non-transient, non-static member. Please mark as transient or provide accessors. 51
Found non-transient, non-static member. Please mark as transient or provide accessors. 60
Found non-transient, non-static member. Please mark as transient or provide accessors. 68
Found non-transient, non-static member. Please mark as transient or provide accessors. 75
Found non-transient, non-static member. Please mark as transient or provide accessors. 80
Found non-transient, non-static member. Please mark as transient or provide accessors. 90
Found non-transient, non-static member. Please mark as transient or provide accessors. 97
Found non-transient, non-static member. Please mark as transient or provide accessors. 106
Found non-transient, non-static member. Please mark as transient or provide accessors. 113
Found non-transient, non-static member. Please mark as transient or provide accessors. 120
Comment is too large: Line too long 123
Found non-transient, non-static member. Please mark as transient or provide accessors. 127
Found non-transient, non-static member. Please mark as transient or provide accessors. 134
Avoid if (x != y) ..; else ..; 142–206
A catch statement should never catch throwable since it includes errors. 183
An instanceof check is being performed on the caught exception. Create a separate catch clause for this exception type. 185

org/andromda/maven/plugin/bootstrap/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 NCSS line count of 110 52–209
The method execute() has an NPath complexity of 3000 52–209
Avoid really long methods. 52–209
Local variable 'sb' could be declared final 66
Avoid variables with short names like sb 66
The String literal "" appears 26 times in this file; the first occurrence is on line 69 69
Parameter 'repeat' is not assigned and could be declared final 220
Parameter 'str' is not assigned and could be declared final 220
Local variable 'buffer' could be declared final 222
Parameter 'indent' is not assigned and could be declared final 240
Avoid variables with short names like sb 240
Parameter 'description' is not assigned and could be declared final 240
Parameter 'sb' is not assigned and could be declared final 240
Local variable 'it' could be declared final 242
Parameter 'lineLength' is not assigned and could be declared final 258
Parameter 'text' is not assigned and could be declared final 258
Parameter 'indentSize' is not assigned and could be declared final 258
Parameter 'indent' is not assigned and could be declared final 258
Local variable 'lines' could be declared final 260
Local variable 'ind' could be declared final 262
Local variable 'plainLines' could be declared final 263
Parameter 'lines' is not assigned and could be declared final 280
Parameter 'line' is not assigned and could be declared final 280
Parameter 'lineLength' is not assigned and could be declared final 280
Parameter 'indentSize' is not assigned and could be declared final 280
Local variable 'lineIndent' could be declared final 282
Local variable 'buf' could be declared final 283
Local variable 'tokens' could be declared final 284
Local variable 'token' could be declared final 287
Avoid variables with short names like c 303
Local variable 'c' could be declared final 303
Parameter 'line' is not assigned and could be declared final 327