PMD Results

The following document contains the results of PMD 5.0.5.

Files

org/andromda/maven/plugin/modelarchiver/BaseArchiveMojo.java

Violation Line
Abstract classes should be named AbstractXXX 22–166
Found non-transient, non-static member. Please mark as transient or provide accessors. 61
Field replaceExtensions has the same name as a method 86
Parameter 'pExtension' is not assigned and could be declared final 94
Parameter 'pPath' is not assigned and could be declared final 94
Local variable 'lFileIter' could be declared final 96
Avoid reassigning parameters such as 'pattern' 110
Avoid reassigning parameters such as 'pattern' 110
Parameter 'pReplacementExtensions' is not assigned and could be declared final 130
Parameter 'pFile' is not assigned and could be declared final 130
Local variable 'replacementExtensions' could be declared final 132
Avoid if (x != y) ..; else ..; 132
Local variable 'replacementExtension' could be declared final 136
Parameter 'newFile' is not assigned and could be declared final 159

org/andromda/maven/plugin/modelarchiver/EmxArchiverMojo.java

Violation Line
Found non-transient, non-static member. Please mark as transient or provide accessors. 28
Found non-transient, non-static member. Please mark as transient or provide accessors. 37
Avoid instantiating new objects inside loops 79–80
A catch statement should never catch throwable since it includes errors. 95

org/andromda/maven/plugin/modelarchiver/EpxArchiverMojo.java

Violation Line
Found non-transient, non-static member. Please mark as transient or provide accessors. 28
Found non-transient, non-static member. Please mark as transient or provide accessors. 37
Avoid instantiating new objects inside loops 80–81
A catch statement should never catch throwable since it includes errors. 95

org/andromda/maven/plugin/modelarchiver/Uml2ArchiverMojo.java

Violation Line
Found non-transient, non-static member. Please mark as transient or provide accessors. 28
Found non-transient, non-static member. Please mark as transient or provide accessors. 37
Avoid instantiating new objects inside loops 80–81
A catch statement should never catch throwable since it includes errors. 95

org/andromda/maven/plugin/modelarchiver/UmlArchiverMojo.java

Violation Line
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. 36
Avoid instantiating new objects inside loops 79–80
A catch statement should never catch throwable since it includes errors. 94

org/andromda/maven/plugin/modelarchiver/UmlLibraryArchiverMojo.java

Violation Line
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. 36
Avoid instantiating new objects inside loops 79–80
A catch statement should never catch throwable since it includes errors. 94

org/andromda/maven/plugin/modelarchiver/UmlProfileArchiverMojo.java

Violation Line
Found non-transient, non-static member. Please mark as transient or provide accessors. 28
Found non-transient, non-static member. Please mark as transient or provide accessors. 37
Avoid instantiating new objects inside loops 80–81
A catch statement should never catch throwable since it includes errors. 95

org/andromda/maven/plugin/modelarchiver/XmiZipArchiverMojo.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. 47
Found non-transient, non-static member. Please mark as transient or provide accessors. 56
Found non-transient, non-static member. Please mark as transient or provide accessors. 64
Found non-transient, non-static member. Please mark as transient or provide accessors. 71
Deeply nested if..then statements are hard to read 126–141
Avoid instantiating new objects inside loops 129–130
Avoid instantiating new objects inside loops 139
Avoid if (x != y) ..; else ..; 148–156
A catch statement should never catch throwable since it includes errors. 158
A catch statement should never catch throwable since it includes errors. 185
An instanceof check is being performed on the caught exception. Create a separate catch clause for this exception type. 187
An instanceof check is being performed on the caught exception. Create a separate catch clause for this exception type. 187

org/andromda/maven/plugin/modelarchiver/XmlArchiverMojo.java

Violation Line
Found non-transient, non-static member. Please mark as transient or provide accessors. 29
Avoid instantiating new objects inside loops 72–73
A catch statement should never catch throwable since it includes errors. 103