PMD Results

The following document contains the results of PMD 5.2.1.

Files

wjhk/jupload2/EmptyApplet.java

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

wjhk/jupload2/JUploadDaemon.java

Violation Line
Use explicit scoping instead of the default package private level 43
Assigning an Object to null is a code smell. Consider refactoring. 82

wjhk/jupload2/context/JUploadContextApplet.java

Violation Line
Use explicit scoping instead of the default package private level 56
It is a good practice to call super() in a constructor 63–74
Use explicit scoping instead of the default package private level 81–122
Found 'DD'-anomaly for variable 'ex' (lines '84'-'105'). 84–105
Found 'DD'-anomaly for variable 'ex' (lines '84'-'91'). 84–91
Found 'DD'-anomaly for variable 'ex' (lines '91'-'105'). 91–105
Found 'DD'-anomaly for variable 'ex' (lines '105'-'111'). 105–111
A method should have only one exit point, and that should be the last statement in the method 134
Found 'DD'-anomaly for variable 'cookie' (lines '204'-'209'). 204–209
Found 'DD'-anomaly for variable 'userAgent' (lines '245'-'250'). 245–250
Found 'DD'-anomaly for variable 'expr' (lines '311'-'315'). 311–315
Useless parentheses. 325
Found 'DU'-anomaly for variable 'target' (lines '332'-'354'). 332–354
Avoid using Literals in Conditional Statements 333
A method should have only one exit point, and that should be the last statement in the method 366
Found 'DD'-anomaly for variable 'uri' (lines '367'-'369'). 367–369
Found 'DU'-anomaly for variable 'dollarReplacement' (lines '392'-'406'). 392–406
Found 'DU'-anomaly for variable 'singleQuoteReplacement' (lines '393'-'406'). 393–406
Found 'DU'-anomaly for variable 'linefeedReplacement' (lines '394'-'406'). 394–406
A method should have only one exit point, and that should be the last statement in the method 397

wjhk/jupload2/context/JUploadContextExecutable.java

Violation Line
Use explicit scoping instead of the default package private level 51
Use explicit scoping instead of the default package private level 53
It is a good practice to call super() in a constructor 75–80
It is a good practice to call super() in a constructor 89–110
Use explicit scoping instead of the default package private level 119–135

wjhk/jupload2/context/JavascriptHandler.java

Violation Line
It is a good practice to call super() in a constructor 71–80
A method should have only one exit point, and that should be the last statement in the method 95
Use explicit scoping instead of the default package private level 117–123
Assigning an Object to null is a code smell. Consider refactoring. 129

wjhk/jupload2/exception/JUploadEOFException.java

Violation Line
Use explicit scoping instead of the default package private level 40
Use explicit scoping instead of the default package private level 42
Avoid using Literals in Conditional Statements 60
A method should have only one exit point, and that should be the last statement in the method 61

wjhk/jupload2/exception/JUploadException.java

Violation Line
A method should have only one exit point, and that should be the last statement in the method 107

wjhk/jupload2/filedata/DefaultFileData.java

Violation Line
Use explicit scoping instead of the default package private level 59
Use explicit scoping instead of the default package private level 66
Use explicit scoping instead of the default package private level 86
Assigning an Object to null is a code smell. Consider refactoring. 201
Found 'DU'-anomaly for variable 'ret' (lines '270'-'307'). 270–307
Found 'DD'-anomaly for variable 'digest' (lines '271'-'281'). 271–281
Found 'DU'-anomaly for variable 'digest' (lines '281'-'307'). 281–307
Avoid assignments in operands 282
Found 'DU'-anomaly for variable 'nbBytes' (lines '282'-'307'). 282–307
Avoid instantiating Boolean objects; reference Boolean.TRUE or Boolean.FALSE or call Boolean.valueOf() instead. 328
Avoid instantiating Boolean objects; reference Boolean.TRUE or Boolean.FALSE or call Boolean.valueOf() instead. 331
Found 'UR'-anomaly for variable 'i' (lines '354'-'377'). 354–377
Found 'UR'-anomaly for variable 'treePath' (lines '354'-'377'). 354–377
Found 'DU'-anomaly for variable 'ret' (lines '355'-'386'). 355–386
Found 'DD'-anomaly for variable 'ret' (lines '355'-'361'). 355–361
Found 'DD'-anomaly for variable 'ret' (lines '355'-'367'). 355–367
Found 'DU'-anomaly for variable 'ret' (lines '361'-'386'). 361–386
Found 'DD'-anomaly for variable 'skip' (lines '363'-'365'). 363–365
Useless parentheses. 366
Useless parentheses. 366
Found 'DU'-anomaly for variable 'ret' (lines '367'-'386'). 367–386
Found 'DD'-anomaly for variable 'pathCurrentFileParentPath' (lines '441'-'447'). 441–447
Assigning an Object to null is a code smell. Consider refactoring. 461

wjhk/jupload2/filedata/PictureFileData.java

Violation Line
Use explicit scoping instead of the default package private level 71
Use explicit scoping instead of the default package private level 77
Use explicit scoping instead of the default package private level 89
Use explicit scoping instead of the default package private level 96
Use explicit scoping instead of the default package private level 101
Use explicit scoping instead of the default package private level 107
Use explicit scoping instead of the default package private level 117
Use explicit scoping instead of the default package private level 125
Use explicit scoping instead of the default package private level 130
Avoid using Literals in Conditional Statements 172
These nested if statements could be combined 191–236
A method should have only one exit point, and that should be the last statement in the method 284
Found 'DU'-anomaly for variable 'localImage' (lines '321'-'380'). 321–380
Found 'DD'-anomaly for variable 'localImage' (lines '321'-'359'). 321–359
A method should have only one exit point, and that should be the last statement in the method 332
A method should have only one exit point, and that should be the last statement in the method 339
A method should have only one exit point, and that should be the last statement in the method 345
Assigning an Object to null is a code smell. Consider refactoring. 356
Found 'DU'-anomaly for variable 'irwh' (lines '356'-'380'). 356–380
Found 'DD'-anomaly for variable 'localImage' (lines '359'-'365'). 359–365
Assigning an Object to null is a code smell. Consider refactoring. 362
Found 'DU'-anomaly for variable 'sourceImage' (lines '362'-'380'). 362–380
Assigning an Object to null is a code smell. Consider refactoring. 365
Assigning an Object to null is a code smell. Consider refactoring. 410
Assigning an Object to null is a code smell. Consider refactoring. 444
Use explicit scoping instead of the default package private level 457–498
Use explicit scoping instead of the default package private level 508–576
Found 'DU'-anomaly for variable 'metadata' (lines '511'-'576'). 511–576
Found 'DD'-anomaly for variable 'metadata' (lines '511'-'546'). 511–546
Found 'DD'-anomaly for variable 'iioImage' (lines '512'-'549'). 512–549
Found 'DU'-anomaly for variable 'iioImage' (lines '512'-'576'). 512–576
Found 'DD'-anomaly for variable 'originalImage' (lines '513'-'540'). 513–540
Found 'DD'-anomaly for variable 'transformedImage' (lines '514'-'541'). 514–541
Found 'DU'-anomaly for variable 'transformedImage' (lines '514'-'576'). 514–576
Found 'DU'-anomaly for variable 'transmitMetadata' (lines '517'-'576'). 517–576
Found 'DD'-anomaly for variable 'nbPictures' (lines '526'-'530'). 526–530
Assigning an Object to null is a code smell. Consider refactoring. 557
Found 'DD'-anomaly for variable 'originalImage' (lines '557'-'540'). 557–540
Assigning an Object to null is a code smell. Consider refactoring. 570
Found 'DU'-anomaly for variable 'originalImage' (lines '570'-'576'). 570–576
Use explicit scoping instead of the default package private level 583–587
Use explicit scoping instead of the default package private level 593–599
Use explicit scoping instead of the default package private level 607–620
Use explicit scoping instead of the default package private level 630–654
Use explicit scoping instead of the default package private level 662–673
Assigning an Object to null is a code smell. Consider refactoring. 686
Found 'DD'-anomaly for variable 'is' (lines '704'-'709'). 704–709
Found 'DD'-anomaly for variable 'os' (lines '705'-'710'). 705–710
Found 'DU'-anomaly for variable 'os' (lines '710'-'744'). 710–744
Avoid assignments in operands 713
Found 'DU'-anomaly for variable 'l' (lines '713'-'744'). 713–744
Assigning an Object to null is a code smell. Consider refactoring. 727
Assigning an Object to null is a code smell. Consider refactoring. 738
Found 'DU'-anomaly for variable 'imageIcon' (lines '781'-'821'). 781–821
Found 'DU'-anomaly for variable 'tmpImage' (lines '784'-'821'). 784–821
Found 'DD'-anomaly for variable 'tmpImage' (lines '784'-'790'). 784–790
These nested if statements could be combined 789–791
Found 'DU'-anomaly for variable 'tmpImage' (lines '790'-'821'). 790–821
Avoid using Literals in Conditional Statements 803
Assigning an Object to null is a code smell. Consider refactoring. 815

