PMD Results

The following document contains the results of PMD 5.0.2.

Files

org/andromda/metafacades/emf/uml22/ActionFacadeLogicImpl.java

Violation Line
Assigning an Object to null is a code smell. Consider refactoring. 37
Assigning an Object to null is a code smell. Consider refactoring. 47

org/andromda/metafacades/emf/uml22/ActionStateFacadeLogicImpl.java

Violation Line
Local variable 'nextNode' could be declared final 41
A method should have only one exit point, and that should be the last statement in the method 45

org/andromda/metafacades/emf/uml22/ActivityGraphFacadeLogicImpl.java

Violation Line
A method should have only one exit point, and that should be the last statement in the method 64

org/andromda/metafacades/emf/uml22/AssociationEndFacadeLogicImpl.java

Violation Line
Unnecessary wrapper object creation 177
Local variable 'type' could be declared final 245
Prefer StringBuffer over += for concatenating strings 253
Useless parentheses. 273
Useless parentheses. 282

org/andromda/metafacades/emf/uml22/AssociationEndImpl.java

Violation Line
A high number of imports can indicate a high degree of coupling within an object. 1 - 2480
This class has a bunch of public methods and attributes 1 - 2480
High amount of different objects as members denotes a high coupling 1 - 2480
Avoid really long classes. 55 - 2480
This class has too many methods, consider refactoring it. 57 - 2480
Use explicit scoping instead of the default package private level 66
Found non-transient, non-static member. Please mark as transient or provide accessors. 66
Local variable 'other' could be declared final 84
A method should have only one exit point, and that should be the last statement in the method 85
A method should have only one exit point, and that should be the last statement in the method 90
Method names should not contain underscores 557
Line too long 1051
Line too long 1364
Line too long 1508
Line too long 1546
Line too long 1558
Line too long 1637
Line too long 1649
Line too long 1702
Line too long 1726
Line too long 1754
Line too long 1771
Line too long 1787
Line too long 1903
Line too long 1920
Line too long 1929
Line too long 1938
Line too long 1947
Line too long 2044
Line too long 2053
Line too long 2205
Line too long 2213
Avoid using short method names 2223
Line too long 2245
Line too long 2254
Line too long 2370
Parameter 'value' is not assigned and could be declared final 2396
Parameter 'value' is not assigned and could be declared final 2416
Parameter 'value' is not assigned and could be declared final 2426
Line too long 2433
Parameter 'diagnostics' is not assigned and could be declared final 2436
Parameter 'context' is not assigned and could be declared final 2436
Line too long 2443
Parameter 'diagnostics' is not assigned and could be declared final 2446
Parameter 'context' is not assigned and could be declared final 2446
Line too long 2453
Parameter 'diagnostics' is not assigned and could be declared final 2456
Parameter 'context' is not assigned and could be declared final 2456
Parameter 'diagnostics' is not assigned and could be declared final 2466
Parameter 'context' is not assigned and could be declared final 2466
Line too long 2473
Parameter 'diagnostics' is not assigned and could be declared final 2476
Parameter 'context' is not assigned and could be declared final 2476

org/andromda/metafacades/emf/uml22/AssociationFacadeLogicImpl.java

Violation Line
Useless parentheses. 75
This statement may have some unnecessary parentheses 75
A method should have only one exit point, and that should be the last statement in the method 107
A method should have only one exit point, and that should be the last statement in the method 120

org/andromda/metafacades/emf/uml22/AttributeFacadeLogicImpl.java

Violation Line
Prefer StringBuffer over += for concatenating strings 83
Prefer StringBuffer over += for concatenating strings 92
Useless parentheses. 127
Useless parentheses. 165
Useless parentheses. 178
Prefer StringBuffer over += for concatenating strings 182
The method handleGetGetterSetterTypeName() has an NPath complexity of 243 230 - 309
Prefer StringBuffer over += for concatenating strings 279
These nested if statements could be combined 291 - 301
Assigning an Object to null is a code smell. Consider refactoring. 363
Avoid reassigning parameters such as 'name' 484

org/andromda/metafacades/emf/uml22/AttributeImpl.java

