PMD Results

The following document contains the results of PMD .

Summary

Files Errors
69 427

Files

Files Violations
ch/elca/dependency/ArgvDialog.java 20
ch/elca/dependency/DPTool.java 15
ch/elca/dependency/DPToolErrorDialog.java 12
ch/elca/dependency/adapter/grappa/EdgeFactory.java 1
ch/elca/dependency/adapter/grappa/GraphFactory.java 1
ch/elca/dependency/adapter/grappa/GrappaSupport.java 1
ch/elca/dependency/adapter/grappa/NodeFactory.java 1
ch/elca/dependency/adapter/grappa/SubgraphFactory.java 1
ch/elca/dependency/core/Analyzer.java 12
ch/elca/dependency/core/DependencyModel.java 15
ch/elca/dependency/core/Filter.java 2
ch/elca/dependency/core/FilterApplicator.java 7
ch/elca/dependency/core/Observer.java 1
ch/elca/dependency/core/RawModel.java 16
ch/elca/dependency/core/Selection.java 1
ch/elca/dependency/core/classinfo/AttributeInfo.java 4
ch/elca/dependency/core/classinfo/ClassInfo.java 10
ch/elca/dependency/core/classinfo/ConstantPoolInfo.java 16
ch/elca/dependency/core/classinfo/FieldInfo.java 4
ch/elca/dependency/core/classinfo/MethodInfo.java 7
ch/elca/dependency/core/classinfo/Reader.java 24
ch/elca/dependency/exception/AnalyseException.java 1
ch/elca/dependency/exception/BadLayerDefinitionException.java 1
ch/elca/dependency/exception/FilterApplicationException.java 1
ch/elca/dependency/exception/FilterListIOException.java 1
ch/elca/dependency/exception/LayoutException.java 1
ch/elca/dependency/graph/AbstractGraphProcessor.java 3
ch/elca/dependency/graph/Aggregator.java 7
ch/elca/dependency/graph/Filter.java 2
ch/elca/dependency/graph/GraphUtils.java 4
ch/elca/dependency/graph/Layer.java 2
ch/elca/dependency/graph/LayerOrder.java 9
ch/elca/dependency/graph/LayeringEventSupport.java 1
ch/elca/dependency/graph/Perspective.java 4
ch/elca/dependency/graph/PerspectiveManager.java 4
ch/elca/dependency/gui/AboutDialog.java 4
ch/elca/dependency/gui/DependencyDialog.java 15
ch/elca/dependency/gui/DependencyTableModel.java 5
ch/elca/dependency/gui/FilterDialog.java 12
ch/elca/dependency/gui/FilterListTableModel.java 3
ch/elca/dependency/gui/FilterManipulationDialog.java 3
ch/elca/dependency/gui/LevelTableModel.java 3
ch/elca/dependency/gui/MainFrame.java 7
ch/elca/dependency/gui/MyProgressBar.java 7
ch/elca/dependency/gui/ReportDialog.java 4
ch/elca/dependency/gui/SnappingDesktopManager.java 9
ch/elca/dependency/layer/UserLayer.java 10
ch/elca/dependency/report/CompositeReportInfo.java 1
ch/elca/dependency/report/LayeringViolationReportInfo.java 8
ch/elca/dependency/report/Report.java 6
ch/elca/dependency/report/ReportManager.java 5
ch/elca/dependency/util/ArrayQueue.java 1
ch/elca/dependency/util/DefaultPropertiesReader.java 2
ch/elca/dependency/util/DiskFileReader.java 2
ch/elca/dependency/util/IOManager.java 2
ch/elca/dependency/util/IconGrabber.java 1
ch/elca/dependency/util/Queue.java 1
ch/elca/dependency/util/Shutdown.java 2
ch/elca/dependency/util/StringManager.java 1
ch/elca/dependency/util/UnZip.java 1
ch/elca/dependency/util/gui/PersistentJFrame.java 1
ch/elca/dependency/util/gui/PersistentJInternalFrame.java 1
ch/elca/dependency/view/ClassDepView.java 14
ch/elca/dependency/view/GraphView.java 19
ch/elca/dependency/view/MyGrappaAdapter.java 32
ch/elca/dependency/view/OverView.java 16
ch/elca/dependency/view/PackageDepView.java 13
ch/elca/dependency/view/StatisticView.java 3
ch/elca/dependency/view/View.java 1