wjhk/jupload2/filedata/helper/ImageFileConversionInfo.java

Violation Line
If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation 36
A method should have only one exit point, and that should be the last statement in the method 64
A method should have only one exit point, and that should be the last statement in the method 89
A method should have only one exit point, and that should be the last statement in the method 93
Found 'UR'-anomaly for variable 'sourceFormat' (lines '105'-'151'). 105–151
Found 'UR'-anomaly for variable 'relation' (lines '105'-'132'). 105–132
Avoid using Literals in Conditional Statements 133
Found 'DU'-anomaly for variable 'targetFormat' (lines '142'-'168'). 142–168
Found 'DD'-anomaly for variable 'targetFormat' (lines '142'-'142'). 142
Found 'UR'-anomaly for variable 'formatRelation' (lines '174'-'178'). 174–178

wjhk/jupload2/filedata/helper/ImageHelper.java

Violation Line
Use explicit scoping instead of the default package private level 138
Found 'DU'-anomaly for variable 'theta' (lines '168'-'234'). 168–234
Found 'DD'-anomaly for variable 'nonScaledRotatedWidth' (lines '173'-'178'). 173–178
Found 'DD'-anomaly for variable 'nonScaledRotatedHeight' (lines '174'-'179'). 174–179
Useless parentheses. 183
Useless parentheses. 184
Avoid using Literals in Conditional Statements 186
Useless parentheses. 189
These nested if statements could be combined 189–194
Useless parentheses. 190
Useless parentheses. 191
Useless parentheses. 192
Avoid using Literals in Conditional Statements 204
Assigning an Object to null is a code smell. Consider refactoring. 252
Found 'DU'-anomaly for variable 'irwh' (lines '252'-'305'). 252–305
Avoid using Literals in Conditional Statements 253
Found 'DU'-anomaly for variable 'msGetBufferedImage' (lines '327'-'451'). 327–451
Found 'DD'-anomaly for variable 'theta' (lines '328'-'359'). 328–359
Found 'DU'-anomaly for variable 'theta' (lines '328'-'451'). 328–451
Found 'DD'-anomaly for variable 'returnedBufferedImage' (lines '330'-'418'). 330–418
Found 'DD'-anomaly for variable 'returnedBufferedImage' (lines '330'-'423'). 330–423
Use one line for each declaration, it enhances code readability. 338
Found 'DD'-anomaly for variable 'translationX' (lines '338'-'349'). 338–349
Found 'DD'-anomaly for variable 'translationX' (lines '338'-'353'). 338–353
Found 'DD'-anomaly for variable 'translationY' (lines '338'-'346'). 338–346
Found 'DD'-anomaly for variable 'translationX' (lines '338'-'345'). 338–345
Found 'DD'-anomaly for variable 'translationY' (lines '338'-'350'). 338–350
Found 'DD'-anomaly for variable 'translationY' (lines '338'-'354'). 338–354
Avoid using Literals in Conditional Statements 366
Found 'DU'-anomaly for variable 'img' (lines '399'-'451'). 399–451
Assigning an Object to null is a code smell. Consider refactoring. 399
Assigning an Object to null is a code smell. Consider refactoring. 405
Found 'DU'-anomaly for variable 'tempBufferedImage' (lines '405'-'451'). 405–451
Found 'DU'-anomaly for variable 'returnedBufferedImage' (lines '418'-'451'). 418–451
Found 'DD'-anomaly for variable 'affineTransformOp' (lines '420'-'422'). 420–422
Assigning an Object to null is a code smell. Consider refactoring. 432
Found 'DU'-anomaly for variable 'affineTransformOp' (lines '432'-'451'). 432–451
Use explicit scoping instead of the default package private level 461–553
Found 'DU'-anomaly for variable 'msGetBufferedImage' (lines '462'-'553'). 462–553
Found 'DU'-anomaly for variable 'dest' (lines '463'-'553'). 463–553
Use one line for each declaration, it enhances code readability. 469–470
Found 'DU'-anomaly for variable 'theta' (lines '472'-'553'). 472–553
Found 'DD'-anomaly for variable 'widthAfterRotation' (lines '480'-'529'). 480–529
Found 'DU'-anomaly for variable 'widthAfterRotation' (lines '480'-'553'). 480–553
Found 'DD'-anomaly for variable 'widthAfterRotation' (lines '480'-'538'). 480–538
Found 'DD'-anomaly for variable 'heigthAfterRotation' (lines '481'-'539'). 481–539
Found 'DU'-anomaly for variable 'heigthAfterRotation' (lines '481'-'553'). 481–553
Found 'DD'-anomaly for variable 'heigthAfterRotation' (lines '481'-'530'). 481–530
Found 'DD'-anomaly for variable 'widthAfterRotation' (lines '489'-'529'). 489–529
Found 'DD'-anomaly for variable 'widthAfterRotation' (lines '489'-'538'). 489–538
Found 'DU'-anomaly for variable 'widthAfterRotation' (lines '489'-'553'). 489–553
Found 'DD'-anomaly for variable 'heigthAfterRotation' (lines '490'-'530'). 490–530
Found 'DD'-anomaly for variable 'heigthAfterRotation' (lines '490'-'539'). 490–539
Found 'DU'-anomaly for variable 'heigthAfterRotation' (lines '490'-'553'). 490–553
Found 'DU'-anomaly for variable 'scaledPicture' (lines '501'-'553'). 501–553
Found 'DD'-anomaly for variable 'scaledPicture' (lines '501'-'514'). 501–514
Avoid using Literals in Conditional Statements 502
Use one line for each declaration, it enhances code readability. 503
Found 'DU'-anomaly for variable 'scaledPicture' (lines '514'-'553'). 514–553
Found 'DU'-anomaly for variable 'maxWidthBeforeRotation' (lines '525'-'553'). 525–553
Found 'DU'-anomaly for variable 'maxHeigthBeforeRotation' (lines '526'-'553'). 526–553
Found 'DU'-anomaly for variable 'widthBeforeRotation' (lines '527'-'553'). 527–553
Found 'DU'-anomaly for variable 'heigthBeforeRotation' (lines '528'-'553'). 528–553
Found 'DU'-anomaly for variable 'widthAfterRotation' (lines '529'-'553'). 529–553
Found 'DU'-anomaly for variable 'heigthAfterRotation' (lines '530'-'553'). 530–553
Found 'DU'-anomaly for variable 'maxWidthBeforeRotation' (lines '534'-'553'). 534–553
Found 'DU'-anomaly for variable 'maxHeigthBeforeRotation' (lines '535'-'553'). 535–553
Found 'DU'-anomaly for variable 'widthBeforeRotation' (lines '536'-'553'). 536–553
Found 'DU'-anomaly for variable 'heigthBeforeRotation' (lines '537'-'553'). 537–553
Found 'DU'-anomaly for variable 'widthAfterRotation' (lines '538'-'553'). 538–553
Found 'DU'-anomaly for variable 'heigthAfterRotation' (lines '539'-'553'). 539–553

wjhk/jupload2/filedata/helper/ImageReaderWriterHelper.java

Violation Line
Use explicit scoping instead of the default package private level 59
Use explicit scoping instead of the default package private level 64
Use explicit scoping instead of the default package private level 69
Use explicit scoping instead of the default package private level 74
Use explicit scoping instead of the default package private level 79
Use explicit scoping instead of the default package private level 84
Use explicit scoping instead of the default package private level 90
Use explicit scoping instead of the default package private level 95
Assigning an Object to null is a code smell. Consider refactoring. 163
Assigning an Object to null is a code smell. Consider refactoring. 164
Assigning an Object to null is a code smell. Consider refactoring. 176
Assigning an Object to null is a code smell. Consider refactoring. 177
Avoid using Literals in Conditional Statements 394
Avoid empty catch blocks 399–401

wjhk/jupload2/gui/DebugDialog.java

Violation Line
Use explicit scoping instead of the default package private level 58
Use explicit scoping instead of the default package private level 63
Use explicit scoping instead of the default package private level 68
It is a good practice to call super() in a constructor 78–119
Assigning an Object to null is a code smell. Consider refactoring. 178
Avoid empty catch blocks 180–182

wjhk/jupload2/gui/DialogUploadRetry.java