Violation Line
This class has a bunch of public methods and attributes 1 - 2487
High amount of different objects as members denotes a high coupling 1 - 2487
A high number of imports can indicate a high degree of coupling within an object. 1 - 2487
Avoid really long classes. 56 - 2487
This class has too many methods, consider refactoring it. 58 - 2487
Found non-transient, non-static member. Please mark as transient or provide accessors. 67
Use explicit scoping instead of the default package private level 67
Local variable 'other' could be declared final 85
A method should have only one exit point, and that should be the last statement in the method 86
A method should have only one exit point, and that should be the last statement in the method 91
Method names should not contain underscores 562
Line too long 1056
Line too long 1369
Line too long 1505
Line too long 1545
Line too long 1557
Line too long 1613
Line too long 1640
Line too long 1652
Line too long 1706
Line too long 1733
Line too long 1761
Line too long 1778
Line too long 1794
Line too long 1910
Line too long 1928
Line too long 1937
Line too long 1946
Line too long 1955
Line too long 2052
Line too long 2061
Line too long 2213
Line too long 2221
Avoid using short method names 2231
Line too long 2253
Line too long 2262
Line too long 2377
Parameter 'value' is not assigned and could be declared final 2403
Parameter 'value' is not assigned and could be declared final 2423
Parameter 'value' is not assigned and could be declared final 2433
Line too long 2440
Parameter 'diagnostics' is not assigned and could be declared final 2443
Parameter 'context' is not assigned and could be declared final 2443
Line too long 2450
Parameter 'diagnostics' is not assigned and could be declared final 2453
Parameter 'context' is not assigned and could be declared final 2453
Line too long 2460
Parameter 'context' is not assigned and could be declared final 2463
Parameter 'diagnostics' is not assigned and could be declared final 2463
Parameter 'context' is not assigned and could be declared final 2473
Parameter 'diagnostics' is not assigned and could be declared final 2473
Line too long 2480
Parameter 'context' is not assigned and could be declared final 2483
Parameter 'diagnostics' is not assigned and could be declared final 2483

org/andromda/metafacades/emf/uml22/AttributeLinkFacadeLogicImpl.java

Violation Line
Assigning an Object to null is a code smell. Consider refactoring. 55

org/andromda/metafacades/emf/uml22/AttributeLinkImpl.java

Violation Line
This class has a bunch of public methods and attributes 1 - 709
Use explicit scoping instead of the default package private level 43
Found non-transient, non-static member. Please mark as transient or provide accessors. 43
A method should have only one exit point, and that should be the last statement in the method 61
Line too long 130

org/andromda/metafacades/emf/uml22/CallEventFacadeLogicImpl.java

Violation Line
Assigning an Object to null is a code smell. Consider refactoring. 40

org/andromda/metafacades/emf/uml22/ClassifierFacadeLogicImpl.java

Violation Line
A high number of imports can indicate a high degree of coupling within an object. 1 - 1355
Avoid really long classes. 58 - 1355
This class has too many methods, consider refactoring it. 60 - 1355
Avoid catching generic exceptions such as NullPointerException, RuntimeException, Exception in try-catch block 93
Local variable 'attribute' could be declared final 143
Avoid excessively long variable names like allRequiredConstructorParameters 193
Local variable 'parent' could be declared final 196
Line too long 278
Avoid catching generic exceptions such as NullPointerException, RuntimeException, Exception in try-catch block 344
Useless parentheses. 546
Useless parentheses. 546
Local variable 'generalization' could be declared final 782
Local variable 'dependency' could be declared final 846
Local variable 'supplier' could be declared final 849
Prefer StringBuffer over += for concatenating strings 958
Assigning an Object to null is a code smell. Consider refactoring. 968
Local variable 'operation' could be declared final 1019
Local variable 'dependency' could be declared final 1045
Avoid using if statements without curly braces 1069
Avoid using if statements without curly braces 1100
Avoid catching generic exceptions such as NullPointerException, RuntimeException, Exception in try-catch block 1158
A method should have only one exit point, and that should be the last statement in the method 1217
Line too long 1223
A method should have only one exit point, and that should be the last statement in the method 1238
Local variable 'param' could be declared final 1241
A method should have only one exit point, and that should be the last statement in the method 1246
A method should have only one exit point, and that should be the last statement in the method 1260
Assigning an Object to null is a code smell. Consider refactoring. 1315
Local variable 'parameter' could be declared final 1341