ch/elca/dependency/ArgvDialog.java

Violation Line
Avoid unused private fields such as 'ROWS' 69
Avoid unused private fields such as 'COLUMNS' 70
Avoid variables with short names 127
Avoid empty 'while' statements 130
Avoid really long methods. 188
Avoid variables with short names 375
Avoid empty catch blocks 380
Avoid variables with short names 388
Avoid variables with short names 393
Avoid variables with short names 398
Avoid variables with short names 403
Avoid variables with short names 408
Avoid variables with short names 413
Avoid variables with short names 418
Avoid variables with short names 423
Avoid variables with short names 448
Avoid empty catch blocks 453
Avoid variables with short names 478
Avoid empty catch blocks 483
Avoid variables with short names 609

ch/elca/dependency/DPTool.java

Violation Line
All methods are static. Consider using Singleton instead. 31
Avoid unused imports such as 'Hashtable' 44
The class 'DPTool' has a Cyclomatic Complexity of 7 (Highest = 10). 56
Avoid really long methods. 89
Avoid reassigning parameters such as 'argv' 89
The method 'main' has a Cyclomatic Complexity of 10. 89
Avoid variables with short names 103
Avoid variables with short names 128
Avoid variables with short names 146
Avoid variables with short names 184
Avoid variables with short names 215
The same String literal appears 9 times in this file; the first occurrence is on line 225 225
Avoid unused local variables such as 'helpRequest' 292
Avoid variables with short names 294
Avoid empty catch blocks 298

ch/elca/dependency/DPToolErrorDialog.java

Violation Line
Avoid unused imports such as 'WindowConstants' 47
Avoid variables with short names 94
Avoid variables with short names 127
Avoid variables with short names 135
Avoid empty 'while' statements 138
Avoid variables with short names 163
Avoid empty 'while' statements 166
Avoid variables with short names 341
Avoid variables with short names 346
Avoid variables with short names 351
Avoid variables with short names 364
Avoid variables with short names 369

ch/elca/dependency/adapter/grappa/EdgeFactory.java

Violation Line
All methods are static. Consider using Singleton instead. 29

ch/elca/dependency/adapter/grappa/GraphFactory.java

Violation Line
All methods are static. Consider using Singleton instead. 29

ch/elca/dependency/adapter/grappa/GrappaSupport.java

Violation Line
All methods are static. Consider using Singleton instead. 29

ch/elca/dependency/adapter/grappa/NodeFactory.java

Violation Line
All methods are static. Consider using Singleton instead. 30

ch/elca/dependency/adapter/grappa/SubgraphFactory.java

Violation Line
All methods are static. Consider using Singleton instead. 29

ch/elca/dependency/core/Analyzer.java

Violation Line
Avoid using implementation types like 'HashMap'; use the interface instead 103
Avoid using implementation types like 'HashMap'; use the interface instead 110
Avoid using implementation types like 'HashMap'; use the interface instead 115
Avoid using implementation types like 'HashMap'; use the interface instead 120
Avoid using implementation types like 'ArrayList'; use the interface instead 141
Avoid variables with short names 164
Avoid variables with short names 188
Avoid variables with short names 190
Avoid using implementation types like 'HashMap'; use the interface instead 204
Avoid using implementation types like 'HashMap'; use the interface instead 224
Avoid variables with short names 259
Avoid variables with short names 311

ch/elca/dependency/core/DependencyModel.java

Violation Line
Avoid using implementation types like 'HashMap'; use the interface instead 106
Avoid using implementation types like 'ArrayList'; use the interface instead 122
Avoid using implementation types like 'HashSet'; use the interface instead 137
Avoid using implementation types like 'HashMap'; use the interface instead 145
Avoid using implementation types like 'HashSet'; use the interface instead 150
Avoid calls to overridable methods during construction 199
Avoid using implementation types like 'HashMap'; use the interface instead 312
Avoid using implementation types like 'ArrayList'; use the interface instead 339
Avoid using implementation types like 'HashSet'; use the interface instead 381
Avoid using implementation types like 'ArrayList'; use the interface instead 460
Avoid using implementation types like 'HashSet'; use the interface instead 509
Avoid using implementation types like 'HashMap'; use the interface instead 518
Avoid using implementation types like 'HashMap'; use the interface instead 537
Avoid using implementation types like 'HashMap'; use the interface instead 555
Avoid using implementation types like 'ArrayList'; use the interface instead 573

