PMD Results
The following document contains the results of PMD 3.7.
Files
net/sf/echobinding/App.java
| Violation | Line |
|---|
| Each class should declare at least one constructor | 7 - 13 |
net/sf/echobinding/binding/AbstractBindingContext.java
| Violation | Line |
|---|
| Error while parsing C:\tmp\EchoBinding\src\main\java\net\sf\echobinding\binding\AbstractBindingContext.java | 0 |
| Error while parsing C:\tmp\EchoBinding\src\main\java\net\sf\echobinding\binding\AbstractBindingContext.java | 0 |
net/sf/echobinding/binding/AbstractPropertyAdapter.java
| Violation | Line |
|---|
| Error while parsing C:\tmp\EchoBinding\src\main\java\net\sf\echobinding\binding\AbstractPropertyAdapter.java | 0 |
| Error while parsing C:\tmp\EchoBinding\src\main\java\net\sf\echobinding\binding\AbstractPropertyAdapter.java | 0 |
net/sf/echobinding/binding/OgnlBindingContext.java
| Violation | Line |
|---|
| Error while parsing C:\tmp\EchoBinding\src\main\java\net\sf\echobinding\binding\OgnlBindingContext.java | 0 |
| Error while parsing C:\tmp\EchoBinding\src\main\java\net\sf\echobinding\binding\OgnlBindingContext.java | 0 |
net/sf/echobinding/binding/OgnlPropertyAdapter.java
| Violation | Line |
|---|
| It is a good practice to call super() in a constructor | 61 - 63 |
| These nested if statements could be combined | 128 - 130 |
| A method should have only one exit point, and that should be the last statement in the method | 266 |
| A method should have only one exit point, and that should be the last statement in the method | 294 |
net/sf/echobinding/BindingConfiguration.java
| Violation | Line |
|---|
| Error while parsing C:\tmp\EchoBinding\src\main\java\net\sf\echobinding\BindingConfiguration.java | 0 |
| Error while parsing C:\tmp\EchoBinding\src\main\java\net\sf\echobinding\BindingConfiguration.java | 0 |
net/sf/echobinding/Configuration.java
| Violation | Line |
|---|
| Each class should declare at least one constructor | 36 - 53 |
net/sf/echobinding/ConfigurationSetup.java
| Violation | Line |
|---|
| Error while parsing C:\tmp\EchoBinding\src\main\java\net\sf\echobinding\ConfigurationSetup.java | 0 |
| Error while parsing C:\tmp\EchoBinding\src\main\java\net\sf\echobinding\ConfigurationSetup.java | 0 |
net/sf/echobinding/controls/BoundButtonGroup.java
| Violation | Line |
|---|
| Error while parsing C:\tmp\EchoBinding\src\main\java\net\sf\echobinding\controls\BoundButtonGroup.java | 0 |
| Error while parsing C:\tmp\EchoBinding\src\main\java\net\sf\echobinding\controls\BoundButtonGroup.java | 0 |
net/sf/echobinding/controls/Button.java
| Violation | Line |
|---|
| Perhaps '_context' could be replaced by a local variable. | 44 |
net/sf/echobinding/controls/CheckBox.java
| Violation | Line |
|---|
| It is a good practice to call super() in a constructor | 93 - 98 |
| Avoid instantiating Boolean objects; reference Boolean.TRUE or Boolean.FALSE or call Boolean.valueOf() instead. | 184 |
net/sf/echobinding/controls/RadioButton.java
| Violation | Line |
|---|
| 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 method | 107 |
net/sf/echobinding/controls/SelectField.java
| Violation | Line |
|---|
| Error while parsing C:\tmp\EchoBinding\src\main\java\net\sf\echobinding\controls\SelectField.java | 0 |
| Error while parsing C:\tmp\EchoBinding\src\main\java\net\sf\echobinding\controls\SelectField.java | 0 |
net/sf/echobinding/datacontrol/FormColumnTable.java
| Violation | Line |
|---|
| Error while parsing C:\tmp\EchoBinding\src\main\java\net\sf\echobinding\datacontrol\FormColumnTable.java | 0 |
| Error while parsing C:\tmp\EchoBinding\src\main\java\net\sf\echobinding\datacontrol\FormColumnTable.java | 0 |
net/sf/echobinding/datacontrol/FormHelper.java
| Violation | Line |
|---|
| Each class should declare at least one constructor | 39 - 141 |
net/sf/echobinding/decorator/DefaultDecorator.java
| Violation | Line |
|---|
| Each class should declare at least one constructor | 34 - 43 |
net/sf/echobinding/format/DateFormat.java
| Violation | Line |
|---|
| Perhaps '_datePattern' could be replaced by a local variable. | 39 |
net/sf/echobinding/format/TimeFormat.java
| Violation | Line |
|---|
| Each class should declare at least one constructor | 38 - 94 |
| A method should have only one exit point, and that should be the last statement in the method | 56 |
net/sf/echobinding/model/AbstractPresentationModel.java
| Violation | Line |
|---|
| Error while parsing C:\tmp\EchoBinding\src\main\java\net\sf\echobinding\model\AbstractPresentationModel.java | 0 |
| Error while parsing C:\tmp\EchoBinding\src\main\java\net\sf\echobinding\model\AbstractPresentationModel.java | 0 |
net/sf/echobinding/persistence/DefaultTablePersistenceManager.java
| Violation | Line |
|---|
| Error while parsing C:\tmp\EchoBinding\src\main\java\net\sf\echobinding\persistence\DefaultTablePersistenceManager.java | 0 |
| Error while parsing C:\tmp\EchoBinding\src\main\java\net\sf\echobinding\persistence\DefaultTablePersistenceManager.java | 0 |
net/sf/echobinding/table/BoundTable.java
| Violation | Line |
|---|
| Error while parsing C:\tmp\EchoBinding\src\main\java\net\sf\echobinding\table\BoundTable.java | 0 |
| Error while parsing C:\tmp\EchoBinding\src\main\java\net\sf\echobinding\table\BoundTable.java | 0 |
net/sf/echobinding/table/BoundTableCollectionCellRenderer.java
| Violation | Line |
|---|
| Each class should declare at least one constructor | 43 - 93 |
net/sf/echobinding/table/BoundTableHeaderRenderer.java
| Violation | Line |
|---|
| Each class should declare at least one constructor | 40 - 77 |
net/sf/echobinding/table/BoundTableModel.java
| Violation | Line |
|---|
| Use explicit scoping instead of the default package private level | 40 |
| Use explicit scoping instead of the default package private level | 41 |
| A method should have only one exit point, and that should be the last statement in the method | 65 |
net/sf/echobinding/table/DefaultDataBoundTableCellRenderer.java
| Violation | Line |
|---|
| Each class should declare at least one constructor | 43 - 84 |
net/sf/echobinding/table/EditableTableCellRenderer.java
| Violation | Line |
|---|
| Each class should declare at least one constructor | 37 - 70 |
| A method should have only one exit point, and that should be the last statement in the method | 51 |
net/sf/echobinding/table/EditableTableHeaderRenderer.java
| Violation | Line |
|---|
| Each class should declare at least one constructor | 36 - 52 |
| A method should have only one exit point, and that should be the last statement in the method | 44 |
net/sf/echobinding/table/EditableTableModel.java
| Violation | Line |
|---|
| Use explicit scoping instead of the default package private level | 38 |
| Use explicit scoping instead of the default package private level | 39 |
| A method should have only one exit point, and that should be the last statement in the method | 65 |
| A method should have only one exit point, and that should be the last statement in the method | 70 |
| A method should have only one exit point, and that should be the last statement in the method | 86 |
| A method should have only one exit point, and that should be the last statement in the method | 102 |
net/sf/echobinding/table/SelectableTable.java
| Violation | Line |
|---|
| Error while parsing C:\tmp\EchoBinding\src\main\java\net\sf\echobinding\table\SelectableTable.java | 0 |
| Error while parsing C:\tmp\EchoBinding\src\main\java\net\sf\echobinding\table\SelectableTable.java | 0 |
net/sf/echobinding/util/Images.java
| Violation | Line |
|---|
| Each class should declare at least one constructor | 35 - 52 |
net/sf/echobinding/util/Util.java
| Violation | Line |
|---|
| Each class should declare at least one constructor | 33 - 44 |
| Assigning an Object to null is a code smell. Consider refactoring. | 42 |
net/sf/echobinding/validation/BeanValidator.java
| Violation | Line |
|---|
| Error while parsing C:\tmp\EchoBinding\src\main\java\net\sf\echobinding\validation\BeanValidator.java | 0 |
| Error while parsing C:\tmp\EchoBinding\src\main\java\net\sf\echobinding\validation\BeanValidator.java | 0 |
net/sf/echobinding/validation/DateValidator.java
| Violation | Line |
|---|
| A method should have only one exit point, and that should be the last statement in the method | 48 |
net/sf/echobinding/validation/DefaultValidationHandler.java
| Violation | Line |
|---|
| A method should have only one exit point, and that should be the last statement in the method | 87 |
net/sf/echobinding/validation/NumberValidator.java
| Violation | Line |
|---|
| A method should have only one exit point, and that should be the last statement in the method | 64 |
net/sf/echobinding/validation/RegexValidator.java
| Violation | Line |
|---|
| A method should have only one exit point, and that should be the last statement in the method | 60 |