org/andromda/metafacades/emf/uml22/EntityAssociationEndLogicImpl.java

Violation Line
Assigning an Object to null is a code smell. Consider refactoring. 73
Assigning an Object to null is a code smell. Consider refactoring. 76
Deeply nested if..then statements are hard to read 152 - 159
Deeply nested if..then statements are hard to read 166 - 171
A catch statement should never catch throwable since it includes errors. 197

org/andromda/metafacades/emf/uml22/EntityAssociationLogicImpl.java

Violation Line
No need to check for null before an instanceof 53
Assigning an Object to null is a code smell. Consider refactoring. 58

org/andromda/metafacades/emf/uml22/EntityAttributeLogicImpl.java

Violation Line
Assigning an Object to null is a code smell. Consider refactoring. 74
Unnecessary use of fully qualified name 'org.andromda.metafacades.uml.TypeMappings' due to existing import 'org.andromda.metafacades.uml.TypeMappings' 88
Unnecessary use of fully qualified name 'org.andromda.metafacades.uml.TypeMappings' due to existing import 'org.andromda.metafacades.uml.TypeMappings' 117
Deeply nested if..then statements are hard to read 137 - 144
Deeply nested if..then statements are hard to read 147 - 152
A catch statement should never catch throwable since it includes errors. 209

org/andromda/metafacades/emf/uml22/EntityLogicImpl.java

Violation Line
A high number of imports can indicate a high degree of coupling within an object. 1 - 1159
Avoid really long classes. 47 - 1159
This class has too many methods, consider refactoring it. 49 - 1159
Local variable 'operation' could be declared final 122
Deeply nested if..then statements are hard to read 223 - 226
Deeply nested if..then statements are hard to read 227 - 230
Deeply nested if..then statements are hard to read 231 - 234
Deeply nested if..then statements are hard to read 240 - 244
Assigning an Object to null is a code smell. Consider refactoring. 278
Local variable 'attribute' could be declared final 325
Deeply nested if..then statements are hard to read 331 - 334
No need to check for null before an instanceof 647
Use bitwise inversion to invert boolean values 649
No need to check for null before an instanceof 674
Use bitwise inversion to invert boolean values 676
Use bitwise inversion to invert boolean values 680
Use bitwise inversion to invert boolean values 773
Do not use the short type 795
Unnecessary wrapper object creation 805
Local variable 'identifier' could be declared final 853
Unnecessary wrapper object creation 948
Local variable 'identifiers' could be declared final 994
Avoid if (x != y) ..; else ..; 1060 - 1079
A method should have only one exit point, and that should be the last statement in the method 1092 - 1093
Avoid if (x != y) ..; else ..; 1098 - 1105
A method should have only one exit point, and that should be the last statement in the method 1100
A method should have only one exit point, and that should be the last statement in the method 1117
Avoid if (x != y) ..; else ..; 1120 - 1139
Local variable 'facade' could be declared final 1122
A method should have only one exit point, and that should be the last statement in the method 1125
A method should have only one exit point, and that should be the last statement in the method 1129
A method should have only one exit point, and that should be the last statement in the method 1133

org/andromda/metafacades/emf/uml22/EnumerationFacadeLogicImpl.java

Violation Line
Useless parentheses. 79
Useless parentheses. 80

org/andromda/metafacades/emf/uml22/EnumerationLiteralFacadeLogicImpl.java

Violation Line
Prefer StringBuffer over += for concatenating strings 98

org/andromda/metafacades/emf/uml22/EventFacadeLogicImpl.java

Violation Line
A method should have only one exit point, and that should be the last statement in the method 41
A method should have only one exit point, and that should be the last statement in the method 64

org/andromda/metafacades/emf/uml22/FrontEndActionLogicImpl.java

