PMD Results

The following document contains the results of PMD 5.0.5.

Files

org/andromda/cartridges/xmlschema/XmlSchemaProfile.java

Violation Line
Variables that are final and static should be all capitals, 'profile' is not all capitals. 17
It is a good practice to call super() in a constructor 4043

org/andromda/cartridges/xmlschema/metafacades/XSDAssociationEndLogicImpl.java

Violation Line
Parameter 'context' is not assigned and could be declared final 18
Parameter 'metaObject' is not assigned and could be declared final 18
Local variable 'isMany' could be declared final 30
Local variable 'isRequired' could be declared final 49

org/andromda/cartridges/xmlschema/metafacades/XSDAttributeLogicImpl.java

Violation Line
Parameter 'metaObject' is not assigned and could be declared final 20
Parameter 'context' is not assigned and could be declared final 20
Local variable 'isMany' could be declared final 32
Local variable 'isRequired' could be declared final 51

org/andromda/cartridges/xmlschema/metafacades/XSDComplexTypeLogicImpl.java

Violation Line
Parameter 'context' is not assigned and could be declared final 16
Parameter 'metaObject' is not assigned and could be declared final 16

org/andromda/cartridges/xmlschema/metafacades/XSDEnumerationTypeLogicImpl.java

Violation Line
Parameter 'context' is not assigned and could be declared final 16
Parameter 'metaObject' is not assigned and could be declared final 16

org/andromda/cartridges/xmlschema/metafacades/XSDAssociationEnd.java

Violation Line
Avoid modifiers which are implied by the context 22
Avoid modifiers which are implied by the context 29
Avoid modifiers which are implied by the context 36
Avoid modifiers which are implied by the context 42

org/andromda/cartridges/xmlschema/metafacades/XSDAssociationEndLogic.java

Violation Line
This class has a bunch of public methods and attributes 5996
Abstract classes should be named AbstractXXX 32996
Found non-transient, non-static member. Please mark as transient or provide accessors. 40
Parameter 'context' is not assigned and could be declared final 46
Parameter 'metaObjectIn' is not assigned and could be declared final 46
Avoid reassigning parameters such as 'context' 63
Private field 'superAssociationEndFacade' could be made final; it is only initialized in the declaration or constructor. 72
Found non-transient, non-static member. Please mark as transient or provide accessors. 72
Found non-transient, non-static member. Please mark as transient or provide accessors. 73
Avoid using redundant field initializer for 'superAssociationEndFacadeInitialized' 73
Avoid excessively long variable names like superAssociationEndFacadeInitialized 73
Avoid reassigning parameters such as 'context' 94
An empty method in an abstract class should be abstract instead 111114
Local variable 'returnValue' could be declared final 155
Local variable 'returnValue' could be declared final 176
An empty method in an abstract class should be abstract instead 185188
An empty method in an abstract class should be abstract instead 194197
Parameter 'element' is not assigned and could be declared final 479
Parameter 'tagName' is not assigned and could be declared final 489
Parameter 'tagName' is not assigned and could be declared final 499
Parameter 'bindedElement' is not assigned and could be declared final 510
Parameter 'kind' is not assigned and could be declared final 529
Parameter 'indent' is not assigned and could be declared final 540
Parameter 'lineLength' is not assigned and could be declared final 552
Parameter 'indent' is not assigned and could be declared final 552
Parameter 'indent' is not assigned and could be declared final 563
Parameter 'htmlStyle' is not assigned and could be declared final 563
Parameter 'lineLength' is not assigned and could be declared final 563
Parameter 'modelName' is not assigned and could be declared final 584
Parameter 'modelName' is not assigned and could be declared final 681
Parameter 'parameterName' is not assigned and could be declared final 775
Parameter 'stereotypeName' is not assigned and could be declared final 804
Parameter 'keywordName' is not assigned and could be declared final 814
Parameter 'stereotypeName' is not assigned and could be declared final 831
Parameter 'translation' is not assigned and could be declared final 900
Parameter 'name' is not assigned and could be declared final 900
Parameter 'translation' is not assigned and could be declared final 909
Parameter 'kind' is not assigned and could be declared final 918
Parameter 'translation' is not assigned and could be declared final 918
Local variable 'owner' could be declared final 939
Consider simply returning the value vs storing it in local variable 'owner' 940
Local variable 'name' could be declared final 950
Consider simply returning the value vs storing it in local variable 'name' 951
Parameter 'validationMessages' is not assigned and could be declared final 959
Avoid appending characters as strings in StringBuffer.append. 977
A catch statement should never catch throwable since it includes errors. 982
A catch statement should never catch throwable since it includes errors. 988
Avoid empty catch blocks 988991
Avoid appending characters as strings in StringBuffer.append. 993

