PMD Results

The following document contains the results of PMD 3.9.

Files

Example - us/semblance/example/ExamplePlugin.java

ViolationLine
Document empty constructor47
Document empty method58

Example - us/semblance/example/book/edition/EditionDetailForm.java

ViolationLine
Avoid if (x != y) ..; else ..;70

Example - us/semblance/example/book/search/BookListForm.java

ViolationLine
No need to check for null before an instanceof52
No need to check for null before an instanceof56
Deeply nested if..then statements are hard to read59 - 60

Example - us/semblance/example/customer/detail/CustomerDetailAction.java

ViolationLine
Caught exception is rethrown, original stack trace may be lost90

Example - us/semblance/example/customer/detail/CustomerDetailForm.java

ViolationLine
Avoid unused imports such as 'us.semblance.foundation.CriteriaBean'22

Example - us/semblance/example/customer/search/CustomerFilterBean.java

ViolationLine
Avoid unused imports such as 'us.semblance.example.model.Customer'22

Example - us/semblance/example/customer/search/CustomerSearchBean.java

ViolationLine
Avoid unused imports such as 'us.semblance.example.model.Customer'22

Example - us/semblance/example/model/Edition.java

ViolationLine
Too many fields32 - 186

Example - us/semblance/example/model/KeywordSelectionGroup.java

ViolationLine
Document empty constructor30

Foundation - us/semblance/config/BeanConfig.java

ViolationLine
This class has a bunch of public methods and attributes17 - 567
Avoid unused imports such as 'us.semblance.formatter.Formatter'28
The class 'BeanConfig' has a Cyclomatic Complexity of 2 (Highest = 10).46 - 567
Too many fields46 - 567
Document empty constructor77
Avoid reassigning parameters such as 'keypath'87
Use block level rather than method level synchronization114 - 117
Use block level rather than method level synchronization119 - 121
Avoid reassigning parameters such as 'keypath'133
Use block level rather than method level synchronization327 - 349
The method 'formatLabelForKeypath' has a Cyclomatic Complexity of 10.457 - 484
Avoid if (x != y) ..; else ..;493
Avoid if (x != y) ..; else ..;514

Foundation - us/semblance/config/ValidationConfig.java

ViolationLine
The class 'ValidationConfig' has a Cyclomatic Complexity of 2 (Highest = 10).42 - 153
Private field 'validationRules' could be made final; it is only initialized in the declaration or constructor.54
The method 'addRule' has a Cyclomatic Complexity of 10.82 - 104

Foundation - us/semblance/formatter/ArrayFormatter.java

ViolationLine
Overriding method merely calls super77 - 79

Foundation - us/semblance/formatter/CollectionFormatter.java

ViolationLine
An instanceof check is being performed on the caught exception. Create a separate catch clause for this exception type.57

Foundation - us/semblance/formatter/CurrencyFormatter.java

ViolationLine
Avoid reassigning parameters such as 'target'50
An operation on an Immutable object (BigDecimal or BigInteger) won''t change the object itself62

Foundation - us/semblance/formatter/DateFormatter.java

ViolationLine
When instantiating a SimpleDateFormat object, specify a Locale56
When instantiating a SimpleDateFormat object, specify a Locale76

Foundation - us/semblance/formatter/DynamicTypeCodeFormatter.java

ViolationLine
The class 'DynamicTypeCodeFormatter' has a Cyclomatic Complexity of 6 (Highest = 12).34 - 189
The method getSortOrdering() has an NPath complexity of 800106 - 133
The method 'createTypeCode' has a Cyclomatic Complexity of 12.145 - 188

Foundation - us/semblance/formatter/Formatter.java

ViolationLine
High amount of different objects as members denotes a high coupling17 - 841
The class 'Formatter' has a Cyclomatic Complexity of 3 (Highest = 10).86 - 841
Avoid empty catch blocks258
The method 'findFormatter' has a Cyclomatic Complexity of 10.342 - 387
The method formatArray() has an NPath complexity of 560600 - 637
The method 'formatArray' has a Cyclomatic Complexity of 10.600 - 637
Avoid if (x != y) ..; else ..;604 - 612
An instanceof check is being performed on the caught exception. Create a separate catch clause for this exception type.650
Avoid reassigning parameters such as 'value'684
Avoid using implementation types like 'ArrayList'; use the interface instead739
Avoid using implementation types like 'HashSet'; use the interface instead741