Violation Line
The class 'FrontEndActionLogicImpl' has a Cyclomatic Complexity of 4 (Highest = 26). 39 - 513
Avoid using redundant field initializer for 'actionStates' 178
Found non-transient, non-static member. Please mark as transient or provide accessors. 178
Found non-transient, non-static member. Please mark as transient or provide accessors. 184
Avoid using redundant field initializer for 'actionForwards' 184
Avoid using redundant field initializer for 'decisionTransitions' 190
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
Avoid using redundant field initializer for 'transitions' 195
The method handleGetFormFields() has an NPath complexity of 6480 358 - 512
The method 'handleGetFormFields' has a Cyclomatic Complexity of 26. 358 - 512
Avoid really long methods. 358 - 512
Local variable 'parameter' could be declared final 502

org/andromda/metafacades/emf/uml22/FrontEndActionStateLogicImpl.java

Violation Line
Local variable 'transition' could be declared final 81
Parameter 'object' is not assigned and could be declared final 116
Consider simply returning the value vs storing it in local variable 'operation' 118
Parameter 'object' is not assigned and could be declared final 148
Consider simply returning the value vs storing it in local variable 'operation' 150

org/andromda/metafacades/emf/uml22/FrontEndActivityGraphLogicImpl.java

Violation Line
Assigning an Object to null is a code smell. Consider refactoring. 77

org/andromda/metafacades/emf/uml22/FrontEndControllerOperationLogicImpl.java

Violation Line
Avoid if (x != y) ..; else ..; 60
These nested if statements could be combined 131 - 136
Deeply nested if..then statements are hard to read 131 - 136
Local variable 'parameter' could be declared final 145
Deeply nested if..then statements are hard to read 203 - 206
Deeply nested if..then statements are hard to read 220 - 241
Avoid if (x != y) ..; else ..; 263
Avoid if (x != y) ..; else ..; 278
Use bitwise inversion to invert boolean values 284
Use bitwise inversion to invert boolean values 286

org/andromda/metafacades/emf/uml22/FrontEndEventLogicImpl.java

Violation Line
A method should have only one exit point, and that should be the last statement in the method 55
These nested if statements could be combined 61 - 64
A method should have only one exit point, and that should be the last statement in the method 63
Useless parentheses. 78
Assigning an Object to null is a code smell. Consider refactoring. 78

org/andromda/metafacades/emf/uml22/FrontEndForwardLogicImpl.java

Violation Line
Assigning an Object to null is a code smell. Consider refactoring. 71
Avoid using redundant field initializer for 'actionStates' 164
Found non-transient, non-static member. Please mark as transient or provide accessors. 164
Assigning an Object to null is a code smell. Consider refactoring. 231
Line too long 269
Deeply nested if..then statements are hard to read 290 - 298
Local variable 'incomingForward' could be declared final 293

org/andromda/metafacades/emf/uml22/FrontEndParameterLogicImpl.java

Violation Line
Deeply nested if..then statements are hard to read 91 - 94
Use bitwise inversion to invert boolean values 147
Local variable 'attribute' could be declared final 238

org/andromda/metafacades/emf/uml22/FrontEndPseudostateLogicImpl.java

Violation Line
Deeply nested if..then statements are hard to read 71 - 74

org/andromda/metafacades/emf/uml22/FrontEndUseCaseLogicImpl.java

Violation Line
Local variable 'useCase' could be declared final 98
Local variable 'user' could be declared final 157
Deeply nested if..then statements are hard to read 257 - 260

org/andromda/metafacades/emf/uml22/FrontEndViewLogicImpl.java

Violation Line
Assigning an Object to null is a code smell. Consider refactoring. 82
Local variable 'action' could be declared final 147
Local variable 'action' could be declared final 162

org/andromda/metafacades/emf/uml22/GeneralizableElementFacadeLogicImpl.java

Violation Line
Local variable 'general' could be declared final 89
Local variable 'element' could be declared final 116
Local variable 'element' could be declared final 131

org/andromda/metafacades/emf/uml22/GuardFacadeLogicImpl.java

Violation Line
A method should have only one exit point, and that should be the last statement in the method 59