org/andromda/cartridges/xmlschema/metafacades/XSDAttribute.java

Violation Line
Avoid modifiers which are implied by the context 22
Avoid modifiers which are implied by the context 29
Avoid modifiers which are implied by the context 36
Avoid modifiers which are implied by the context 42
Avoid modifiers which are implied by the context 48
Avoid modifiers which are implied by the context 54

org/andromda/cartridges/xmlschema/metafacades/XSDAttributeLogic.java

Violation Line
This class has a bunch of public methods and attributes 51006
Abstract classes should be named AbstractXXX 321006
Found non-transient, non-static member. Please mark as transient or provide accessors. 40
Parameter 'metaObjectIn' is not assigned and could be declared final 46
Parameter 'context' is not assigned and could be declared final 46
Avoid reassigning parameters such as 'context' 63
Found non-transient, non-static member. Please mark as transient or provide accessors. 72
Private field 'superAttributeFacade' could be made final; it is only initialized in the declaration or constructor. 72
Avoid using redundant field initializer for 'superAttributeFacadeInitialized' 73
Found non-transient, non-static member. Please mark as transient or provide accessors. 73
Avoid reassigning parameters such as 'context' 94
An empty method in an abstract class should be abstract instead 111114
Local variable 'returnValue' could be declared final 155
Local variable 'returnValue' could be declared final 176
Local variable 'returnValue' could be declared final 195
Local variable 'returnValue' could be declared final 214
An empty method in an abstract class should be abstract instead 223226
An empty method in an abstract class should be abstract instead 232235
Parameter 'follow' is not assigned and could be declared final 244
Parameter 'name' is not assigned and could be declared final 244
Parameter 'element' is not assigned and could be declared final 489
Parameter 'tagName' is not assigned and could be declared final 499
Parameter 'tagName' is not assigned and could be declared final 509
Parameter 'bindedElement' is not assigned and could be declared final 520
Parameter 'kind' is not assigned and could be declared final 539
Parameter 'indent' is not assigned and could be declared final 550
Parameter 'lineLength' is not assigned and could be declared final 562
Parameter 'indent' is not assigned and could be declared final 562
Parameter 'lineLength' is not assigned and could be declared final 573
Parameter 'htmlStyle' is not assigned and could be declared final 573
Parameter 'indent' is not assigned and could be declared final 573
Parameter 'modelName' is not assigned and could be declared final 594
Parameter 'modelName' is not assigned and could be declared final 691
Parameter 'parameterName' is not assigned and could be declared final 785
Parameter 'stereotypeName' is not assigned and could be declared final 814
Parameter 'keywordName' is not assigned and could be declared final 824
Parameter 'stereotypeName' is not assigned and could be declared final 841
Parameter 'translation' is not assigned and could be declared final 910
Parameter 'name' is not assigned and could be declared final 910
Parameter 'translation' is not assigned and could be declared final 919
Parameter 'translation' is not assigned and could be declared final 928
Parameter 'kind' is not assigned and could be declared final 928
Local variable 'owner' could be declared final 949
Consider simply returning the value vs storing it in local variable 'owner' 950
Local variable 'name' could be declared final 960
Consider simply returning the value vs storing it in local variable 'name' 961
Parameter 'validationMessages' is not assigned and could be declared final 969
Avoid appending characters as strings in StringBuffer.append. 987
A catch statement should never catch throwable since it includes errors. 992
Avoid empty catch blocks 9981001
A catch statement should never catch throwable since it includes errors. 998
Avoid appending characters as strings in StringBuffer.append. 1003

