The following document contains the results of PMD 4.2.5.
| Violation | Line |
|---|---|
| Avoid instantiation through private constructors from outside of the constructor's class. | 59 |
| Avoid if (x != y) ..; else ..; | 114 - 118 |
| Avoid if (x != y) ..; else ..; | 197 - 201 |
| Avoid if (x != y) ..; else ..; | 238 - 260 |
| Violation | Line |
|---|---|
| Use ArrayList instead of Vector | 125 |
| Violation | Line |
|---|---|
| Avoid really long methods. | 43 - 158 |
| Avoid unused method parameters such as 'evt'. | 160 - 164 |
| Avoid unused method parameters such as 'evt'. | 170 - 174 |
| Prefer StringBuffer over += for concatenating strings | 182 |
| Avoid if (x != y) ..; else ..; | 185 - 190 |
| Prefer StringBuffer over += for concatenating strings | 186 |
| Prefer StringBuffer over += for concatenating strings | 189 |
| Prefer StringBuffer over += for concatenating strings | 191 |
| Perhaps 'degreeLabel' could be replaced by a local variable. | 220 |
| Perhaps 'okButton' could be replaced by a local variable. | 222 |
| Violation | Line |
|---|---|
| Avoid really long methods. | 40 - 236 |
| Avoid unused method parameters such as 'evt'. | 238 - 242 |
| Avoid unused method parameters such as 'evt'. | 248 - 252 |
| Prefer StringBuffer over += for concatenating strings | 262 |
| Prefer StringBuffer over += for concatenating strings | 264 |
| Prefer StringBuffer over += for concatenating strings | 268 |
| Prefer StringBuffer over += for concatenating strings | 271 |
| Perhaps 'buttonGroupType' could be replaced by a local variable. | 313 |
| Perhaps 'cancelButton' could be replaced by a local variable. | 314 |
| Perhaps 'columnsLabel' could be replaced by a local variable. | 315 |
| Perhaps 'okButton' could be replaced by a local variable. | 319 |
| Perhaps 'rowsLabel' could be replaced by a local variable. | 320 |
| Perhaps 'typeLabel' could be replaced by a local variable. | 323 |
| Violation | Line |
|---|---|
| This class has too many methods, consider refactoring it. | 71 - 1206 |
| Avoid really long classes. | 71 - 1206 |
| Too many fields | 71 - 1206 |
| Overridable method 'getJJMenuBar' called during object construction | 185 |
| Avoid empty catch blocks | 615 - 617 |
| Avoid empty catch blocks | 628 - 630 |
| Prefer StringBuffer over += for concatenating strings | 1202 |
| Violation | Line |
|---|---|
| The class 'MathViewer' has a Cyclomatic Complexity of 6 (Highest = 10). | 40 - 110 |
| The method 'main' has a Cyclomatic Complexity of 10. | 70 - 109 |
| Violation | Line |
|---|---|
| This class has too many methods, consider refactoring it. | 62 - 282 |
| The class 'ParametersDialog' has a Cyclomatic Complexity of 4 (Highest = 10). | 62 - 282 |
| Avoid instantiating new objects inside loops | 111 |
| The method 'createInputForParameter' has a Cyclomatic Complexity of 10. | 159 - 214 |
| Violation | Line |
|---|---|
| The class 'BiNode' has a Cyclomatic Complexity of 4 (Highest = 10). | 32 - 506 |
| The method 'insert' has a Cyclomatic Complexity of 10. | 143 - 185 |
| Violation | Line |
|---|---|
| StringBuffers can grow quite a lot, and so may become a source of memory leak (if the owning class has a long life time). | 47 |
| Violation | Line |
|---|---|
| Document empty constructor | 31 - 33 |
| Violation | Line |
|---|---|
| Avoid instantiation through private constructors from outside of the constructor's class. | 48 |
| Violation | Line |
|---|---|
| This final field could be made static | 40 |