org/andromda/metafacades/emf/uml22/InstanceFacadeLogicImpl.java

Violation Line
Found non-transient, non-static member. Please mark as transient or provide accessors. 31
Avoid using redundant field initializer for 'value' 31
Found non-transient, non-static member. Please mark as transient or provide accessors. 32
Avoid using redundant field initializer for 'valueSet' 32
Useless parentheses. 78

org/andromda/metafacades/emf/uml22/LinkEndFacadeLogicImpl.java

Violation Line
Useless parentheses. 36
Assigning an Object to null is a code smell. Consider refactoring. 36

org/andromda/metafacades/emf/uml22/LinkEndImpl.java

Violation Line
This class has a bunch of public methods and attributes 1 - 698
Found non-transient, non-static member. Please mark as transient or provide accessors. 38
Use explicit scoping instead of the default package private level 38
A method should have only one exit point, and that should be the last statement in the method 60
Line too long 125

org/andromda/metafacades/emf/uml22/LinkInstanceImpl.java

Violation Line
High amount of different objects as members denotes a high coupling 1 - 1241
A high number of imports can indicate a high degree of coupling within an object. 1 - 1241
This class has a bunch of public methods and attributes 1 - 1241
Avoid really long classes. 43 - 1241
This class has too many methods, consider refactoring it. 44 - 1241
Use explicit scoping instead of the default package private level 48
Found non-transient, non-static member. Please mark as transient or provide accessors. 48
A method should have only one exit point, and that should be the last statement in the method 70
Method names should not contain underscores 335
Method names should not contain underscores 343
Line too long 458
Line too long 476
Line too long 492
Line too long 516
Parameter 'notification' is not assigned and could be declared final 968
Line too long 974
Line too long 982
Line too long 990
Line too long 999
Line too long 1007
Parameter 'qualifiedName' is not assigned and could be declared final 1074
Parameter 'stereotype' is not assigned and could be declared final 1074
Parameter 'stereotype' is not assigned and could be declared final 1082
Parameter 'class1' is not assigned and could be declared final 1098
Parameter 'qualifiedName' is not assigned and could be declared final 1106
Parameter 'eClass' is not assigned and could be declared final 1130
Parameter 'stereotype' is not assigned and could be declared final 1138
Parameter 'eClass' is not assigned and could be declared final 1162
Parameter 'stereotype' is not assigned and could be declared final 1170
Parameter 'stereotype' is not assigned and could be declared final 1178
Parameter 'stereotype' is not assigned and could be declared final 1186
Parameter 'parameter' is not assigned and could be declared final 1202
Parameter 'value' is not assigned and could be declared final 1218
Parameter 'arg1' is not assigned and could be declared final 1227
Parameter 'arg0' is not assigned and could be declared final 1227
Line too long 1234
Parameter 'context' is not assigned and could be declared final 1237
Parameter 'diagnostics' is not assigned and could be declared final 1237

org/andromda/metafacades/emf/uml22/ManageableEntityLogicImpl.java

Violation Line
The class 'ManageableEntityLogicImpl' has a Cyclomatic Complexity of 3 (Highest = 20). 36 - 710
Prefer StringBuffer over += for concatenating strings 83
Local variable 'identifiers' could be declared final 262
The method createListWithManageableMembers() has an NPath complexity of 561 289 - 376
The method 'createListWithManageableMembers' has a Cyclomatic Complexity of 20. 289 - 376
Local variable 'attribute' could be declared final 293
Local variable 'associationEnd' could be declared final 317
Local variable 'identifier' could be declared final 339
Assigning an Object to null is a code smell. Consider refactoring. 393
These nested if statements could be combined 419 - 426
Useless parentheses. 419
Line too long 433
Line too long 451
Avoid if (x != y) ..; else ..; 467 - 482
Local variable 'facade' could be declared final 469
Unnecessary wrapper object creation 641
Unnecessary wrapper object creation 659

org/andromda/metafacades/emf/uml22/ModelElementFacadeLogicImpl.java

