PMD Results

The following document contains the results of PMD 3.7.

Files

net/sf/echobinding/App.java

ViolationLine
Each class should declare at least one constructor7 - 13

net/sf/echobinding/binding/AbstractBindingContext.java

ViolationLine
Error while parsing C:\tmp\EchoBinding\src\main\java\net\sf\echobinding\binding\AbstractBindingContext.java0
Error while parsing C:\tmp\EchoBinding\src\main\java\net\sf\echobinding\binding\AbstractBindingContext.java0

net/sf/echobinding/binding/AbstractPropertyAdapter.java

ViolationLine
Error while parsing C:\tmp\EchoBinding\src\main\java\net\sf\echobinding\binding\AbstractPropertyAdapter.java0
Error while parsing C:\tmp\EchoBinding\src\main\java\net\sf\echobinding\binding\AbstractPropertyAdapter.java0

net/sf/echobinding/binding/OgnlBindingContext.java

ViolationLine
Error while parsing C:\tmp\EchoBinding\src\main\java\net\sf\echobinding\binding\OgnlBindingContext.java0
Error while parsing C:\tmp\EchoBinding\src\main\java\net\sf\echobinding\binding\OgnlBindingContext.java0

net/sf/echobinding/binding/OgnlPropertyAdapter.java

ViolationLine
It is a good practice to call super() in a constructor61 - 63
These nested if statements could be combined128 - 130
A method should have only one exit point, and that should be the last statement in the method266
A method should have only one exit point, and that should be the last statement in the method294

net/sf/echobinding/BindingConfiguration.java

ViolationLine
Error while parsing C:\tmp\EchoBinding\src\main\java\net\sf\echobinding\BindingConfiguration.java0
Error while parsing C:\tmp\EchoBinding\src\main\java\net\sf\echobinding\BindingConfiguration.java0

net/sf/echobinding/Configuration.java

ViolationLine
Each class should declare at least one constructor36 - 53

net/sf/echobinding/ConfigurationSetup.java

ViolationLine
Error while parsing C:\tmp\EchoBinding\src\main\java\net\sf\echobinding\ConfigurationSetup.java0
Error while parsing C:\tmp\EchoBinding\src\main\java\net\sf\echobinding\ConfigurationSetup.java0

net/sf/echobinding/controls/BoundButtonGroup.java

ViolationLine
Error while parsing C:\tmp\EchoBinding\src\main\java\net\sf\echobinding\controls\BoundButtonGroup.java0
Error while parsing C:\tmp\EchoBinding\src\main\java\net\sf\echobinding\controls\BoundButtonGroup.java0

net/sf/echobinding/controls/Button.java

ViolationLine
Perhaps '_context' could be replaced by a local variable.44

net/sf/echobinding/controls/CheckBox.java

ViolationLine
It is a good practice to call super() in a constructor93 - 98
Avoid instantiating Boolean objects; reference Boolean.TRUE or Boolean.FALSE or call Boolean.valueOf() instead.184

net/sf/echobinding/controls/RadioButton.java

ViolationLine
Perhaps '_option' could be replaced by a local variable.47
A method should have only one exit point, and that should be the last statement in the method107

net/sf/echobinding/controls/SelectField.java

ViolationLine
Error while parsing C:\tmp\EchoBinding\src\main\java\net\sf\echobinding\controls\SelectField.java0
Error while parsing C:\tmp\EchoBinding\src\main\java\net\sf\echobinding\controls\SelectField.java0

net/sf/echobinding/datacontrol/FormColumnTable.java

ViolationLine
Error while parsing C:\tmp\EchoBinding\src\main\java\net\sf\echobinding\datacontrol\FormColumnTable.java0
Error while parsing C:\tmp\EchoBinding\src\main\java\net\sf\echobinding\datacontrol\FormColumnTable.java0

net/sf/echobinding/datacontrol/FormHelper.java

ViolationLine
Each class should declare at least one constructor39 - 141

net/sf/echobinding/decorator/DefaultDecorator.java

ViolationLine
Each class should declare at least one constructor34 - 43

net/sf/echobinding/format/DateFormat.java