Foundation - us/semblance/formatter/NumberFormatter.java

ViolationLine
The class 'NumberFormatter' has a Cyclomatic Complexity of 5 (Highest = 11).28 - 72
Avoid reassigning parameters such as 'target'42
The method convertToObject() has an NPath complexity of 51442 - 62
The method 'convertToObject' has a Cyclomatic Complexity of 11.42 - 62

Foundation - us/semblance/formatter/SelectionGroupFormatter.java

ViolationLine
Avoid unused imports such as 'us.semblance.selection.SelectionGroup'26
The class 'SelectionGroupFormatter' has a Cyclomatic Complexity of 5 (Highest = 11).36 - 122
The method 'createMultipleSelectionGroup' has a Cyclomatic Complexity of 11.77 - 116

Foundation - us/semblance/formatter/TimestampFormatter.java

ViolationLine
Caught exception is rethrown, original stack trace may be lost48

Foundation - us/semblance/foundation/CriteriaBean.java

ViolationLine
The class 'CriteriaBean' has a Cyclomatic Complexity of 5 (Highest = 11).52 - 250
The method 'qualifier' has a Cyclomatic Complexity of 11.175 - 218
No need to check for null before an instanceof204
Consider simply returning the value vs storing it in local variable 'andQual'217

Foundation - us/semblance/foundation/FetchResult.java

ViolationLine
Document empty constructor38

Foundation - us/semblance/foundation/FetchSpecification.java

ViolationLine
Document empty constructor48

Foundation - us/semblance/foundation/KeyValueCoder.java

ViolationLine
This class has a bunch of public methods and attributes17 - 1658
Avoid really long classes.164 - 1658
The class 'KeyValueCoder' has a Cyclomatic Complexity of 4 (Highest = 11).164 - 1658
Use block level rather than method level synchronization181
Use block level rather than method level synchronization182 - 184
Document empty constructor197
Avoid if (x != y) ..; else ..;413 - 416
Avoid if (x != y) ..; else ..;435 - 438
Avoid reassigning parameters such as 'key'504
Avoid reassigning parameters such as 'key'514
Avoid if (x != y) ..; else ..;687 - 693
Avoid if (x != y) ..; else ..;772 - 775
Avoid if (x != y) ..; else ..;789 - 792
The method valueAtIndex() has an NPath complexity of 1024868 - 883
The method 'valueAtIndex' has a Cyclomatic Complexity of 11.868 - 883
Avoid instantiating Boolean objects; reference Boolean.TRUE or Boolean.FALSE or call Boolean.valueOf() instead.873
The method 'setValueAtIndex' has a Cyclomatic Complexity of 10.884 - 896
Avoid if (x != y) ..; else ..;887 - 895
The method typeAtIndex() has an NPath complexity of 1024897 - 912
The method 'typeAtIndex' has a Cyclomatic Complexity of 11.897 - 912
The class 'DefaultImplementation' has a Cyclomatic Complexity of 5 (Highest = 19).955 - 1557
The method '_valueForKey' has a Cyclomatic Complexity of 10.1062 - 1090
The method _valueForKey() has an NPath complexity of 96001104 - 1140
The method '_valueForKey' has a Cyclomatic Complexity of 19.1104 - 1140
Avoid if (x != y) ..; else ..;1386 - 1389

Foundation - us/semblance/foundation/KeyValueComparator.java

ViolationLine
Use equals() to compare object references.98

Foundation - us/semblance/foundation/LabelValueBean.java

ViolationLine
Document empty constructor32

Foundation - us/semblance/foundation/Qualifier.java