Violation Line
Use explicit scoping instead of the default package private level 60
Use explicit scoping instead of the default package private level 62
Use explicit scoping instead of the default package private level 64
Use explicit scoping instead of the default package private level 66
Use explicit scoping instead of the default package private level 68
Use explicit scoping instead of the default package private level 70
Use explicit scoping instead of the default package private level 72
Use explicit scoping instead of the default package private level 78
Use explicit scoping instead of the default package private level 84
Use explicit scoping instead of the default package private level 91

wjhk/jupload2/gui/DnDListener.java

Violation Line
Use explicit scoping instead of the default package private level 64
Found 'UR'-anomaly for variable 's' (lines '96'-'116'). 96–116

wjhk/jupload2/gui/JUploadDebugPopupMenu.java

Violation Line
Use explicit scoping instead of the default package private level 54
Use explicit scoping instead of the default package private level 56
Use explicit scoping instead of the default package private level 58
Use explicit scoping instead of the default package private level 60
Use explicit scoping instead of the default package private level 62
Use explicit scoping instead of the default package private level 64
It is a good practice to call super() in a constructor 71–113

wjhk/jupload2/gui/JUploadFileFilter.java

Violation Line
Use explicit scoping instead of the default package private level 38
It is a good practice to call super() in a constructor 40–42

wjhk/jupload2/gui/JUploadFileView.java

Violation Line
Use explicit scoping instead of the default package private level 66
Use explicit scoping instead of the default package private level 69
Use explicit scoping instead of the default package private level 75
Use explicit scoping instead of the default package private level 81
Use explicit scoping instead of the default package private level 89
Use explicit scoping instead of the default package private level 92
Use explicit scoping instead of the default package private level 95
Use explicit scoping instead of the default package private level 98
Use explicit scoping instead of the default package private level 101
Use explicit scoping instead of the default package private level 104
Use explicit scoping instead of the default package private level 107
Use explicit scoping instead of the default package private level 128–141
A method should have only one exit point, and that should be the last statement in the method 131
A method should have only one exit point, and that should be the last statement in the method 137
Use explicit scoping instead of the default package private level 148–180
Assigning an Object to null is a code smell. Consider refactoring. 178
Use explicit scoping instead of the default package private level 207
Avoid using java.lang.ThreadGroup; it is not thread safe 207
Use explicit scoping instead of the default package private level 210
Use explicit scoping instead of the default package private level 213
Use explicit scoping instead of the default package private level 216–217
Use explicit scoping instead of the default package private level 225
It is a good practice to call super() in a constructor 238–261
Use explicit scoping instead of the default package private level 263–269
Assigning an Object to null is a code smell. Consider refactoring. 280
Found 'DD'-anomaly for variable 'iw' (lines '289'-'291'). 289–291
Found 'DU'-anomaly for variable 'iw' (lines '289'-'296'). 289–296
Useless parentheses. 340–341
Useless parentheses. 340
A method should have only one exit point, and that should be the last statement in the method 342
A method should have only one exit point, and that should be the last statement in the method 347
A method should have only one exit point, and that should be the last statement in the method 360

wjhk/jupload2/gui/JUploadMainPopupMenu.java

Violation Line
It is a good practice to call super() in a constructor 39–55

wjhk/jupload2/gui/JUploadPanelImpl.java

Violation Line
It is a good practice to call super() in a constructor 120–160
Useless parentheses. 241
Useless parentheses. 242–243
These nested if statements could be combined 320–325
Avoid using Literals in Conditional Statements 417
A method should have only one exit point, and that should be the last statement in the method 434
A method should have only one exit point, and that should be the last statement in the method 440
Useless parentheses. 464

wjhk/jupload2/gui/JUploadTextArea.java

Violation Line
Use explicit scoping instead of the default package private level 55
Use explicit scoping instead of the default package private level 62
It is a good practice to call super() in a constructor 92–95
Found 'DD'-anomaly for variable 'nextMessage' (lines '100'-'112'). 100–112
Found 'DU'-anomaly for variable 'nextMessage' (lines '100'-'148'). 100–148
Avoid using Literals in Conditional Statements 102
Use explicit scoping instead of the default package private level 154
Use explicit scoping instead of the default package private level 286–294
Useless parentheses. 293

wjhk/jupload2/gui/JUploadTransferHandler.java

Violation Line
Use explicit scoping instead of the default package private level 29
Use explicit scoping instead of the default package private level 32
Use explicit scoping instead of the default package private level 37
Use explicit scoping instead of the default package private level 42
It is a good practice to call super() in a constructor 51–59
Found 'DD'-anomaly for variable 'importAccepted' (lines '68'-'86'). 68–86
Found 'DD'-anomaly for variable 'importAccepted' (lines '68'-'74'). 68–74
Found 'DD'-anomaly for variable 'fileList' (lines '71'-'73'). 71–73
Avoid assignments in operands 79
Found 'DU'-anomaly for variable 'uriStr' (lines '79'-'99'). 79–99
A method should have only one exit point, and that should be the last statement in the method 90
Found 'UR'-anomaly for variable 'flavor' (lines '123'-'125'). 123–125
A method should have only one exit point, and that should be the last statement in the method 126
Found 'UR'-anomaly for variable 'flavor' (lines '138'-'140'). 138–140
A method should have only one exit point, and that should be the last statement in the method 141

wjhk/jupload2/gui/filepanel/ColumnComparator.java

Violation Line
This statement may have some unnecessary parentheses 68
Useless parentheses. 68
This statement may have some unnecessary parentheses 69
Useless parentheses. 69
Useless parentheses. 72
This statement may have some unnecessary parentheses 72
This statement may have some unnecessary parentheses 73
Useless parentheses. 73
Useless parentheses. 76
This statement may have some unnecessary parentheses 76
This statement may have some unnecessary parentheses 77
Useless parentheses. 77
This statement may have some unnecessary parentheses 80
Useless parentheses. 80
This statement may have some unnecessary parentheses 81
Useless parentheses. 81
A method should have only one exit point, and that should be the last statement in the method 84
A method should have only one exit point, and that should be the last statement in the method 87–88

wjhk/jupload2/gui/filepanel/FilePanelFlatDataModel2.java

Violation Line
Avoid using Literals in Conditional Statements 149
Found 'UR'-anomaly for variable 'fd' (lines '165'-'169'). 165–169
Found 'DD'-anomaly for variable 'foundFileData' (lines '166'-'170'). 166–170
A method should have only one exit point, and that should be the last statement in the method 190
A method should have only one exit point, and that should be the last statement in the method 220
A method should have only one exit point, and that should be the last statement in the method 251
Useless parentheses. 298
A method should have only one exit point, and that should be the last statement in the method 319
A method should have only one exit point, and that should be the last statement in the method 321
A method should have only one exit point, and that should be the last statement in the method 323
A method should have only one exit point, and that should be the last statement in the method 325
A method should have only one exit point, and that should be the last statement in the method 327
A method should have only one exit point, and that should be the last statement in the method 330
Found 'DU'-anomaly for variable 'fd' (lines '347'-'356'). 347–356

wjhk/jupload2/gui/filepanel/FilePanelJFlatTable.java

Violation Line
Use explicit scoping instead of the default package private level 53
Use explicit scoping instead of the default package private level 56
It is a good practice to call super() in a constructor 64–80
Avoid using Literals in Conditional Statements 134
Use bitwise inversion to invert boolean values 154

wjhk/jupload2/gui/filepanel/FilePanelTableImp.java

Violation Line
Use explicit scoping instead of the default package private level 62
Use explicit scoping instead of the default package private level 65
Use explicit scoping instead of the default package private level 68
Use explicit scoping instead of the default package private level 71
Use explicit scoping instead of the default package private level 74
Use explicit scoping instead of the default package private level 77
Use explicit scoping instead of the default package private level 80
Use explicit scoping instead of the default package private level 83
Use explicit scoping instead of the default package private level 86
Use explicit scoping instead of the default package private level 89
It is a good practice to call super() in a constructor 97–134
Found 'DU'-anomaly for variable 'startTime' (lines '173'-'223'). 173–223
Found 'DU'-anomaly for variable 'nbFiles' (lines '174'-'223'). 174–223
Found 'UR'-anomaly for variable 'fd' (lines '275'-'277'). 275–277
Use explicit scoping instead of the default package private level 321–357
Use explicit scoping instead of the default package private level 364–381
Found 'DU'-anomaly for variable 'flatColumnModel' (lines '434'-'451'). 434–451
Found 'DU'-anomaly for variable 'treeviewColumnModel' (lines '443'-'451'). 443–451

wjhk/jupload2/gui/filepanel/NameRenderer.java

Violation Line
Each class should declare at least one constructor 36–56

wjhk/jupload2/gui/filepanel/SizeRenderer.java