ViolationLine
Perhaps '_datePattern' could be replaced by a local variable.39

net/sf/echobinding/format/TimeFormat.java

ViolationLine
Each class should declare at least one constructor38 - 94
A method should have only one exit point, and that should be the last statement in the method56

net/sf/echobinding/model/AbstractPresentationModel.java

ViolationLine
Error while parsing C:\tmp\EchoBinding\src\main\java\net\sf\echobinding\model\AbstractPresentationModel.java0
Error while parsing C:\tmp\EchoBinding\src\main\java\net\sf\echobinding\model\AbstractPresentationModel.java0

net/sf/echobinding/persistence/DefaultTablePersistenceManager.java

ViolationLine
Error while parsing C:\tmp\EchoBinding\src\main\java\net\sf\echobinding\persistence\DefaultTablePersistenceManager.java0
Error while parsing C:\tmp\EchoBinding\src\main\java\net\sf\echobinding\persistence\DefaultTablePersistenceManager.java0

net/sf/echobinding/table/BoundTable.java

ViolationLine
Error while parsing C:\tmp\EchoBinding\src\main\java\net\sf\echobinding\table\BoundTable.java0
Error while parsing C:\tmp\EchoBinding\src\main\java\net\sf\echobinding\table\BoundTable.java0

net/sf/echobinding/table/BoundTableCollectionCellRenderer.java

ViolationLine
Each class should declare at least one constructor43 - 93

net/sf/echobinding/table/BoundTableHeaderRenderer.java

ViolationLine
Each class should declare at least one constructor40 - 77

net/sf/echobinding/table/BoundTableModel.java

ViolationLine
Use explicit scoping instead of the default package private level40
Use explicit scoping instead of the default package private level41
A method should have only one exit point, and that should be the last statement in the method65

net/sf/echobinding/table/DefaultDataBoundTableCellRenderer.java

ViolationLine
Each class should declare at least one constructor43 - 84

net/sf/echobinding/table/EditableTableCellRenderer.java

ViolationLine
Each class should declare at least one constructor37 - 70
A method should have only one exit point, and that should be the last statement in the method51

net/sf/echobinding/table/EditableTableHeaderRenderer.java

ViolationLine
Each class should declare at least one constructor36 - 52
A method should have only one exit point, and that should be the last statement in the method44

net/sf/echobinding/table/EditableTableModel.java

ViolationLine
Use explicit scoping instead of the default package private level38
Use explicit scoping instead of the default package private level39
A method should have only one exit point, and that should be the last statement in the method65
A method should have only one exit point, and that should be the last statement in the method70
A method should have only one exit point, and that should be the last statement in the method86
A method should have only one exit point, and that should be the last statement in the method102

net/sf/echobinding/table/SelectableTable.java

ViolationLine
Error while parsing C:\tmp\EchoBinding\src\main\java\net\sf\echobinding\table\SelectableTable.java0
Error while parsing C:\tmp\EchoBinding\src\main\java\net\sf\echobinding\table\SelectableTable.java0

net/sf/echobinding/util/Images.java

ViolationLine
Each class should declare at least one constructor35 - 52

net/sf/echobinding/util/Util.java

ViolationLine
Each class should declare at least one constructor33 - 44
Assigning an Object to null is a code smell. Consider refactoring.42

net/sf/echobinding/validation/BeanValidator.java

ViolationLine
Error while parsing C:\tmp\EchoBinding\src\main\java\net\sf\echobinding\validation\BeanValidator.java0
Error while parsing C:\tmp\EchoBinding\src\main\java\net\sf\echobinding\validation\BeanValidator.java0

net/sf/echobinding/validation/DateValidator.java

ViolationLine
A method should have only one exit point, and that should be the last statement in the method48

net/sf/echobinding/validation/DefaultValidationHandler.java

ViolationLine
A method should have only one exit point, and that should be the last statement in the method87

net/sf/echobinding/validation/NumberValidator.java

ViolationLine
A method should have only one exit point, and that should be the last statement in the method64

net/sf/echobinding/validation/RegexValidator.java

ViolationLine
A method should have only one exit point, and that should be the last statement in the method60