ViolationLine
The class 'KeyValueQualifier' has a Cyclomatic Complexity of 3 (Highest = 16).146 - 233
Avoid unused constructor parameters such as 'caseSensitive'.160 - 164
The method isMatchingValue() has an NPath complexity of 504181 - 212
The method 'isMatchingValue' has a Cyclomatic Complexity of 16.181 - 212
Switch statements should have a default label202 - 208
The method isMatchingString() has an NPath complexity of 200214 - 232
When doing a String.toLowerCase()/toUpperCase() call, use a Locale225
When doing a String.toLowerCase()/toUpperCase() call, use a Locale226
Private field 'key' could be made final; it is only initialized in the declaration or constructor.238
Avoid reassigning parameters such as 'val'257
This abstract class does not have any abstract methods287 - 310
Private field 'qualifiers' could be made final; it is only initialized in the declaration or constructor.290
Document empty constructor292
Document empty constructor316
Document empty constructor330
Document empty constructor344
Document empty constructor358
Private field 'nestedQualifier' could be made final; it is only initialized in the declaration or constructor.371
Document empty constructor373
Private field 'values' could be made final; it is only initialized in the declaration or constructor.389
Private field 'key' could be made final; it is only initialized in the declaration or constructor.390
Document empty constructor392

Foundation - us/semblance/foundation/Range.java

ViolationLine
The class 'Range' has a Cyclomatic Complexity of 3 (Highest = 12).27 - 88
Document empty constructor33
The method isEqualToRange() has an NPath complexity of 28867 - 79
The method 'isEqualToRange' has a Cyclomatic Complexity of 12.67 - 79

Foundation - us/semblance/foundation/SLValueBinder.java

ViolationLine
The class 'SLValueBinder' has a Cyclomatic Complexity of 2 (Highest = 11).45 - 333
Avoid reassigning parameters such as 'value'113
The method bindValues() has an NPath complexity of 292148 - 197
The method 'bindValues' has a Cyclomatic Complexity of 11.148 - 197
Avoid if (x != y) ..; else ..;253 - 259

Foundation - us/semblance/foundation/SortOrdering.java

ViolationLine
Document empty constructor38

Foundation - us/semblance/foundation/ValueRange.java

ViolationLine
The class 'ValueRange' has a Cyclomatic Complexity of 3 (Highest = 15).34 - 132
The method isEqualToRange() has an NPath complexity of 544110 - 131
The method 'isEqualToRange' has a Cyclomatic Complexity of 15.110 - 131

Foundation - us/semblance/layout/Anchor.java

ViolationLine
The class 'Anchor' has a Cyclomatic Complexity of 3 (Highest = 10).31 - 162
The method 'renderGenericAttributes' has a Cyclomatic Complexity of 10.94 - 126
Position literals first in String comparisons123
Caught exception is rethrown, original stack trace may be lost143

Foundation - us/semblance/layout/Control.java

ViolationLine
Consider simply returning the value vs storing it in local variable 'spec'190
Avoid if (x != y) ..; else ..;207 - 211

Foundation - us/semblance/layout/Div.java

ViolationLine
No need to check for null before an instanceof60

Foundation - us/semblance/layout/Element.java

ViolationLine
This class has a bunch of public methods and attributes17 - 730
The class 'Element' has a Cyclomatic Complexity of 2 (Highest = 18).83 - 730
The method elementValueForKey() has an NPath complexity of 1176416 - 461
The method 'elementValueForKey' has a Cyclomatic Complexity of 18.416 - 461
Position literals first in String comparisons422
No need to check for null before an instanceof440
Avoid reassigning parameters such as 'val'517
Avoid reassigning parameters such as 'val'517
Avoid reassigning parameters such as 'val'517
Avoid reassigning parameters such as 'obj'620
The method formattedValueForKeypath() has an NPath complexity of 288620 - 657
The method 'formattedValueForKeypath' has a Cyclomatic Complexity of 10.620 - 657
Avoid if (x != y) ..; else ..;639 - 644

Foundation - us/semblance/layout/ElementGroup.java

ViolationLine
Avoid if (x != y) ..; else ..;165 - 173
Avoid if (x != y) ..; else ..;169 - 172
Use equals() to compare object references.297

Foundation - us/semblance/layout/Input.java