Violation Line
A high number of imports can indicate a high degree of coupling within an object. 1 - 1262
The class 'ModelElementFacadeLogicImpl' has a Cyclomatic Complexity of 3 (Highest = 28). 67 - 1262
Avoid really long classes. 67 - 1262
This class has too many methods, consider refactoring it. 69 - 1262
Use explicit scoping instead of the default package private level 74
Line too long 130
A method should have only one exit point, and that should be the last statement in the method 131
Avoid catching generic exceptions such as NullPointerException, RuntimeException, Exception in try-catch block 165
A method should have only one exit point, and that should be the last statement in the method 171
Local variable 'packageName' could be declared final 198
Avoid catching generic exceptions such as NullPointerException, RuntimeException, Exception in try-catch block 207
Unnecessary use of fully qualified name 'org.andromda.metafacades.uml.TypeMappings' due to existing import 'org.andromda.metafacades.uml.TypeMappings' 251
A catch statement should never catch throwable since it includes errors. 267
Avoid using if statements without curly braces 300 - 301
A method should have only one exit point, and that should be the last statement in the method 301
Local variable 'xmiResource' could be declared final 303
Local variable 'rtn' could be declared final 304
Assigning an Object to null is a code smell. Consider refactoring. 344
Line too long 389
Local variable 'constraint' could be declared final 511
Useless parentheses. 513
Useless parentheses. 514
Useless parentheses. 515
Useless parentheses. 516
Useless parentheses. 517
Avoid reassigning parameters such as 'name' 529
Avoid reassigning parameters such as 'lineLength' 578
StringBuffer constructor is initialized with size 16, but has at least 26 characters appended. 581
Local variable 'end' could be declared final 616
Avoid concatenating nonliterals in a StringBuffer/StringBuilder constructor or append(). 625
Local variable 'comment' could be declared final 647
Use explicit scoping instead of the default package private level 835 - 851
Useless parentheses. 838
Useless parentheses. 838
Line too long 942
Avoid reassigning parameters such as 'parameterName' 954
The method handleGetBindedFullyQualifiedName() has an NPath complexity of 23880 1103 - 1248
Avoid really long methods. 1103 - 1248
The method 'handleGetBindedFullyQualifiedName' has a Cyclomatic Complexity of 28. 1103 - 1248
Avoid excessively long variable names like metafacadeNamespaceScopeOperator 1105
No need to check for null before an instanceof 1114
Prefer StringBuffer over += for concatenating strings 1117
No need to check for null before an instanceof 1119
Prefer StringBuffer over += for concatenating strings 1121
No need to check for null before an instanceof 1129
Prefer StringBuffer over += for concatenating strings 1131
Prefer StringBuffer over += for concatenating strings 1136
Local variable 'templateParameterFacade' could be declared final 1189
Local variable 'parameter' could be declared final 1206
Avoid if (x != y) ..; else ..; 1211 - 1237
Local variable 'typeName' could be declared final 1213

org/andromda/metafacades/emf/uml22/ModelFacadeLogicImpl.java

Violation Line
Line too long 200
Useless parentheses. 217
Line too long 225
Useless parentheses. 302
Local variable 'facades' could be declared final 324
Local variable 'classes' could be declared final 325 - 327
Local variable 'metaClass' could be declared final 331
Local variable 'metafacade' could be declared final 333

org/andromda/metafacades/emf/uml22/ObjectInstanceImpl.java

Violation Line
High amount of different objects as members denotes a high coupling 1 - 1194
This class has a bunch of public methods and attributes 1 - 1194
A high number of imports can indicate a high degree of coupling within an object. 1 - 1194
Avoid really long classes. 43 - 1194
This class has too many methods, consider refactoring it. 44 - 1194
Use explicit scoping instead of the default package private level 48
Found non-transient, non-static member. Please mark as transient or provide accessors. 48
A method should have only one exit point, and that should be the last statement in the method 66
Method names should not contain underscores 318
Method names should not contain underscores 326
Line too long 424
Line too long 442
Line too long 458
Line too long 482
Line too long 927
Line too long 935
Line too long 943
Line too long 952
Line too long 960
Line too long 1187
Parameter 'diagnostics' is not assigned and could be declared final 1190
Parameter 'context' is not assigned and could be declared final 1190