org/andromda/cartridges/xmlschema/metafacades/XSDComplexType.java

Violation Line
Avoid modifiers which are implied by the context 22

org/andromda/cartridges/xmlschema/metafacades/XSDComplexTypeLogic.java

Violation Line
This class has a bunch of public methods and attributes 51303
Abstract classes should be named AbstractXXX 361303
Avoid really long classes. 361303
Found non-transient, non-static member. Please mark as transient or provide accessors. 44
Parameter 'context' is not assigned and could be declared final 50
Parameter 'metaObjectIn' is not assigned and could be declared final 50
Avoid reassigning parameters such as 'context' 67
Private field 'superClassifierFacade' could be made final; it is only initialized in the declaration or constructor. 76
Found non-transient, non-static member. Please mark as transient or provide accessors. 76
Avoid excessively long variable names like superClassifierFacadeInitialized 77
Found non-transient, non-static member. Please mark as transient or provide accessors. 77
Avoid using redundant field initializer for 'superClassifierFacadeInitialized' 77
Avoid reassigning parameters such as 'context' 98
An empty method in an abstract class should be abstract instead 115118
An empty method in an abstract class should be abstract instead 124127
An empty method in an abstract class should be abstract instead 133136
An empty method in an abstract class should be abstract instead 142145
Parameter 'name' is not assigned and could be declared final 152
Parameter 'follow' is not assigned and could be declared final 248
Parameter 'follow' is not assigned and could be declared final 345
Parameter 'follow' is not assigned and could be declared final 397
Parameter 'follow' is not assigned and could be declared final 703
Parameter 'tagName' is not assigned and could be declared final 703
Parameter 'element' is not assigned and could be declared final 786
Parameter 'tagName' is not assigned and could be declared final 796
Parameter 'tagName' is not assigned and could be declared final 806
Parameter 'bindedElement' is not assigned and could be declared final 817
Parameter 'kind' is not assigned and could be declared final 836
Parameter 'indent' is not assigned and could be declared final 847
Parameter 'indent' is not assigned and could be declared final 859
Parameter 'lineLength' is not assigned and could be declared final 859
Parameter 'htmlStyle' is not assigned and could be declared final 870
Parameter 'indent' is not assigned and could be declared final 870
Parameter 'lineLength' is not assigned and could be declared final 870
Parameter 'modelName' is not assigned and could be declared final 891
Parameter 'modelName' is not assigned and could be declared final 988
Parameter 'parameterName' is not assigned and could be declared final 1082
Parameter 'stereotypeName' is not assigned and could be declared final 1111
Parameter 'keywordName' is not assigned and could be declared final 1121
Parameter 'stereotypeName' is not assigned and could be declared final 1138
Parameter 'name' is not assigned and could be declared final 1207
Parameter 'translation' is not assigned and could be declared final 1207
Parameter 'translation' is not assigned and could be declared final 1216
Parameter 'translation' is not assigned and could be declared final 1225
Parameter 'kind' is not assigned and could be declared final 1225
Local variable 'owner' could be declared final 1246
Consider simply returning the value vs storing it in local variable 'owner' 1247
Local variable 'name' could be declared final 1257
Consider simply returning the value vs storing it in local variable 'name' 1258
Parameter 'validationMessages' is not assigned and could be declared final 1266
Avoid appending characters as strings in StringBuffer.append. 1284
A catch statement should never catch throwable since it includes errors. 1289
A catch statement should never catch throwable since it includes errors. 1295
Avoid empty catch blocks 12951298
Avoid appending characters as strings in StringBuffer.append. 1300