ViolationLine
The class 'Input' has a Cyclomatic Complexity of 5 (Highest = 18).50 - 359
The method initMinMax() has an NPath complexity of 30090 - 128
The method 'initMinMax' has a Cyclomatic Complexity of 13.90 - 128
The method 'renderAttribute' has a Cyclomatic Complexity of 11.192 - 242
Deeply nested if..then statements are hard to read223 - 235
An empty statement (semicolon) not part of a loop273
The method renderGenericAttributes() has an NPath complexity of 6601283 - 333
The method 'renderGenericAttributes' has a Cyclomatic Complexity of 18.283 - 333

Foundation - us/semblance/layout/KeypathSpecification.java

ViolationLine
Overridable method 'findPrefix' called during object construction50
Overridable method 'findPrefix' called during object construction58

Foundation - us/semblance/layout/Layout.java

ViolationLine
Consider simply returning the value vs storing it in local variable 'elements'79
Document empty constructor95

Foundation - us/semblance/layout/Option.java

ViolationLine
The class 'Option' has a Cyclomatic Complexity of 6 (Highest = 14).42 - 221
The method renderGenericAttributes() has an NPath complexity of 976107 - 158
The method 'renderGenericAttributes' has a Cyclomatic Complexity of 14.107 - 158
Avoid if (x != y) ..; else ..;142 - 144

Foundation - us/semblance/layout/RenderingContext.java

ViolationLine
Document empty constructor69
Avoid if (x != y) ..; else ..;211

Foundation - us/semblance/layout/XHTMLCoder.java

ViolationLine
The class 'XHTMLCoder' has a Cyclomatic Complexity of 4 (Highest = 13).43 - 428
Overridable method 'initElementDefinitions' called during object construction127
Overridable method 'initElementDefinitions' called during object construction132
Avoid if (x != y) ..; else ..;237 - 244
The method 'addChildToParent' has a Cyclomatic Complexity of 13.274 - 315
Document empty constructor327
Avoid if (x != y) ..; else ..;402 - 408

Foundation - us/semblance/listview/SLDisplayGroup.java

ViolationLine
This class has a bunch of public methods and attributes17 - 390

Foundation - us/semblance/listview/SLObjectSource.java

ViolationLine
This abstract class does not have any abstract methods49 - 427

Foundation - us/semblance/objectadaptor/KeypathGenerator.java

ViolationLine
All methods are static. Consider using Singleton instead. Alternatively, you could add a private constructor or make the class abstract to silence this warning.17 - 121

Foundation - us/semblance/objectadaptor/SLGenericAdaptor.java

ViolationLine
The class 'SLGenericAdaptor' has a Cyclomatic Complexity of 5 (Highest = 18).47 - 237
The method addNestedObjectsIfNeeded() has an NPath complexity of 2269125 - 187
The method 'addNestedObjectsIfNeeded' has a Cyclomatic Complexity of 18.125 - 187
Avoid using implementation types like 'ArrayList'; use the interface instead203
Avoid using implementation types like 'HashMap'; use the interface instead204
Avoid using implementation types like 'HashSet'; use the interface instead205
Avoid using implementation types like 'ArrayList'; use the interface instead206

Foundation - us/semblance/selection/DynamicTypeCode.java

ViolationLine
Avoid unused imports such as 'us.semblance.foundation.ConfigurationException'26
Avoid unused imports such as 'us.semblance.foundation.SortOrdering'29
Avoid unused imports such as 'us.semblance.selection.ValueGroup.Value'30
The class 'DynamicTypeCode' has a Cyclomatic Complexity of 2 (Highest = 11).110 - 373
Document empty constructor121
Overridable method 'setSelectedValue' called during object construction142
The method equals() has an NPath complexity of 432347 - 364
The method 'equals' has a Cyclomatic Complexity of 11.347 - 364
Use equals() to compare object references.360

Foundation - us/semblance/selection/MultipleSelectionGroup.java

ViolationLine
These nested if statements could be combined223 - 224

Foundation - us/semblance/selection/SelectionGroup.java

ViolationLine
Avoid unused imports such as 'us.semblance.foundation.ConfigurationException'27
Document empty constructor102