Violation Line
Found 'DU'-anomaly for variable 'sizeunit_gigabytes' (lines '142'-'167'). 142–167
Found 'DU'-anomaly for variable 'sizeunit_megabytes' (lines '144'-'167'). 144–167
Found 'DU'-anomaly for variable 'sizeunit_kilobytes' (lines '146'-'167'). 146–167
Found 'DU'-anomaly for variable 'sizeunit_bytes' (lines '148'-'167'). 148–167

wjhk/jupload2/gui/filepanel/SortArrowIcon.java

Violation Line
Found 'DU'-anomaly for variable 'light' (lines '87'-'109'). 87–109
Found 'DU'-anomaly for variable 'shade' (lines '88'-'109'). 88–109
Found 'DU'-anomaly for variable 'h' (lines '91'-'109'). 91–109
Found 'DU'-anomaly for variable 'm' (lines '92'-'109'). 92–109

wjhk/jupload2/gui/filepanel/SortHeaderRenderer.java

Violation Line
It is a good practice to call super() in a constructor 53–56
Found 'DD'-anomaly for variable 'index' (lines '65'-'69'). 65–69
Found 'DD'-anomaly for variable 'ascending' (lines '66'-'70'). 66–70

wjhk/jupload2/gui/filepanel/treeview/FileDataNode.java

Violation Line
Use explicit scoping instead of the default package private level 49
Use explicit scoping instead of the default package private level 58
Use explicit scoping instead of the default package private level 61
Use explicit scoping instead of the default package private level 67
This statement may have some unnecessary parentheses 136
Useless parentheses. 136
A method should have only one exit point, and that should be the last statement in the method 236

wjhk/jupload2/gui/filepanel/treeview/FileDataTreeViewModel.java

Violation Line
Use explicit scoping instead of the default package private level 30
Use explicit scoping instead of the default package private level 36
Use explicit scoping instead of the default package private level 38
Use explicit scoping instead of the default package private level 40
Use explicit scoping instead of the default package private level 42
Use explicit scoping instead of the default package private level 45
Use explicit scoping instead of the default package private level 48
Use explicit scoping instead of the default package private level 51
Use explicit scoping instead of the default package private level 54
Use explicit scoping instead of the default package private level 56
Use explicit scoping instead of the default package private level 64
Use explicit scoping instead of the default package private level 69
Use explicit scoping instead of the default package private level 71
A method should have only one exit point, and that should be the last statement in the method 130
A method should have only one exit point, and that should be the last statement in the method 132
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 136
A method should have only one exit point, and that should be the last statement in the method 138
A method should have only one exit point, and that should be the last statement in the method 141
A method should have only one exit point, and that should be the last statement in the method 208
A method should have only one exit point, and that should be the last statement in the method 213
A method should have only one exit point, and that should be the last statement in the method 226
Found 'DD'-anomaly for variable 'parentNode' (lines '286'-'292'). 286–292
Found 'DD'-anomaly for variable 'parentNode' (lines '286'-'299'). 286–299
Found 'DD'-anomaly for variable 'parentNode' (lines '286'-'294'). 286–294

wjhk/jupload2/gui/filepanel/treeview/FolderNode.java

Violation Line
Use explicit scoping instead of the default package private level 54
Use explicit scoping instead of the default package private level 63
Use explicit scoping instead of the default package private level 66
Use explicit scoping instead of the default package private level 72
Use explicit scoping instead of the default package private level 75
Use explicit scoping instead of the default package private level 77
Use explicit scoping instead of the default package private level 79
Use explicit scoping instead of the default package private level 81
Found 'DD'-anomaly for variable 'filename' (lines '92'-'94'). 92–94
Useless parentheses. 94
Found 'UR'-anomaly for variable 'o' (lines '109'-'113'). 109–113
Found 'DU'-anomaly for variable 'depth' (lines '111'-'130'). 111–130
Found 'DD'-anomaly for variable 'ints' (lines '133'-'135'). 133–135
Found 'DD'-anomaly for variable 'ints' (lines '135'-'135'). 135
Found 'DD'-anomaly for variable 'treeNodes' (lines '141'-'143'). 141–143
Found 'DD'-anomaly for variable 'treeNodes' (lines '143'-'143'). 143
Found 'UR'-anomaly for variable 'child' (lines '174'-'177'). 174–177
Found 'UR'-anomaly for variable 'tfdn' (lines '250'-'254'). 250–254
Useless parentheses. 261
This statement may have some unnecessary parentheses 261
Assigning an Object to null is a code smell. Consider refactoring. 316
Found 'UR'-anomaly for variable 'node' (lines '320'-'322'). 320–322
A method should have only one exit point, and that should be the last statement in the method 323
A method should have only one exit point, and that should be the last statement in the method 412
A method should have only one exit point, and that should be the last statement in the method 468
A method should have only one exit point, and that should be the last statement in the method 489
Found 'UR'-anomaly for variable 'file' (lines '503'-'527'). 503–527
Found 'DU'-anomaly for variable 'child' (lines '510'-'531'). 510–531
Found 'UR'-anomaly for variable 'tfdn' (lines '539'-'543'). 539–543

wjhk/jupload2/gui/filepanel/treeview/MyAbstractTreeTableModel.java

Violation Line
Use explicit scoping instead of the default package private level 40
Use explicit scoping instead of the default package private level 43
Use explicit scoping instead of the default package private level 123–141
Use explicit scoping instead of the default package private level 150–171
A method should have only one exit point, and that should be the last statement in the method 178
A method should have only one exit point, and that should be the last statement in the method 180
Found 'DU'-anomaly for variable 'e' (lines '198'-'219'). 198–219
Found 'UR'-anomaly for variable 'child' (lines '245'-'257'). 245–257
Found 'UR'-anomaly for variable 'child' (lines '245'-'253'). 245–253
Found 'DD'-anomaly for variable 'ret' (lines '246'-'258'). 246–258
Found 'DD'-anomaly for variable 'ret' (lines '246'-'263'). 246–263
These nested if statements could be combined 248–266
Found 'DD'-anomaly for variable 'ret' (lines '258'-'258'). 258
Found 'DD'-anomaly for variable 'ret' (lines '258'-'263'). 258–263
Found 'DD'-anomaly for variable 'ret' (lines '263'-'258'). 263–258
Found 'DD'-anomaly for variable 'ret' (lines '263'-'263'). 263
Found 'DD'-anomaly for variable 'childIdx' (lines '285'-'289'). 285–289
Found 'DD'-anomaly for variable 'children' (lines '286'-'290'). 286–290
Found 'DD'-anomaly for variable 'childIdx' (lines '289'-'289'). 289
Found 'DD'-anomaly for variable 'children' (lines '290'-'290'). 290
A method should have only one exit point, and that should be the last statement in the method 306
A method should have only one exit point, and that should be the last statement in the method 312

wjhk/jupload2/gui/filepanel/treeview/MyTreeTableCellEditor.java

Violation Line
It is a good practice to call super() in a constructor 33–36

wjhk/jupload2/gui/filepanel/treeview/MyTreeTableModelAdapter.java

Violation Line
Use explicit scoping instead of the default package private level 34
Use explicit scoping instead of the default package private level 36
It is a good practice to call super() in a constructor 38–72
Assigning an Object to null is a code smell. Consider refactoring. 93

wjhk/jupload2/gui/filepanel/treeview/MyTreeTableSelectionModel.java

Violation Line
Use explicit scoping instead of the default package private level 34–36

wjhk/jupload2/gui/image/DialogPicturePanel.java

Violation Line
Use explicit scoping instead of the default package private level 44
Assigning an Object to null is a code smell. Consider refactoring. 73

wjhk/jupload2/gui/image/JUploadImagePreview.java

Violation Line
Use explicit scoping instead of the default package private level 49
Use explicit scoping instead of the default package private level 54
Use explicit scoping instead of the default package private level 59
It is a good practice to call super() in a constructor 66–69
Found 'DD'-anomaly for variable 'thumbnail' (lines '81'-'83'). 81–83
Use explicit scoping instead of the default package private level 114
Use explicit scoping instead of the default package private level 119
Use explicit scoping instead of the default package private level 124
Use explicit scoping instead of the default package private level 131
Use explicit scoping instead of the default package private level 137
It is a good practice to call super() in a constructor 146–153
Use explicit scoping instead of the default package private level 162–165
Use explicit scoping instead of the default package private level 172–200
Assigning an Object to null is a code smell. Consider refactoring. 174
Assigning an Object to null is a code smell. Consider refactoring. 180
Assigning an Object to null is a code smell. Consider refactoring. 187
Avoid using Literals in Conditional Statements 233

wjhk/jupload2/gui/image/PictureDialog.java

Violation Line
Use explicit scoping instead of the default package private level 51
Use explicit scoping instead of the default package private level 53
Use explicit scoping instead of the default package private level 55
Use explicit scoping instead of the default package private level 57