org/andromda/metafacades/emf/uml22/OperationFacadeLogicImpl.java

Violation Line
This class has too many methods, consider refactoring it. 42 - 849
A method should have only one exit point, and that should be the last statement in the method 196
Avoid using if statements without curly braces 222
Use bitwise inversion to invert boolean values 245 - 246
Avoid reassigning parameters such as 'initialExceptions' 476
Avoid if (x != y) ..; else ..; 549 - 560
Avoid if (x != y) ..; else ..; 553 - 560
Prefer StringBuffer over += for concatenating strings 632
Useless parentheses. 643

org/andromda/metafacades/emf/uml22/PackageFacadeLogicImpl.java

Violation Line
Local variable 'element' could be declared final 91
Useless parentheses. 95

org/andromda/metafacades/emf/uml22/ParameterFacadeLogicImpl.java

Violation Line
Prefer StringBuffer over += for concatenating strings 57
Useless parentheses. 109
A method should have only one exit point, and that should be the last statement in the method 122
A method should have only one exit point, and that should be the last statement in the method 255
A method should have only one exit point, and that should be the last statement in the method 269
Prefer StringBuffer over += for concatenating strings 340
Prefer StringBuffer over += for concatenating strings 392
Unnecessary wrapper object creation 435
Avoid if (x != y) ..; else ..; 435
Useless parentheses. 435

org/andromda/metafacades/emf/uml22/PartitionFacadeLogicImpl.java

Violation Line
A method should have only one exit point, and that should be the last statement in the method 39

org/andromda/metafacades/emf/uml22/PseudostateFacadeLogicImpl.java

Violation Line
Useless parentheses. 34
Useless parentheses. 61
Useless parentheses. 70
Useless parentheses. 79
Useless parentheses. 88
Useless parentheses. 97
Useless parentheses. 106

org/andromda/metafacades/emf/uml22/RoleLogicImpl.java

Violation Line
Use bitwise inversion to invert boolean values 107

org/andromda/metafacades/emf/uml22/ServiceOperationLogicImpl.java

Violation Line
Useless parentheses. 147

org/andromda/metafacades/emf/uml22/StateMachineFacadeLogicImpl.java

Violation Line
Useless parentheses. 136
Useless parentheses. 137
Local variable 'region' could be declared final 185
Local variable 'object' could be declared final 199

org/andromda/metafacades/emf/uml22/TagDefinitionImpl.java

Violation Line
A high number of imports can indicate a high degree of coupling within an object. 1 - 1193
High amount of different objects as members denotes a high coupling 1 - 1193
This class has a bunch of public methods and attributes 1 - 1193
Avoid really long classes. 45 - 1193
This class has too many methods, consider refactoring it. 47 - 1193
Found non-transient, non-static member. Please mark as transient or provide accessors. 56
Assigning an Object to null is a code smell. Consider refactoring. 99
Useless parentheses. 99
Avoid if (x != y) ..; else ..; 99
Line too long 1023
Line too long 1033

org/andromda/metafacades/emf/uml22/TemplateParameterFacadeLogicImpl.java

Violation Line
Local variable 'param' could be declared final 89
Local variable 'element' could be declared final 90
Line too long 117
Avoid if (x != y) ..; else ..; 133 - 140
Prefer StringBuffer over += for concatenating strings 155
Prefer StringBuffer over += for concatenating strings 160
Local variable 'param' could be declared final 203
A method should have only one exit point, and that should be the last statement in the method 207
Parameter 'indent' is not assigned and could be declared final 229
Parameter 'indent' is not assigned and could be declared final 240
Parameter 'lineLength' is not assigned and could be declared final 240
Line too long 242
Line too long 251
Avoid reassigning parameters such as 'lineLength' 254
The method handleGetDocumentation() has an NPath complexity of 216 254 - 298
Parameter 'indent' is not assigned and could be declared final 254
Parameter 'htmlStyle' is not assigned and could be declared final 254
StringBuffer constructor is initialized with size 16, but has at least 26 characters appended. 256
Local variable 'commentString' could be declared final 277
These nested if statements could be combined 286 - 290
Local variable 'todoTag' could be declared final 288
Avoid concatenating nonliterals in a StringBuffer/StringBuilder constructor or append(). 289