ch/elca/dependency/core/Filter.java

Violation Line
The same String literal appears 5 times in this file; the first occurrence is on line 92 92
Avoid variables with short names 153

ch/elca/dependency/core/FilterApplicator.java

Violation Line
The class 'FilterApplicator' has a Cyclomatic Complexity of 8 (Highest = 12). 50
Avoid really long methods. 93
Avoid using implementation types like 'ArrayList'; use the interface instead 93
Avoid variables with short names 93
The method 'apply' has a Cyclomatic Complexity of 12. 93
Avoid variables with short names 245
Avoid using implementation types like 'ArrayList'; use the interface instead 261

ch/elca/dependency/core/Observer.java

Violation Line
Avoid using implementation types like 'ArrayList'; use the interface instead 68

ch/elca/dependency/core/RawModel.java

Violation Line
The class 'RawModel' has a Cyclomatic Complexity of 3 (Highest = 19). 68
Avoid using implementation types like 'HashMap'; use the interface instead 88
Avoid using implementation types like 'HashMap'; use the interface instead 93
Avoid calls to overridable methods during construction 113
Avoid really long methods. 124
The method 'createModel' has a Cyclomatic Complexity of 19. 124
Avoid variables with short names 189
Deeply nested if..then statements are hard to read 216
Deeply nested if..then statements are hard to read 219
Deeply nested if..then statements are hard to read 249
Deeply nested if..then statements are hard to read 255
Avoid using implementation types like 'ArrayList'; use the interface instead 288
Avoid using implementation types like 'HashMap'; use the interface instead 288
Avoid using implementation types like 'ArrayList'; use the interface instead 330
Avoid using implementation types like 'HashMap'; use the interface instead 339
Avoid using implementation types like 'HashMap'; use the interface instead 369

ch/elca/dependency/core/Selection.java

Violation Line
Avoid using implementation types like 'ArrayList'; use the interface instead 48

ch/elca/dependency/core/classinfo/AttributeInfo.java

Violation Line
Avoid variables with short names 48
Avoid variables with short names 69
Avoid variables with short names 75
Avoid using if statements without curly braces 94

ch/elca/dependency/core/classinfo/ClassInfo.java

Violation Line
Avoid using implementation types like 'ArrayList'; use the interface instead 67
Avoid using implementation types like 'ArrayList'; use the interface instead 72
Avoid using implementation types like 'HashSet'; use the interface instead 78
Avoid using implementation types like 'ArrayList'; use the interface instead 94
Avoid using implementation types like 'ArrayList'; use the interface instead 95
Avoid using implementation types like 'HashSet'; use the interface instead 95
Avoid using implementation types like 'ArrayList'; use the interface instead 198
Avoid using implementation types like 'ArrayList'; use the interface instead 209
Avoid using implementation types like 'HashSet'; use the interface instead 220
Avoid variables with short names 248

ch/elca/dependency/core/classinfo/ConstantPoolInfo.java

Violation Line
The class 'ConstantPoolInfo' has a Cyclomatic Complexity of 4 (Highest = 15). 35
The method 'read' has a Cyclomatic Complexity of 15. 139
Avoid variables with short names 143
The method 'toString' has a Cyclomatic Complexity of 10. 255
Avoid variables with short names 256
Avoid using 'if...else' statements without curly braces 281
Avoid using if statements without curly braces 283
Avoid using 'if...else' statements without curly braces 285
Avoid using if statements without curly braces 287
Avoid using if statements without curly braces 296
Avoid variables with short names 305
The method 'isEqual' has a Cyclomatic Complexity of 10. 305
Avoid using if statements without curly braces 306
Avoid using if statements without curly braces 309
Switch statements should have a default label 311
Avoid using if statements without curly braces 341

ch/elca/dependency/core/classinfo/FieldInfo.java

Violation Line
Avoid variables with short names 44
Avoid using if statements without curly braces 56
Avoid variables with short names 80
Avoid variables with short names 91