wjhk/jupload2/gui/image/PicturePanel.java

Violation Line
Assigning an Object to null is a code smell. Consider refactoring. 136
Assigning an Object to null is a code smell. Consider refactoring. 139
Found 'DU'-anomaly for variable 'enableButton' (lines '148'-'167'). 148–167
Found 'DD'-anomaly for variable 'enableButton' (lines '148'-'153'). 148–153
Found 'DU'-anomaly for variable 'enableButton' (lines '153'-'167'). 153–167
Assigning an Object to null is a code smell. Consider refactoring. 209
Assigning an Object to null is a code smell. Consider refactoring. 229
Assigning an Object to null is a code smell. Consider refactoring. 244
Assigning an Object to null is a code smell. Consider refactoring. 245
Assigning an Object to null is a code smell. Consider refactoring. 318

wjhk/jupload2/policies/AAFUploadPolicy.java

Violation Line
Found 'DD'-anomaly for variable 'files' (lines '22'-'23'). 22–23

wjhk/jupload2/policies/CoppermineUploadPolicy.java

Violation Line
A method should have only one exit point, and that should be the last statement in the method 179
Avoid using Literals in Conditional Statements 202
These nested if statements could be combined 272–276

wjhk/jupload2/policies/DefaultUploadPolicy.java

Violation Line
Use explicit scoping instead of the default package private level 125
Found 'DU'-anomaly for variable 'bReturn' (lines '539'-'644'). 539–644
Found 'DD'-anomaly for variable 'bReturn' (lines '539'-'588'). 539–588
Avoid using Literals in Conditional Statements 541
Useless parentheses. 554
Useless parentheses. 554
Avoid using Literals in Conditional Statements 560
A method should have only one exit point, and that should be the last statement in the method 561
Use one line for each declaration, it enhances code readability. 566
These nested if statements could be combined 586–589
Found 'DU'-anomaly for variable 'bReturn' (lines '588'-'644'). 588–644
Found 'DD'-anomaly for variable 'bReturn' (lines '588'-'588'). 588
Found 'DD'-anomaly for variable 'errmsg' (lines '596'-'601'). 596–601
These nested if statements could be combined 598–602
Found 'DD'-anomaly for variable 'warnmsg' (lines '613'-'616'). 613–616
These nested if statements could be combined 615–617
A method should have only one exit point, and that should be the last statement in the method 628
Found 'DU'-anomaly for variable 'j' (lines '659'-'676'). 659–676
Use one line for each declaration, it enhances code readability. 659
Found 'DD'-anomaly for variable 'j' (lines '659'-'659'). 659
A method should have only one exit point, and that should be the last statement in the method 726
A method should have only one exit point, and that should be the last statement in the method 729
A method should have only one exit point, and that should be the last statement in the method 788
Found 'DD'-anomaly for variable 'exceptionMsg' (lines '897'-'916'). 897–916
Found 'DD'-anomaly for variable 'exceptionMsg' (lines '897'-'914'). 897–914
Found 'DU'-anomaly for variable 'exceptionMsg' (lines '897'-'934'). 897–934
Found 'DD'-anomaly for variable 'exceptionClassName' (lines '898'-'903'). 898–903
Found 'DD'-anomaly for variable 'exceptionClassName' (lines '898'-'907'). 898–907
Found 'DD'-anomaly for variable 'exceptionClassName' (lines '898'-'905'). 898–905
Found 'DD'-anomaly for variable 'logMsg' (lines '899'-'918'). 899–918
Found 'DD'-anomaly for variable 'alertMsg' (lines '945'-'958'). 945–958
Found 'DD'-anomaly for variable 'alertMsg' (lines '945'-'952'). 945–952
Found 'DD'-anomaly for variable 'alertMsg' (lines '945'-'950'). 945–950
Found 'DD'-anomaly for variable 'alertMsg' (lines '950'-'958'). 950–958
Found 'DD'-anomaly for variable 'alertMsg' (lines '952'-'958'). 952–958
Found 'DD'-anomaly for variable 'buttonClicked' (lines '963'-'979'). 963–979
Found 'DD'-anomaly for variable 'buttonClicked' (lines '963'-'968'). 963–968
Found 'DD'-anomaly for variable 'buttonClicked' (lines '963'-'973'). 963–973
Avoid using final local variables, turn them into fields 1021
Found 'DU'-anomaly for variable 'tag' (lines '1 021'-'1 030'). 1021–1030
Found 'DU'-anomaly for variable 'ret' (lines '1 034'-'1 063'). 1034–1063
A method should have only one exit point, and that should be the last statement in the method 1039
A method should have only one exit point, and that should be the last statement in the method 1041
A method should have only one exit point, and that should be the last statement in the method 1043
A method should have only one exit point, and that should be the last statement in the method 1045
A method should have only one exit point, and that should be the last statement in the method 1047
A method should have only one exit point, and that should be the last statement in the method 1049
A method should have only one exit point, and that should be the last statement in the method 1051
A method should have only one exit point, and that should be the last statement in the method 1081
A method should have only one exit point, and that should be the last statement in the method 1093
A method should have only one exit point, and that should be the last statement in the method 1095
Avoid using Literals in Conditional Statements 1098
Avoid using Literals in Conditional Statements 1167
These nested if statements could be combined 1167–1170
Found 'DD'-anomaly for variable 'action' (lines '1 233'-'1 287'). 1233–1287
Found 'DD'-anomaly for variable 'action' (lines '1 233'-'1 307'). 1233–1307
Found 'DD'-anomaly for variable 'connectionHelper' (lines '1 235'-'1 240'). 1235–1240
Found 'DU'-anomaly for variable 'localDebugOk' (lines '1 236'-'1 382'). 1236–1382
Found 'DD'-anomaly for variable 'exceptionClass' (lines '1 251'-'1 255'). 1251–1255
Found 'DD'-anomaly for variable 'exceptionCause' (lines '1 252'-'1 257'). 1252–1257
Found 'DD'-anomaly for variable 'exceptionStackTrace' (lines '1 253'-'1 273'). 1253–1273
Found 'DD'-anomaly for variable 'action' (lines '1 287'-'1 294'). 1287–1294
Found 'DD'-anomaly for variable 'action' (lines '1 294'-'1 310'). 1294–1310
Avoid assignments in operands 1296
Found 'DU'-anomaly for variable 'line' (lines '1 296'-'1 382'). 1296–1382
Found 'DD'-anomaly for variable 'action' (lines '1 307'-'1 310'). 1307–1310
Found 'DD'-anomaly for variable 'action' (lines '1 310'-'1 321'). 1310–1321
Found 'DD'-anomaly for variable 'action' (lines '1 321'-'1 333'). 1321–1333
Found 'DU'-anomaly for variable 'action' (lines '1 333'-'1 382'). 1333–1382
Avoid empty if statements 1423–1427
Assigning an Object to null is a code smell. Consider refactoring. 1555
Assigning an Object to null is a code smell. Consider refactoring. 1593
Assigning an Object to null is a code smell. Consider refactoring. 1610
Assigning an Object to null is a code smell. Consider refactoring. 1799
Avoid using Literals in Conditional Statements 1854
Assigning an Object to null is a code smell. Consider refactoring. 1896
A method should have only one exit point, and that should be the last statement in the method 2018
Found 'DD'-anomaly for variable 'val' (lines '2 094'-'2 100'). 2094–2100
Found 'DD'-anomaly for variable 'val' (lines '2 094'-'2 096'). 2094–2096
Found 'DD'-anomaly for variable 'val' (lines '2 094'-'2 098'). 2094–2098
Found 'DD'-anomaly for variable 'val' (lines '2 094'-'2 102'). 2094–2102
Found 'DD'-anomaly for variable 'val' (lines '2 096'-'2 102'). 2096–2102
Found 'DD'-anomaly for variable 'val' (lines '2 096'-'2 100'). 2096–2100
Found 'DD'-anomaly for variable 'val' (lines '2 096'-'2 098'). 2096–2098
Found 'DD'-anomaly for variable 'val' (lines '2 098'-'2 102'). 2098–2102
Found 'DD'-anomaly for variable 'val' (lines '2 098'-'2 100'). 2098–2100
Found 'DD'-anomaly for variable 'val' (lines '2 100'-'2 102'). 2100–2102
Assigning an Object to null is a code smell. Consider refactoring. 2213
Assigning an Object to null is a code smell. Consider refactoring. 2219
Found 'DU'-anomaly for variable 'message' (lines '2 278'-'2 284'). 2278–2284
A method should have only one exit point, and that should be the last statement in the method 2313
A method should have only one exit point, and that should be the last statement in the method 2315
A method should have only one exit point, and that should be the last statement in the method 2335
A method should have only one exit point, and that should be the last statement in the method 2337

