PMD Results

The following document contains the results of PMD 5.0.2.

Files

org/andromda/maven/plugin/AbstractAndroMDAMojo.java

Violation Line
Found non-transient, non-static member. Please mark as transient or provide accessors. 37
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. 77
Found non-transient, non-static member. Please mark as transient or provide accessors. 86
Found non-transient, non-static member. Please mark as transient or provide accessors. 93
Line too long 123
A catch statement should never catch throwable since it includes errors. 131
An instanceof check is being performed on the caught exception. Create a separate catch clause for this exception type. 135
An instanceof check is being performed on the caught exception. Create a separate catch clause for this exception type. 135
A method/constructor shouldnt explicitly throw java.lang.Exception 151
A method/constructor shouldnt explicitly throw java.lang.Exception 151
Parameter 'configurationUri' is not assigned and could be declared final 156
Parameter 'project' is not assigned and could be declared final 164
Parameter 'settings' is not assigned and could be declared final 174
Parameter 'propertyFiles' is not assigned and could be declared final 184

org/andromda/maven/plugin/AndroMDAMojo.java

Violation Line
Found non-transient, non-static member. Please mark as transient or provide accessors. 30
Private field 'skipProcessing' could be made final; it is only initialized in the declaration or constructor. 39
Found non-transient, non-static member. Please mark as transient or provide accessors. 39
Avoid using redundant field initializer for 'skipProcessing' 39
Found non-transient, non-static member. Please mark as transient or provide accessors. 46
Found non-transient, non-static member. Please mark as transient or provide accessors. 54
Parameter 'lastModifiedCheck' is not assigned and could be declared final 59
Parameter 'buildSourceDirectory' is not assigned and could be declared final 67
Parameter 'modelOutputHistory' is not assigned and could be declared final 75
Local variable 'date' could be declared final 96
Deeply nested if..then statements are hard to read 99 - 102
Avoid if (x != y) ..; else ..; 107
Local variable 'repositoryCount' could be declared final 131
Local variable 'newDate' could be declared final 142

org/andromda/maven/plugin/MagicDrawExportEMFXMIMojo.java

Violation Line
Found non-transient, non-static member. Please mark as transient or provide accessors. 32
This final field could be made static 32
Found non-transient, non-static member. Please mark as transient or provide accessors. 39
Local variable 'repositoryCount' could be declared final 55
Local variable 'uris' could be declared final 68
A catch statement should never catch throwable since it includes errors. 79
The method exportFile() has an NPath complexity of 960 85 - 180
A method/constructor shouldnt explicitly throw java.lang.Exception 85
Parameter 'dest' is not assigned and could be declared final 85
A method/constructor shouldnt explicitly throw java.lang.Exception 85
Avoid declaring a variable if it is unreferenced before a possible exit point. 87
Avoid declaring a variable if it is unreferenced before a possible exit point. 88
A method should have only one exit point, and that should be the last statement in the method 94
New exception is thrown in catch block, original stack trace may be lost 104
New exception is thrown in catch block, original stack trace may be lost 115
New exception is thrown in catch block, original stack trace may be lost 129
Local variable 'destFile' could be declared final 132
Avoid if (x != y) ..; else ..; 133 - 150
When instantiating a SimpleDateFormat object, specify a Locale 141
Local variable 'formatter' could be declared final 141
Local variable 'command' could be declared final 157 - 160
Local variable 'process' could be declared final 161
Avoid variables with short names like is 165
Local variable 'is' could be declared final 165
Avoid assignments in operands 168
Avoid instantiating new objects inside loops 170
Local variable 'err' could be declared final 174
Avoid using redundant field initializer for 'checkedMagicDraw' 185
Found non-transient, non-static member. Please mark as transient or provide accessors. 185
Found non-transient, non-static member. Please mark as transient or provide accessors. 190
Found non-transient, non-static member. Please mark as transient or provide accessors. 195
Local variable 'home' could be declared final 211
Avoid variables with short names like os 218
Local variable 'os' could be declared final 218
Local variable 'pluginName115' could be declared final 225
Local variable 'pluginName116' could be declared final 226
Local variable 'pluginName15' could be declared final 227