Foundation - us/semblance/selection/ValueGroup.java

ViolationLine
This class has a bunch of public methods and attributes17 - 741
This usage of the Collection.toArray() method will throw a ClassCastException.207
Avoid reassigning parameters such as 'vals'517
Document empty constructor555
Position literals first in String comparisons631
Position literals first in String comparisons642
Use equals() to compare object references.716

Foundation - us/semblance/serviceadaptor/SLServiceAdaptorException.java

ViolationLine
Document empty constructor29 - 30

Foundation - us/semblance/serviceadaptor/ValueGroupServiceAdaptor.java

ViolationLine
Avoid unused imports such as 'us.semblance.selection.DynamicTypeCode'27
Document empty constructor53

Foundation - us/semblance/serviceadaptor/ValueGroupXMLServiceAdaptor.java

ViolationLine
Avoid unused imports such as 'us.semblance.selection.DynamicTypeCode'26

Foundation - us/semblance/serviceadaptor/XMLServiceAdaptor.java

ViolationLine
Caught exception is rethrown, original stack trace may be lost83
Caught exception is rethrown, original stack trace may be lost94

Foundation - us/semblance/util/MethodCache.java

ViolationLine
Document empty constructor81

Foundation - us/semblance/util/SLArrayUtils.java

ViolationLine
All methods are static. Consider using Singleton instead. Alternatively, you could add a private constructor or make the class abstract to silence this warning.17 - 85
The class 'SLArrayUtils' has a Cyclomatic Complexity of 10 (Highest = 11).24 - 85
The method 'setArrayValue' has a Cyclomatic Complexity of 10.33 - 52
Avoid if (x != y) ..; else ..;35 - 51
The method toObject() has an NPath complexity of 102454 - 70
The method 'toObject' has a Cyclomatic Complexity of 11.54 - 70
The method toString() has an NPath complexity of 25672 - 84

Foundation - us/semblance/util/SLLoggingFormatter.java

ViolationLine
Document empty constructor29
Use block level rather than method level synchronization31 - 59
Avoid empty catch blocks54
Avoid empty catch blocks55

Foundation - us/semblance/util/SLStringUtils.java

ViolationLine
This abstract class does not have any abstract methods41 - 212

Foundation - us/semblance/util/SLUtils.java

ViolationLine
All methods are static. Consider using Singleton instead. Alternatively, you could add a private constructor or make the class abstract to silence this warning.17 - 699
The class 'SLUtils' has a Cyclomatic Complexity of 4 (Highest = 13).53 - 699
Avoid empty catch blocks76
The method 'fileInputStreamFromFile' has a Cyclomatic Complexity of 13.87 - 126
The method 'inputStreamForPath' has a Cyclomatic Complexity of 10.143 - 182
Avoid empty catch blocks156
Avoid empty catch blocks164
Avoid empty catch blocks171
The method 'invokeMethod' has a Cyclomatic Complexity of 10.411 - 443
Use equals() to compare object references.570
The method objectType() has an NPath complexity of 512578 - 590
The method 'objectType' has a Cyclomatic Complexity of 10.578 - 590
Singleton is not thread safe688 - 695

Foundation - us/semblance/util/SLUtilsBase.java

ViolationLine
This abstract class does not have any abstract methods25 - 40
Consider simply returning the value vs storing it in local variable 'stream'38

Foundation - us/semblance/validation/LengthValidation.java

ViolationLine
Document empty constructor33

Foundation - us/semblance/validation/RangeValidation.java

ViolationLine
Document empty constructor50
Overridable method 'initWithValues' called during object construction55

Foundation - us/semblance/validation/RequiredFieldValidation.java

ViolationLine
Document empty constructor43
Overridable method 'initWithValues' called during object construction53

Foundation - us/semblance/validation/RuleGroup.java

ViolationLine
Document empty constructor56
Document empty constructor169
Document empty constructor205

Foundation - us/semblance/validation/ValidationRule.java

ViolationLine
Avoid empty catch blocks99
Document empty constructor106
Overridable method 'initWithValues' called during object construction122

Foundation - us/semblance/xml/GenericAttributedObject.java