ch/elca/dependency/core/classinfo/MethodInfo.java

Violation Line
Avoid variables with short names 48
Avoid using 'for' statements without curly braces 81
Avoid variables with short names 92
Avoid unused local variables such as 'isArray' 93
Avoid unused local variables such as 'ndx' 96
Avoid variables with short names 102
Avoid using 'if...else' statements without curly braces 108

ch/elca/dependency/core/classinfo/Reader.java

Violation Line
The class 'Reader' has a Cyclomatic Complexity of 6 (Highest = 23). 68
Avoid really long methods. 110
Avoid variables with short names 110
The method 'read' has a Cyclomatic Complexity of 23. 110
Avoid variables with short names 112
Avoid using if statements without curly braces 135
Avoid using if statements without curly braces 181
The method 'typeString' has a Cyclomatic Complexity of 15. 277
Avoid variables with short names 280
Switch statements should have a default label 287
Avoid using 'if...else' statements without curly braces 308
The method 'accessString' has a Cyclomatic Complexity of 12. 338
Avoid variables with short names 339
Avoid variables with short names 394
Avoid using 'while' statements without curly braces 396
Avoid using 'while' statements without curly braces 400
Avoid variables with short names 432
Avoid variables with short names 433
Avoid using implementation types like 'ArrayList'; use the interface instead 461
Avoid using implementation types like 'ArrayList'; use the interface instead 494
Avoid variables with short names 498
Avoid variables with short names 525
Avoid using implementation types like 'HashSet'; use the interface instead 568
Avoid unused private methods such as 'makeNiceString' 592

ch/elca/dependency/exception/AnalyseException.java

Violation Line
Avoid variables with short names 74

ch/elca/dependency/exception/BadLayerDefinitionException.java

Violation Line
Avoid variables with short names 68

ch/elca/dependency/exception/FilterApplicationException.java

Violation Line
Avoid variables with short names 71

ch/elca/dependency/exception/FilterListIOException.java

Violation Line
Avoid variables with short names 72

ch/elca/dependency/exception/LayoutException.java

Violation Line
Avoid variables with short names 69

ch/elca/dependency/graph/AbstractGraphProcessor.java

Violation Line
Avoid unused private fields such as 'LOG' 52
Avoid unused private fields such as 'STR' 57
Avoid calls to overridable methods during construction 80

ch/elca/dependency/graph/Aggregator.java

Violation Line
The class 'Aggregator' has a Cyclomatic Complexity of 3 (Highest = 12). 49
Avoid calls to overridable methods during construction 88
Avoid variables with short names 100
The method 'process' has a Cyclomatic Complexity of 12. 133
Deeply nested if..then statements are hard to read 165
Deeply nested if..then statements are hard to read 183
Deeply nested if..then statements are hard to read 208

ch/elca/dependency/graph/Filter.java

Violation Line
Avoid calls to overridable methods during construction 93
Avoid variables with short names 105

ch/elca/dependency/graph/GraphUtils.java

Violation Line
All methods are static. Consider using Singleton instead. 30
The class 'GraphUtils' has a Cyclomatic Complexity of 4 (Highest = 16). 48
Avoid really long methods. 58
The method 'copy' has a Cyclomatic Complexity of 16. 58

ch/elca/dependency/graph/Layer.java

Violation Line
Avoid using implementation types like 'ArrayList'; use the interface instead 59
Avoid calls to overridable methods during construction 73

ch/elca/dependency/graph/LayerOrder.java

Violation Line
The class 'LayerOrder' has a Cyclomatic Complexity of 2 (Highest = 13). 53
Avoid using implementation types like 'ArrayList'; use the interface instead 68
Avoid calls to overridable methods during construction 92
Avoid really long methods. 121
The method 'process' has a Cyclomatic Complexity of 13. 121
Avoid unused local variables such as 'nodeName' 130
Avoid variables with short names 141
Avoid variables with short names 246
Avoid empty catch blocks 248

ch/elca/dependency/graph/LayeringEventSupport.java

Violation Line
Avoid using implementation types like 'ArrayList'; use the interface instead 51

ch/elca/dependency/graph/Perspective.java