wjhk/jupload2/policies/PictureUploadPolicy.java

Violation Line
A method should have only one exit point, and that should be the last statement in the method 241
Found 'DD'-anomaly for variable 'pfd' (lines '244'-'246'). 244–246
A method should have only one exit point, and that should be the last statement in the method 253
A method should have only one exit point, and that should be the last statement in the method 258
Avoid empty if statements 384–386
Use explicit scoping instead of the default package private level 439–441
Use explicit scoping instead of the default package private level 451–458
Use explicit scoping instead of the default package private level 468–475
Use explicit scoping instead of the default package private level 488–490
Use explicit scoping instead of the default package private level 503–505
Use explicit scoping instead of the default package private level 515–517
Use explicit scoping instead of the default package private level 527–529
Use explicit scoping instead of the default package private level 549–552
Use explicit scoping instead of the default package private level 569–571
A method should have only one exit point, and that should be the last statement in the method 680

wjhk/jupload2/policies/SilverpeasUploadPolicy.java

Violation Line
Found 'DD'-anomaly for variable 'content' (lines '139'-'154'). 139–154
Found 'DD'-anomaly for variable 'connection' (lines '140'-'144'). 140–144
Found 'DD'-anomaly for variable 'in' (lines '141'-'146'). 141–146
Found 'DD'-anomaly for variable 'size' (lines '149'-'151'). 149–151
Avoid assignments in operands 151
Found 'DU'-anomaly for variable 'size' (lines '151'-'168'). 151–168

wjhk/jupload2/policies/SuperSimpleUploadPolicy.java

Violation Line
Use explicit scoping instead of the default package private level 95
Use explicit scoping instead of the default package private level 96
Use explicit scoping instead of the default package private level 97

wjhk/jupload2/policies/UploadPolicyFactory.java

Violation Line
Found 'DD'-anomaly for variable 'action' (lines '70'-'76'). 70–76
Found 'DU'-anomaly for variable 'usingDefaultUploadPolicy' (lines '71'-'162'). 71–162
Found 'DD'-anomaly for variable 'usingDefaultUploadPolicy' (lines '71'-'106'). 71–106
Found 'DD'-anomaly for variable 'action' (lines '76'-'114'). 76–114
Found 'DD'-anomaly for variable 'uploadPolicyClass' (lines '77'-'83'). 77–83
Found 'DD'-anomaly for variable 'uploadPolicyClass' (lines '83'-'92'). 83–92
Assigning an Object to null is a code smell. Consider refactoring. 92
Found 'DD'-anomaly for variable 'uploadPolicyClass' (lines '98'-'107'). 98–107
Found 'DU'-anomaly for variable 'usingDefaultUploadPolicy' (lines '106'-'162'). 106–162
Found 'DD'-anomaly for variable 'action' (lines '114'-'123'). 114–123
Found 'DU'-anomaly for variable 'uploadPolicy' (lines '124'-'162'). 124–162

wjhk/jupload2/upload/DefaultFileUploadThread.java

Violation Line
Use explicit scoping instead of the default package private level 58
Use explicit scoping instead of the default package private level 65
Use explicit scoping instead of the default package private level 75
Use explicit scoping instead of the default package private level 84
Use explicit scoping instead of the default package private level 89
Use explicit scoping instead of the default package private level 134
Use explicit scoping instead of the default package private level 142
Use explicit scoping instead of the default package private level 160–161
Use explicit scoping instead of the default package private level 169
Use explicit scoping instead of the default package private level 175
Use explicit scoping instead of the default package private level 185
Use explicit scoping instead of the default package private level 193
Use explicit scoping instead of the default package private level 199
Use explicit scoping instead of the default package private level 205
Use explicit scoping instead of the default package private level 212
Use explicit scoping instead of the default package private level 246–248
Use explicit scoping instead of the default package private level 255–257
Found 'DD'-anomaly for variable 'packet' (lines '275'-'279'). 275–279
Found 'DU'-anomaly for variable 'errMsg' (lines '367'-'398'). 367–398
Use explicit scoping instead of the default package private level 410–472
Found 'UR'-anomaly for variable 'uploadFileData' (lines '410'-'429'). 410–429
Found 'DU'-anomaly for variable 'bChunkEnabled' (lines '411'-'472'). 411–472
Found 'DD'-anomaly for variable 'bChunkEnabled' (lines '411'-'450'). 411–450
Found 'DU'-anomaly for variable 'totalContentLength' (lines '412'-'472'). 412–472
Found 'DU'-anomaly for variable 'totalFileLength' (lines '413'-'472'). 413–472
Avoid using Literals in Conditional Statements 446
Avoid using Literals in Conditional Statements 457
Use explicit scoping instead of the default package private level 481–574
Found 'DU'-anomaly for variable 'bLastChunk' (lines '482'-'574'). 482–574
Found 'DU'-anomaly for variable 'chunkPart' (lines '483'-'574'). 483–574
Found 'DD'-anomaly for variable 'contentLength' (lines '485'-'512'). 485–512
Found 'DU'-anomaly for variable 'contentLength' (lines '485'-'574'). 485–574
Found 'DD'-anomaly for variable 'thisChunkSize' (lines '486'-'508'). 486–508
Found 'DU'-anomaly for variable 'thisChunkSize' (lines '486'-'574'). 486–574
Found 'DD'-anomaly for variable 'thisChunkSize' (lines '486'-'510'). 486–510
Avoid using Literals in Conditional Statements 488
Useless parentheses. 504
Use explicit scoping instead of the default package private level 585–642
Found 'UR'-anomaly for variable 'uploadFileData' (lines '585'-'601'). 585–601
Found 'UR'-anomaly for variable 'uploadFileData' (lines '648'-'651'). 648–651

wjhk/jupload2/upload/FilePreparationThread.java

Violation Line
Use explicit scoping instead of the default package private level 28
Use explicit scoping instead of the default package private level 34
Use explicit scoping instead of the default package private level 40
Use explicit scoping instead of the default package private level 46
Use explicit scoping instead of the default package private level 53
Use explicit scoping instead of the default package private level 58
Use explicit scoping instead of the default package private level 61
Use explicit scoping instead of the default package private level 64
Use explicit scoping instead of the default package private level 67
Use explicit scoping instead of the default package private level 70
Use explicit scoping instead of the default package private level 75
Found 'DU'-anomaly for variable 'uploadFileRoot' (lines '125'-'220'). 125–220
Found 'DU'-anomaly for variable 'numFileInCurrentUpload' (lines '135'-'220'). 135–220

wjhk/jupload2/upload/FileUploadManagerThreadImpl.java

Violation Line
Use explicit scoping instead of the default package private level 32
Use explicit scoping instead of the default package private level 39
Use explicit scoping instead of the default package private level 45
Use explicit scoping instead of the default package private level 51
Use explicit scoping instead of the default package private level 56
Use explicit scoping instead of the default package private level 61
Use explicit scoping instead of the default package private level 67
Use explicit scoping instead of the default package private level 75
Use explicit scoping instead of the default package private level 82
Use explicit scoping instead of the default package private level 85
Use explicit scoping instead of the default package private level 88
Use explicit scoping instead of the default package private level 91
Found 'UR'-anomaly for variable 'fd' (lines '166'-'204'). 166–204
Useless parentheses. 225
Useless parentheses. 234
Useless parentheses. 268
Avoid empty if statements 287–291
Found 'UR'-anomaly for variable 'fileData' (lines '389'-'397'). 389–397
Useless parentheses. 405

wjhk/jupload2/upload/FileUploadThreadFTP.java

