PMD Results

The following document contains the results of PMD 5.0.5.

Files

org/andromda/repositories/emf/uml22/EMFUML2RepositoryFacade.java

Violation Line
A high number of imports can indicate a high degree of coupling within an object. 1–413
Variables that are final and static should be all capitals, 'logger' is not all capitals. 50
Avoid really long methods. 58–247
Comment is too large: Line too long 69
Local variable 'packageRegistry' could be declared final 70
Comment is too large: Line too long 140
Comment is too large: Line too long 141
Comment is too large: Line too long 143
Avoid if (x != y) ..; else ..; 146–168
Local variable 'path' could be declared final 149
Local variable 'uri' could be declared final 150
The String literal "" appears 7 times in this file; the first occurrence is on line 157 157
The String literal "profiles" appears 5 times in this file; the first occurrence is on line 163 163
Local variable 'uriMap' could be declared final 170
Local variable 'path' could be declared final 200
Local variable 'uri' could be declared final 201
Comment is too large: Line too long 202
Comment is too large: Line too long 203
Local variable 'path' could be declared final 211
Local variable 'uri' could be declared final 212
Comment is too large: Line too long 213
Local variable 'm2repository' could be declared final 222
Local variable 'uri' could be declared final 225
Parameter 'registry' is not assigned and could be declared final 257
Parameter 'ePackageClassName' is not assigned and could be declared final 274
Parameter 'registry' is not assigned and could be declared final 274
Local variable 'ePackageClass' could be declared final 281
Local variable 'nsURI' could be declared final 285
Local variable 'eInstance' could be declared final 286
Avoid catching generic exceptions such as NullPointerException, RuntimeException, Exception in try-catch block 295
Avoid empty catch blocks 295–298
Parameter 'modelUris' is not assigned and could be declared final 307
Parameter 'moduleSearchPaths' is not assigned and could be declared final 308
Local variable 'modelResource' could be declared final 324
Local variable 'modelPackage' could be declared final 334–335
Comment is too large: Line too long 342
Comment is too large: Line too long 344
Local variable 'pkg' could be declared final 352
Parameter 'uri' is not assigned and could be declared final 374
A catch statement should never catch throwable since it includes errors. 383
Local variable 'resource' could be declared final 390
Local variable 'uriModel' could be declared final 391
Avoid if (x != y) ..; else ..; 394–401
Assigning an Object to null is a code smell. Consider refactoring. 400

org/andromda/repositories/emf/uml22/EMXProxyResolvingResourceSet.java

Violation Line
Variables that are final and static should be all capitals, 'logger' is not all capitals. 25
Parameter 'loadOnDemand' is not assigned and could be declared final 34
The method getEObject() has an NPath complexity of 276 34–105
Parameter 'uri' is not assigned and could be declared final 34
A method should have only one exit point, and that should be the last statement in the method 41
Local variable 'now' could be declared final 43
Comment is too large: Line too long 46
Avoid catching generic exceptions such as NullPointerException, RuntimeException, Exception in try-catch block 55
The String literal " " appears 4 times in this file; the first occurrence is on line 64 64
Avoid if (x != y) ..; else ..; 65
Avoid if (x != y) ..; else ..; 70
Local variable 'separatorIndex' could be declared final 82
Avoid catching generic exceptions such as NullPointerException, RuntimeException, Exception in try-catch block 92
Deeply nested if..then statements are hard to read 96–99