Parent Project
Overview
Project Documentation
The following document contains the results of PMD 5.0.2.
| Violation | Line |
|---|---|
| Parameter 'metaObject' is not assigned and could be declared final | 24 |
| Parameter 'context' is not assigned and could be declared final | 25 |
| Local variable 'transition' could be declared final | 40 |
| Violation | Line |
|---|---|
| Parameter 'metaObject' is not assigned and could be declared final | 19 |
| Parameter 'context' is not assigned and could be declared final | 20 |
| Violation | Line |
|---|---|
| Parameter 'metaObject' is not assigned and could be declared final | 25 |
| Parameter 'context' is not assigned and could be declared final | 26 |
| Parameter 'object' is not assigned and could be declared final | 40 |
| Parameter 'object' is not assigned and could be declared final | 57 |
| Violation | Line |
|---|---|
| Parameter 'metaObject' is not assigned and could be declared final | 24 |
| Parameter 'context' is not assigned and could be declared final | 25 |
| Violation | Line |
|---|---|
| Parameter 'metaObject' is not assigned and could be declared final | 20 |
| Parameter 'context' is not assigned and could be declared final | 21 |
| Violation | Line |
|---|---|
| Parameter 'metaObject' is not assigned and could be declared final | 22 |
| Parameter 'context' is not assigned and could be declared final | 23 |
| Violation | Line |
|---|---|
| Parameter 'metaObject' is not assigned and could be declared final | 37 |
| Unnecessary use of fully qualified name 'org.omg.uml.foundation.core.AssociationEnd' due to existing import 'org.omg.uml.foundation.core.AssociationEnd' | 37 |
| Parameter 'context' is not assigned and could be declared final | 38 |
| A method should have only one exit point, and that should be the last statement in the method | 62 |
| Local variable 'multiplicityRange' could be declared final | 185 |
| Local variable 'type' could be declared final | 353 |
| Prefer StringBuffer over += for concatenating strings | 361 |
| Local variable 'multiplicityRange' could be declared final | 405 |
| Local variable 'multiplicityRange' could be declared final | 432 |
| This call to String.startsWith can be rewritten using String.charAt(0) | 441 |
| Violation | Line |
|---|---|
| Parameter 'metaObject' is not assigned and could be declared final | 25 |
| Parameter 'context' is not assigned and could be declared final | 26 |
| This statement may have some unnecessary parentheses | 96 |
| Useless parentheses. | 96 |
| Local variable 'ends' could be declared final | 114 |
| Avoid if (x != y) ..; else ..; | 115 - 122 |
| A method should have only one exit point, and that should be the last statement in the method | 117 |
| Local variable 'ends' could be declared final | 131 |
| A method should have only one exit point, and that should be the last statement in the method | 134 |
| Violation | Line |
|---|---|
| Parameter 'metaObject' is not assigned and could be declared final | 37 |
| Parameter 'context' is not assigned and could be declared final | 38 |
| Local variable 'typeName' could be declared final | 84 |
| Prefer StringBuffer over += for concatenating strings | 87 |
| Avoid using if statements without curly braces | 95 |
| Avoid reassigning parameters such as 'name' | 158 |
| Parameter 'follow' is not assigned and could be declared final | 159 |
| Local variable 'lower' could be declared final | 181 |
| Local variable 'multiplicityRange' could be declared final | 200 |
| Local variable 'multiplicityRange' could be declared final | 230 |
| This call to String.startsWith can be rewritten using String.charAt(0) | 249 |
| Local variable 'multiplicityRange' could be declared final | 276 |
| Assigning an Object to null is a code smell. Consider refactoring. | 306 |
| Useless parentheses. | 316 |
| Prefer StringBuffer over += for concatenating strings | 333 |
| Useless parentheses. | 413 |
| Unnecessary wrapper object creation | 431 |
| Local variable 'type' could be declared final | 502 |
| Prefer StringBuffer over += for concatenating strings | 510 |
| Violation | Line |
|---|---|
| Parameter 'metaObject' is not assigned and could be declared final | 25 |
| Parameter 'context' is not assigned and could be declared final | 26 |
| Violation | Line |
|---|---|
| Parameter 'context' is not assigned and could be declared final | 20 |
| Parameter 'metaObject' is not assigned and could be declared final | 20 |
| Violation | Line |
|---|---|
| Parameter 'metaObject' is not assigned and could be declared final | 22 |
| Parameter 'context' is not assigned and could be declared final | 23 |
| Violation | Line |
|---|---|
| Parameter 'metaObject' is not assigned and could be declared final | 25 |
| Parameter 'context' is not assigned and could be declared final | 26 |
| Violation | Line |
|---|---|
| A high number of imports can indicate a high degree of coupling within an object. | 1 - 1103 |
| Avoid really long classes. | 41 - 1103 |
| This class has too many methods, consider refactoring it. | 43 - 1103 |
| Parameter 'metaObject' is not assigned and could be declared final | 50 |
| Parameter 'context' is not assigned and could be declared final | 51 |
| Variables that are final and static should be all capitals, 'logger' is not all capitals. | 59 |
| Parameter 'object' is not assigned and could be declared final | 82 |
| Unnecessary use of fully qualified name 'org.omg.uml.foundation.core.Operation' due to existing import 'org.omg.uml.foundation.core.Operation' | 84 |
| Parameter 'object' is not assigned and could be declared final | 104 |
| Local variable 'interfaceIterator' could be declared final | 113 |
| Parameter 'interfaceClassifier' is not assigned and could be declared final | 123 |
| Local variable 'generalizationIterator' could be declared final | 128 |
| Local variable 'participantAssociation' could be declared final | 147 - 148 |
| These nested if statements could be combined | 226 - 229 |
| A catch statement should never catch throwable since it includes errors. | 256 |
| Local variable 'characterType' could be declared final | 318 |
| Local variable 'charType' could be declared final | 320 |
| Local variable 'integerType' could be declared final | 375 |
| Local variable 'intType' could be declared final | 377 |
| Parameter 'follow' is not assigned and could be declared final | 478 |
| Avoid excessively long variable names like allRequiredConstructorParameters | 532 |
| Local variable 'parents' could be declared final | 536 |
| Local variable 'propertyIterator' could be declared final | 557 |
| Parameter 'follow' is not assigned and could be declared final | 585 |
| Local variable 'attribute' could be declared final | 627 |
| Local variable 'typeName' could be declared final | 630 |
| Parameter 'object' is not assigned and could be declared final | 658 |
| Local variable 'abstractionIterator' could be declared final | 674 |
| Parameter 'object' is not assigned and could be declared final | 733 |
| Parameter 'object' is not assigned and could be declared final | 749 |
| Prefer StringBuffer over += for concatenating strings | 803 |
| Parameter 'object' is not assigned and could be declared final | 898 |
| Parameter 'object' is not assigned and could be declared final | 914 |
| Parameter 'object' is not assigned and could be declared final | 931 |
| Avoid if (x != y) ..; else ..; | 965 - 972 |
| Parameter 'follow' is not assigned and could be declared final | 1009 |
| Local variable 'parentIterator' could be declared final | 1078 |
| Assigning an Object to null is a code smell. Consider refactoring. | 1093 |
| Violation | Line |
|---|---|
| Parameter 'metaObject' is not assigned and could be declared final | 27 |
| Parameter 'context' is not assigned and could be declared final | 28 |
| Parameter 'language' is not assigned and could be declared final | 121 |
| Parameter 'indent' is not assigned and could be declared final | 135 |
| Parameter 'lineLength' is not assigned and could be declared final | 136 |
| Parameter 'htmlStyle' is not assigned and could be declared final | 137 |
| Violation | Line |
|---|---|
| Parameter 'metaObject' is not assigned and could be declared final | 19 |
| Parameter 'context' is not assigned and could be declared final | 19 |
| Local variable 'suppliers' could be declared final | 67 - 68 |
| Local variable 'clients' could be declared final | 83 - 84 |
| Violation | Line |
|---|---|
| Parameter 'metaObject' is not assigned and could be declared final | 17 |
| Parameter 'context' is not assigned and could be declared final | 17 |
| Violation | Line |
|---|---|
| Parameter 'context' is not assigned and could be declared final | 30 |
| Parameter 'metaObject' is not assigned and could be declared final | 30 |
| Variables that are final and static should be all capitals, 'logger' is not all capitals. | 51 |
| Assigning an Object to null is a code smell. Consider refactoring. | 65 |
| Avoid if (x != y) ..; else ..; | 122 |
| Deeply nested if..then statements are hard to read | 140 - 147 |
| Local variable 'attribute' could be declared final | 142 |
| Local variable 'typeName' could be declared final | 151 |
| Deeply nested if..then statements are hard to read | 154 - 157 |
| A catch statement should never catch throwable since it includes errors. | 182 |
| Avoid if (x != y) ..; else ..; | 218 |
| Violation | Line |
|---|---|
| Parameter 'metaObject' is not assigned and could be declared final | 28 |
| Parameter 'context' is not assigned and could be declared final | 29 |
| Assigning an Object to null is a code smell. Consider refactoring. | 52 |
| Violation | Line |
|---|---|
| Parameter 'metaObject' is not assigned and could be declared final | 26 |
| Parameter 'context' is not assigned and could be declared final | 26 |
| Variables that are final and static should be all capitals, 'logger' is not all capitals. | 34 |
| Assigning an Object to null is a code smell. Consider refactoring. | 59 |
| Assigning an Object to null is a code smell. Consider refactoring. | 62 |
| Avoid if (x != y) ..; else ..; | 104 |
| Deeply nested if..then statements are hard to read | 121 - 128 |
| Local variable 'literalType' could be declared final | 123 |
| Deeply nested if..then statements are hard to read | 131 - 134 |
| A catch statement should never catch throwable since it includes errors. | 199 |
| Local variable 'errMsg' could be declared final | 201 |
| Avoid if (x != y) ..; else ..; | 227 |
| Violation | Line |
|---|---|
| A high number of imports can indicate a high degree of coupling within an object. | 1 - 1052 |
| Avoid really long classes. | 41 - 1052 |
| This class has too many methods, consider refactoring it. | 43 - 1052 |
| Variables that are final and static should be all capitals, 'dynamicIdentifiersPresent' is not all capitals. | 61 |
| Local variable 'operation' could be declared final | 114 |
| Local variable 'entity' could be declared final | 123 |
| Avoid using if statements without curly braces | 192 |
| Assigning an Object to null is a code smell. Consider refactoring. | 242 |
| Deeply nested if..then statements are hard to read | 295 - 298 |
| Parameter 'object' is not assigned and could be declared final | 462 |
| Parameter 'object' is not assigned and could be declared final | 495 |
| Parameter 'object' is not assigned and could be declared final | 514 |
| Parameter 'object' is not assigned and could be declared final | 537 |
| Parameter 'object' is not assigned and could be declared final | 546 |
| Parameter 'object' is not assigned and could be declared final | 577 |
| Local variable 'targetElement' could be declared final | 579 |
| Parameter 'follow' is not assigned and could be declared final | 590 |
| Parameter 'follow' is not assigned and could be declared final | 601 |
| Parameter 'object' is not assigned and could be declared final | 610 |
| Use bitwise inversion to invert boolean values | 615 |
| Use bitwise inversion to invert boolean values | 619 |
| Parameter 'follow' is not assigned and could be declared final | 632 |
| Parameter 'object' is not assigned and could be declared final | 640 |
| Use bitwise inversion to invert boolean values | 645 |
| Parameter 'follow' is not assigned and could be declared final | 658 |
| Parameter 'object' is not assigned and could be declared final | 668 |
| Use bitwise inversion to invert boolean values | 674 |
| Local variable 'attribute' could be declared final | 702 |
| Use bitwise inversion to invert boolean values | 706 |
| Local variable 'sortedProperties' could be declared final | 717 |
| Do not use the short type | 726 |
| Unnecessary wrapper object creation | 736 |
| Local variable 'identifier' could be declared final | 787 |
| Local variable 'identifier' could be declared final | 795 |
| Parameter 'object' is not assigned and could be declared final | 820 |
| Parameter 'object' is not assigned and could be declared final | 845 |
| Local variable 'end' could be declared final | 850 |
| Local variable 'facade' could be declared final | 868 |
| Unnecessary wrapper object creation | 873 |
| Avoid if (x != y) ..; else ..; | 915 |
| Useless parentheses. | 915 |
| Avoid if (x != y) ..; else ..; | 918 |
| Parameter 'object' is not assigned and could be declared final | 936 |
| A method should have only one exit point, and that should be the last statement in the method | 975 |
| Local variable 'facade' could be declared final | 979 |
| A method should have only one exit point, and that should be the last statement in the method | 982 |
| A method should have only one exit point, and that should be the last statement in the method | 986 |
| A method should have only one exit point, and that should be the last statement in the method | 1002 |
| A method should have only one exit point, and that should be the last statement in the method | 1017 |
| Local variable 'facade' could be declared final | 1021 |
| A method should have only one exit point, and that should be the last statement in the method | 1024 |
| A method should have only one exit point, and that should be the last statement in the method | 1028 |
| Violation | Line |
|---|---|
| Parameter 'metaObject' is not assigned and could be declared final | 19 |
| Parameter 'context' is not assigned and could be declared final | 20 |
| Parameter 'translation' is not assigned and could be declared final | 29 |
| Violation | Line |
|---|---|
| Parameter 'metaObject' is not assigned and could be declared final | 29 |
| Parameter 'context' is not assigned and could be declared final | 29 |
| Local variable 'literals' could be declared final | 53 |
| Parameter 'object' is not assigned and could be declared final | 58 |
| Local variable 'variables' could be declared final | 79 |
| Parameter 'object' is not assigned and could be declared final | 84 |
| Violation | Line |
|---|---|
| Parameter 'metaObject' is not assigned and could be declared final | 24 |
| Parameter 'context' is not assigned and could be declared final | 25 |
| Parameter 'modelName' is not assigned and could be declared final | 52 |
| Parameter 'modelValue' is not assigned and could be declared final | 67 |
| Prefer StringBuffer over += for concatenating strings | 84 |
| Violation | Line |
|---|---|
| Parameter 'metaObject' is not assigned and could be declared final | 25 |
| Parameter 'context' is not assigned and could be declared final | 26 |
| Violation | Line |
|---|---|
| Parameter 'metaObject' is not assigned and could be declared final | 22 |
| Parameter 'context' is not assigned and could be declared final | 22 |
| Violation | Line |
|---|---|
| Parameter 'context' is not assigned and could be declared final | 20 |
| Parameter 'metaObject' is not assigned and could be declared final | 20 |
| Violation | Line |
|---|---|
| Parameter 'metaObject' is not assigned and could be declared final | 19 |
| Parameter 'context' is not assigned and could be declared final | 20 |
| Violation | Line |
|---|---|
| The class 'FrontEndActionLogicImpl' has a Cyclomatic Complexity of 4 (Highest = 25). | 40 - 499 |
| Parameter 'metaObject' is not assigned and could be declared final | 50 |
| Parameter 'context' is not assigned and could be declared final | 51 |
| Parameter 'object' is not assigned and could be declared final | 103 |
| Avoid using redundant field initializer for 'actionStates' | 178 |
| Found non-transient, non-static member. Please mark as transient or provide accessors. | 178 |
| Avoid using redundant field initializer for 'actionForwards' | 183 |
| Found non-transient, non-static member. Please mark as transient or provide accessors. | 183 |
| Found non-transient, non-static member. Please mark as transient or provide accessors. | 188 |
| Avoid using redundant field initializer for 'decisionTransitions' | 188 |
| Found non-transient, non-static member. Please mark as transient or provide accessors. | 193 |
| Avoid using redundant field initializer for 'transitions' | 193 |
| Parameter 'transition' is not assigned and could be declared final | 217 |
| Parameter 'processedTransitions' is not assigned and could be declared final | 218 |
| The method 'handleGetFormFields' has a Cyclomatic Complexity of 25. | 352 - 498 |
| Avoid really long methods. | 352 - 498 |
| The method handleGetFormFields() has an NPath complexity of 5400 | 352 - 498 |
| Violation | Line |
|---|---|
| Parameter 'metaObject' is not assigned and could be declared final | 44 |
| Parameter 'context' is not assigned and could be declared final | 45 |
| Parameter 'object' is not assigned and could be declared final | 120 |
| Consider simply returning the value vs storing it in local variable 'operation' | 122 |
| Parameter 'object' is not assigned and could be declared final | 159 |
| Consider simply returning the value vs storing it in local variable 'operation' | 161 |
| Violation | Line |
|---|---|
| Parameter 'metaObject' is not assigned and could be declared final | 28 |
| Parameter 'context' is not assigned and could be declared final | 28 |
| Assigning an Object to null is a code smell. Consider refactoring. | 70 |
| Violation | Line |
|---|---|
| Parameter 'metaObject' is not assigned and could be declared final | 36 |
| Parameter 'context' is not assigned and could be declared final | 37 |
| Parameter 'object' is not assigned and could be declared final | 52 |
| Violation | Line |
|---|---|
| Parameter 'metaObject' is not assigned and could be declared final | 47 |
| Parameter 'context' is not assigned and could be declared final | 48 |
| Variables that are final and static should be all capitals, 'logger' is not all capitals. | 56 |
| Avoid if (x != y) ..; else ..; | 66 |
| Deeply nested if..then statements are hard to read | 128 - 131 |
| These nested if statements could be combined | 128 - 131 |
| Deeply nested if..then statements are hard to read | 195 - 198 |
| Deeply nested if..then statements are hard to read | 212 - 232 |
| Avoid if (x != y) ..; else ..; | 254 |
| Avoid if (x != y) ..; else ..; | 273 |
| Use bitwise inversion to invert boolean values | 279 |
| Use bitwise inversion to invert boolean values | 286 |
| Violation | Line |
|---|---|
| Parameter 'context' is not assigned and could be declared final | 25 |
| Parameter 'metaObject' is not assigned and could be declared final | 25 |
| A method should have only one exit point, and that should be the last statement in the method | 49 |
| Avoid catching generic exceptions such as NullPointerException, RuntimeException, Exception in try-catch block | 51 |
| Local variable 'transition' could be declared final | 64 |
| Local variable 'controllerCall' could be declared final | 78 |
| Violation | Line |
|---|---|
| Parameter 'metaObject' is not assigned and could be declared final | 21 |
| Parameter 'context' is not assigned and could be declared final | 22 |
| Violation | Line |
|---|---|
| Parameter 'context' is not assigned and could be declared final | 33 |
| Parameter 'metaObject' is not assigned and could be declared final | 33 |
| Local variable 'useCase' could be declared final | 56 |
| Violation | Line |
|---|---|
| Parameter 'metaObject' is not assigned and could be declared final | 40 |
| Parameter 'context' is not assigned and could be declared final | 41 |
| Assigning an Object to null is a code smell. Consider refactoring. | 62 |
| Avoid using redundant field initializer for 'actionStates' | 153 |
| Found non-transient, non-static member. Please mark as transient or provide accessors. | 153 |
| Useless parentheses. | 217 |
| Assigning an Object to null is a code smell. Consider refactoring. | 217 |
| Line too long | 251 |
| Deeply nested if..then statements are hard to read | 271 - 279 |
| Local variable 'incomingForward' could be declared final | 274 |
| Violation | Line |
|---|---|
| Parameter 'metaObject' is not assigned and could be declared final | 25 |
| Parameter 'context' is not assigned and could be declared final | 25 |
| Violation | Line |
|---|---|
| Parameter 'metaObject' is not assigned and could be declared final | 40 |
| Parameter 'context' is not assigned and could be declared final | 41 |
| Deeply nested if..then statements are hard to read | 92 - 95 |
| Use bitwise inversion to invert boolean values | 148 |
| Violation | Line |
|---|---|
| Parameter 'metaObject' is not assigned and could be declared final | 33 |
| Parameter 'context' is not assigned and could be declared final | 34 |
| Local variable 'transition' could be declared final | 71 |
| Deeply nested if..then statements are hard to read | 72 - 75 |
| Violation | Line |
|---|---|
| Parameter 'metaObject' is not assigned and could be declared final | 45 |
| Parameter 'context' is not assigned and could be declared final | 46 |
| A method should have only one exit point, and that should be the last statement in the method | 96 |
| Avoid if (x != y) ..; else ..; | 267 |
| Avoid if (x != y) ..; else ..; | 268 |
| These nested if statements could be combined | 318 - 321 |
| Violation | Line |
|---|---|
| Parameter 'metaObject' is not assigned and could be declared final | 36 |
| Parameter 'context' is not assigned and could be declared final | 37 |
| Assigning an Object to null is a code smell. Consider refactoring. | 84 |
| Violation | Line |
|---|---|
| Parameter 'metaObject' is not assigned and could be declared final | 30 |
| Parameter 'context' is not assigned and could be declared final | 31 |
| Local variable 'generalizations' could be declared final | 61 |
| Local variable 'iterator' could be declared final | 64 |
| Local variable 'parents' could be declared final | 79 |
| Local variable 'generalizations' could be declared final | 80 |
| Local variable 'generalization' could be declared final | 83 |
| Local variable 'specializations' could be declared final | 106 - 107 |
| Parameter 'object' is not assigned and could be declared final | 110 |
| Parameter 'follow' is not assigned and could be declared final | 162 |
| Violation | Line |
|---|---|
| Parameter 'metaObject' is not assigned and could be declared final | 22 |
| Parameter 'context' is not assigned and could be declared final | 23 |
| Violation | Line |
|---|---|
| Parameter 'metaObject' is not assigned and could be declared final | 22 |
| Parameter 'context' is not assigned and could be declared final | 23 |
| Assigning an Object to null is a code smell. Consider refactoring. | 42 |
| Violation | Line |
|---|---|
| Parameter 'context' is not assigned and could be declared final | 20 |
| Parameter 'metaObject' is not assigned and could be declared final | 20 |
| Violation | Line |
|---|---|
| Parameter 'context' is not assigned and could be declared final | 20 |
| Parameter 'metaObject' is not assigned and could be declared final | 20 |
| Violation | Line |
|---|---|
| Parameter 'context' is not assigned and could be declared final | 18 |
| Parameter 'metaObject' is not assigned and could be declared final | 18 |
| Violation | Line |
|---|---|
| Parameter 'context' is not assigned and could be declared final | 26 |
| Parameter 'metaObject' is not assigned and could be declared final | 26 |
| Violation | Line |
|---|---|
| Parameter 'metaObject' is not assigned and could be declared final | 21 |
| Parameter 'context' is not assigned and could be declared final | 21 |
| Violation | Line |
|---|---|
| Parameter 'metaObject' is not assigned and could be declared final | 23 |
| Parameter 'context' is not assigned and could be declared final | 23 |
| Violation | Line |
|---|---|
| Parameter 'metaObject' is not assigned and could be declared final | 22 |
| Parameter 'context' is not assigned and could be declared final | 23 |
| Violation | Line |
|---|---|
| The class 'ManageableEntityLogicImpl' has a Cyclomatic Complexity of 3 (Highest = 20). | 33 - 637 |
| Parameter 'metaObject' is not assigned and could be declared final | 42 |
| Parameter 'context' is not assigned and could be declared final | 42 |
| Prefer StringBuffer over += for concatenating strings | 72 |
| Parameter 'manageableAssociationEnds' is not assigned and could be declared final | 102 |
| Parameter 'entity' is not assigned and could be declared final | 102 |
| The method 'createListWithManageableMembers' has a Cyclomatic Complexity of 20. | 237 - 332 |
| The method createListWithManageableMembers() has an NPath complexity of 561 | 237 - 332 |
| Parameter 'listType' is not assigned and could be declared final | 237 |
| Deeply nested if..then statements are hard to read | 318 - 321 |
| Parameter 'withTypes' is not assigned and could be declared final | 345 |
| Assigning an Object to null is a code smell. Consider refactoring. | 347 |
| Unnecessary wrapper object creation | 355 |
| These nested if statements could be combined | 372 - 379 |
| Useless parentheses. | 372 |
| Line too long | 402 |
| Avoid if (x != y) ..; else ..; | 417 - 424 |
| Parameter 'actors' is not assigned and could be declared final | 452 |
| Parameter 'actor' is not assigned and could be declared final | 452 |
| Avoid if (x != y) ..; else ..; | 475 - 489 |
| Avoid if (x != y) ..; else ..; | 519 - 533 |
| Avoid if (x != y) ..; else ..; | 563 - 577 |
| Unnecessary wrapper object creation | 567 |
| Unnecessary wrapper object creation | 589 |
| Parameter 'left' is not assigned and could be declared final | 629 |
| Parameter 'right' is not assigned and could be declared final | 629 |
| Violation | Line |
|---|---|
| A high number of imports can indicate a high degree of coupling within an object. | 1 - 1190 |
| Avoid really long classes. | 55 - 1190 |
| This class has too many methods, consider refactoring it. | 57 - 1190 |
| Unnecessary use of fully qualified name 'org.omg.uml.foundation.core.ModelElement' due to existing import 'org.omg.uml.foundation.core.ModelElement' | 64 |
| Parameter 'metaObject' is not assigned and could be declared final | 64 |
| Parameter 'context' is not assigned and could be declared final | 65 |
| Variables that are final and static should be all capitals, 'logger' is not all capitals. | 73 |
| Local variable 'packageName' could be declared final | 91 - 94 |
| Avoid catching generic exceptions such as NullPointerException, RuntimeException, Exception in try-catch block | 103 |
| Parameter 'modelName' is not assigned and could be declared final | 117 |
| Parameter 'modelName' is not assigned and could be declared final | 135 |
| Parameter 'modelName' is not assigned and could be declared final | 148 |
| Avoid reassigning parameters such as 'name' | 172 |
| Local variable 'tagName' could be declared final | 186 |
| Deeply nested if..then statements are hard to read | 200 - 212 |
| Local variable 'valueString' could be declared final | 203 |
| Parameter 'name' is not assigned and could be declared final | 225 |
| Local variable 'taggedValues' could be declared final | 227 |
| Assigning an Object to null is a code smell. Consider refactoring. | 228 |
| Local variable 'stereotypes' could be declared final | 237 |
| Parameter 'object' is not assigned and could be declared final | 247 |
| Local variable 'name' could be declared final | 251 |
| A method should have only one exit point, and that should be the last statement in the method | 258 |
| Parameter 'stereotypeName' is not assigned and could be declared final | 281 |
| Local variable 'stereotypeNames' could be declared final | 327 |
| Local variable 'stereotypes' could be declared final | 329 |
| Parameter 'indent' is not assigned and could be declared final | 368 |
| Parameter 'indent' is not assigned and could be declared final | 380 |
| Parameter 'lineLength' is not assigned and could be declared final | 381 |
| Parameter 'indent' is not assigned and could be declared final | 394 |
| Avoid reassigning parameters such as 'lineLength' | 395 |
| Parameter 'htmlStyle' is not assigned and could be declared final | 396 |
| StringBuffer constructor is initialized with size 16, but has at least 26 characters appended. | 398 |
| Local variable 'comment' could be declared final | 408 |
| These nested if statements could be combined | 431 - 435 |
| Local variable 'todoTag' could be declared final | 433 |
| Avoid concatenating nonliterals in a StringBuffer/StringBuilder constructor or append(). | 434 |
| Local variable 'rtn' could be declared final | 439 - 443 |
| Local variable 'comment' could be declared final | 460 |
| Parameter 'keyword' is not assigned and could be declared final | 529 |
| Avoid catching generic exceptions such as NullPointerException, RuntimeException, Exception in try-catch block | 549 |
| Local variable 'property' could be declared final | 577 |
| A catch statement should never catch throwable since it includes errors. | 591 |
| Local variable 'feature' could be declared final | 626 |
| A method should have only one exit point, and that should be the last statement in the method | 627 |
| Local variable 'classifier' could be declared final | 641 |
| A method should have only one exit point, and that should be the last statement in the method | 642 |
| Local variable 'namespace' could be declared final | 646 |
| A method should have only one exit point, and that should be the last statement in the method | 647 |
| Local variable 'dependencies' could be declared final | 676 |
| Local variable 'clientDependencies' could be declared final | 677 - 678 |
| Parameter 'object' is not assigned and could be declared final | 687 |
| Useless parentheses. | 689 |
| Local variable 'dependencies' could be declared final | 701 |
| Local variable 'supplierDependencies' could be declared final | 702 - 704 |
| Parameter 'object' is not assigned and could be declared final | 713 |
| Useless parentheses. | 715 |
| Parameter 'object' is not assigned and could be declared final | 758 |
| Local variable 'constraint' could be declared final | 762 |
| Useless parentheses. | 763 - 767 |
| Useless parentheses. | 763 |
| Useless parentheses. | 764 |
| Useless parentheses. | 765 |
| Useless parentheses. | 766 |
| Useless parentheses. | 767 |
| Parameter 'translation' is not assigned and could be declared final | 780 |
| Local variable 'constraint' could be declared final | 783 - 793 |
| Parameter 'object' is not assigned and could be declared final | 788 |
| Parameter 'translation' is not assigned and could be declared final | 806 |
| Parameter 'constraints' is not assigned and could be declared final | 822 |
| Parameter 'translation' is not assigned and could be declared final | 823 |
| Local variable 'constraintIt' could be declared final | 829 |
| Local variable 'constraint' could be declared final | 832 |
| Parameter 'translation' is not assigned and could be declared final | 845 |
| Local variable 'constraints' could be declared final | 847 |
| Parameter 'object' is not assigned and could be declared final | 852 |
| Local variable 'constraint' could be declared final | 854 |
| Local variable 'dependencies' could be declared final | 949 |
| Parameter 'object' is not assigned and could be declared final | 954 |
| Parameter 'element' is not assigned and could be declared final | 985 |
| Unnecessary use of fully qualified name 'org.omg.uml.foundation.core.ModelElement' due to existing import 'org.omg.uml.foundation.core.ModelElement' | 987 |
| Avoid reassigning parameters such as 'parameterName' | 1004 |
| Parameter 'bindedElement' is not assigned and could be declared final | 1038 |
| Parameter 'object' is not assigned and could be declared final | 1047 |
| Parameter 'modelName' is not assigned and could be declared final | 1070 |
| The method handleGetBindedFullyQualifiedName() has an NPath complexity of 1002 | 1070 - 1176 |
| Avoid really long methods. | 1070 - 1176 |
| Parameter 'bindingFacades' is not assigned and could be declared final | 1070 |
| Avoid excessively long variable names like metafacadeNamespaceScopeOperator | 1074 |
| Prefer StringBuffer over += for concatenating strings | 1077 |
| Local variable 'bindedParameters' could be declared final | 1105 |
| Local variable 'bindingFacade' could be declared final | 1106 |
| Local variable 'templateParametersIterator' could be declared final | 1121 |
| Local variable 'templateArgumentsIterator' could be declared final | 1122 |
| Local variable 'templateParameterFacade' could be declared final | 1134 |
| Avoid concatenating nonliterals in a StringBuffer/StringBuilder constructor or append(). | 1142 |
| Violation | Line |
|---|---|
| Parameter 'metaObject' is not assigned and could be declared final | 31 |
| Parameter 'context' is not assigned and could be declared final | 32 |
| Local variable 'rootPackages' could be declared final | 43 |
| Parameter 'tag' is not assigned and could be declared final | 97 |
| Parameter 'value' is not assigned and could be declared final | 98 |
| Parameter 'tag' is not assigned and could be declared final | 108 |
| Parameter 'value' is not assigned and could be declared final | 109 |
| Parameter 'name' is not assigned and could be declared final | 118 |
| Parameter 'name' is not assigned and could be declared final | 128 |
| Parameter 'stereotypeName' is not assigned and could be declared final | 129 |
| Parameter 'name' is not assigned and could be declared final | 139 |
| Parameter 'name' is not assigned and could be declared final | 149 |
| Parameter 'stereotypeName' is not assigned and could be declared final | 150 |
| Line too long | 157 |
| Parameter 'useCase' is not assigned and could be declared final | 160 |
| Local variable 'useCaseMetaClass' could be declared final | 162 |
| Line too long | 167 |
| Parameter 'activityGraph' is not assigned and could be declared final | 171 |
| Parameter 'stereotypeName' is not assigned and could be declared final | 172 |
| Local variable 'activityGraphMetaClass' could be declared final | 174 |
| Local variable 'compositeState' could be declared final | 176 |
| Found non-transient, non-static member. Please mark as transient or provide accessors. | 193 |
| Private field 'stereotypeName' could be made final; it is only initialized in the declaration or constructor. | 193 |
| Avoid using redundant field initializer for 'stereotypeName' | 193 |
| Useless parentheses. | 202 |
| Parameter 'collection' is not assigned and could be declared final | 208 |
| Parameter 'collectionFilter' is not assigned and could be declared final | 209 |
| Local variable 'object' could be declared final | 212 |
| Violation | Line |
|---|---|
| Unnecessary use of fully qualified name 'org.omg.uml.foundation.core.Namespace' due to existing import 'org.omg.uml.foundation.core.Namespace' | 22 |
| Parameter 'metaObject' is not assigned and could be declared final | 22 |
| Parameter 'context' is not assigned and could be declared final | 23 |
| Violation | Line |
|---|---|
| Parameter 'metaObject' is not assigned and could be declared final | 19 |
| Parameter 'context' is not assigned and could be declared final | 19 |
| Violation | Line |
|---|---|
| Parameter 'metaObject' is not assigned and could be declared final | 22 |
| Parameter 'context' is not assigned and could be declared final | 23 |
| Violation | Line |
|---|---|
| This class has too many methods, consider refactoring it. | 39 - 804 |
| Parameter 'context' is not assigned and could be declared final | 45 |
| Parameter 'metaObject' is not assigned and could be declared final | 45 |
| Variables that are final and static should be all capitals, 'logger' is not all capitals. | 53 |
| Parameter 'withArgumentNames' is not assigned and could be declared final | 95 |
| Parameter 'withArgumentNames' is not assigned and could be declared final | 109 |
| Parameter 'name' is not assigned and could be declared final | 129 |
| Local variable 'buffer' could be declared final | 131 |
| Local variable 'buffer' could be declared final | 145 |
| Local variable 'iterator' could be declared final | 147 |
| Local variable 'parameter' could be declared final | 152 |
| Local variable 'facade' could be declared final | 160 |
| Local variable 'buffer' could be declared final | 174 |
| Local variable 'iterator' could be declared final | 176 |
| Local variable 'parameter' could be declared final | 181 |
| Local variable 'facade' could be declared final | 189 |
| A method should have only one exit point, and that should be the last statement in the method | 206 |
| Parameter 'object' is not assigned and could be declared final | 243 |
| Parameter 'follow' is not assigned and could be declared final | 285 |
| Avoid reassigning parameters such as 'name' | 285 |
| Local variable 'exceptions' could be declared final | 388 |
| Parameter 'object' is not assigned and could be declared final | 394 |
| Local variable 'dependency' could be declared final | 399 |
| Local variable 'targetElement' could be declared final | 407 |
| A method should have only one exit point, and that should be the last statement in the method | 412 |
| Parameter 'object' is not assigned and could be declared final | 436 |
| Use bitwise inversion to invert boolean values | 462 - 465 |
| Avoid reassigning parameters such as 'initialExceptions' | 474 |
| Local variable 'exceptionList' could be declared final | 477 |
| Local variable 'exceptions' could be declared final | 478 |
| Local variable 'exceptionIt' could be declared final | 485 |
| Local variable 'exception' could be declared final | 488 |
| Parameter 'modifier' is not assigned and could be declared final | 504 |
| Parameter 'argumentModifier' is not assigned and could be declared final | 513 |
| Parameter 'modifier' is not assigned and could be declared final | 518 |
| Parameter 'withArgumentNames' is not assigned and could be declared final | 518 |
| Local variable 'paramter' could be declared final | 526 |
| Parameter 'object' is not assigned and could be declared final | 700 |
| A method should have only one exit point, and that should be the last statement in the method | 717 |
| A method should have only one exit point, and that should be the last statement in the method | 732 |
| Local variable 'operationIterator' could be declared final | 777 |
| Violation | Line |
|---|---|
| Parameter 'metaObject' is not assigned and could be declared final | 26 |
| Parameter 'context' is not assigned and could be declared final | 27 |
| Parameter 'object' is not assigned and could be declared final | 41 |
| Parameter 'object' is not assigned and could be declared final | 57 |
| Violation | Line |
|---|---|
| Parameter 'metaObject' is not assigned and could be declared final | 31 |
| Parameter 'context' is not assigned and could be declared final | 32 |
| Local variable 'name' could be declared final | 46 - 48 |
| Local variable 'typeName' could be declared final | 123 |
| Prefer StringBuffer over += for concatenating strings | 126 |
| Line too long | 133 |
| Avoid using if statements without curly braces | 135 |
| A method should have only one exit point, and that should be the last statement in the method | 156 |
| A method should have only one exit point, and that should be the last statement in the method | 354 |
| A method should have only one exit point, and that should be the last statement in the method | 369 |
| Violation | Line |
|---|---|
| Parameter 'metaObject' is not assigned and could be declared final | 26 |
| Parameter 'context' is not assigned and could be declared final | 26 |
| Local variable 'vertices' could be declared final | 45 |
| Violation | Line |
|---|---|
| Parameter 'metaObject' is not assigned and could be declared final | 19 |
| Parameter 'context' is not assigned and could be declared final | 20 |
| Violation | Line |
|---|---|
| Parameter 'context' is not assigned and could be declared final | 18 |
| Parameter 'metaObject' is not assigned and could be declared final | 18 |
| Violation | Line |
|---|---|
| Parameter 'context' is not assigned and could be declared final | 22 |
| Parameter 'metaObject' is not assigned and could be declared final | 22 |
| Violation | Line |
|---|---|
| Parameter 'metaObject' is not assigned and could be declared final | 36 |
| Parameter 'context' is not assigned and could be declared final | 37 |
| Local variable 'value' could be declared final | 48 |
| Avoid if (x != y) ..; else ..; | 49 - 60 |
| Local variable 'mask' could be declared final | 56 - 58 |
| Parameter 'object' is not assigned and could be declared final | 82 |
| Use bitwise inversion to invert boolean values | 104 |
| Violation | Line |
|---|---|
| Parameter 'metaObject' is not assigned and could be declared final | 33 |
| Parameter 'context' is not assigned and could be declared final | 33 |
| Parameter 'object' is not assigned and could be declared final | 47 |
| Local variable 'targetElement' could be declared final | 49 |
| Parameter 'object' is not assigned and could be declared final | 64 |
| Local variable 'targetElement' could be declared final | 66 |
| Parameter 'object' is not assigned and could be declared final | 86 |
| Local variable 'dependency' could be declared final | 109 |
| Parameter 'object' is not assigned and could be declared final | 141 |
| Parameter 'object' is not assigned and could be declared final | 161 |
| Parameter 'object' is not assigned and could be declared final | 194 |
| Parameter 'object' is not assigned and could be declared final | 220 |
| Violation | Line |
|---|---|
| Parameter 'metaObject' is not assigned and could be declared final | 33 |
| Parameter 'context' is not assigned and could be declared final | 34 |
| Parameter 'object' is not assigned and could be declared final | 55 |
| Local variable 'dependency' could be declared final | 57 |
| Parameter 'object' is not assigned and could be declared final | 66 |
| Parameter 'object' is not assigned and could be declared final | 79 |
| Parameter 'object' is not assigned and could be declared final | 142 |
| Avoid if (x != y) ..; else ..; | 146 |
| Parameter 'object' is not assigned and could be declared final | 160 |
| Avoid if (x != y) ..; else ..; | 164 |
| Violation | Line |
|---|---|
| Parameter 'metaObject' is not assigned and could be declared final | 18 |
| Parameter 'context' is not assigned and could be declared final | 18 |
| Violation | Line |
|---|---|
| Parameter 'context' is not assigned and could be declared final | 33 |
| Parameter 'metaObject' is not assigned and could be declared final | 33 |
| Parameter 'object' is not assigned and could be declared final | 56 |
| Parameter 'object' is not assigned and could be declared final | 73 |
| Parameter 'object' is not assigned and could be declared final | 120 |
| Useless parentheses. | 122 |
| Useless parentheses. | 123 |
| Parameter 'object' is not assigned and could be declared final | 155 |
| Useless parentheses. | 157 |
| Parameter 'collectionFilter' is not assigned and could be declared final | 167 |
| Parameter 'collection' is not assigned and could be declared final | 174 |
| Parameter 'collectionFilter' is not assigned and could be declared final | 175 |
| Violation | Line |
|---|---|
| Parameter 'metaObject' is not assigned and could be declared final | 25 |
| Parameter 'context' is not assigned and could be declared final | 26 |
| A method should have only one exit point, and that should be the last statement in the method | 39 |
| Violation | Line |
|---|---|
| Parameter 'metaObject' is not assigned and could be declared final | 21 |
| Parameter 'context' is not assigned and could be declared final | 21 |
| Violation | Line |
|---|---|
| Parameter 'metaObject' is not assigned and could be declared final | 22 |
| Parameter 'context' is not assigned and could be declared final | 23 |
| Violation | Line |
|---|---|
| Parameter 'metaObject' is not assigned and could be declared final | 22 |
| Parameter 'context' is not assigned and could be declared final | 22 |
| Deeply nested if..then statements are hard to read | 40 - 43 |
| Assigning an Object to null is a code smell. Consider refactoring. | 68 |
| Useless parentheses. | 68 |
| Violation | Line |
|---|---|
| Parameter 'metaObject' is not assigned and could be declared final | 21 |
| Parameter 'context' is not assigned and could be declared final | 21 |
| Violation | Line |
|---|---|
| Parameter 'context' is not assigned and could be declared final | 29 |
| Parameter 'metaObject' is not assigned and could be declared final | 29 |
| A method should have only one exit point, and that should be the last statement in the method | 64 |
| Parameter 'indent' is not assigned and could be declared final | 115 |
| Parameter 'lineLength' is not assigned and could be declared final | 123 |
| Parameter 'indent' is not assigned and could be declared final | 123 |
| Parameter 'htmlStyle' is not assigned and could be declared final | 136 |
| Parameter 'indent' is not assigned and could be declared final | 136 |
| Avoid reassigning parameters such as 'lineLength' | 136 |
| StringBuffer constructor is initialized with size 16, but has at least 26 characters appended. | 138 |
| Local variable 'comment' could be declared final | 148 |
| These nested if statements could be combined | 165 - 169 |
| Local variable 'todoTag' could be declared final | 167 |
| Avoid concatenating nonliterals in a StringBuffer/StringBuilder constructor or append(). | 168 |
| Local variable 'rtn' could be declared final | 173 - 177 |
| Violation | Line |
|---|---|
| Parameter 'context' is not assigned and could be declared final | 18 |
| Parameter 'metaObject' is not assigned and could be declared final | 18 |
| Violation | Line |
|---|---|
| Parameter 'metaObject' is not assigned and could be declared final | 25 |
| Parameter 'context' is not assigned and could be declared final | 26 |
| Local variable 'sourceVertex' could be declared final | 118 |
| Violation | Line |
|---|---|
| A high number of imports can indicate a high degree of coupling within an object. | 1 - 707 |
| All methods are static. Consider using Singleton instead. Alternatively, you could add a private constructor or make the class abstract to silence this warning. | 47 - 707 |
| Use explicit scoping instead of the default package private level | 61 - 88 |
| Local variable 'elements' could be declared final | 64 - 66 |
| Parameter 'object' is not assigned and could be declared final | 69 |
| Local variable 'element' could be declared final | 71 |
| Local variable 'fullName' could be declared final | 72 |
| Local variable 'name' could be declared final | 73 |
| Use explicit scoping instead of the default package private level | 101 - 114 |
| Parameter 'separator' is not assigned and could be declared final | 101 |
| Parameter 'modelName' is not assigned and could be declared final | 101 |
| Parameter 'metaObject' is not assigned and could be declared final | 101 |
| Useless parentheses. | 104 |
| Prefer StringBuffer over += for concatenating strings | 107 |
| Use explicit scoping instead of the default package private level | 122 - 125 |
| Use explicit scoping instead of the default package private level | 134 - 148 |
| Parameter 'object' is not assigned and could be declared final | 141 |
| Use explicit scoping instead of the default package private level | 155 - 170 |
| Local variable 'rootPackages' could be declared final | 158 |
| Local variable 'rootPackage' could be declared final | 159 |
| Use explicit scoping instead of the default package private level | 177 - 180 |
| Avoid reassigning parameters such as 'visibility' | 188 |
| Use explicit scoping instead of the default package private level | 188 - 209 |
| Parameter 'fullyQualifiedTypeName' is not assigned and could be declared final | 223 |
| Parameter 'separator' is not assigned and could be declared final | 223 |
| Parameter 'name' is not assigned and could be declared final | 223 |
| Use explicit scoping instead of the default package private level | 223 - 234 |
| Parameter 'visibility' is not assigned and could be declared final | 223 |
| Local variable 'attribute' could be declared final | 225 |
| Local variable 'type' could be declared final | 228 |
| Parameter 'name' is not assigned and could be declared final | 243 |
| Use explicit scoping instead of the default package private level | 243 - 266 |
| Parameter 'classifier' is not assigned and could be declared final | 243 |
| Local variable 'features' could be declared final | 248 |
| Local variable 'feature' could be declared final | 253 |
| Parameter 'name' is not assigned and could be declared final | 274 |
| Use explicit scoping instead of the default package private level | 274 - 283 |
| Parameter 'name' is not assigned and could be declared final | 290 |
| Use explicit scoping instead of the default package private level | 290 - 293 |
| Parameter 'name' is not assigned and could be declared final | 302 |
| Parameter 'stereotypeName' is not assigned and could be declared final | 302 |
| Use explicit scoping instead of the default package private level | 302 - 321 |
| Local variable 'useCases' could be declared final | 306 |
| Local variable 'useCase' could be declared final | 310 |
| These nested if statements could be combined | 313 - 316 |
| Use explicit scoping instead of the default package private level | 328 - 331 |
| Parameter 'name' is not assigned and could be declared final | 328 |
| Parameter 'stereotypeName' is not assigned and could be declared final | 340 |
| Parameter 'name' is not assigned and could be declared final | 340 |
| Use explicit scoping instead of the default package private level | 340 - 359 |
| Local variable 'graphs' could be declared final | 344 |
| Local variable 'graph' could be declared final | 348 |
| These nested if statements could be combined | 351 - 354 |
| Parameter 'tag' is not assigned and could be declared final | 368 |
| Use explicit scoping instead of the default package private level | 368 - 403 |
| Parameter 'element' is not assigned and could be declared final | 368 |
| Parameter 'value' is not assigned and could be declared final | 368 |
| Local variable 'taggedValues' could be declared final | 372 |
| Local variable 'taggedValue' could be declared final | 375 |
| Local variable 'tagName' could be declared final | 378 |
| Local variable 'dataValue' could be declared final | 385 |
| Local variable 'referenceValue' could be declared final | 394 |
| Use explicit scoping instead of the default package private level | 411 - 414 |
| Parameter 'element' is not assigned and could be declared final | 411 |
| Parameter 'value' is not assigned and could be declared final | 411 |
| Parameter 'stereotypeName' is not assigned and could be declared final | 421 |
| Use explicit scoping instead of the default package private level | 421 - 436 |
| Parameter 'element' is not assigned and could be declared final | 421 |
| Local variable 'stereotypes' could be declared final | 425 |
| Local variable 'stereotype' could be declared final | 429 |
| Use explicit scoping instead of the default package private level | 445 - 462 |
| Parameter 'value' is not assigned and could be declared final | 445 |
| Parameter 'tag' is not assigned and could be declared final | 445 |
| Local variable 'useCases' could be declared final | 449 |
| Local variable 'useCase' could be declared final | 454 |
| Use explicit scoping instead of the default package private level | 471 - 487 |
| Parameter 'tag' is not assigned and could be declared final | 471 |
| Parameter 'value' is not assigned and could be declared final | 471 |
| Local variable 'classes' could be declared final | 475 |
| Local variable 'clazz' could be declared final | 479 |
| Parameter 'useCase' is not assigned and could be declared final | 493 |
| Use explicit scoping instead of the default package private level | 493 - 529 |
| Local variable 'finalStates' could be declared final | 495 |
| Local variable 'useCaseName' could be declared final | 499 |
| Local variable 'allFinalStates' could be declared final | 500 |
| Local variable 'finalState' could be declared final | 503 |
| Avoid if (x != y) ..; else ..; | 504 - 524 |
| Use explicit scoping instead of the default package private level | 537 - 555 |
| Parameter 'facade' is not assigned and could be declared final | 537 |
| Avoid variables with short names like id | 543 |
| Local variable 'id' could be declared final | 543 |
| Local variable 'graphs' could be declared final | 544 |
| Local variable 'element' could be declared final | 547 |
| Use explicit scoping instead of the default package private level | 563 - 581 |
| Parameter 'facade' is not assigned and could be declared final | 563 |
| Avoid variables with short names like id | 569 |
| Local variable 'id' could be declared final | 569 |
| Local variable 'useCases' could be declared final | 570 |
| Local variable 'element' could be declared final | 573 |
| Use explicit scoping instead of the default package private level | 589 - 607 |
| Parameter 'facade' is not assigned and could be declared final | 589 |
| Avoid variables with short names like id | 595 |
| Local variable 'id' could be declared final | 595 |
| Local variable 'parameters' could be declared final | 596 |
| Local variable 'element' could be declared final | 599 |
| Parameter 'facade' is not assigned and could be declared final | 615 |
| Use explicit scoping instead of the default package private level | 615 - 633 |
| Local variable 'id' could be declared final | 621 |
| Avoid variables with short names like id | 621 |
| Local variable 'events' could be declared final | 622 |
| Local variable 'element' could be declared final | 625 |
| Parameter 'facade' is not assigned and could be declared final | 641 |
| Use explicit scoping instead of the default package private level | 641 - 659 |
| Avoid variables with short names like id | 647 |
| Local variable 'id' could be declared final | 647 |
| Local variable 'modelElements' could be declared final | 648 |
| Local variable 'element' could be declared final | 651 |
| Parameter 'classifier' is not assigned and could be declared final | 670 |
| Local variable 'serialVersionString' could be declared final | 673 - 674 |
| Local variable 'element' could be declared final | 692 |
| Violation | Line |
|---|---|
| Variables that are final and static should be all capitals, 'logger' is not all capitals. | 33 |
| The String literal "modelElement" appears 5 times in this file; the first occurrence is on line 44 | 44 |
| Local variable 'element' could be declared final | 108 |
| Local variable 'stereotypes' could be declared final | 109 |
| Local variable 'stereotype' could be declared final | 112 |
| Avoid reassigning parameters such as 'stereotype' | 126 |
| Local variable 'stereotypeNames' could be declared final | 140 |
| Deeply nested if..then statements are hard to read | 141 - 144 |
| Found non-transient, non-static member. Please mark as transient or provide accessors. | 177 |
| Deeply nested if..then statements are hard to read | 213 - 217 |
| Violation | Line |
|---|---|
| Parameter 'metaObject' is not assigned and could be declared final | 24 |
| Parameter 'context' is not assigned and could be declared final | 25 |
| Violation | Line |
|---|---|
| Parameter 'context' is not assigned and could be declared final | 21 |
| Parameter 'metaObject' is not assigned and could be declared final | 21 |
| Local variable 'useArrays' could be declared final | 37 - 38 |
| These nested if statements could be combined | 41 - 44 |
| Deeply nested if..then statements are hard to read | 41 - 44 |
| Violation | Line |
|---|---|
| Parameter 'context' is not assigned and could be declared final | 25 |
| Parameter 'metaObject' is not assigned and could be declared final | 25 |
| Local variable 'stereotypes' could be declared final | 37 |
| Parameter 'stereotypes' is not assigned and could be declared final | 49 |
| Local variable 'specializations' could be declared final | 51 |
| Local variable 'stereotypeList' could be declared final | 52 |