Violation Line
Avoid using Literals in Conditional Statements 166
Use explicit scoping instead of the default package private level 183–273
Assigning an Object to null is a code smell. Consider refactoring. 224
Use explicit scoping instead of the default package private level 276–279
Use explicit scoping instead of the default package private level 282–353
Found 'DD'-anomaly for variable 'workingDir' (lines '285'-'307'). 285–307
Found 'DD'-anomaly for variable 'workingDir' (lines '285'-'296'). 285–296
Found 'DD'-anomaly for variable 'action' (lines '286'-'326'). 286–326
Found 'DD'-anomaly for variable 'action' (lines '286'-'293'). 286–293
Found 'DD'-anomaly for variable 'action' (lines '286'-'312'). 286–312
Found 'DU'-anomaly for variable 'action' (lines '286'-'353'). 286–353
Found 'DD'-anomaly for variable 'action' (lines '293'-'326'). 293–326
Found 'DD'-anomaly for variable 'action' (lines '293'-'312'). 293–312
Found 'DU'-anomaly for variable 'action' (lines '293'-'353'). 293–353
Found 'DD'-anomaly for variable 'workingDir' (lines '296'-'300'). 296–300
Found 'DD'-anomaly for variable 'action' (lines '312'-'314'). 312–314
Found 'DU'-anomaly for variable 'action' (lines '314'-'353'). 314–353
Found 'DD'-anomaly for variable 'action' (lines '314'-'326'). 314–326
Found 'DD'-anomaly for variable 'action' (lines '326'-'328'). 326–328
Found 'DD'-anomaly for variable 'action' (lines '328'-'336'). 328–336
Found 'DD'-anomaly for variable 'action' (lines '336'-'339'). 336–339
Found 'DU'-anomaly for variable 'action' (lines '339'-'353'). 339–353
Use explicit scoping instead of the default package private level 356–372
Assigning an Object to null is a code smell. Consider refactoring. 369
Assigning an Object to null is a code smell. Consider refactoring. 370
Use explicit scoping instead of the default package private level 375–391
Assigning an Object to null is a code smell. Consider refactoring. 388
Use explicit scoping instead of the default package private level 397–412
Use explicit scoping instead of the default package private level 415–419
Use explicit scoping instead of the default package private level 422–425
Use explicit scoping instead of the default package private level 428–432
Found 'UR'-anomaly for variable 'uploadFileData' (lines '465'-'477'). 465–477
Found 'DU'-anomaly for variable 'foldersToCreate' (lines '467'-'532'). 467–532
Found 'DD'-anomaly for variable 'folderName' (lines '477'-'478'). 477–478
Found 'DU'-anomaly for variable 'intermediateFolderName' (lines '483'-'532'). 483–532
Found 'DD'-anomaly for variable 'intermediateFolderName' (lines '483'-'483'). 483
Use explicit scoping instead of the default package private level 535–538

wjhk/jupload2/upload/FileUploadThreadHTTP.java

Violation Line
Use explicit scoping instead of the default package private level 95–104
Use explicit scoping instead of the default package private level 107–112
Use explicit scoping instead of the default package private level 115–123
Use explicit scoping instead of the default package private level 126–148
Use explicit scoping instead of the default package private level 151–154
Use explicit scoping instead of the default package private level 157–166
Use explicit scoping instead of the default package private level 168–182
Avoid using Literals in Conditional Statements 170
Use explicit scoping instead of the default package private level 187–220
Found 'UR'-anomaly for variable 'uploadFileData' (lines '188'-'199'). 188–199
Assigning an Object to null is a code smell. Consider refactoring. 194
Assigning an Object to null is a code smell. Consider refactoring. 204
Assigning an Object to null is a code smell. Consider refactoring. 213
Use explicit scoping instead of the default package private level 228–231
Use explicit scoping instead of the default package private level 234–297
Found 'UR'-anomaly for variable 'uploadFileData' (lines '378'-'384'). 378–384
Found 'DU'-anomaly for variable 'numInCurrentUpload' (lines '382'-'387'). 382–387
Found 'UR'-anomaly for variable 'param' (lines '430'-'451'). 430–451
Found 'DU'-anomaly for variable 'bae' (lines '435'-'486'). 435–486
Avoid using Literals in Conditional Statements 453

wjhk/jupload2/upload/PacketConstructionThread.java

Violation Line
Use explicit scoping instead of the default package private level 14
Use explicit scoping instead of the default package private level 17
Use explicit scoping instead of the default package private level 23
Use explicit scoping instead of the default package private level 28
Use explicit scoping instead of the default package private level 33
Use explicit scoping instead of the default package private level 38

wjhk/jupload2/upload/UploadFileData.java

Violation Line
Use explicit scoping instead of the default package private level 53
Use explicit scoping instead of the default package private level 59
Use explicit scoping instead of the default package private level 135–137
Use explicit scoping instead of the default package private level 148–202
Avoid using Literals in Conditional Statements 149
Found 'DU'-anomaly for variable 'inputStream' (lines '156'-'202'). 156–202
Useless parentheses. 165
Found 'DD'-anomaly for variable 'towrite' (lines '166'-'169'). 166–169
Found 'DU'-anomaly for variable 'towrite' (lines '169'-'202'). 169–202
Avoid using Literals in Conditional Statements 184
Avoid empty catch blocks 213–215
Assigning an Object to null is a code smell. Consider refactoring. 216
Assigning an Object to null is a code smell. Consider refactoring. 237

wjhk/jupload2/upload/UploadFilePacket.java

Violation Line
Use explicit scoping instead of the default package private level 45
Use explicit scoping instead of the default package private level 50
Use explicit scoping instead of the default package private level 53
Use explicit scoping instead of the default package private level 56
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 89
A method should have only one exit point, and that should be the last statement in the method 92
Useless parentheses. 107
Useless parentheses. 107
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 120
An empty statement (semicolon) not part of a loop 124

wjhk/jupload2/upload/helper/ByteArrayEncoderHTTP.java

Violation Line
Use explicit scoping instead of the default package private level 56
Use explicit scoping instead of the default package private level 75
Found 'DD'-anomaly for variable 'propertySuffix' (lines '220'-'229'). 220–229
Found 'DD'-anomaly for variable 'propertySuffix' (lines '220'-'226'). 220–226
Found 'DD'-anomaly for variable 'action' (lines '271'-'283'). 271–283
Found 'DD'-anomaly for variable 'action' (lines '283'-'286'). 283–286
Found 'DD'-anomaly for variable 'action' (lines '286'-'289'). 286–289
Found 'DD'-anomaly for variable 'action' (lines '289'-'292'). 289–292
Found 'DU'-anomaly for variable 'elementType' (lines '290'-'374'). 290–374
Found 'DD'-anomaly for variable 'elementType' (lines '290'-'290'). 290
Found 'DD'-anomaly for variable 'action' (lines '292'-'299'). 292–299
Found 'DD'-anomaly for variable 'action' (lines '299'-'305'). 299–305
These nested if statements could be combined 311–319
Found 'DD'-anomaly for variable 'i' (lines '355'-'281'). 355–281

wjhk/jupload2/upload/helper/CookieJar.java

Violation Line
Use explicit scoping instead of the default package private level 45–46
Use explicit scoping instead of the default package private level 59
Use explicit scoping instead of the default package private level 61
Use explicit scoping instead of the default package private level 63
Use explicit scoping instead of the default package private level 65
Use explicit scoping instead of the default package private level 67
Use explicit scoping instead of the default package private level 69
Use explicit scoping instead of the default package private level 71
Useless parentheses. 119
Useless parentheses. 120
A method should have only one exit point, and that should be the last statement in the method 158
A method should have only one exit point, and that should be the last statement in the method 164
Found 'UR'-anomaly for variable 'key' (lines '199'-'209'). 199–209
Found 'DU'-anomaly for variable 'path' (lines '205'-'219'). 205–219
Found 'DU'-anomaly for variable 'secure' (lines '206'-'219'). 206–219
A method should have only one exit point, and that should be the last statement in the method 215
Useless parentheses. 236

wjhk/jupload2/upload/helper/HTTPConnectionHelper.java

Violation Line
It is a good practice to call super() in a constructor 199–201
It is a good practice to call super() in a constructor 215–219
Assigning an Object to null is a code smell. Consider refactoring. 297
Assigning an Object to null is a code smell. Consider refactoring. 364
Assigning an Object to null is a code smell. Consider refactoring. 374
These nested if statements could be combined 379–381
Assigning an Object to null is a code smell. Consider refactoring. 386
A method should have only one exit point, and that should be the last statement in the method 459
A method should have only one exit point, and that should be the last statement in the method 461
A method should have only one exit point, and that should be the last statement in the method 463
A method should have only one exit point, and that should be the last statement in the method 465
A method should have only one exit point, and that should be the last statement in the method 467
Use explicit scoping instead of the default package private level 477–479
Avoid using Literals in Conditional Statements 540
Assigning an Object to null is a code smell. Consider refactoring. 643
Assigning an Object to null is a code smell. Consider refactoring. 647
Useless parentheses. 653
Useless parentheses. 653
Useless parentheses. 653
Useless parentheses. 663
A method should have only one exit point, and that should be the last statement in the method 722
A method should have only one exit point, and that should be the last statement in the method 724
A method should have only one exit point, and that should be the last statement in the method 728
Found 'DU'-anomaly for variable 'alphaNum' (lines '743'-'750'). 743–750

wjhk/jupload2/upload/helper/HTTPInputStreamReader.java

