PMD Results

The following document contains the results of PMD 4.1.

Files

org/paneris/rimauresq/Display.java

Violation Line
Found 'DD'-anomaly for variable 'id' (lines '79'-'81'). 79 - 81
Found 'DD'-anomaly for variable 'id' (lines '81'-'89'). 81 - 89
Found 'DD'-anomaly for variable 'value' (lines '85'-'86'). 85 - 86
Found 'DD'-anomaly for variable 'url' (lines '103'-'104'). 103 - 104
A method should have only one exit point, and that should be the last statement in the method 107
A method should have only one exit point, and that should be the last statement in the method 115
A method should have only one exit point, and that should be the last statement in the method 119
A method should have only one exit point, and that should be the last statement in the method 123
A method should have only one exit point, and that should be the last statement in the method 128
Found 'DD'-anomaly for variable 'url' (lines '132'-'133'). 132 - 133
A method should have only one exit point, and that should be the last statement in the method 134
A method should have only one exit point, and that should be the last statement in the method 135
A method should have only one exit point, and that should be the last statement in the method 139
A method should have only one exit point, and that should be the last statement in the method 140

org/paneris/rimauresq/RimauresqServlet.java

Violation Line
Found 'DD'-anomaly for variable 'value' (lines '111'-'112'). 111 - 112
Found 'DU'-anomaly for variable 'value' (lines '112'-'137'). 112 - 137

org/paneris/rimauresq/Trolley.java

Violation Line
Use explicit scoping instead of the default package private level 84
Avoid unnecessary constructors - the compiler will generate these for you 89 - 91
Found 'DU'-anomaly for variable 'orderedItems' (lines '105'-'219'). 105 - 219
Found 'DU'-anomaly for variable 'orderedItems' (lines '105'-'219'). 105 - 219
Found 'DU'-anomaly for variable 'items' (lines '106'-'219'). 106 - 219
Found 'DU'-anomaly for variable 'items' (lines '106'-'219'). 106 - 219
Avoid using final local variables, turn them into fields 107
Avoid using final local variables, turn them into fields 108
A method should have only one exit point, and that should be the last statement in the method 118
A method should have only one exit point, and that should be the last statement in the method 144

org/paneris/rimauresq/model/Product.java

Violation Line
Avoid unnecessary constructors - the compiler will generate these for you 39
It is a good practice to call super() in a constructor 39

org/paneris/rimauresq/model/Setting.java

Violation Line
Avoid unnecessary constructors - the compiler will generate these for you 27
It is a good practice to call super() in a constructor 27

org/paneris/rimauresq/model/SettingTable.java

Violation Line
Do not hard code IPv4 or IPv6 addresses, even 127.0.0.1 ! 51

org/paneris/rimauresq/model/ShopOrder.java

Violation Line
Avoid unnecessary constructors - the compiler will generate these for you 58
It is a good practice to call super() in a constructor 58

org/paneris/rimauresq/model/ShopOrderItem.java

Violation Line
Avoid unnecessary constructors - the compiler will generate these for you 43
It is a good practice to call super() in a constructor 43

org/paneris/rimauresq/model/User.java

Violation Line
Avoid unnecessary constructors - the compiler will generate these for you 47
It is a good practice to call super() in a constructor 47