ViolationLine
Document empty constructor38

Foundation - us/semblance/xml/XMLCoder.java

ViolationLine
This class has a bunch of public methods and attributes17 - 1284
Avoid really long classes.66 - 1284
The class 'XMLCoder' has a Cyclomatic Complexity of 4 (Highest = 19).66 - 1284
Document empty constructor87
Avoid empty catch blocks113
The method '_addNestedElements' has a Cyclomatic Complexity of 16.303 - 366
The method parseElement() has an NPath complexity of 5400369 - 450
The method 'parseElement' has a Cyclomatic Complexity of 19.369 - 450
Avoid if (x != y) ..; else ..;435 - 438
The method 'addChildToParent' has a Cyclomatic Complexity of 15.473 - 509
The method 'parseAttributes' has a Cyclomatic Complexity of 10.587 - 630
Avoid if (x != y) ..; else ..;620 - 628
The method 'createObject' has a Cyclomatic Complexity of 10.690 - 729
Avoid using implementation types like 'HashMap'; use the interface instead930
Avoid using implementation types like 'ArrayList'; use the interface instead931
Avoid using implementation types like 'HashSet'; use the interface instead932
Avoid using implementation types like 'HashMap'; use the interface instead1086

StrutsLive - org/apache/struts/action/SLMessageGroup.java

ViolationLine
The class 'SLMessageGroup' has a Cyclomatic Complexity of 3 (Highest = 10).43 - 197
The method addErrorMessagesForKeys() has an NPath complexity of 154594 - 127
The method 'addErrorMessagesForKeys' has a Cyclomatic Complexity of 10.94 - 127
No need to check for null before an instanceof161

StrutsLive - org/apache/struts/tiles/SLTilesRequestProcessor.java

ViolationLine
No need to import a type that lives in the same package25
Caught exception is rethrown, original stack trace may be lost51

StrutsLive - us/semblance/strutslive/SLAction.java

ViolationLine
Avoid if (x != y) ..; else ..;159 - 160

StrutsLive - us/semblance/strutslive/SLActionForm.java

ViolationLine
High amount of different objects as members denotes a high coupling17 - 1139
A high number of imports can indicate a high degree of coupling within an object.17 - 1139
This class has a bunch of public methods and attributes17 - 1139
Avoid really long classes.89 - 1139
Document empty constructor112
Avoid unused method parameters such as 'request'.267 - 278
Deeply nested if..then statements are hard to read342 - 345
Avoid reassigning parameters such as 'keypath'456
These nested if statements could be combined708
Avoid if (x != y) ..; else ..;875 - 884

StrutsLive - us/semblance/strutslive/SLLayoutServlet.java

ViolationLine
Avoid unused imports such as 'org.apache.struts.action.ActionServlet'34
The class 'SLLayoutServlet' has a Cyclomatic Complexity of 7 (Highest = 21).50 - 230
The method getLayoutForm() has an NPath complexity of 19200107 - 162
The method 'getLayoutForm' has a Cyclomatic Complexity of 21.107 - 162
No need to check for null before an instanceof127
No need to check for null before an instanceof143
No need to check for null before an instanceof153
Deeply nested if..then statements are hard to read155
The method 'renderLayout' has a Cyclomatic Complexity of 12.168 - 229
Avoid empty catch blocks226

StrutsLive - us/semblance/strutslive/SLListViewAction.java

ViolationLine
The class 'SLListViewAction' has a Cyclomatic Complexity of 3 (Highest = 10).43 - 231
The method 'modifySelected' has a Cyclomatic Complexity of 10.73 - 103
No need to check for null before an instanceof225

StrutsLive - us/semblance/strutslive/SLListViewForm.java

ViolationLine
This class has a bunch of public methods and attributes17 - 610
Avoid unused imports such as 'org.apache.struts.action.ActionForm'28
Avoid if (x != y) ..; else ..;189
Position literals first in String comparisons268
No need to check for null before an instanceof298
The method getHeadingClass() has an NPath complexity of 1250307 - 319
Avoid if (x != y) ..; else ..;309
Avoid if (x != y) ..; else ..;310
No need to check for null before an instanceof362
Document empty constructor419