Violation Line
Use explicit scoping instead of the default package private level 34
Use explicit scoping instead of the default package private level 41
Use explicit scoping instead of the default package private level 46
Use explicit scoping instead of the default package private level 54
Use explicit scoping instead of the default package private level 219–224
Use explicit scoping instead of the default package private level 234–241
Assigning an Object to null is a code smell. Consider refactoring. 260
Found 'UR'-anomaly for variable 'CR' (lines '287'-'287'). 287
Found 'UR'-anomaly for variable 'EOS' (lines '287'-'287'). 287
Found 'UR'-anomaly for variable 'LF' (lines '287'-'287'). 287
Found 'DU'-anomaly for variable 'EOS' (lines '289'-'378'). 289–378
Avoid using final local variables, turn them into fields 290
Found 'DU'-anomaly for variable 'CR' (lines '290'-'378'). 290–378
Avoid using final local variables, turn them into fields 291
Found 'DU'-anomaly for variable 'LF' (lines '291'-'378'). 291–378
Found 'DU'-anomaly for variable 'len' (lines '292'-'378'). 292–378
Found 'DU'-anomaly for variable 'buf' (lines '294'-'378'). 294–378
Found 'DD'-anomaly for variable 'ret' (lines '295'-'375'). 295–375
Found 'DU'-anomaly for variable 'ret' (lines '295'-'378'). 295–378
Found 'DD'-anomaly for variable 'ret' (lines '295'-'368'). 295–368
Found 'DU'-anomaly for variable 'b' (lines '301'-'378'). 301–378
A method should have only one exit point, and that should be the last statement in the method 316
Found 'DD'-anomaly for variable 'buf' (lines '363'-'364'). 363–364
Found 'DU'-anomaly for variable 'buf' (lines '364'-'378'). 364–378
Found 'DU'-anomaly for variable 'sbHeaders' (lines '389'-'461'). 389–461
Useless parentheses. 474
Useless parentheses. 474
Useless parentheses. 504
Found 'DD'-anomaly for variable 'ofs' (lines '505'-'505'). 505
Found 'DU'-anomaly for variable 'ofs' (lines '505'-'591'). 505–591
Useless parentheses. 537

wjhk/jupload2/upload/helper/HttpConnect.java

Violation Line
Each class should declare at least one constructor 67–79
Found 'DD'-anomaly for variable 'ret' (lines '115'-'220'). 115–220
Found 'DD'-anomaly for variable 'ret' (lines '115'-'206'). 115–206
Found 'DD'-anomaly for variable 'ret' (lines '115'-'195'). 115–195
Found 'DD'-anomaly for variable 'ret' (lines '115'-'188'). 115–188
Found 'DU'-anomaly for variable 'ret' (lines '115'-'237'). 115–237
Found 'DD'-anomaly for variable 'ret' (lines '115'-'226'). 115–226
Found 'DD'-anomaly for variable 'ret' (lines '115'-'233'). 115–233
Found 'DD'-anomaly for variable 'host' (lines '116'-'214'). 116–214
Found 'DU'-anomaly for variable 'host' (lines '116'-'237'). 116–237
Useless parentheses. 118
Useless parentheses. 118
Useless parentheses. 118
Found 'DU'-anomaly for variable 'useProxy' (lines '118'-'237'). 118–237
Found 'DU'-anomaly for variable 'port' (lines '122'-'237'). 122–237
Found 'DU'-anomaly for variable 'context' (lines '123'-'237'). 123–237
Useless parentheses. 168
Found 'DU'-anomaly for variable 'port' (lines '210'-'237'). 210–237
Found 'DD'-anomaly for variable 'port' (lines '210'-'216'). 210–216
Useless parentheses. 214
Found 'DD'-anomaly for variable 'protocol' (lines '289'-'342'). 289–342
Avoid using Literals in Conditional Statements 300
Found 'DD'-anomaly for variable 'newPostURL' (lines '402'-'438'). 402–438
Found 'DU'-anomaly for variable 'newPostURL' (lines '402'-'445'). 402–445
Found 'DD'-anomaly for variable 'newPostURL' (lines '402'-'427'). 402–427
Found 'DD'-anomaly for variable 'newPostURL' (lines '405'-'427'). 405–427
Found 'DU'-anomaly for variable 'newPostURL' (lines '405'-'445'). 405–445
Found 'DD'-anomaly for variable 'newPostURL' (lines '405'-'438'). 405–438
Avoid empty if statements 422–425

wjhk/jupload2/upload/helper/HttpProtocolFinderThread.java

Violation Line
Use explicit scoping instead of the default package private level 36
Use explicit scoping instead of the default package private level 39
Found 'DD'-anomaly for variable 'computedProtocol' (lines '78'-'86'). 78–86
Found 'DD'-anomaly for variable 'computedProtocol' (lines '78'-'98'). 78–98
Found 'DD'-anomaly for variable 'computedProtocol' (lines '78'-'84'). 78–84
Found 'DD'-anomaly for variable 'computedProtocol' (lines '78'-'124'). 78–124
Found 'DU'-anomaly for variable 'postURL' (lines '79'-'136'). 79–136
Found 'DD'-anomaly for variable 'computedProtocol' (lines '98'-'108'). 98–108

wjhk/jupload2/upload/helper/InteractiveTrustManager.java

Violation Line
A method should have only one exit point, and that should be the last statement in the method 149
Assigning an Object to null is a code smell. Consider refactoring. 164
Found 'DD'-anomaly for variable 'pw' (lines '165'-'163'). 165–163
Found 'DU'-anomaly for variable 'pw' (lines '165'-'171'). 165–171
Assigning an Object to null is a code smell. Consider refactoring. 165
Useless parentheses. 260
Useless parentheses. 261–262
Assigning an Object to null is a code smell. Consider refactoring. 285
Avoid using Literals in Conditional Statements 333
Found 'DD'-anomaly for variable 'ok' (lines '340'-'342'). 340–342
Found 'DD'-anomaly for variable 'expired' (lines '385'-'392'). 385–392
Found 'DD'-anomaly for variable 'notyet' (lines '386'-'396'). 386–396
Found 'DU'-anomaly for variable 'fp' (lines '451'-'578'). 451–578
Found 'DU'-anomaly for variable 'd' (lines '453'-'578'). 453–578
Useless parentheses. 563
Avoid empty catch blocks 599–601

wjhk/jupload2/upload/helper/ProgressBarManager.java

Violation Line
Use explicit scoping instead of the default package private level 59
Use explicit scoping instead of the default package private level 65
Use explicit scoping instead of the default package private level 70
Use explicit scoping instead of the default package private level 76
Use explicit scoping instead of the default package private level 82
Use explicit scoping instead of the default package private level 88
Use explicit scoping instead of the default package private level 94
Use explicit scoping instead of the default package private level 97
Use explicit scoping instead of the default package private level 102
Use explicit scoping instead of the default package private level 107
Use explicit scoping instead of the default package private level 114
Use explicit scoping instead of the default package private level 117
Use explicit scoping instead of the default package private level 122
Use explicit scoping instead of the default package private level 127
Found 'DD'-anomaly for variable 'currentRequestDuration' (lines '220'-'223'). 220–223
Found 'DD'-anomaly for variable 'msg' (lines '322'-'325'). 322–325
Found 'DD'-anomaly for variable 'msg' (lines '322'-'330'). 322–330
Found 'DD'-anomaly for variable 'msg' (lines '322'-'343'). 322–343
Found 'DD'-anomaly for variable 'msg' (lines '322'-'341'). 322–341
Found 'DD'-anomaly for variable 'msg' (lines '322'-'350'). 322–350
Useless parentheses. 330
Found 'DU'-anomaly for variable 'firstFileInPacket' (lines '337'-'365'). 337–365
Avoid using Literals in Conditional Statements 340
Found 'DD'-anomaly for variable 'msg' (lines '341'-'345'). 341–345
Found 'DD'-anomaly for variable 'msg' (lines '343'-'345'). 343–345
Useless parentheses. 350
This statement may have some unnecessary parentheses 350
Found 'DD'-anomaly for variable 'percent' (lines '380'-'385'). 380–385
Found 'DD'-anomaly for variable 'percent' (lines '380'-'390'). 380–390
Found 'DD'-anomaly for variable 'percent' (lines '380'-'387'). 380–387
Found 'DD'-anomaly for variable 'percent' (lines '380'-'395'). 380–395
Avoid using Literals in Conditional Statements 399
Found 'DD'-anomaly for variable 'percent' (lines '426'-'429'). 426–429
Found 'DD'-anomaly for variable 'globalCPS' (lines '441'-'444'). 441–444
Avoid using Literals in Conditional Statements 450
Found 'DD'-anomaly for variable 'eta' (lines '451'-'459'). 451–459
Avoid using Literals in Conditional Statements 453
Found 'DD'-anomaly for variable 'eta' (lines '454'-'459'). 454–459
Found 'DD'-anomaly for variable 'eta' (lines '457'-'459'). 457–459