The following document contains the results of PMD 3.7.
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 |
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 |
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 |
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 |
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 |
Violation | Line |
---|---|
Each class should declare at least one constructor | 36 - 53 |
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 |
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 |
Violation | Line |
---|---|
Perhaps '_context' could be replaced by a local variable. | 44 |
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 |
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 |
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 |
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 |
Violation | Line |
---|---|
Each class should declare at least one constructor | 39 - 141 |
Violation | Line |
---|---|
Each class should declare at least one constructor | 34 - 43 |
Violation | Line |
---|---|
Perhaps '_datePattern' could be replaced by a local variable. | 39 |
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 |
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 |
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 |
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 |
Violation | Line |
---|---|
Each class should declare at least one constructor | 43 - 93 |
Violation | Line |
---|---|
Each class should declare at least one constructor | 40 - 77 |
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 |
Violation | Line |
---|---|
Each class should declare at least one constructor | 43 - 84 |
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 |
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 |
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 |
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 |
Violation | Line |
---|---|
Each class should declare at least one constructor | 35 - 52 |
Violation | Line |
---|---|
Each class should declare at least one constructor | 33 - 44 |
Assigning an Object to null is a code smell. Consider refactoring. | 42 |
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 |
Violation | Line |
---|---|
A method should have only one exit point, and that should be the last statement in the method | 48 |
Violation | Line |
---|---|
A method should have only one exit point, and that should be the last statement in the method | 87 |
Violation | Line |
---|---|
A method should have only one exit point, and that should be the last statement in the method | 64 |
Violation | Line |
---|---|
A method should have only one exit point, and that should be the last statement in the method | 60 |