Violation Line
Avoid using implementation types like 'ArrayList'; use the interface instead 71
Avoid using implementation types like 'ArrayList'; use the interface instead 76
Avoid calls to overridable methods during construction 93
Avoid using implementation types like 'ArrayList'; use the interface instead 131

ch/elca/dependency/graph/PerspectiveManager.java

Violation Line
All methods are static. Consider using Singleton instead. 30
Avoid variables with short names 87
Avoid variables with short names 120
Avoid using implementation types like 'ArrayList'; use the interface instead 135

ch/elca/dependency/gui/AboutDialog.java

Violation Line
Avoid variables with short names 93
Avoid variables with short names 138
Avoid variables with short names 139
Avoid variables with short names 141

ch/elca/dependency/gui/DependencyDialog.java

Violation Line
The class 'DependencyDialog' has a Cyclomatic Complexity of 4 (Highest = 16). 65
Avoid calls to overridable methods during construction 93
Avoid variables with short names 103
Avoid variables with short names 166
Avoid variables with short names 176
Avoid really long methods. 208
The method 'initializeTable' has a Cyclomatic Complexity of 16. 208
Avoid unused local variables such as 'keys' 229
Avoid variables with short names 251
The same String literal appears 9 times in this file; the first occurrence is on line 305 305
Avoid variables with short names 312
Avoid variables with short names 326
Avoid variables with short names 345
Avoid unused local variables such as 'graph' 371
Avoid variables with short names 404

ch/elca/dependency/gui/DependencyTableModel.java

Violation Line
Avoid using implementation types like 'ArrayList'; use the interface instead 65
Avoid variables with short names 168
Avoid variables with short names 252
Avoid variables with short names 255
Avoid variables with short names 256

ch/elca/dependency/gui/FilterDialog.java

Violation Line
Avoid unused imports such as 'Iterator' 36
Avoid using implementation types like 'ArrayList'; use the interface instead 88
Avoid calls to overridable methods during construction 114
Avoid variables with short names 125
Avoid unused local variables such as 'pref' 133
Avoid variables with short names 167
Avoid variables with short names 177
Avoid variables with short names 387
Avoid variables with short names 444
Avoid variables with short names 481
Avoid variables with short names 580
Avoid empty 'if' statements 587

ch/elca/dependency/gui/FilterListTableModel.java

Violation Line
Avoid using implementation types like 'ArrayList'; use the interface instead 75
Avoid empty catch blocks 250
Avoid variables with short names 250

ch/elca/dependency/gui/FilterManipulationDialog.java

Violation Line
Avoid unused imports such as 'Insets' 33
Avoid calls to overridable methods during construction 66
The same String literal appears 4 times in this file; the first occurrence is on line 123 123

ch/elca/dependency/gui/LevelTableModel.java

Violation Line
Avoid using implementation types like 'ArrayList'; use the interface instead 63
Avoid empty catch blocks 198
Avoid variables with short names 198

ch/elca/dependency/gui/MainFrame.java

Violation Line
Avoid importing anything from the package 'java.lang' 42
Avoid unused private fields such as 'm_dependencyDialog' 74
Avoid variables with short names 86
Avoid variables with short names 125
Avoid variables with short names 384
Avoid variables with short names 426
Avoid variables with short names 468

ch/elca/dependency/gui/MyProgressBar.java

Violation Line
Avoid calls to overridable methods during construction 86
Avoid calls to overridable methods during construction 98
Avoid variables with short names 139
Avoid variables with short names 168
Avoid unused private fields such as 'count' 202
Avoid variables with short names 232
Avoid variables with short names 242

ch/elca/dependency/gui/ReportDialog.java

Violation Line
Avoid variables with short names 72
Avoid variables with short names 247
Avoid variables with short names 252
Avoid variables with short names 291

ch/elca/dependency/gui/SnappingDesktopManager.java

Violation Line
The class 'SnappingDesktopManager' has a Cyclomatic Complexity of 6 (Highest = 19). 44
Avoid variables with short names 55
Avoid variables with short names 60
The method 'findSnapping' has a Cyclomatic Complexity of 19. 60
Avoid variables with short names 61
Avoid variables with short names 62
Deeply nested if..then statements are hard to read 85
Deeply nested if..then statements are hard to read 94
Avoid variables with short names 150