StrutsLive - us/semblance/strutslive/SLMessageResources.java

ViolationLine
The method getMessage() has an NPath complexity of 22556 - 92
Use block level rather than method level synchronization102 - 138

StrutsLive - us/semblance/strutslive/SLPlugin.java

ViolationLine
Document empty constructor41
Document empty method70

StrutsLive - us/semblance/strutslive/SLRequestProcessor.java

ViolationLine
The class 'SLRequestProcessor' has a Cyclomatic Complexity of 6 (Highest = 13).50 - 527
Document empty constructor66
Avoid reassigning parameters such as 'request'68
The method process() has an NPath complexity of 51268 - 120
The method 'process' has a Cyclomatic Complexity of 10.68 - 120
The method processForm() has an NPath complexity of 252137 - 183
The method 'processForm' has a Cyclomatic Complexity of 13.137 - 183
An instanceof check is being performed on the caught exception. Create a separate catch clause for this exception type.157
An instanceof check is being performed on the caught exception. Create a separate catch clause for this exception type.158
An instanceof check is being performed on the caught exception. Create a separate catch clause for this exception type.159
Avoid unnecessary comparisons in boolean expressions255
Avoid empty catch blocks319 - 321
Avoid empty catch blocks323
The method processActionPerform() has an NPath complexity of 440429 - 471
An instanceof check is being performed on the caught exception. Create a separate catch clause for this exception type.462
The method processForwardConfig() has an NPath complexity of 240473 - 496
Avoid if (x != y) ..; else ..;490 - 495
Avoid reassigning parameters such as 'request'498

StrutsLive - us/semblance/strutslive/tableview/BatchFetchingDataSource.java

ViolationLine
Avoid unused imports such as 'us.semblance.listview.SLBatchingPersistentObjectSource'20

StrutsLive - us/semblance/strutslive/tableview/DataSource.java

ViolationLine
This abstract class does not have any abstract methods28 - 36

StrutsLive - us/semblance/strutslive/tableview/SLSearchAction.java

ViolationLine
The class 'SLSearchAction' has a Cyclomatic Complexity of 5 (Highest = 11).39 - 124
The method search() has an NPath complexity of 51255 - 123
The method 'search' has a Cyclomatic Complexity of 11.55 - 123

StrutsLive - us/semblance/strutslive/tableview/TableViewAction.java

ViolationLine
The class 'TableViewAction' has a Cyclomatic Complexity of 3 (Highest = 10).43 - 196
The method search() has an NPath complexity of 256132 - 195
The method 'search' has a Cyclomatic Complexity of 10.132 - 195

StrutsLive - us/semblance/strutslive/tableview/TableViewConfig.java

ViolationLine
Too many fields18 - 240
Caught exception is rethrown, original stack trace may be lost201

StrutsLive - us/semblance/strutslive/tag/LayoutTag.java

ViolationLine
The class 'LayoutTag' has a Cyclomatic Complexity of 3 (Highest = 11).45 - 123
The method 'doStartTag' has a Cyclomatic Complexity of 11.62 - 99

StrutsLive - us/semblance/strutslive/util/FormConfig.java

ViolationLine
Avoid unused imports such as 'us.semblance.strutslive.SLActionForm'26

StrutsLive - us/semblance/strutslive/util/KeypathBinder.java

ViolationLine
The class 'KeypathBinder' has a Cyclomatic Complexity of 3 (Highest = 16).65 - 340
Avoid empty catch blocks186
Avoid reassigning parameters such as 'value'235
The method convertValue() has an NPath complexity of 1500235 - 286
The method 'convertValue' has a Cyclomatic Complexity of 16.235 - 286
Avoid if (x != y) ..; else ..;320 - 322
Caught exception is rethrown, original stack trace may be lost326

StrutsLive - us/semblance/strutslive/util/SLWebUtils.java

ViolationLine
All methods are static. Consider using Singleton instead. Alternatively, you could add a private constructor or make the class abstract to silence this warning.17 - 80
Avoid unused imports such as 'us.semblance.util.FileException'26