org/andromda/metafacades/emf/uml22/UMLModelAccessFacade.java

Violation Line
Found non-transient, non-static member. Please mark as transient or provide accessors. 46
Local variable 'modelResource' could be declared final 68
Local variable 'modelResource' could be declared final 216

org/andromda/metafacades/emf/uml22/UmlUtilities.java

Violation Line
A high number of imports can indicate a high degree of coupling within an object. 1 - 1625
Avoid really long classes. 64 - 1625
All methods are static. Consider using Singleton instead. Alternatively, you could add a private constructor or make the class abstract to silence this warning. 65 - 1625
A method should have only one exit point, and that should be the last statement in the method 210
A method should have only one exit point, and that should be the last statement in the method 262
A method should have only one exit point, and that should be the last statement in the method 304
Avoid reassigning parameters such as 'text' 326
Avoid reassigning parameters such as 'text' 326
A method should have only one exit point, and that should be the last statement in the method 330
A method should have only one exit point, and that should be the last statement in the method 361
Local variable 'nextCandidate' could be declared final 371
Local variable 'property' could be declared final 431
Local variable 'parent' could be declared final 464
Avoid declaring a variable if it is unreferenced before a possible exit point. 500
A method should have only one exit point, and that should be the last statement in the method 503
Line too long 513
Local variable 'property' could be declared final 536
Line too long 548
A method should have only one exit point, and that should be the last statement in the method 649
A method should have only one exit point, and that should be the last statement in the method 688
Local variable 'stereotype' could be declared final 693
A method should have only one exit point, and that should be the last statement in the method 715
Local variable 'itStereo' could be declared final 733
A method should have only one exit point, and that should be the last statement in the method 737
Avoid really long methods. 775 - 876
A method should have only one exit point, and that should be the last statement in the method 780
Avoid instantiating new objects inside loops 810 - 812
Avoid instantiating new objects inside loops 822 - 824
Avoid instantiating new objects inside loops 844 - 850
Avoid instantiating new objects inside loops 851
Avoid instantiating new objects inside loops 861
Use explicit scoping instead of the default package private level 883 - 901
A method should have only one exit point, and that should be the last statement in the method 918
Local variable 'stereotype' could be declared final 926
A method should have only one exit point, and that should be the last statement in the method 953
Local variable 'stereotype' could be declared final 961
Use explicit scoping instead of the default package private level 983 - 990
A method should have only one exit point, and that should be the last statement in the method 1003
A method should have only one exit point, and that should be the last statement in the method 1037
A method should have only one exit point, and that should be the last statement in the method 1058
Local variable 'resource' could be declared final 1060
A method should have only one exit point, and that should be the last statement in the method 1096
A method should have only one exit point, and that should be the last statement in the method 1121
The method getPackageName() has an NPath complexity of 270 1153 - 1222
A method should have only one exit point, and that should be the last statement in the method 1160
A method should have only one exit point, and that should be the last statement in the method 1244
No need to check for null before an instanceof 1301
A method should have only one exit point, and that should be the last statement in the method 1303
Prefer StringBuffer over += for concatenating strings 1316
Avoid variables with short names like rs 1339
A method should have only one exit point, and that should be the last statement in the method 1344
A method should have only one exit point, and that should be the last statement in the method 1388
Line too long 1430
Useless parentheses. 1444
A method should have only one exit point, and that should be the last statement in the method 1549
All methods are static. Consider using Singleton instead. Alternatively, you could add a private constructor or make the class abstract to silence this warning. 1608 - 1613

org/andromda/metafacades/emf/uml22/ValueObjectAssociationEndLogicImpl.java

Violation Line
Unnecessary wrapper object creation 40
These nested if statements could be combined 46 - 49
Deeply nested if..then statements are hard to read 46 - 49

org/andromda/metafacades/emf/uml22/ValueObjectLogicImpl.java

Violation Line
Local variable 'stereotype' could be declared final 37
Local variable 'classifier' could be declared final 52