ch/elca/dependency/layer/UserLayer.java

Violation Line
The class 'UserLayer' has a Cyclomatic Complexity of 4 (Highest = 16). 54
Avoid using implementation types like 'HashMap'; use the interface instead 81
Avoid using implementation types like 'HashMap'; use the interface instead 86
Avoid using implementation types like 'ArrayList'; use the interface instead 91
Avoid using implementation types like 'ArrayList'; use the interface instead 174
Avoid really long methods. 223
The method 'parseLayerFile' has a Cyclomatic Complexity of 16. 223
Avoid variables with short names 362
Avoid using implementation types like 'ArrayList'; use the interface instead 371
Avoid variables with short names 392

ch/elca/dependency/report/CompositeReportInfo.java

Violation Line
Avoid using implementation types like 'ArrayList'; use the interface instead 62

ch/elca/dependency/report/LayeringViolationReportInfo.java

Violation Line
The class 'LayeringViolationReportInfo' has a Cyclomatic Complexity of 10 (Highest = 16). 60
Avoid really long methods. 142
The method 'layeringViolated' has a Cyclomatic Complexity of 16. 142
Avoid unused local variables such as 'tmpObject' 146
The same String literal appears 9 times in this file; the first occurrence is on line 218 218
Avoid variables with short names 224
Avoid variables with short names 240
Avoid variables with short names 260

ch/elca/dependency/report/Report.java

Violation Line
Avoid variables with short names 179
The same String literal appears 4 times in this file; the first occurrence is on line 194 194
Avoid variables with short names 216
Avoid variables with short names 318
Avoid variables with short names 342
Avoid variables with short names 361

ch/elca/dependency/report/ReportManager.java

Violation Line
All methods are static. Consider using Singleton instead. 30
Avoid variables with short names 133
Avoid variables with short names 152
Avoid variables with short names 165
Avoid variables with short names 200

ch/elca/dependency/util/ArrayQueue.java

Violation Line
Avoid variables with short names 94

ch/elca/dependency/util/DefaultPropertiesReader.java

Violation Line
Avoid unused imports such as 'File' 32
Avoid variables with short names 103

ch/elca/dependency/util/DiskFileReader.java

Violation Line
Avoid variables with short names 80
Avoid variables with short names 97

ch/elca/dependency/util/IOManager.java

Violation Line
All methods are static. Consider using Singleton instead. 30
Avoid unused private fields such as 'SEPARATOR' 71

ch/elca/dependency/util/IconGrabber.java

Violation Line
All methods are static. Consider using Singleton instead. 31

ch/elca/dependency/util/Queue.java

Violation Line
Avoid variables with short names 49

ch/elca/dependency/util/Shutdown.java

Violation Line
All methods are static. Consider using Singleton instead. 30
Avoid using implementation types like 'ArrayList'; use the interface instead 55

ch/elca/dependency/util/StringManager.java

Violation Line
Avoid variables with short names 99

ch/elca/dependency/util/UnZip.java

Violation Line
Avoid variables with short names 148

ch/elca/dependency/util/gui/PersistentJFrame.java

Violation Line
Avoid calls to overridable methods during construction 73

ch/elca/dependency/util/gui/PersistentJInternalFrame.java

Violation Line
Avoid calls to overridable methods during construction 81

ch/elca/dependency/view/ClassDepView.java

Violation Line
The class 'ClassDepView' has a Cyclomatic Complexity of 4 (Highest = 10). 57
Avoid using implementation types like 'HashMap'; use the interface instead 113
Avoid using implementation types like 'HashMap'; use the interface instead 129
Avoid variables with short names 187
Avoid variables with short names 202
Avoid variables with short names 220
Avoid using implementation types like 'ArrayList'; use the interface instead 281
The same String literal appears 4 times in this file; the first occurrence is on line 467 467
Avoid empty catch blocks 480
The method 'addIsUsedAncestors' has a Cyclomatic Complexity of 10. 492
Avoid empty catch blocks 560
Avoid variables with short names 654
Avoid variables with short names 691
Avoid variables with short names 748

ch/elca/dependency/view/GraphView.java