org/andromda/cartridges/xmlschema/metafacades/XSDEnumerationType.java

Violation Line
Avoid modifiers which are implied by the context 22

org/andromda/cartridges/xmlschema/metafacades/XSDEnumerationTypeLogic.java

Violation Line
This class has a bunch of public methods and attributes 51370
Avoid really long classes. 371370
Abstract classes should be named AbstractXXX 371370
Found non-transient, non-static member. Please mark as transient or provide accessors. 45
Parameter 'metaObjectIn' is not assigned and could be declared final 51
Parameter 'context' is not assigned and could be declared final 51
Avoid reassigning parameters such as 'context' 68
Found non-transient, non-static member. Please mark as transient or provide accessors. 77
Private field 'superEnumerationFacade' could be made final; it is only initialized in the declaration or constructor. 77
Found non-transient, non-static member. Please mark as transient or provide accessors. 78
Avoid using redundant field initializer for 'superEnumerationFacadeInitialized' 78
Avoid excessively long variable names like superEnumerationFacadeInitialized 78
Avoid reassigning parameters such as 'context' 99
An empty method in an abstract class should be abstract instead 116119
An empty method in an abstract class should be abstract instead 125128
An empty method in an abstract class should be abstract instead 134137
An empty method in an abstract class should be abstract instead 143146
An empty method in an abstract class should be abstract instead 152155
Parameter 'name' is not assigned and could be declared final 162
Parameter 'follow' is not assigned and could be declared final 258
Parameter 'follow' is not assigned and could be declared final 355
Parameter 'follow' is not assigned and could be declared final 407
Parameter 'follow' is not assigned and could be declared final 770
Parameter 'tagName' is not assigned and could be declared final 770
Parameter 'element' is not assigned and could be declared final 853
Parameter 'tagName' is not assigned and could be declared final 863
Parameter 'tagName' is not assigned and could be declared final 873
Parameter 'bindedElement' is not assigned and could be declared final 884
Parameter 'kind' is not assigned and could be declared final 903
Parameter 'indent' is not assigned and could be declared final 914
Parameter 'indent' is not assigned and could be declared final 926
Parameter 'lineLength' is not assigned and could be declared final 926
Parameter 'indent' is not assigned and could be declared final 937
Parameter 'htmlStyle' is not assigned and could be declared final 937
Parameter 'lineLength' is not assigned and could be declared final 937
Parameter 'modelName' is not assigned and could be declared final 958
Parameter 'modelName' is not assigned and could be declared final 1055
Parameter 'parameterName' is not assigned and could be declared final 1149
Parameter 'stereotypeName' is not assigned and could be declared final 1178
Parameter 'keywordName' is not assigned and could be declared final 1188
Parameter 'stereotypeName' is not assigned and could be declared final 1205
Parameter 'name' is not assigned and could be declared final 1274
Parameter 'translation' is not assigned and could be declared final 1274
Parameter 'translation' is not assigned and could be declared final 1283
Parameter 'translation' is not assigned and could be declared final 1292
Parameter 'kind' is not assigned and could be declared final 1292
Local variable 'owner' could be declared final 1313
Consider simply returning the value vs storing it in local variable 'owner' 1314
Local variable 'name' could be declared final 1324
Consider simply returning the value vs storing it in local variable 'name' 1325
Parameter 'validationMessages' is not assigned and could be declared final 1333
Avoid appending characters as strings in StringBuffer.append. 1351
A catch statement should never catch throwable since it includes errors. 1356
A catch statement should never catch throwable since it includes errors. 1362
Avoid empty catch blocks 13621365
Avoid appending characters as strings in StringBuffer.append. 1367