Violation Line
Avoid unused imports such as 'Edge' 35
Avoid unused imports such as 'Element' 36
Avoid unused imports such as 'GrappaPoint' 39
Avoid unused imports such as 'Node' 42
Avoid duplicate imports such as 'java.awt.event.InputEvent' 53
The class 'GraphView' has a Cyclomatic Complexity of 3 (Highest = 11). 82
Avoid using implementation types like 'ArrayList'; use the interface instead 166
Avoid really long methods. 206
Avoid unused local variables such as 'pref' 286
Avoid using implementation types like 'ArrayList'; use the interface instead 386
Avoid variables with short names 522
The method 'doLayoutOverWeb' has a Cyclomatic Complexity of 11. 532
Avoid variables with short names 544
Switch statements should have a default label 548
Avoid variables with short names 598
Avoid variables with short names 625
The same String literal appears 4 times in this file; the first occurrence is on line 705 705
Avoid variables with short names 758
Avoid variables with short names 790

ch/elca/dependency/view/MyGrappaAdapter.java

Violation Line
The class 'MyGrappaAdapter' has a Cyclomatic Complexity of 12 (Highest = 31). 54
Avoid unused private fields such as 'CAT' 59
Avoid really long methods. 121
Avoid variables with short names 121
The method 'grappaClicked' has a Cyclomatic Complexity of 31. 121
Avoid using if statements without curly braces 153
Deeply nested if..then statements are hard to read 155
Deeply nested if..then statements are hard to read 184
Deeply nested if..then statements are hard to read 194
Avoid using if statements without curly braces 229
Avoid empty 'if' statements 288
Avoid variables with short names 301
The same String literal appears 4 times in this file; the first occurrence is on line 314 314
Avoid variables with short names 365
Avoid unused local variables such as 'centerPoint' 400
Avoid variables with short names 468
The method 'grappaTip' has a Cyclomatic Complexity of 17. 468
Avoid variables with short names 480
Avoid really long methods. 537
The method 'actionPerformed' has a Cyclomatic Complexity of 31. 537
Avoid variables with short names 544
Avoid using if statements without curly braces 549
Avoid using if statements without curly braces 576
Avoid using if statements without curly braces 590
Avoid variables with short names 626
Avoid variables with short names 627
Avoid using if statements without curly braces 628
Avoid variables with short names 636
Avoid using implementation types like 'Vector'; use the interface instead 675
The method 'drillDown' has a Cyclomatic Complexity of 17. 675
A high ratio of statements to labels in a switch statement. Consider refactoring. 691
Switch statements should have a default label 691

ch/elca/dependency/view/OverView.java

Violation Line
The class 'OverView' has a Cyclomatic Complexity of 3 (Highest = 10). 74
Avoid unused private fields such as 'm_preOrderList' 123
Avoid using implementation types like 'ArrayList'; use the interface instead 123
Avoid unused private fields such as 'm_foundOne' 130
Avoid variables with short names 225
Avoid using implementation types like 'ArrayList'; use the interface instead 276
Avoid variables with short names 307
Avoid variables with short names 340
Avoid variables with short names 353
Avoid empty catch blocks 357
Avoid variables with short names 366
Avoid empty catch blocks 370
The method 'analyze' has a Cyclomatic Complexity of 10. 387
Avoid variables with short names 408
Avoid variables with short names 491
Avoid variables with short names 535

ch/elca/dependency/view/PackageDepView.java

Violation Line
Avoid using implementation types like 'HashMap'; use the interface instead 108
Avoid really long methods. 124
Avoid variables with short names 148
Avoid variables with short names 163
Avoid variables with short names 181
Avoid using implementation types like 'ArrayList'; use the interface instead 269
Avoid empty catch blocks 336
Avoid variables with short names 349
Avoid empty catch blocks 475
Avoid empty catch blocks 543
Avoid variables with short names 614
Avoid variables with short names 643
Avoid variables with short names 694

ch/elca/dependency/view/StatisticView.java

Violation Line
Avoid unused private fields such as 'm_projLibPack' 75
Avoid using implementation types like 'ArrayList'; use the interface instead 265
Avoid using implementation types like 'ArrayList'; use the interface instead 284

ch/elca/dependency/view/View.java

Violation Line
Avoid calls to overridable methods during construction 104