The following document contains the results of PMD 5.3.5.
| Violation | Line |
|---|---|
| Comment is too large: Too many lines | 1–15 |
| All methods are static. Consider using a utility class instead. Alternatively, you could add a private constructor or make the class abstract to silence this warning. | 29–88 |
| Field unmap has the same name as a method | 30 |
| Variables that are final and static should be all capitals, 'unmap' is not all capitals. | 30 |
| fieldCommentRequirement Required | 30 |
| Variables that are final and static should be all capitals, 'getCleaner' is not all capitals. | 32 |
| fieldCommentRequirement Required | 32 |
| Variables that are final and static should be all capitals, 'clean' is not all capitals. | 33 |
| fieldCommentRequirement Required | 33 |
| Local variable 'unmapMethod' could be declared final | 38–39 |
| Avoid catching generic exceptions such as NullPointerException, RuntimeException, Exception in try-catch block | 43 |
| New exception is thrown in catch block, original stack trace may be lost | 56 |
| Parameter 'buffer' is not assigned and could be declared final | 65 |
| Parameter 'buffer' is not assigned and could be declared final | 79 |
| Avoid catching generic exceptions such as NullPointerException, RuntimeException, Exception in try-catch block | 84 |
| Violation | Line |
|---|---|
| Comment is too large: Too many lines | 1–15 |
| Avoid unused imports such as 'com.metamx.common.guava.CloseQuietly' | 25 |
| headerCommentRequirement Required | 44–497 |
| All methods are static. Consider using a utility class instead. Alternatively, you could add a private constructor or make the class abstract to silence this warning. | 45–497 |
| This class has too many methods, consider refactoring it. | 45–497 |
| fieldCommentRequirement Required | 46 |
| Variables that are final and static should be all capitals, 'log' is not all capitals. | 46 |
| fieldCommentRequirement Required | 47 |
| Avoid excessively long variable names like DEFAULT_RETRY_COUNT | 47 |
| fieldCommentRequirement Required | 49 |
| fieldCommentRequirement Required | 50 |
| Comment is too large: Too many lines | 52–61 |
| Comment is too large: Line too long | 53 |
| Comment is too large: Line too long | 55 |
| Parameter 'directory' is not assigned and could be declared final | 62 |
| Parameter 'outputZipFile' is not assigned and could be declared final | 62 |
| Comment is too large: Too many lines | 73–82 |
| Comment is too large: Line too long | 74 |
| Comment is too large: Line too long | 76 |
| Comment is too large: Line too long | 77 |
| Parameter 'out' is not assigned and could be declared final | 83 |
| Avoid prefixing parameters by in, out or inOut. Uses Javadoc to document this behavior. | 83 |
| Parameter 'directory' is not assigned and could be declared final | 83 |
| Found 'UR'-anomaly for variable 'file' (lines '83'-'93'). | 83–93 |
| Local variable 'file' could be declared final | 92 |
| Avoid instantiating new objects inside loops | 98 |
| Potential violation of Law of Demeter (method chain calls) | 99 |
| Comment is too large: Too many lines | 109–122 |
| Comment is too large: Line too long | 110 |
| Comment is too large: Line too long | 111 |
| Comment is too large: Line too long | 116 |
| Comment is too large: Line too long | 117 |
| Comment is too large: Line too long | 119 |
| Avoid prefixing parameters by in, out or inOut. Uses Javadoc to document this behavior. | 125 |
| Parameter 'cacheLocally' is not assigned and could be declared final | 127 |
| Avoid if (x != y) ..; else ..; | 130–164 |
| A method should have only one exit point, and that should be the last statement in the method | 132–143 |
| publicMethodCommentRequirement Required | 136–139 |
| A method/constructor shouldnt explicitly throw java.lang.Exception | 136 |
| Avoid catching generic exceptions such as NullPointerException, RuntimeException, Exception in try-catch block | 145 |
| Found 'DU'-anomaly for variable 'copyResult' (lines '151'-'165'). | 151–165 |
| Avoid unused local variables such as 'copyResult'. | 151 |
| Found 'DU'-anomaly for variable 'copyResult' (lines '151'-'165'). | 151–165 |
| Local variable 'copyResult' could be declared final | 151–156 |
| Avoid declaring a variable if it is unreferenced before a possible exit point. | 151–156 |
| Potential violation of Law of Demeter (object not created locally) | 160 |
| Potential violation of Law of Demeter (object not created locally) | 161 |
| Comment is too large: Too many lines | 167–178 |
| Comment is too large: Line too long | 168 |
| Comment is too large: Line too long | 169 |
| Comment is too large: Line too long | 173 |
| Comment is too large: Line too long | 175 |
| Avoid prefixing parameters by in, out or inOut. Uses Javadoc to document this behavior. | 181 |
| Parameter 'cacheLocally' is not assigned and could be declared final | 182 |
| Comment is too large: Too many lines | 188–197 |
| Comment is too large: Line too long | 189 |
| Avoid prefixing parameters by in, out or inOut. Uses Javadoc to document this behavior. | 198 |
| Potential violation of Law of Demeter (object not created locally) | 207 |
| Potential violation of Law of Demeter (object not created locally) | 208 |
| Avoid instantiating new objects inside loops | 211–218 |
| publicMethodCommentRequirement Required | 214–217 |
| Avoid instantiating new objects inside loops | 219 |
| Potential violation of Law of Demeter (object not created locally) | 219 |
| Comment is too large: Too many lines | 229–240 |
| Comment is too large: Line too long | 230 |
| Comment is too large: Line too long | 237 |
| Avoid variables with short names like in | 241 |
| Avoid prefixing parameters by in, out or inOut. Uses Javadoc to document this behavior. | 241 |
| Avoid prefixing parameters by in, out or inOut. Uses Javadoc to document this behavior. | 241 |
| Parameter 'in' is not assigned and could be declared final | 241 |
| Parameter 'outDir' is not assigned and could be declared final | 241 |
| Found 'DU'-anomaly for variable 'entry' (lines '246'-'254'). | 246–254 |
| Potential violation of Law of Demeter (object not created locally) | 246 |
| Avoid assignments in operands | 246 |
| Avoid instantiating new objects inside loops | 247 |
| Potential violation of Law of Demeter (object not created locally) | 247 |
| Potential violation of Law of Demeter (method chain calls) | 248 |
| Potential violation of Law of Demeter (object not created locally) | 249 |
| Potential violation of Law of Demeter (object not created locally) | 250 |
| Comment is too large: Too many lines | 256–265 |
| Parameter 'outFile' is not assigned and could be declared final | 266 |
| Avoid prefixing parameters by in, out or inOut. Uses Javadoc to document this behavior. | 266 |
| Comment is too large: Line too long | 272 |
| Comment is too large: Line too long | 274 |
| Avoid variables with short names like in | 279 |
| Avoid prefixing parameters by in, out or inOut. Uses Javadoc to document this behavior. | 279 |
| Parameter 'outFile' is not assigned and could be declared final | 279 |
| Avoid prefixing parameters by in, out or inOut. Uses Javadoc to document this behavior. | 279 |
| Parameter 'in' is not assigned and could be declared final | 279 |
| Potential violation of Law of Demeter (method chain calls) | 282 |
| Comment is too large: Line too long | 288 |
| Comment is too large: Line too long | 292 |
| Avoid variables with short names like in | 294 |
| Avoid prefixing parameters by in, out or inOut. Uses Javadoc to document this behavior. | 294 |
| publicMethodCommentRequirement Required | 300–304 |
| Comment is too large: Line too long | 302 |
| Comment is too large: Too many lines | 309–318 |
| Parameter 'out' is not assigned and could be declared final | 319 |
| Avoid prefixing parameters by in, out or inOut. Uses Javadoc to document this behavior. | 319 |
| Avoid variables with short names like in | 319 |
| Parameter 'in' is not assigned and could be declared final | 319 |
| Avoid prefixing parameters by in, out or inOut. Uses Javadoc to document this behavior. | 319 |
| Comment is too large: Too many lines | 331–339 |
| Avoid variables with short names like in | 341 |
| Avoid prefixing parameters by in, out or inOut. Uses Javadoc to document this behavior. | 341 |
| Avoid prefixing parameters by in, out or inOut. Uses Javadoc to document this behavior. | 342 |
| Parameter 'shouldRetry' is not assigned and could be declared final | 343 |
| publicMethodCommentRequirement Required | 350–353 |
| Avoid prefixing parameters by in, out or inOut. Uses Javadoc to document this behavior. | 370 |
| Avoid variables with short names like in | 370 |
| Avoid prefixing parameters by in, out or inOut. Uses Javadoc to document this behavior. | 370 |
| Parameter 'outFile' is not assigned and could be declared final | 370 |
| Comment is too large: Too many lines | 375–385 |
| Comment is too large: Line too long | 380 |
| Parameter 'inputStream' is not assigned and could be declared final | 386 |
| Avoid prefixing parameters by in, out or inOut. Uses Javadoc to document this behavior. | 386 |
| Parameter 'out' is not assigned and could be declared final | 386 |
| Comment is too large: Too many lines | 398–408 |
| Comment is too large: Line too long | 403 |
| Parameter 'shouldRetry' is not assigned and could be declared final | 409 |
| Avoid prefixing parameters by in, out or inOut. Uses Javadoc to document this behavior. | 409 |
| Avoid prefixing parameters by in, out or inOut. Uses Javadoc to document this behavior. | 409 |
| Parameter 'shouldRetry' is not assigned and could be declared final | 416 |
| Avoid variables with short names like in | 416 |
| Avoid prefixing parameters by in, out or inOut. Uses Javadoc to document this behavior. | 416 |
| publicMethodCommentRequirement Required | 416–432 |
| Avoid prefixing parameters by in, out or inOut. Uses Javadoc to document this behavior. | 416 |
| publicMethodCommentRequirement Required | 424–427 |
| Comment is too large: Too many lines | 435–444 |
| Avoid prefixing parameters by in, out or inOut. Uses Javadoc to document this behavior. | 445 |
| Avoid prefixing parameters by in, out or inOut. Uses Javadoc to document this behavior. | 445 |
| Parameter 'fName' is not assigned and could be declared final | 457 |
| A method should have only one exit point, and that should be the last statement in the method | 460 |
| Parameter 'fName' is not assigned and could be declared final | 472 |
| A method should have only one exit point, and that should be the last statement in the method | 475 |
| Comment is too large: Too many lines | 480–488 |
| Parameter 'fname' is not assigned and could be declared final | 489 |
| Potential violation of Law of Demeter (object not created locally) | 492 |
| Violation | Line |
|---|---|
| Comment is too large: Too many lines | 1–15 |
| headerCommentRequirement Required | 29–128 |
| All methods are static. Consider using a utility class instead. Alternatively, you could add a private constructor or make the class abstract to silence this warning. | 30–128 |
| Comment is too large: Line too long | 32 |
| Parameter 'input' is not assigned and could be declared final | 37 |
| publicMethodCommentRequirement Required | 37–40 |
| Comment is too large: Too many lines | 42–51 |
| Comment is too large: Line too long | 43 |
| Comment is too large: Line too long | 45 |
| Comment is too large: Line too long | 47 |
| Comment is too large: Line too long | 48 |
| Avoid prefixing parameters by in, out or inOut. Uses Javadoc to document this behavior. | 54 |
| Avoid catching generic exceptions such as NullPointerException, RuntimeException, Exception in try-catch block | 68 |
| Comment is too large: Line too long | 74 |
| Comment is too large: Line too long | 76 |
| Comment is too large: Line too long | 77 |
| fieldCommentRequirement Required | 81 |
| Field size has the same name as a method | 82 |
| fieldCommentRequirement Required | 82 |
| Avoid using redundant field initializer for 'size' | 82 |
| Found non-transient, non-static member. Please mark as transient or provide accessors. | 82 |
| publicMethodCommentRequirement Required | 84–87 |
| publicMethodCommentRequirement Required | 90–93 |
| publicMethodCommentRequirement Required | 95–98 |
| Parameter 'files' is not assigned and could be declared final | 95 |
| publicMethodCommentRequirement Required | 100–103 |
| Parameter 'files' is not assigned and could be declared final | 100 |
| Overridable method 'addSizedFiles' called during object construction | 102 |
| protectedMethodCommentRequirement Required | 105–116 |
| Found 'UR'-anomaly for variable 'file' (lines '105'-'112'). | 105–112 |
| Parameter 'files' is not assigned and could be declared final | 105 |
| Local variable 'file' could be declared final | 111 |
| publicMethodCommentRequirement Required | 118–121 |
| Parameter 'files' is not assigned and could be declared final | 118 |
| publicMethodCommentRequirement Required | 123–126 |
| Parameter 'file' is not assigned and could be declared final | 123 |
| Violation | Line |
|---|---|
| Comment is too large: Too many lines | 1–15 |
| Possible God class (WMC=140, ATFD=50, TCC=0.0) | 17–885 |
| This class has a bunch of public methods and attributes | 17–885 |
| fieldCommentRequirement Required | 45 |
| fieldCommentRequirement Required | 46 |
| Avoid excessively long variable names like lowerDefaultFormat | 47 |
| fieldCommentRequirement Required | 47 |
| Parameter 'type' is not assigned and could be declared final | 50 |
| publicMethodCommentRequirement Required | 50–62 |
| A method should have only one exit point, and that should be the last statement in the method | 54 |
| A method should have only one exit point, and that should be the last statement in the method | 56 |
| The String literal "There is no format for type %s at granularity %s" appears 6 times in this file; the first occurrence is on line 60 | 60 |
| Parameter 'count' is not assigned and could be declared final | 65 |
| publicMethodCommentRequirement Required | 65–68 |
| publicMethodCommentRequirement Required | 71–78 |
| Parameter 'time' is not assigned and could be declared final | 71 |
| Potential violation of Law of Demeter (object not created locally) | 75 |
| Potential violation of Law of Demeter (object not created locally) | 77 |
| publicMethodCommentRequirement Required | 81–84 |
| Parameter 'interval' is not assigned and could be declared final | 81 |
| Potential violation of Law of Demeter (method chain calls) | 83 |
| Parameter 'formatter' is not assigned and could be declared final | 87 |
| Parameter 'filePath' is not assigned and could be declared final | 87 |
| publicMethodCommentRequirement Required | 87–102 |
| Local variable 'vals' could be declared final | 89 |
| fieldCommentRequirement Required | 106 |
| fieldCommentRequirement Required | 107 |
| Avoid excessively long variable names like lowerDefaultFormat | 108 |
| fieldCommentRequirement Required | 108 |
| Parameter 'type' is not assigned and could be declared final | 111 |
| publicMethodCommentRequirement Required | 111–123 |
| A method should have only one exit point, and that should be the last statement in the method | 115 |
| A method should have only one exit point, and that should be the last statement in the method | 117 |
| Parameter 'count' is not assigned and could be declared final | 126 |
| publicMethodCommentRequirement Required | 126–129 |
| Parameter 'time' is not assigned and could be declared final | 132 |
| publicMethodCommentRequirement Required | 132–140 |
| Potential violation of Law of Demeter (object not created locally) | 136 |
| Potential violation of Law of Demeter (object not created locally) | 137 |
| Potential violation of Law of Demeter (object not created locally) | 139 |
| publicMethodCommentRequirement Required | 143–146 |
| Parameter 'interval' is not assigned and could be declared final | 143 |
| Potential violation of Law of Demeter (method chain calls) | 145 |
| Parameter 'formatter' is not assigned and could be declared final | 149 |
| Parameter 'filePath' is not assigned and could be declared final | 149 |
| publicMethodCommentRequirement Required | 149–159 |
| Local variable 'vals' could be declared final | 151 |
| Parameter 'type' is not assigned and could be declared final | 164 |
| publicMethodCommentRequirement Required | 164–167 |
| publicMethodCommentRequirement Required | 170–173 |
| Parameter 'count' is not assigned and could be declared final | 170 |
| Parameter 'time' is not assigned and could be declared final | 176 |
| publicMethodCommentRequirement Required | 176–185 |
| Potential violation of Law of Demeter (object not created locally) | 180 |
| Potential violation of Law of Demeter (object not created locally) | 181 |
| Potential violation of Law of Demeter (object not created locally) | 182 |
| Potential violation of Law of Demeter (object not created locally) | 182 |
| Potential violation of Law of Demeter (object not created locally) | 182 |
| Potential violation of Law of Demeter (object not created locally) | 184 |
| publicMethodCommentRequirement Required | 188–191 |
| Parameter 'interval' is not assigned and could be declared final | 188 |
| Potential violation of Law of Demeter (method chain calls) | 190 |
| Parameter 'filePath' is not assigned and could be declared final | 194 |
| publicMethodCommentRequirement Required | 194–203 |
| Parameter 'formatter' is not assigned and could be declared final | 194 |
| Local variable 'vals' could be declared final | 196 |
| A method should have only one exit point, and that should be the last statement in the method | 199 |
| Parameter 'type' is not assigned and could be declared final | 208 |
| publicMethodCommentRequirement Required | 208–211 |
| publicMethodCommentRequirement Required | 214–217 |
| Parameter 'count' is not assigned and could be declared final | 214 |
| Parameter 'time' is not assigned and could be declared final | 220 |
| publicMethodCommentRequirement Required | 220–229 |
| Potential violation of Law of Demeter (object not created locally) | 224 |
| Potential violation of Law of Demeter (object not created locally) | 225 |
| Potential violation of Law of Demeter (object not created locally) | 226 |
| Potential violation of Law of Demeter (object not created locally) | 226 |
| Potential violation of Law of Demeter (object not created locally) | 226 |
| Potential violation of Law of Demeter (object not created locally) | 228 |
| publicMethodCommentRequirement Required | 232–235 |
| Parameter 'interval' is not assigned and could be declared final | 232 |
| Potential violation of Law of Demeter (method chain calls) | 234 |
| publicMethodCommentRequirement Required | 238–247 |
| Parameter 'filePath' is not assigned and could be declared final | 238 |
| Parameter 'formatter' is not assigned and could be declared final | 238 |
| Local variable 'vals' could be declared final | 240 |
| A method should have only one exit point, and that should be the last statement in the method | 243 |
| Parameter 'type' is not assigned and could be declared final | 252 |
| publicMethodCommentRequirement Required | 252–255 |
| Parameter 'count' is not assigned and could be declared final | 258 |
| publicMethodCommentRequirement Required | 258–261 |
| Parameter 'time' is not assigned and could be declared final | 264 |
| publicMethodCommentRequirement Required | 264–273 |
| Potential violation of Law of Demeter (object not created locally) | 268 |
| Potential violation of Law of Demeter (object not created locally) | 269 |
| Potential violation of Law of Demeter (object not created locally) | 270 |
| Potential violation of Law of Demeter (object not created locally) | 270 |
| Potential violation of Law of Demeter (object not created locally) | 270 |
| Potential violation of Law of Demeter (object not created locally) | 272 |
| Parameter 'interval' is not assigned and could be declared final | 276 |
| publicMethodCommentRequirement Required | 276–279 |
| Potential violation of Law of Demeter (method chain calls) | 278 |
| Parameter 'filePath' is not assigned and could be declared final | 282 |
| publicMethodCommentRequirement Required | 282–291 |
| Parameter 'formatter' is not assigned and could be declared final | 282 |
| Local variable 'vals' could be declared final | 284 |
| A method should have only one exit point, and that should be the last statement in the method | 287 |
| fieldCommentRequirement Required | 295 |
| fieldCommentRequirement Required | 296 |
| Avoid excessively long variable names like lowerDefaultFormat | 297 |
| fieldCommentRequirement Required | 297 |
| Parameter 'type' is not assigned and could be declared final | 300 |
| publicMethodCommentRequirement Required | 300–312 |
| A method should have only one exit point, and that should be the last statement in the method | 304 |
| A method should have only one exit point, and that should be the last statement in the method | 306 |
| Parameter 'n' is not assigned and could be declared final | 315 |
| Avoid variables with short names like n | 315 |
| publicMethodCommentRequirement Required | 315–318 |
| Parameter 'time' is not assigned and could be declared final | 321 |
| publicMethodCommentRequirement Required | 321–330 |
| Potential violation of Law of Demeter (object not created locally) | 325 |
| Potential violation of Law of Demeter (object not created locally) | 326 |
| Potential violation of Law of Demeter (object not created locally) | 327 |
| Potential violation of Law of Demeter (object not created locally) | 329 |
| publicMethodCommentRequirement Required | 333–336 |
| Parameter 'interval' is not assigned and could be declared final | 333 |
| Potential violation of Law of Demeter (method chain calls) | 335 |
| publicMethodCommentRequirement Required | 339–349 |
| Parameter 'filePath' is not assigned and could be declared final | 339 |
| Parameter 'formatter' is not assigned and could be declared final | 339 |
| Local variable 'vals' could be declared final | 341 |
| publicMethodCommentRequirement Required | 354–357 |
| Parameter 'type' is not assigned and could be declared final | 354 |
| Avoid variables with short names like n | 360 |
| publicMethodCommentRequirement Required | 360–363 |
| Parameter 'n' is not assigned and could be declared final | 360 |
| Parameter 'time' is not assigned and could be declared final | 366 |
| publicMethodCommentRequirement Required | 366–376 |
| Potential violation of Law of Demeter (object not created locally) | 370 |
| Potential violation of Law of Demeter (object not created locally) | 371 |
| Potential violation of Law of Demeter (object not created locally) | 372 |
| Potential violation of Law of Demeter (object not created locally) | 373 |
| Potential violation of Law of Demeter (object not created locally) | 373 |
| Potential violation of Law of Demeter (object not created locally) | 373 |
| Potential violation of Law of Demeter (object not created locally) | 375 |
| Parameter 'interval' is not assigned and could be declared final | 379 |
| publicMethodCommentRequirement Required | 379–382 |
| Potential violation of Law of Demeter (method chain calls) | 381 |
| Parameter 'formatter' is not assigned and could be declared final | 385 |
| Parameter 'filePath' is not assigned and could be declared final | 385 |
| publicMethodCommentRequirement Required | 385–393 |
| Local variable 'vals' could be declared final | 387 |
| A method should have only one exit point, and that should be the last statement in the method | 390 |
| fieldCommentRequirement Required | 397 |
| The String literal "'y'=yyyy/'m'=MM/'d'=dd" appears 4 times in this file; the first occurrence is on line 397 | 397 |
| fieldCommentRequirement Required | 398 |
| fieldCommentRequirement Required | 399 |
| Avoid excessively long variable names like lowerDefaultFormat | 399 |
| Parameter 'type' is not assigned and could be declared final | 402 |
| publicMethodCommentRequirement Required | 402–414 |
| A method should have only one exit point, and that should be the last statement in the method | 406 |
| A method should have only one exit point, and that should be the last statement in the method | 408 |
| Avoid variables with short names like n | 417 |
| Parameter 'n' is not assigned and could be declared final | 417 |
| publicMethodCommentRequirement Required | 417–420 |
| Parameter 'time' is not assigned and could be declared final | 423 |
| publicMethodCommentRequirement Required | 423–430 |
| Potential violation of Law of Demeter (object not created locally) | 427 |
| Potential violation of Law of Demeter (object not created locally) | 429 |
| Parameter 'interval' is not assigned and could be declared final | 433 |
| publicMethodCommentRequirement Required | 433–436 |
| Potential violation of Law of Demeter (method chain calls) | 435 |
| Parameter 'filePath' is not assigned and could be declared final | 439 |
| publicMethodCommentRequirement Required | 439–449 |
| Parameter 'formatter' is not assigned and could be declared final | 439 |
| Local variable 'vals' could be declared final | 441 |
| fieldCommentRequirement Required | 453 |
| fieldCommentRequirement Required | 454 |
| Avoid excessively long variable names like lowerDefaultFormat | 455 |
| fieldCommentRequirement Required | 455 |
| Parameter 'type' is not assigned and could be declared final | 458 |
| publicMethodCommentRequirement Required | 458–461 |
| Avoid variables with short names like n | 464 |
| publicMethodCommentRequirement Required | 464–467 |
| Parameter 'n' is not assigned and could be declared final | 464 |
| Parameter 'time' is not assigned and could be declared final | 470 |
| publicMethodCommentRequirement Required | 470–478 |
| Potential violation of Law of Demeter (object not created locally) | 474 |
| Potential violation of Law of Demeter (object not created locally) | 475 |
| Potential violation of Law of Demeter (object not created locally) | 477 |
| Parameter 'interval' is not assigned and could be declared final | 481 |
| publicMethodCommentRequirement Required | 481–484 |
| Potential violation of Law of Demeter (method chain calls) | 483 |
| Parameter 'formatter' is not assigned and could be declared final | 487 |
| publicMethodCommentRequirement Required | 487–497 |
| Parameter 'filePath' is not assigned and could be declared final | 487 |
| Local variable 'vals' could be declared final | 489 |
| fieldCommentRequirement Required | 501 |
| fieldCommentRequirement Required | 502 |
| fieldCommentRequirement Required | 503 |
| Avoid excessively long variable names like lowerDefaultFormat | 503 |
| Parameter 'type' is not assigned and could be declared final | 506 |
| publicMethodCommentRequirement Required | 506–518 |
| A method should have only one exit point, and that should be the last statement in the method | 510 |
| A method should have only one exit point, and that should be the last statement in the method | 512 |
| publicMethodCommentRequirement Required | 521–524 |
| Parameter 'n' is not assigned and could be declared final | 521 |
| Avoid variables with short names like n | 521 |
| Parameter 'time' is not assigned and could be declared final | 527 |
| publicMethodCommentRequirement Required | 527–535 |
| Potential violation of Law of Demeter (object not created locally) | 531 |
| Potential violation of Law of Demeter (object not created locally) | 532 |
| Potential violation of Law of Demeter (object not created locally) | 534 |
| publicMethodCommentRequirement Required | 538–541 |
| Parameter 'interval' is not assigned and could be declared final | 538 |
| Potential violation of Law of Demeter (method chain calls) | 540 |
| Parameter 'formatter' is not assigned and could be declared final | 544 |
| publicMethodCommentRequirement Required | 544–554 |
| Parameter 'filePath' is not assigned and could be declared final | 544 |
| Local variable 'vals' could be declared final | 546 |
| fieldCommentRequirement Required | 558 |
| fieldCommentRequirement Required | 559 |
| Avoid excessively long variable names like lowerDefaultFormat | 560 |
| fieldCommentRequirement Required | 560 |
| publicMethodCommentRequirement Required | 563–575 |
| Parameter 'type' is not assigned and could be declared final | 563 |
| A method should have only one exit point, and that should be the last statement in the method | 567 |
| A method should have only one exit point, and that should be the last statement in the method | 569 |
| Parameter 'n' is not assigned and could be declared final | 578 |
| Avoid variables with short names like n | 578 |
| publicMethodCommentRequirement Required | 578–581 |
| publicMethodCommentRequirement Required | 584–593 |
| Parameter 'time' is not assigned and could be declared final | 584 |
| Potential violation of Law of Demeter (object not created locally) | 588 |
| Potential violation of Law of Demeter (object not created locally) | 589 |
| Potential violation of Law of Demeter (object not created locally) | 590 |
| Potential violation of Law of Demeter (object not created locally) | 592 |
| Parameter 'interval' is not assigned and could be declared final | 596 |
| publicMethodCommentRequirement Required | 596–599 |
| Potential violation of Law of Demeter (method chain calls) | 598 |
| Parameter 'formatter' is not assigned and could be declared final | 602 |
| Parameter 'filePath' is not assigned and could be declared final | 602 |
| publicMethodCommentRequirement Required | 602–612 |
| Local variable 'vals' could be declared final | 604 |
| fieldCommentRequirement Required | 617–620 |
| Avoid excessively long variable names like defaultPathPattern | 617 |
| fieldCommentRequirement Required | 621–622 |
| publicMethodCommentRequirement Required | 625 |
| Avoid variables with short names like n | 627 |
| publicMethodCommentRequirement Required | 627 |
| publicMethodCommentRequirement Required | 629 |
| publicMethodCommentRequirement Required | 631 |
| publicMethodCommentRequirement Required | 633 |
| Parameter 'filePath' is not assigned and could be declared final | 635 |
| publicMethodCommentRequirement Required | 635–638 |
| Parameter 'filePath' is not assigned and could be declared final | 641 |
| Parameter 'formatter' is not assigned and could be declared final | 641 |
| protectedMethodCommentRequirement Required | 641–665 |
| Potential violation of Law of Demeter (object not created locally) | 655 |
| Local variable 'matcher' could be declared final | 655 |
| Potential violation of Law of Demeter (object not created locally) | 658 |
| Potential violation of Law of Demeter (object not created locally) | 659 |
| Assigning an Object to null is a code smell. Consider refactoring. | 660 |
| Potential violation of Law of Demeter (object not created locally) | 660 |
| Avoid if (x != y) ..; else ..; | 660 |
| Potential violation of Law of Demeter (object not created locally) | 660 |
| publicMethodCommentRequirement Required | 668–671 |
| Parameter 'type' is not assigned and could be declared final | 668 |
| When doing a String.toLowerCase()/toUpperCase() call, use a Locale | 670 |
| publicMethodCommentRequirement Required | 673–676 |
| Parameter 'time' is not assigned and could be declared final | 673 |
| Parameter 'time' is not assigned and could be declared final | 678 |
| Parameter 'count' is not assigned and could be declared final | 678 |
| publicMethodCommentRequirement Required | 678–681 |
| Parameter 'time' is not assigned and could be declared final | 683 |
| publicMethodCommentRequirement Required | 683–686 |
| publicMethodCommentRequirement Required | 688–691 |
| Parameter 'count' is not assigned and could be declared final | 688 |
| Parameter 'time' is not assigned and could be declared final | 688 |
| Parameter 'time' is not assigned and could be declared final | 693 |
| publicMethodCommentRequirement Required | 693–696 |
| Parameter 'type' is not assigned and could be declared final | 698 |
| Parameter 'time' is not assigned and could be declared final | 698 |
| publicMethodCommentRequirement Required | 698–701 |
| When doing a String.toLowerCase()/toUpperCase() call, use a Locale | 700 |
| Parameter 'time' is not assigned and could be declared final | 703 |
| publicMethodCommentRequirement Required | 703–706 |
| Parameter 'type' is not assigned and could be declared final | 703 |
| Potential violation of Law of Demeter (method chain calls) | 705 |
| Parameter 't' is not assigned and could be declared final | 711 |
| Avoid variables with short names like t | 711 |
| Local variable 'start' could be declared final | 713 |
| Parameter 'interval' is not assigned and could be declared final | 720 |
| Avoid using final local variables, turn them into fields | 723 |
| Potential violation of Law of Demeter (method chain calls) | 725 |
| Potential violation of Law of Demeter (method chain calls) | 728 |
| publicMethodCommentRequirement Required | 740–743 |
| publicMethodCommentRequirement Required | 745–748 |
| publicMethodCommentRequirement Required | 750–753 |
| publicMethodCommentRequirement Required | 755–758 |
| headerCommentRequirement Required | 760–775 |
| fieldCommentRequirement Required | 762 |
| Found non-transient, non-static member. Please mark as transient or provide accessors. | 762 |
| publicMethodCommentRequirement Required | 764–767 |
| Parameter 'inputInterval' is not assigned and could be declared final | 764 |
| publicMethodCommentRequirement Required | 770–773 |
| headerCommentRequirement Required | 777–792 |
| fieldCommentRequirement Required | 779 |
| Found non-transient, non-static member. Please mark as transient or provide accessors. | 779 |
| Parameter 'inputInterval' is not assigned and could be declared final | 781 |
| publicMethodCommentRequirement Required | 781–784 |
| publicMethodCommentRequirement Required | 787–790 |
| headerCommentRequirement Required | 794–834 |
| Found non-transient, non-static member. Please mark as transient or provide accessors. | 796 |
| fieldCommentRequirement Required | 796 |
| fieldCommentRequirement Required | 798 |
| Found non-transient, non-static member. Please mark as transient or provide accessors. | 798 |
| fieldCommentRequirement Required | 799 |
| Found non-transient, non-static member. Please mark as transient or provide accessors. | 799 |
| publicMethodCommentRequirement Required | 801–807 |
| Parameter 'inputInterval' is not assigned and could be declared final | 801 |
| publicMethodCommentRequirement Required | 810–813 |
| publicMethodCommentRequirement Required | 816–827 |
| Local variable 'retVal' could be declared final | 821 |
| publicMethodCommentRequirement Required | 830–833 |
| headerCommentRequirement Required | 836–877 |
| Found non-transient, non-static member. Please mark as transient or provide accessors. | 838 |
| fieldCommentRequirement Required | 838 |
| fieldCommentRequirement Required | 840 |
| Found non-transient, non-static member. Please mark as transient or provide accessors. | 840 |
| Found non-transient, non-static member. Please mark as transient or provide accessors. | 841 |
| fieldCommentRequirement Required | 841 |
| publicMethodCommentRequirement Required | 843–850 |
| Parameter 'inputInterval' is not assigned and could be declared final | 843 |
| publicMethodCommentRequirement Required | 853–856 |
| publicMethodCommentRequirement Required | 859–870 |
| Local variable 'retVal' could be declared final | 864 |
| publicMethodCommentRequirement Required | 873–876 |
| enumCommentRequirement Required | 879–884 |
| Violation | Line |
|---|---|
| Comment is too large: Too many lines | 1–15 |
| Avoid short class names like IAE | 21–32 |
| Parameter 'formatText' is not assigned and could be declared final | 23 |
| publicMethodCommentRequirement Required | 23–26 |
| Parameter 'arguments' is not assigned and could be declared final | 23 |
| Parameter 'cause' is not assigned and could be declared final | 28 |
| publicMethodCommentRequirement Required | 28–31 |
| Parameter 'formatText' is not assigned and could be declared final | 28 |
| Parameter 'arguments' is not assigned and could be declared final | 28 |
| Violation | Line |
|---|---|
| Comment is too large: Too many lines | 1–15 |
| Avoid short class names like ISE | 21–32 |
| Parameter 'arguments' is not assigned and could be declared final | 23 |
| publicMethodCommentRequirement Required | 23–26 |
| Parameter 'formatText' is not assigned and could be declared final | 23 |
| Parameter 'cause' is not assigned and could be declared final | 28 |
| publicMethodCommentRequirement Required | 28–31 |
| Parameter 'formatText' is not assigned and could be declared final | 28 |
| Parameter 'arguments' is not assigned and could be declared final | 28 |
| Violation | Line |
|---|---|
| Comment is too large: Too many lines | 1–15 |
| All methods are static. Consider using a utility class instead. Alternatively, you could add a private constructor or make the class abstract to silence this warning. | 34–138 |
| publicMethodCommentRequirement Required | 35–63 |
| Found 'UR'-anomaly for variable 'interval' (lines '35'-'41'). | 35–41 |
| Parameter 'intervals' is not assigned and could be declared final | 35 |
| Avoid using implementation types like 'ArrayList'; use the interface instead | 35 |
| Avoid declaring a variable if it is unreferenced before a possible exit point. | 37 |
| Local variable 'retVal' could be declared final | 37 |
| Found 'DU'-anomaly for variable 'retVal' (lines '37'-'63'). | 37–63 |
| Local variable 'sortedIntervals' could be declared final | 39 |
| Local variable 'interval' could be declared final | 40 |
| Potential violation of Law of Demeter (object not created locally) | 41 |
| Potential violation of Law of Demeter (object not created locally) | 44 |
| A method should have only one exit point, and that should be the last statement in the method | 45 |
| Potential violation of Law of Demeter (object not created locally) | 48 |
| Local variable 'intervalsIter' could be declared final | 48 |
| Local variable 'next' could be declared final | 51 |
| Potential violation of Law of Demeter (object not created locally) | 53 |
| Potential violation of Law of Demeter (object not created locally) | 53 |
| Potential violation of Law of Demeter (object not created locally) | 54 |
| Avoid instantiating new objects inside loops | 54 |
| Potential violation of Law of Demeter (object not created locally) | 54 |
| Potential violation of Law of Demeter (object not created locally) | 56 |
| Potential violation of Law of Demeter (object not created locally) | 60 |
| publicMethodCommentRequirement Required | 65–82 |
| Parameter 'intervals' is not assigned and could be declared final | 65 |
| Found 'UR'-anomaly for variable 'interval' (lines '65'-'71'). | 65–71 |
| Local variable 'startDates' could be declared final | 67 |
| Local variable 'endDates' could be declared final | 68 |
| Local variable 'interval' could be declared final | 70 |
| Potential violation of Law of Demeter (object not created locally) | 71 |
| Potential violation of Law of Demeter (object not created locally) | 72 |
| Local variable 'minStart' could be declared final | 75 |
| Local variable 'maxEnd' could be declared final | 76 |
| Avoid variables with short names like i | 84 |
| publicMethodCommentRequirement Required | 84–97 |
| Parameter 'intervals' is not assigned and could be declared final | 84 |
| Parameter 'input' is not assigned and could be declared final | 90 |
| publicMethodCommentRequirement Required | 90–93 |
| publicMethodCommentRequirement Required | 99–117 |
| Parameter 'times' is not assigned and could be declared final | 99 |
| A method should have only one exit point, and that should be the last statement in the method | 102 |
| A method should have only one exit point, and that should be the last statement in the method | 107 |
| A method should have only one exit point, and that should be the last statement in the method | 109 |
| Found 'DD'-anomaly for variable 'min' (lines '111'-'113'). | 111–113 |
| Potential violation of Law of Demeter (method chain calls) | 113 |
| Potential violation of Law of Demeter (object not created locally) | 113 |
| publicMethodCommentRequirement Required | 119–137 |
| Parameter 'times' is not assigned and could be declared final | 119 |
| A method should have only one exit point, and that should be the last statement in the method | 122 |
| A method should have only one exit point, and that should be the last statement in the method | 127 |
| A method should have only one exit point, and that should be the last statement in the method | 129 |
| Found 'DD'-anomaly for variable 'max' (lines '131'-'133'). | 131–133 |
| Potential violation of Law of Demeter (object not created locally) | 133 |
| Potential violation of Law of Demeter (method chain calls) | 133 |
| Violation | Line |
|---|---|
| Comment is too large: Too many lines | 1–15 |
| All methods are static. Consider using a utility class instead. Alternatively, you could add a private constructor or make the class abstract to silence this warning. | 27–196 |
| publicMethodCommentRequirement Required | 28–31 |
| Avoid variables with short names like in | 28 |
| Avoid prefixing parameters by in, out or inOut. Uses Javadoc to document this behavior. | 28 |
| Parameter 'in' is not assigned and could be declared final | 28 |
| Parameter 'key' is not assigned and could be declared final | 28 |
| Avoid prefixing parameters by in, out or inOut. Uses Javadoc to document this behavior. | 33 |
| Avoid variables with short names like in | 33 |
| Parameter 'defaultValue' is not assigned and could be declared final | 33 |
| publicMethodCommentRequirement Required | 33–46 |
| Parameter 'key' is not assigned and could be declared final | 33 |
| Parameter 'in' is not assigned and could be declared final | 33 |
| Local variable 'retVal' could be declared final | 35 |
| The String literal "Key[%s] is required in map[%s]" appears 6 times in this file; the first occurrence is on line 39 | 39 |
| A method should have only one exit point, and that should be the last statement in the method | 42 |
| Potential violation of Law of Demeter (object not created locally) | 45 |
| publicMethodCommentRequirement Required | 48–58 |
| Parameter 'map' is not assigned and could be declared final | 53 |
| publicMethodCommentRequirement Required | 53–56 |
| publicMethodCommentRequirement Required | 60–70 |
| Parameter 'map' is not assigned and could be declared final | 60 |
| Parameter 'key' is not assigned and could be declared final | 60 |
| Parameter 'lookupMap' is not assigned and could be declared final | 60 |
| Local variable 'lookupKey' could be declared final | 62 |
| Local variable 'retVal' could be declared final | 63 |
| Avoid prefixing parameters by in, out or inOut. Uses Javadoc to document this behavior. | 72 |
| Avoid variables with short names like in | 72 |
| Parameter 'key' is not assigned and could be declared final | 72 |
| Parameter 'in' is not assigned and could be declared final | 72 |
| publicMethodCommentRequirement Required | 72–75 |
| Avoid prefixing parameters by in, out or inOut. Uses Javadoc to document this behavior. | 77 |
| Parameter 'key' is not assigned and could be declared final | 77 |
| Parameter 'defaultValue' is not assigned and could be declared final | 77 |
| Parameter 'in' is not assigned and could be declared final | 77 |
| publicMethodCommentRequirement Required | 77–95 |
| Avoid variables with short names like in | 77 |
| Local variable 'retVal' could be declared final | 79 |
| A method should have only one exit point, and that should be the last statement in the method | 86 |
| Potential violation of Law of Demeter (object not created locally) | 90 |
| Avoid prefixing parameters by in, out or inOut. Uses Javadoc to document this behavior. | 97 |
| Parameter 'key' is not assigned and could be declared final | 97 |
| Parameter 'in' is not assigned and could be declared final | 97 |
| publicMethodCommentRequirement Required | 97–100 |
| Avoid variables with short names like in | 97 |
| Parameter 'defaultValue' is not assigned and could be declared final | 102 |
| Parameter 'in' is not assigned and could be declared final | 102 |
| Avoid prefixing parameters by in, out or inOut. Uses Javadoc to document this behavior. | 102 |
| Parameter 'key' is not assigned and could be declared final | 102 |
| publicMethodCommentRequirement Required | 102–120 |
| Avoid variables with short names like in | 102 |
| Local variable 'retVal' could be declared final | 104 |
| A method should have only one exit point, and that should be the last statement in the method | 111 |
| Potential violation of Law of Demeter (object not created locally) | 115 |
| publicMethodCommentRequirement Required | 122–125 |
| Avoid variables with short names like in | 122 |
| Parameter 'in' is not assigned and could be declared final | 122 |
| Parameter 'key' is not assigned and could be declared final | 122 |
| Avoid prefixing parameters by in, out or inOut. Uses Javadoc to document this behavior. | 122 |
| Parameter 'key' is not assigned and could be declared final | 127 |
| Avoid prefixing parameters by in, out or inOut. Uses Javadoc to document this behavior. | 127 |
| publicMethodCommentRequirement Required | 127–145 |
| Avoid variables with short names like in | 127 |
| Parameter 'defaultValue' is not assigned and could be declared final | 127 |
| Parameter 'in' is not assigned and could be declared final | 127 |
| Local variable 'retVal' could be declared final | 129 |
| A method should have only one exit point, and that should be the last statement in the method | 136 |
| Potential violation of Law of Demeter (object not created locally) | 140 |
| Avoid prefixing parameters by in, out or inOut. Uses Javadoc to document this behavior. | 147 |
| Parameter 'in' is not assigned and could be declared final | 147 |
| publicMethodCommentRequirement Required | 147–150 |
| Avoid variables with short names like in | 147 |
| Parameter 'key' is not assigned and could be declared final | 147 |
| Parameter 'key' is not assigned and could be declared final | 152 |
| Parameter 'in' is not assigned and could be declared final | 152 |
| Parameter 'defaultValue' is not assigned and could be declared final | 152 |
| Avoid prefixing parameters by in, out or inOut. Uses Javadoc to document this behavior. | 152 |
| publicMethodCommentRequirement Required | 152–170 |
| Avoid variables with short names like in | 152 |
| Local variable 'retVal' could be declared final | 154 |
| A method should have only one exit point, and that should be the last statement in the method | 161 |
| New exception is thrown in catch block, original stack trace may be lost | 168 |
| publicMethodCommentRequirement Required | 172–175 |
| Avoid prefixing parameters by in, out or inOut. Uses Javadoc to document this behavior. | 172 |
| Parameter 'in' is not assigned and could be declared final | 172 |
| Parameter 'key' is not assigned and could be declared final | 172 |
| Avoid variables with short names like in | 172 |
| Parameter 'in' is not assigned and could be declared final | 177 |
| Avoid prefixing parameters by in, out or inOut. Uses Javadoc to document this behavior. | 177 |
| publicMethodCommentRequirement Required | 177–195 |
| Parameter 'defaultValue' is not assigned and could be declared final | 177 |
| Avoid variables with short names like in | 177 |
| Parameter 'key' is not assigned and could be declared final | 177 |
| Local variable 'retVal' could be declared final | 179 |
| A method should have only one exit point, and that should be the last statement in the method | 186 |
| New exception is thrown in catch block, original stack trace may be lost | 193 |
| Violation | Line |
|---|---|
| Comment is too large: Too many lines | 1–15 |
| Avoid short class names like Pair | 25–115 |
| Generics names should be a one letter long and upper case. | 25 |
| Generics names should be a one letter long and upper case. | 25 |
| publicMethodCommentRequirement Required | 28–30 |
| Avoid using short method names | 28 |
| Parameter 'rhs' is not assigned and could be declared final | 28 |
| Parameter 'lhs' is not assigned and could be declared final | 28 |
| Found non-transient, non-static member. Please mark as transient or provide accessors. | 32 |
| Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. | 32 |
| fieldCommentRequirement Required | 32 |
| Found non-transient, non-static member. Please mark as transient or provide accessors. | 33 |
| fieldCommentRequirement Required | 33 |
| Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. | 33 |
| Parameter 'rhs' is not assigned and could be declared final | 35 |
| Parameter 'lhs' is not assigned and could be declared final | 35 |
| publicMethodCommentRequirement Required | 35–39 |
| publicMethodCommentRequirement Required | 42–61 |
| Avoid variables with short names like o | 42 |
| Parameter 'o' is not assigned and could be declared final | 42 |
| A method should have only one exit point, and that should be the last statement in the method | 45 |
| A method should have only one exit point, and that should be the last statement in the method | 48 |
| Local variable 'pair' could be declared final | 51 |
| Avoid if (x != y) ..; else ..; | 53 |
| A method should have only one exit point, and that should be the last statement in the method | 54 |
| Avoid unnecessary if..then..else statements when returning booleans | 56–58 |
| Avoid if (x != y) ..; else ..; | 56 |
| A method should have only one exit point, and that should be the last statement in the method | 57 |
| publicMethodCommentRequirement Required | 64–69 |
| Found 'DD'-anomaly for variable 'result' (lines '66'-'67'). | 66–67 |
| Avoid if (x != y) ..; else ..; | 66 |
| Avoid if (x != y) ..; else ..; | 67 |
| publicMethodCommentRequirement Required | 72–78 |
| publicMethodCommentRequirement Required | 80–90 |
| Parameter 'input' is not assigned and could be declared final | 85 |
| publicMethodCommentRequirement Required | 85–88 |
| publicMethodCommentRequirement Required | 92–102 |
| Parameter 'input' is not assigned and could be declared final | 97 |
| publicMethodCommentRequirement Required | 97–100 |
| publicMethodCommentRequirement Required | 104–114 |
| Avoid variables with short names like o2 | 109 |
| Avoid variables with short names like o1 | 109 |
| publicMethodCommentRequirement Required | 109–112 |
| Parameter 'o1' is not assigned and could be declared final | 109 |
| Parameter 'o2' is not assigned and could be declared final | 109 |
| Violation | Line |
|---|---|
| Comment is too large: Too many lines | 1–15 |
| All methods are static. Consider using a utility class instead. Alternatively, you could add a private constructor or make the class abstract to silence this warning. | 23–31 |
| headerCommentRequirement Required | 23–31 |
| Parameter 'filename' is not assigned and could be declared final | 25 |
| publicMethodCommentRequirement Required | 25–29 |
| Violation | Line |
|---|---|
| Comment is too large: Too many lines | 1–15 |
| Avoid short class names like RE | 21–32 |
| publicMethodCommentRequirement Required | 23–26 |
| Parameter 'arguments' is not assigned and could be declared final | 23 |
| Parameter 'formatText' is not assigned and could be declared final | 23 |
| Parameter 'formatText' is not assigned and could be declared final | 28 |
| Parameter 'cause' is not assigned and could be declared final | 28 |
| Parameter 'arguments' is not assigned and could be declared final | 28 |
| publicMethodCommentRequirement Required | 28–31 |
| Violation | Line |
|---|---|
| Comment is too large: Too many lines | 1–15 |
| headerCommentRequirement Required | 27–95 |
| All methods are static. Consider using a utility class instead. Alternatively, you could add a private constructor or make the class abstract to silence this warning. | 28–95 |
| fieldCommentRequirement Required | 29 |
| Variables that are final and static should be all capitals, 'log' is not all capitals. | 29 |
| Comment is too large: Too many lines | 31–47 |
| Comment is too large: Line too long | 32 |
| Comment is too large: Line too long | 33 |
| Comment is too large: Line too long | 36 |
| Comment is too large: Line too long | 40 |
| Comment is too large: Line too long | 41 |
| Avoid variables with short names like f | 49 |
| Parameter 'shouldRetry' is not assigned and could be declared final | 50 |
| A method/constructor shouldnt explicitly throw java.lang.Exception | 53 |
| Found 'DU'-anomaly for variable 'nTry' (lines '56'-'71'). | 56–71 |
| Found 'DD'-anomaly for variable 'nTry' (lines '56'-'59'). | 56–59 |
| Found 'DU'-anomaly for variable 'nTry' (lines '59'-'71'). | 59–71 |
| A catch statement should never catch throwable since it includes errors. | 62 |
| A method/constructor shouldnt explicitly throw java.lang.Exception | 76 |
| Parameter 'shouldRetry' is not assigned and could be declared final | 76 |
| Avoid variables with short names like f | 76 |
| Avoid variables with short names like e | 81 |
| Avoid using final local variables, turn them into fields | 83 |
| Avoid using final local variables, turn them into fields | 84 |
| Violation | Line |
|---|---|
| Comment is too large: Too many lines | 1–15 |
| Avoid unused imports such as 'com.metamx.common.guava.CloseQuietly' | 24 |
| All methods are static. Consider using a utility class instead. Alternatively, you could add a private constructor or make the class abstract to silence this warning. | 38–188 |
| Avoid excessively long variable names like DEFAULT_BUFFER_SIZE | 40 |
| fieldCommentRequirement Required | 40 |
| Comment is too large: Too many lines | 42–53 |
| Comment is too large: Line too long | 45 |
| Comment is too large: Line too long | 47 |
| Comment is too large: Line too long | 48 |
| Parameter 'file' is not assigned and could be declared final | 54 |
| Parameter 'is' is not assigned and could be declared final | 54 |
| Avoid variables with short names like is | 54 |
| Potential violation of Law of Demeter (method chain calls) | 56 |
| Avoid variables with short names like os | 57 |
| Comment is too large: Too many lines | 68–80 |
| Comment is too large: Line too long | 69 |
| Comment is too large: Line too long | 72 |
| Parameter 'timeout' is not assigned and could be declared final | 81 |
| Parameter 'is' is not assigned and could be declared final | 81 |
| Parameter 'file' is not assigned and could be declared final | 81 |
| Avoid variables with short names like is | 81 |
| Potential violation of Law of Demeter (method chain calls) | 83 |
| Avoid variables with short names like os | 84 |
| Comment is too large: Too many lines | 95–104 |
| Avoid variables with short names like os | 105 |
| Avoid variables with short names like is | 105 |
| Parameter 'os' is not assigned and could be declared final | 105 |
| Parameter 'is' is not assigned and could be declared final | 105 |
| Comment is too large: Too many lines | 119–131 |
| Comment is too large: Line too long | 120 |
| Parameter 'is' is not assigned and could be declared final | 132 |
| Avoid variables with short names like os | 132 |
| Parameter 'os' is not assigned and could be declared final | 132 |
| Avoid variables with short names like is | 132 |
| Parameter 'timeout' is not assigned and could be declared final | 132 |
| Local variable 'buffer' could be declared final | 134 |
| Found 'DD'-anomaly for variable 'n' (lines '135'-'138'). | 135–138 |
| Avoid variables with short names like n | 135 |
| Local variable 'startTime' could be declared final | 136 |
| Found 'DU'-anomaly for variable 'startTime' (lines '136'-'146'). | 136–146 |
| Found 'DU'-anomaly for variable 'size' (lines '137'-'146'). | 137–146 |
| Avoid assignments in operands | 138 |
| Found 'DU'-anomaly for variable 'n' (lines '138'-'146'). | 138–146 |
| Comment is too large: Line too long | 149 |
| Comment is too large: Line too long | 151 |
| Comment is too large: Line too long | 152 |
| Comment is too large: Line too long | 153 |
| A method/constructor shouldnt explicitly throw java.lang.Exception | 168 |
| publicMethodCommentRequirement Required | 168–178 |
| Avoid catching generic exceptions such as NullPointerException, RuntimeException, Exception in try-catch block | 184 |
| Violation | Line |
|---|---|
| Comment is too large: Too many lines | 1–15 |
| Comment is too large: Line too long | 28 |
| All methods are static. Consider using a utility class instead. Alternatively, you could add a private constructor or make the class abstract to silence this warning. | 31–87 |
| Variables that are final and static should be all capitals, 'log' is not all capitals. | 32 |
| Avoid unused private fields such as 'log'. | 32 |
| fieldCommentRequirement Required | 32 |
| Comment is too large: Line too long | 33 |
| fieldCommentRequirement Required | 34 |
| fieldCommentRequirement Required | 35 |
| publicMethodCommentRequirement Required | 37–46 |
| publicMethodCommentRequirement Required | 48–53 |
| publicMethodCommentRequirement Required | 55–58 |
| publicMethodCommentRequirement Required | 60–69 |
| Parameter 'formatArgs' is not assigned and could be declared final | 71 |
| Parameter 'message' is not assigned and could be declared final | 71 |
| publicMethodCommentRequirement Required | 71–86 |
| A method should have only one exit point, and that should be the last statement in the method | 74 |
| A method should have only one exit point, and that should be the last statement in the method | 77 |
| Local variable 'bob' could be declared final | 80 |
| Local variable 'formatArg' could be declared final | 81 |
| Violation | Line |
|---|---|
| Comment is too large: Too many lines | 1–15 |
| All methods are static. Consider using a utility class instead. Alternatively, you could add a private constructor or make the class abstract to silence this warning. | 25–46 |
| Parameter 'callable' is not assigned and could be declared final | 26 |
| Parameter 'prefix' is not assigned and could be declared final | 26 |
| publicMethodCommentRequirement Required | 26–32 |
| Avoid catching generic exceptions such as NullPointerException, RuntimeException, Exception in try-catch block | 29 |
| Avoid throwing raw exception types. | 30 |
| Parameter 'log' is not assigned and could be declared final | 35 |
| publicMethodCommentRequirement Required | 35–45 |
| Parameter 'prefix' is not assigned and could be declared final | 35 |
| A method/constructor shouldnt explicitly throw java.lang.Exception | 35 |
| Parameter 'callable' is not assigned and could be declared final | 35 |
| Local variable 'startTime' could be declared final | 38 |
| Local variable 'endTime' could be declared final | 40 |
| There is log block not surrounded by if | 42 |
| There is log block not surrounded by if | 42 |
| Violation | Line |
|---|---|
| Comment is too large: Too many lines | 1–15 |
| Avoid short class names like UOE | 21–32 |
| Parameter 'formatText' is not assigned and could be declared final | 23 |
| publicMethodCommentRequirement Required | 23–26 |
| Parameter 'arguments' is not assigned and could be declared final | 23 |
| Parameter 'formatText' is not assigned and could be declared final | 28 |
| publicMethodCommentRequirement Required | 28–31 |
| Parameter 'cause' is not assigned and could be declared final | 28 |
| Parameter 'arguments' is not assigned and could be declared final | 28 |
| Violation | Line |
|---|---|
| Comment is too large: Too many lines | 1–15 |
| headerCommentRequirement Required | 23–36 |
| Each class should declare at least one constructor | 23–36 |
| Avoid variables with short names like n | 25 |
| Avoid variables with short names like k | 25 |
| Parameter 'n' is not assigned and could be declared final | 25 |
| publicMethodCommentRequirement Required | 25–35 |
| Parameter 'k' is not assigned and could be declared final | 25 |
| Violation | Line |
|---|---|
| Comment is too large: Too many lines | 1–15 |
| headerCommentRequirement Required | 29–55 |
| Each class should declare at least one constructor | 29–55 |
| Found non-transient, non-static member. Please mark as transient or provide accessors. | 31 |
| fieldCommentRequirement Required | 31 |
| Avoid using implementation types like 'HashMap'; use the interface instead | 31 |
| publicMethodCommentRequirement Required | 33–39 |
| Avoid variables with short names like k | 33 |
| Avoid variables with short names like n | 33 |
| Parameter 'k' is not assigned and could be declared final | 33 |
| Parameter 'n' is not assigned and could be declared final | 33 |
| Potential violation of Law of Demeter (method chain calls) | 38 |
| publicMethodCommentRequirement Required | 41–54 |
| Parameter 'n' is not assigned and could be declared final | 48 |
| publicMethodCommentRequirement Required | 48–51 |
| Avoid variables with short names like n | 48 |
| Violation | Line |
|---|---|
| Comment is too large: Too many lines | 1–15 |
| headerCommentRequirement Required | 22–123 |
| All methods are static. Consider using a utility class instead. Alternatively, you could add a private constructor or make the class abstract to silence this warning. | 23–123 |
| To be compliant to J2EE, a webapp should not use any thread. | 24 |
| publicMethodCommentRequirement Required | 24–27 |
| To be compliant to J2EE, a webapp should not use any thread. | 24 |
| Avoid variables with short names like f | 34 |
| To be compliant to J2EE, a webapp should not use any thread. | 34 |
| To be compliant to J2EE, a webapp should not use any thread. | 38 |
| fieldCommentRequirement Required | 38 |
| publicMethodCommentRequirement Required | 41–45 |
| publicMethodCommentRequirement Required | 48–52 |
| publicMethodCommentRequirement Required | 55–59 |
| Avoid variables with short names like f | 67 |
| To be compliant to J2EE, a webapp should not use any thread. | 67 |
| To be compliant to J2EE, a webapp should not use any thread. | 71 |
| fieldCommentRequirement Required | 71 |
| publicMethodCommentRequirement Required | 74–81 |
| publicMethodCommentRequirement Required | 84–93 |
| publicMethodCommentRequirement Required | 96–99 |
| To be compliant to J2EE, a webapp should not use any thread. | 103 |
| headerCommentRequirement Required | 103–122 |
| fieldCommentRequirement Required | 105 |
| Avoid variables with short names like f | 105 |
| To be compliant to J2EE, a webapp should not use any thread. | 105 |
| Found non-transient, non-static member. Please mark as transient or provide accessors. | 105 |
| fieldCommentRequirement Required | 107 |
| Avoid using redundant field initializer for 'hasRun' | 107 |
| Found non-transient, non-static member. Please mark as transient or provide accessors. | 107 |
| Use of modifier volatile is not recommended. | 107 |
| Parameter 'f' is not assigned and could be declared final | 109 |
| publicMethodCommentRequirement Required | 109–112 |
| Avoid variables with short names like f | 109 |
| To be compliant to J2EE, a webapp should not use any thread. | 109 |
| publicMethodCommentRequirement Required | 115–121 |
| Violation | Line |
|---|---|
| Comment is too large: Too many lines | 1–15 |
| Avoid importing anything from the package java.lang | 23 |
| headerCommentRequirement Required | 28–94 |
| All methods are static. Consider using a utility class instead. Alternatively, you could add a private constructor or make the class abstract to silence this warning. | 29–94 |
| publicMethodCommentRequirement Required | 30–36 |
| Parameter 'keys' is not assigned and could be declared final | 30 |
| Consider using varargs for methods or constructors which take an array the last parameter. | 30 |
| Parameter 'values' is not assigned and could be declared final | 30 |
| Parameter 'values' is not assigned and could be declared final | 38 |
| Consider using varargs for methods or constructors which take an array the last parameter. | 38 |
| Parameter 'keys' is not assigned and could be declared final | 38 |
| publicMethodCommentRequirement Required | 38–49 |
| Local variable 'retVal' could be declared final | 44 |
| If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation | 44 |
| Avoid using for statements without curly braces | 46 |
| Potential violation of Law of Demeter (method chain calls) | 46 |
| Potential violation of Law of Demeter (method chain calls) | 46 |
| Comment is too large: Line too long | 51 |
| Parameter 'values' is not assigned and could be declared final | 53 |
| Parameter 'keys' is not assigned and could be declared final | 53 |
| If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation | 54 |
| Local variable 'retVal' could be declared final | 54 |
| Local variable 'keysIter' could be declared final | 56 |
| Local variable 'valsIter' could be declared final | 57 |
| Comment is too large: Line too long | 76 |
| Parameter 'keys' is not assigned and could be declared final | 78 |
| Parameter 'values' is not assigned and could be declared final | 78 |
| Local variable 'retVal' could be declared final | 80 |
| If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation | 80 |
| Local variable 'keysIter' could be declared final | 82 |
| Local variable 'valsIter' could be declared final | 83 |
| Found 'DU'-anomaly for variable 'valsIter' (lines '83'-'93'). | 83–93 |
| Found 'DU'-anomaly for variable 'key' (lines '86'-'93'). | 86–93 |
| Avoid using if...else statements without curly braces | 88 |
| Avoid using if...else statements without curly braces | 89 |
| Violation | Line |
|---|---|
| Comment is too large: Too many lines | 1–15 |
| Each class should declare at least one constructor | 24–35 |
| Abstract classes should be named AbstractXXX | 24–35 |
| publicMethodCommentRequirement Required | 28 |
| publicMethodCommentRequirement Required | 31–34 |
| Potential violation of Law of Demeter (method chain calls) | 33 |
| Violation | Line |
|---|---|
| Comment is too large: Too many lines | 1–15 |
| headerCommentRequirement Required | 26–63 |
| All methods are static. Consider using a utility class instead. Alternatively, you could add a private constructor or make the class abstract to silence this warning. | 27–63 |
| publicMethodCommentRequirement Required | 28–37 |
| Parameter 'config' is not assigned and could be declared final | 28 |
| Parameter 'lifecycle' is not assigned and could be declared final | 28 |
| publicMethodCommentRequirement Required | 39–62 |
| Parameter 'lifecycle' is not assigned and could be declared final | 39 |
| A method/constructor shouldnt explicitly throw java.lang.Exception | 46 |
| publicMethodCommentRequirement Required | 46–48 |
| Document empty method body | 47–48 |
| publicMethodCommentRequirement Required | 51–54 |
| Avoid catching generic exceptions such as NullPointerException, RuntimeException, Exception in try-catch block | 58 |
| Violation | Line |
|---|---|
| Comment is too large: Too many lines | 1–15 |
| Found non-transient, non-static member. Please mark as transient or provide accessors. | 27 |
| fieldCommentRequirement Required | 27 |
| publicMethodCommentRequirement Required | 29–41 |
| To be compliant to J2EE, a webapp should not use any thread. | 35 |
| To be compliant to J2EE, a webapp should not use any thread. | 35 |
| Parameter 'runnable' is not assigned and could be declared final | 35 |
| publicMethodCommentRequirement Required | 35–38 |
| To be compliant to J2EE, a webapp should not use any thread. | 37 |
| publicMethodCommentRequirement Required | 43–46 |
| Parameter 'delegate' is not assigned and could be declared final | 43 |
| publicMethodCommentRequirement Required | 49–52 |
| To be compliant to J2EE, a webapp should not use any thread. | 49 |
| Parameter 'runnable' is not assigned and could be declared final | 49 |
| To be compliant to J2EE, a webapp should not use any thread. | 49 |
| Parameter 'fn' is not assigned and could be declared final | 54 |
| publicMethodCommentRequirement Required | 54–57 |
| Avoid variables with short names like fn | 54 |
| To be compliant to J2EE, a webapp should not use any thread. | 59 |
| Avoid variables with short names like fn | 59 |
| publicMethodCommentRequirement Required | 59–69 |
| To be compliant to J2EE, a webapp should not use any thread. | 59 |
| publicMethodCommentRequirement Required | 64–67 |
| Parameter 'runnable' is not assigned and could be declared final | 64 |
| To be compliant to J2EE, a webapp should not use any thread. | 64 |
| To be compliant to J2EE, a webapp should not use any thread. | 64 |
| publicMethodCommentRequirement Required | 71–84 |
| To be compliant to J2EE, a webapp should not use any thread. | 74 |
| To be compliant to J2EE, a webapp should not use any thread. | 74 |
| To be compliant to J2EE, a webapp should not use any thread. | 77 |
| To be compliant to J2EE, a webapp should not use any thread. | 77 |
| Parameter 'input' is not assigned and could be declared final | 77 |
| publicMethodCommentRequirement Required | 77–81 |
| Violation | Line |
|---|---|
| Comment is too large: Too many lines | 1–15 |
| headerCommentRequirement Required | 21–24 |
| Avoid modifiers which are implied by the context | 23 |
| publicMethodCommentRequirement Required | 23 |
| Violation | Line |
|---|---|
| Comment is too large: Too many lines | 1–15 |
| headerCommentRequirement Required | 29–207 |
| All methods are static. Consider using a utility class instead. Alternatively, you could add a private constructor or make the class abstract to silence this warning. | 30–207 |
| This class has too many methods, consider refactoring it. | 30–207 |
| Variables that are final and static should be all capitals, 'log' is not all capitals. | 31 |
| fieldCommentRequirement Required | 31 |
| Parameter 'delay' is not assigned and could be declared final | 37 |
| To be compliant to J2EE, a webapp should not use any thread. | 37 |
| Parameter 'exec' is not assigned and could be declared final | 37 |
| Parameter 'runnable' is not assigned and could be declared final | 37 |
| To be compliant to J2EE, a webapp should not use any thread. | 50 |
| publicMethodCommentRequirement Required | 60–64 |
| Parameter 'exec' is not assigned and could be declared final | 74 |
| Parameter 'callable' is not assigned and could be declared final | 74 |
| Parameter 'delay' is not assigned and could be declared final | 74 |
| To be compliant to J2EE, a webapp should not use any thread. | 92 |
| publicMethodCommentRequirement Required | 95–109 |
| A catch statement should never catch throwable since it includes errors. | 106 |
| Parameter 'exec' is not assigned and could be declared final | 119 |
| To be compliant to J2EE, a webapp should not use any thread. | 119 |
| Parameter 'runnable' is not assigned and could be declared final | 119 |
| Parameter 'rate' is not assigned and could be declared final | 119 |
| To be compliant to J2EE, a webapp should not use any thread. | 132 |
| A method/constructor shouldnt explicitly throw java.lang.Exception | 138 |
| publicMethodCommentRequirement Required | 138–142 |
| Parameter 'exec' is not assigned and could be declared final | 146 |
| Parameter 'rate' is not assigned and could be declared final | 146 |
| publicMethodCommentRequirement Required | 146–149 |
| Parameter 'callable' is not assigned and could be declared final | 146 |
| publicMethodCommentRequirement Required | 151–183 |
| To be compliant to J2EE, a webapp should not use any thread. | 160 |
| Avoid using redundant field initializer for 'prevSignal' | 162 |
| Use of modifier volatile is not recommended. | 162 |
| Found 'DU'-anomaly for variable 'prevSignal' (lines '162'-'183'). | 162–183 |
| fieldCommentRequirement Required | 162 |
| Found 'DU'-anomaly for variable 'prevSignal' (lines '162'-'183'). | 162–183 |
| publicMethodCommentRequirement Required | 165–178 |
| A catch statement should never catch throwable since it includes errors. | 175 |
| enumCommentRequirement Required | 185–188 |
| publicMethodCommentRequirement Required | 190–199 |
| Parameter 'corePoolSize' is not assigned and could be declared final | 194 |
| Parameter 'nameFormat' is not assigned and could be declared final | 194 |
| publicMethodCommentRequirement Required | 194–197 |
| publicMethodCommentRequirement Required | 201–206 |
| Parameter 'nameFormat' is not assigned and could be declared final | 201 |
| Parameter 'corePoolSize' is not assigned and could be declared final | 201 |
| Violation | Line |
|---|---|
| Comment is too large: Too many lines | 1–15 |
| fieldCommentRequirement Required | 23 |
| fieldCommentRequirement Required | 24 |
| It is a good practice to call super() in a constructor | 26–33 |
| publicMethodCommentRequirement Required | 26–33 |
| Parameter 'formatString' is not assigned and could be declared final | 27 |
| Parameter 'numThreads' is not assigned and could be declared final | 28 |
| publicMethodCommentRequirement Required | 36–39 |
| publicMethodCommentRequirement Required | 42–45 |
| Violation | Line |
|---|---|
| Comment is too large: Too many lines | 1–15 |
| All methods are static. Consider using a utility class instead. Alternatively, you could add a private constructor or make the class abstract to silence this warning. | 26–33 |
| publicMethodCommentRequirement Required | 27–32 |
| Parameter 'props' is not assigned and could be declared final | 27 |
| Local variable 'configFactory' could be declared final | 29 |
| Violation | Line |
|---|---|
| Comment is too large: Too many lines | 1–15 |
| Each class should declare at least one constructor | 26–44 |
| publicMethodCommentRequirement Required | 29–43 |
| Parameter 'clazz' is not assigned and could be declared final | 29 |
| A method should have only one exit point, and that should be the last statement in the method | 32 |
| Parameter 'value' is not assigned and could be declared final | 38 |
| publicMethodCommentRequirement Required | 38–41 |
| Violation | Line |
|---|---|
| Comment is too large: Too many lines | 1–15 |
| Generics names should be a one letter long and upper case. | 21 |
| Generics names should be a one letter long and upper case. | 21 |
| Avoid prefixing parameters by in, out or inOut. Uses Javadoc to document this behavior. | 23 |
| Avoid modifiers which are implied by the context | 23 |
| Avoid variables with short names like in | 23 |
| publicMethodCommentRequirement Required | 23 |
| Violation | Line |
|---|---|
| Comment is too large: Too many lines | 1–15 |
| All methods are static. Consider using a utility class instead. Alternatively, you could add a private constructor or make the class abstract to silence this warning. | 24–37 |
| publicMethodCommentRequirement Required | 25–36 |
| Avoid prefixing parameters by in, out or inOut. Uses Javadoc to document this behavior. | 30 |
| Avoid variables with short names like in | 30 |
| publicMethodCommentRequirement Required | 30–34 |
| Parameter 'accumulated' is not assigned and could be declared final | 30 |
| Parameter 'in' is not assigned and could be declared final | 30 |
| Violation | Line |
|---|---|
| Comment is too large: Too many lines | 1–15 |
| Avoid unused imports such as 'com.google.common.io.Closeables' | 20 |
| The class 'BaseSequence' has a Modified Cyclomatic Complexity of 4 (Highest = 10). | 29–170 |
| The class 'BaseSequence' has a Cyclomatic Complexity of 4 (Highest = 11). | 29–170 |
| Generics names should be a one letter long and upper case. | 29 |
| The class 'BaseSequence' has a Standard Cyclomatic Complexity of 4 (Highest = 10). | 29–170 |
| This class has too many methods, consider refactoring it. | 30–170 |
| Variables that are final and static should be all capitals, 'log' is not all capitals. | 31 |
| fieldCommentRequirement Required | 31 |
| fieldCommentRequirement Required | 33 |
| Found non-transient, non-static member. Please mark as transient or provide accessors. | 33 |
| publicMethodCommentRequirement Required | 35–53 |
| publicMethodCommentRequirement Required | 41–44 |
| Parameter 'iterFromMake' is not assigned and could be declared final | 47 |
| publicMethodCommentRequirement Required | 47–50 |
| Document empty method body | 48–50 |
| publicMethodCommentRequirement Required | 55–60 |
| Parameter 'maker' is not assigned and could be declared final | 56 |
| Parameter 'initValue' is not assigned and could be declared final | 63 |
| Avoid variables with short names like fn | 63 |
| publicMethodCommentRequirement Required | 63–73 |
| Found 'DD'-anomaly for variable 'yielder' (lines '65'-'67'). | 65–67 |
| Potential violation of Law of Demeter (object not created locally) | 68 |
| Parameter 'accumulator' is not assigned and could be declared final | 76 |
| publicMethodCommentRequirement Required | 76–94 |
| Parameter 'initValue' is not assigned and could be declared final | 76 |
| Avoid catching generic exceptions such as NullPointerException, RuntimeException, Exception in try-catch block | 83 |
| Comment is too large: Line too long | 84 |
| Comment is too large: Line too long | 85 |
| Avoid catching generic exceptions such as NullPointerException, RuntimeException, Exception in try-catch block | 89 |
| The method 'makeYielder' has a Standard Cyclomatic Complexity of 10. | 96–162 |
| The method 'makeYielder' has a Cyclomatic Complexity of 11. | 96–162 |
| The method 'makeYielder' has a Modified Cyclomatic Complexity of 10. | 96–162 |
| Parameter 'initValue' is not assigned and could be declared final | 97 |
| Found 'DD'-anomaly for variable 'retVal' (lines '102'-'104'). | 102–104 |
| Found 'DD'-anomaly for variable 'retVal' (lines '102'-'104'). | 102–104 |
| Found 'DD'-anomaly for variable 'retVal' (lines '102'-'104'). | 102–104 |
| Found 'DD'-anomaly for variable 'retVal' (lines '102'-'104'). | 102–104 |
| Found 'DD'-anomaly for variable 'retVal' (lines '102'-'104'). | 102–104 |
| Found 'DD'-anomaly for variable 'retVal' (lines '102'-'104'). | 102–104 |
| A method should have only one exit point, and that should be the last statement in the method | 108–118 |
| publicMethodCommentRequirement Required | 113–116 |
| Found 'DU'-anomaly for variable 'finalRetVal' (lines '121'-'162'). | 121–162 |
| Found 'DU'-anomaly for variable 'finalRetVal' (lines '121'-'162'). | 121–162 |
| Found 'DU'-anomaly for variable 'finalRetVal' (lines '121'-'162'). | 121–162 |
| Found 'DU'-anomaly for variable 'finalRetVal' (lines '121'-'162'). | 121–162 |
| Found 'DU'-anomaly for variable 'finalRetVal' (lines '121'-'162'). | 121–162 |
| Found 'DU'-anomaly for variable 'finalRetVal' (lines '121'-'162'). | 121–162 |
| publicMethodCommentRequirement Required | 125–128 |
| Parameter 'initValue' is not assigned and could be declared final | 131 |
| publicMethodCommentRequirement Required | 131–148 |
| Avoid catching generic exceptions such as NullPointerException, RuntimeException, Exception in try-catch block | 137 |
| Comment is too large: Line too long | 138 |
| Comment is too large: Line too long | 139 |
| Avoid catching generic exceptions such as NullPointerException, RuntimeException, Exception in try-catch block | 143 |
| publicMethodCommentRequirement Required | 151–154 |
| publicMethodCommentRequirement Required | 157–160 |
| Avoid modifiers which are implied by the context | 164–169 |
| headerCommentRequirement Required | 164–169 |
| publicMethodCommentRequirement Required | 166 |
| Avoid modifiers which are implied by the context | 166 |
| publicMethodCommentRequirement Required | 168 |
| Avoid modifiers which are implied by the context | 168 |
| Violation | Line |
|---|---|
| Comment is too large: Too many lines | 1–15 |
| All methods are static. Consider using a utility class instead. Alternatively, you could add a private constructor or make the class abstract to silence this warning. | 27–42 |
| Variables that are final and static should be all capitals, 'log' is not all capitals. | 28 |
| fieldCommentRequirement Required | 28 |
| Parameter 'closeable' is not assigned and could be declared final | 30 |
| publicMethodCommentRequirement Required | 30–41 |
| Violation | Line |
|---|---|
| Comment is too large: Too many lines | 1–15 |
| All methods are static. Consider using a utility class instead. Alternatively, you could add a private constructor or make the class abstract to silence this warning. | 27–113 |
| Comment is too large: Line too long | 29 |
| Comment is too large: Line too long | 30 |
| publicMethodCommentRequirement Required | 41–44 |
| Parameter 't' is not assigned and could be declared final | 41 |
| Avoid variables with short names like t1 | 41 |
| Parameter 't1' is not assigned and could be declared final | 41 |
| Avoid variables with short names like t | 41 |
| Avoid variables with short names like t | 60 |
| Parameter 't' is not assigned and could be declared final | 60 |
| publicMethodCommentRequirement Required | 60–63 |
| Avoid variables with short names like t1 | 60 |
| Parameter 't1' is not assigned and could be declared final | 60 |
| fieldCommentRequirement Required | 67–80 |
| Avoid excessively long variable names like INTERVAL_BY_START_THEN_END | 67 |
| fieldCommentRequirement Required | 69 |
| Parameter 'rhs' is not assigned and could be declared final | 72 |
| Parameter 'lhs' is not assigned and could be declared final | 72 |
| publicMethodCommentRequirement Required | 72–79 |
| fieldCommentRequirement Required | 82–95 |
| Avoid excessively long variable names like INTERVAL_BY_END_THEN_START | 82 |
| fieldCommentRequirement Required | 84 |
| Parameter 'rhs' is not assigned and could be declared final | 87 |
| publicMethodCommentRequirement Required | 87–94 |
| Parameter 'lhs' is not assigned and could be declared final | 87 |
| publicMethodCommentRequirement Required | 98–101 |
| publicMethodCommentRequirement Required | 103–106 |
| publicMethodCommentRequirement Required | 108–111 |
| Violation | Line |
|---|---|
| Comment is too large: Too many lines | 1–15 |
| Avoid unused imports such as 'com.google.common.io.Closeables' | 20 |
| fieldCommentRequirement Required | 28 |
| Found non-transient, non-static member. Please mark as transient or provide accessors. | 28 |
| publicMethodCommentRequirement Required | 30–35 |
| Parameter 'baseSequences' is not assigned and could be declared final | 31 |
| publicMethodCommentRequirement Required | 38–50 |
| Parameter 'initValue' is not assigned and could be declared final | 38 |
| Parameter 'accumulated' is not assigned and could be declared final | 44 |
| Avoid variables with short names like in | 44 |
| Parameter 'in' is not assigned and could be declared final | 44 |
| publicMethodCommentRequirement Required | 44–47 |
| Avoid prefixing parameters by in, out or inOut. Uses Javadoc to document this behavior. | 44 |
| publicMethodCommentRequirement Required | 53–80 |
| Found 'DU'-anomaly for variable 'yielderYielder' (lines '58'-'80'). | 58–80 |
| Local variable 'yielderYielder' could be declared final | 58–69 |
| Found 'DU'-anomaly for variable 'yielderYielder' (lines '58'-'80'). | 58–80 |
| Parameter 'accumulated' is not assigned and could be declared final | 63 |
| Parameter 'in' is not assigned and could be declared final | 63 |
| Avoid prefixing parameters by in, out or inOut. Uses Javadoc to document this behavior. | 63 |
| Avoid variables with short names like in | 63 |
| publicMethodCommentRequirement Required | 63–67 |
| Avoid catching generic exceptions such as NullPointerException, RuntimeException, Exception in try-catch block | 74 |
| Comment is too large: Line too long | 75 |
| Comment is too large: Line too long | 76 |
| publicMethodCommentRequirement Required | 82–110 |
| Avoid reassigning parameters such as 'yielderYielder' | 83 |
| Avoid reassigning parameters such as 'initValue' | 84 |
| Parameter 'accumulator' is not assigned and could be declared final | 85 |
| A method should have only one exit point, and that should be the last statement in the method | 89 |
| Local variable 'yielder' could be declared final | 93 |
| Potential violation of Law of Demeter (method chain calls) | 93 |
| A method should have only one exit point, and that should be the last statement in the method | 95 |
| Potential violation of Law of Demeter (object not created locally) | 98 |
| Potential violation of Law of Demeter (object not created locally) | 100 |
| Found 'DU'-anomaly for variable 'nextInit' (lines '119'-'157'). | 119–157 |
| Found 'DU'-anomaly for variable 'nextInit' (lines '119'-'157'). | 119–157 |
| Found 'DU'-anomaly for variable 'nextInit' (lines '119'-'157'). | 119–157 |
| Found 'DU'-anomaly for variable 'nextInit' (lines '119'-'157'). | 119–157 |
| Local variable 'nextInit' could be declared final | 119 |
| Found 'DU'-anomaly for variable 'nextInit' (lines '119'-'157'). | 119–157 |
| Avoid declaring a variable if it is unreferenced before a possible exit point. | 119 |
| A method should have only one exit point, and that should be the last statement in the method | 127 |
| publicMethodCommentRequirement Required | 133–136 |
| Parameter 'initValue' is not assigned and could be declared final | 139 |
| publicMethodCommentRequirement Required | 139–142 |
| publicMethodCommentRequirement Required | 145–148 |
| publicMethodCommentRequirement Required | 151–155 |
| Violation | Line |
|---|---|
| Comment is too large: Too many lines | 1–15 |
| Found non-transient, non-static member. Please mark as transient or provide accessors. | 27 |
| fieldCommentRequirement Required | 27 |
| publicMethodCommentRequirement Required | 29–34 |
| It is a good practice to call super() in a constructor | 29–34 |
| Parameter 'supplier' is not assigned and could be declared final | 30 |
| Parameter 'o' is not assigned and could be declared final | 37 |
| Avoid variables with short names like o | 37 |
| publicMethodCommentRequirement Required | 37–47 |
| Violation | Line |
|---|---|
| Comment is too large: Too many lines | 1–15 |
| Generics names should be a one letter long and upper case. | 21 |
| Found non-transient, non-static member. Please mark as transient or provide accessors. | 23 |
| fieldCommentRequirement Required | 23 |
| It is a good practice to call super() in a constructor | 25–30 |
| publicMethodCommentRequirement Required | 25–30 |
| Parameter 'delegate' is not assigned and could be declared final | 26 |
| publicMethodCommentRequirement Required | 33–36 |
| publicMethodCommentRequirement Required | 39–42 |
| publicMethodCommentRequirement Required | 45–48 |
| Avoid prefixing parameters by in, out or inOut. Uses Javadoc to document this behavior. | 51 |
| publicMethodCommentRequirement Required | 51–54 |
| Parameter 'in' is not assigned and could be declared final | 51 |
| Parameter 'accumulated' is not assigned and could be declared final | 51 |
| Avoid variables with short names like in | 51 |
| Violation | Line |
|---|---|
| Comment is too large: Too many lines | 1–15 |
| fieldCommentRequirement Required | 25 |
| Found non-transient, non-static member. Please mark as transient or provide accessors. | 25 |
| fieldCommentRequirement Required | 26 |
| Found non-transient, non-static member. Please mark as transient or provide accessors. | 26 |
| publicMethodCommentRequirement Required | 28–35 |
| Parameter 'delegate' is not assigned and could be declared final | 29 |
| Parameter 'numToDrop' is not assigned and could be declared final | 30 |
| publicMethodCommentRequirement Required | 37–40 |
| Violation | Line |
|---|---|
| Comment is too large: Too many lines | 1–15 |
| Found non-transient, non-static member. Please mark as transient or provide accessors. | 25 |
| fieldCommentRequirement Required | 25 |
| Found non-transient, non-static member. Please mark as transient or provide accessors. | 26 |
| fieldCommentRequirement Required | 26 |
| Avoid using redundant field initializer for 'dropped' | 27 |
| Found non-transient, non-static member. Please mark as transient or provide accessors. | 27 |
| fieldCommentRequirement Required | 27 |
| publicMethodCommentRequirement Required | 29–36 |
| Parameter 'delegate' is not assigned and could be declared final | 30 |
| Parameter 'numToDrop' is not assigned and could be declared final | 31 |
| publicMethodCommentRequirement Required | 38–48 |
| publicMethodCommentRequirement Required | 50–59 |
| publicMethodCommentRequirement Required | 61–64 |
| Violation | Line |
|---|---|
| Comment is too large: Too many lines | 1–15 |
| headerCommentRequirement Required | 23–62 |
| fieldCommentRequirement Required | 25 |
| Found non-transient, non-static member. Please mark as transient or provide accessors. | 25 |
| fieldCommentRequirement Required | 26 |
| Found non-transient, non-static member. Please mark as transient or provide accessors. | 26 |
| To be compliant to J2EE, a webapp should not use any thread. | 26 |
| fieldCommentRequirement Required | 27 |
| Found non-transient, non-static member. Please mark as transient or provide accessors. | 27 |
| publicMethodCommentRequirement Required | 29–34 |
| Parameter 'runnable' is not assigned and could be declared final | 29 |
| Parameter 'baseYielder' is not assigned and could be declared final | 29 |
| Parameter 'executor' is not assigned and could be declared final | 29 |
| To be compliant to J2EE, a webapp should not use any thread. | 29 |
| publicMethodCommentRequirement Required | 37–40 |
| Parameter 'initValue' is not assigned and could be declared final | 43 |
| publicMethodCommentRequirement Required | 43–46 |
| publicMethodCommentRequirement Required | 49–52 |
| publicMethodCommentRequirement Required | 55–61 |
| Violation | Line |
|---|---|
| Comment is too large: Too many lines | 1–15 |
| Found non-transient, non-static member. Please mark as transient or provide accessors. | 31 |
| fieldCommentRequirement Required | 31 |
| fieldCommentRequirement Required | 32 |
| Found non-transient, non-static member. Please mark as transient or provide accessors. | 32 |
| publicMethodCommentRequirement Required | 34–41 |
| Parameter 'sequence' is not assigned and could be declared final | 35 |
| Parameter 'exec' is not assigned and could be declared final | 36 |
| publicMethodCommentRequirement Required | 44–65 |
| Found 'DU'-anomaly for variable 'future' (lines '46'-'65'). | 46–65 |
| Local variable 'future' could be declared final | 46–55 |
| Found 'DU'-anomaly for variable 'future' (lines '46'-'65'). | 46–65 |
| publicMethodCommentRequirement Required | 50–53 |
| A method/constructor shouldnt explicitly throw java.lang.Exception | 50 |
| publicMethodCommentRequirement Required | 68–89 |
| Local variable 'future' could be declared final | 70–79 |
| Found 'DU'-anomaly for variable 'future' (lines '70'-'89'). | 70–89 |
| Found 'DU'-anomaly for variable 'future' (lines '70'-'89'). | 70–89 |
| publicMethodCommentRequirement Required | 74–77 |
| A method/constructor shouldnt explicitly throw java.lang.Exception | 74 |
| publicMethodCommentRequirement Required | 96–99 |
| publicMethodCommentRequirement Required | 102–123 |
| Local variable 'future' could be declared final | 104–113 |
| publicMethodCommentRequirement Required | 108–111 |
| A method/constructor shouldnt explicitly throw java.lang.Exception | 108 |
| publicMethodCommentRequirement Required | 126–129 |
| publicMethodCommentRequirement Required | 132–135 |
| Violation | Line |
|---|---|
| Comment is too large: Too many lines | 1–15 |
| fieldCommentRequirement Required | 27 |
| Found non-transient, non-static member. Please mark as transient or provide accessors. | 27 |
| Found non-transient, non-static member. Please mark as transient or provide accessors. | 28 |
| fieldCommentRequirement Required | 28 |
| publicMethodCommentRequirement Required | 30–37 |
| Parameter 'baseSequence' is not assigned and could be declared final | 31 |
| Parameter 'pred' is not assigned and could be declared final | 32 |
| Parameter 'accumulator' is not assigned and could be declared final | 40 |
| Parameter 'initValue' is not assigned and could be declared final | 40 |
| publicMethodCommentRequirement Required | 40–43 |
| publicMethodCommentRequirement Required | 46–53 |
| Parameter 'accumulator' is not assigned and could be declared final | 46 |
| Parameter 'initValue' is not assigned and could be declared final | 46 |
| Avoid excessively long variable names like filteringAccumulator | 48 |
| publicMethodCommentRequirement Required | 62–65 |
| publicMethodCommentRequirement Required | 68–71 |
| Parameter 'initValue' is not assigned and could be declared final | 68 |
| publicMethodCommentRequirement Required | 74–77 |
| publicMethodCommentRequirement Required | 80–83 |
| Violation | Line |
|---|---|
| Comment is too large: Too many lines | 1–15 |
| Generics names should be a one letter long and upper case. | 23 |
| Found non-transient, non-static member. Please mark as transient or provide accessors. | 25 |
| fieldCommentRequirement Required | 25 |
| Found non-transient, non-static member. Please mark as transient or provide accessors. | 26 |
| fieldCommentRequirement Required | 26 |
| publicMethodCommentRequirement Required | 28–34 |
| Parameter 'pred' is not assigned and could be declared final | 29 |
| Parameter 'accumulator' is not assigned and could be declared final | 30 |
| Parameter 'in' is not assigned and could be declared final | 37 |
| Avoid variables with short names like in | 37 |
| Avoid prefixing parameters by in, out or inOut. Uses Javadoc to document this behavior. | 37 |
| Parameter 'accumulated' is not assigned and could be declared final | 37 |
| publicMethodCommentRequirement Required | 37–43 |
| A method should have only one exit point, and that should be the last statement in the method | 40 |
| Violation | Line |
|---|---|
| Comment is too large: Too many lines | 1–15 |
| Generics names should be a one letter long and upper case. | 23 |
| fieldCommentRequirement Required | 25 |
| Found non-transient, non-static member. Please mark as transient or provide accessors. | 25 |
| fieldCommentRequirement Required | 26 |
| Found non-transient, non-static member. Please mark as transient or provide accessors. | 26 |
| Found non-transient, non-static member. Please mark as transient or provide accessors. | 28 |
| Avoid using redundant field initializer for 'didSomething' | 28 |
| Use of modifier volatile is not recommended. | 28 |
| Field didSomething has the same name as a method | 28 |
| fieldCommentRequirement Required | 28 |
| publicMethodCommentRequirement Required | 30–36 |
| It is a good practice to call super() in a constructor | 30–36 |
| Parameter 'pred' is not assigned and could be declared final | 31 |
| Parameter 'accumulator' is not assigned and could be declared final | 32 |
| publicMethodCommentRequirement Required | 39–42 |
| publicMethodCommentRequirement Required | 45–48 |
| publicMethodCommentRequirement Required | 51–55 |
| publicMethodCommentRequirement Required | 57–60 |
| Avoid variables with short names like in | 63 |
| Parameter 'accumulated' is not assigned and could be declared final | 63 |
| Parameter 'in' is not assigned and could be declared final | 63 |
| Avoid prefixing parameters by in, out or inOut. Uses Javadoc to document this behavior. | 63 |
| publicMethodCommentRequirement Required | 63–72 |
| A method should have only one exit point, and that should be the last statement in the method | 69 |
| Violation | Line |
|---|---|
| Comment is too large: Too many lines | 1–15 |
| Avoid short class names like Fns | 25–49 |
| All methods are static. Consider using a utility class instead. Alternatively, you could add a private constructor or make the class abstract to silence this warning. | 26–49 |
| publicMethodCommentRequirement Required | 27–36 |
| publicMethodCommentRequirement Required | 31–34 |
| Parameter 'input' is not assigned and could be declared final | 31 |
| publicMethodCommentRequirement Required | 38–48 |
| publicMethodCommentRequirement Required | 43–46 |
| Parameter 'in' is not assigned and could be declared final | 43 |
| Avoid variables with short names like in | 43 |
| Avoid prefixing parameters by in, out or inOut. Uses Javadoc to document this behavior. | 43 |
| Violation | Line |
|---|---|
| Comment is too large: Too many lines | 1–15 |
| This class has too many methods, consider refactoring it. | 33–121 |
| Found non-transient, non-static member. Please mark as transient or provide accessors. | 34 |
| fieldCommentRequirement Required | 34 |
| Parameter 'delegate' is not assigned and could be declared final | 36 |
| publicMethodCommentRequirement Required | 36–39 |
| publicMethodCommentRequirement Required | 41–44 |
| Parameter 'delegates' is not assigned and could be declared final | 41 |
| Parameter 'delegates' is not assigned and could be declared final | 46 |
| publicMethodCommentRequirement Required | 46–49 |
| publicMethodCommentRequirement Required | 51–56 |
| Parameter 'delegate' is not assigned and could be declared final | 52 |
| publicMethodCommentRequirement Required | 58–61 |
| Avoid variables with short names like fn | 63 |
| Parameter 'fn' is not assigned and could be declared final | 63 |
| publicMethodCommentRequirement Required | 63–66 |
| Parameter 'fn' is not assigned and could be declared final | 68 |
| Avoid variables with short names like fn | 68 |
| publicMethodCommentRequirement Required | 68–71 |
| publicMethodCommentRequirement Required | 73–76 |
| Avoid variables with short names like fn | 73 |
| Parameter 'fn' is not assigned and could be declared final | 73 |
| publicMethodCommentRequirement Required | 78–81 |
| Parameter 'pred' is not assigned and could be declared final | 78 |
| Parameter 'numToDrop' is not assigned and could be declared final | 83 |
| publicMethodCommentRequirement Required | 83–86 |
| Parameter 'limit' is not assigned and could be declared final | 88 |
| publicMethodCommentRequirement Required | 88–91 |
| publicMethodCommentRequirement Required | 93–99 |
| Parameter 'toConcat' is not assigned and could be declared final | 93 |
| Avoid using Literals in Conditional Statements | 95 |
| A method should have only one exit point, and that should be the last statement in the method | 96 |
| Potential violation of Law of Demeter (method chain calls) | 96 |
| publicMethodCommentRequirement Required | 101–104 |
| Parameter 'toConcat' is not assigned and could be declared final | 101 |
| publicMethodCommentRequirement Required | 106–111 |
| publicMethodCommentRequirement Required | 113–120 |
| Violation | Line |
|---|---|
| Comment is too large: Too many lines | 1–15 |
| This class has too many methods, consider refactoring it. | 33–131 |
| Found non-transient, non-static member. Please mark as transient or provide accessors. | 34 |
| fieldCommentRequirement Required | 34 |
| publicMethodCommentRequirement Required | 36–39 |
| Parameter 'delegate' is not assigned and could be declared final | 36 |
| publicMethodCommentRequirement Required | 41–44 |
| Parameter 'toConcat' is not assigned and could be declared final | 41 |
| publicMethodCommentRequirement Required | 46–49 |
| Parameter 'toConcat' is not assigned and could be declared final | 46 |
| publicMethodCommentRequirement Required | 51–56 |
| Parameter 'delegate' is not assigned and could be declared final | 52 |
| publicMethodCommentRequirement Required | 58–61 |
| publicMethodCommentRequirement Required | 63–66 |
| publicMethodCommentRequirement Required | 68–71 |
| publicMethodCommentRequirement Required | 73–76 |
| Avoid variables with short names like fn | 73 |
| Parameter 'fn' is not assigned and could be declared final | 73 |
| Avoid variables with short names like fn | 78 |
| Parameter 'fn' is not assigned and could be declared final | 78 |
| publicMethodCommentRequirement Required | 78–81 |
| Parameter 'fn' is not assigned and could be declared final | 83 |
| publicMethodCommentRequirement Required | 83–86 |
| Avoid variables with short names like fn | 83 |
| Parameter 'pred' is not assigned and could be declared final | 88 |
| publicMethodCommentRequirement Required | 88–91 |
| publicMethodCommentRequirement Required | 93–96 |
| Parameter 'numToDrop' is not assigned and could be declared final | 93 |
| publicMethodCommentRequirement Required | 98–101 |
| Parameter 'limit' is not assigned and could be declared final | 98 |
| Parameter 'toConcat' is not assigned and could be declared final | 103 |
| publicMethodCommentRequirement Required | 103–109 |
| Avoid using Literals in Conditional Statements | 105 |
| Potential violation of Law of Demeter (method chain calls) | 106 |
| A method should have only one exit point, and that should be the last statement in the method | 106 |
| publicMethodCommentRequirement Required | 111–114 |
| Parameter 'toConcat' is not assigned and could be declared final | 111 |
| publicMethodCommentRequirement Required | 116–121 |
| publicMethodCommentRequirement Required | 123–130 |
| Violation | Line |
|---|---|
| Comment is too large: Too many lines | 1–15 |
| Found non-transient, non-static member. Please mark as transient or provide accessors. | 29 |
| fieldCommentRequirement Required | 29 |
| Found non-transient, non-static member. Please mark as transient or provide accessors. | 30 |
| fieldCommentRequirement Required | 30 |
| publicMethodCommentRequirement Required | 32–39 |
| Parameter 'baseIter' is not assigned and could be declared final | 33 |
| Parameter 'baggage' is not assigned and could be declared final | 34 |
| publicMethodCommentRequirement Required | 42–45 |
| publicMethodCommentRequirement Required | 48–51 |
| publicMethodCommentRequirement Required | 54–57 |
| publicMethodCommentRequirement Required | 60–63 |
| Violation | Line |
|---|---|
| Comment is too large: Too many lines | 1–15 |
| fieldCommentRequirement Required | 25 |
| Found non-transient, non-static member. Please mark as transient or provide accessors. | 25 |
| publicMethodCommentRequirement Required | 27–32 |
| Parameter 'provider' is not assigned and could be declared final | 28 |
| Parameter 'accumulator' is not assigned and could be declared final | 35 |
| Parameter 'initValue' is not assigned and could be declared final | 35 |
| publicMethodCommentRequirement Required | 35–38 |
| Potential violation of Law of Demeter (method chain calls) | 37 |
| publicMethodCommentRequirement Required | 41–44 |
| Parameter 'initValue' is not assigned and could be declared final | 41 |
| Parameter 'accumulator' is not assigned and could be declared final | 41 |
| Potential violation of Law of Demeter (method chain calls) | 43 |
| Violation | Line |
|---|---|
| Comment is too large: Too many lines | 1–15 |
| Comment is too large: Line too long | 24 |
| Found non-transient, non-static member. Please mark as transient or provide accessors. | 29 |
| fieldCommentRequirement Required | 29 |
| fieldCommentRequirement Required | 30 |
| Found non-transient, non-static member. Please mark as transient or provide accessors. | 30 |
| publicMethodCommentRequirement Required | 32–42 |
| It is a good practice to call super() in a constructor | 32–42 |
| Parameter 'baseSequence' is not assigned and could be declared final | 33 |
| Parameter 'limit' is not assigned and could be declared final | 34 |
| Parameter 'accumulator' is not assigned and could be declared final | 45 |
| publicMethodCommentRequirement Required | 45–52 |
| Parameter 'initValue' is not assigned and could be declared final | 45 |
| Avoid excessively long variable names like limitedAccumulator | 47 |
| headerCommentRequirement Required | 54–103 |
| fieldCommentRequirement Required | 56 |
| Found non-transient, non-static member. Please mark as transient or provide accessors. | 56 |
| Avoid excessively long variable names like limitedAccumulator | 57 |
| fieldCommentRequirement Required | 57 |
| Found non-transient, non-static member. Please mark as transient or provide accessors. | 57 |
| publicMethodCommentRequirement Required | 59–66 |
| Parameter 'subYielder' is not assigned and could be declared final | 60 |
| Avoid excessively long variable names like limitedAccumulator | 61 |
| Parameter 'limitedAccumulator' is not assigned and could be declared final | 61 |
| publicMethodCommentRequirement Required | 69–72 |
| Parameter 'initValue' is not assigned and could be declared final | 75 |
| publicMethodCommentRequirement Required | 75–87 |
| A method should have only one exit point, and that should be the last statement in the method | 78 |
| Found 'DD'-anomaly for variable 'next' (lines '81'-'84'). | 81–84 |
| Potential violation of Law of Demeter (object not created locally) | 84 |
| publicMethodCommentRequirement Required | 90–96 |
| Useless parentheses. | 93–94 |
| publicMethodCommentRequirement Required | 99–102 |
| headerCommentRequirement Required | 105–147 |
| Use explicit scoping instead of the default package private level | 107 |
| fieldCommentRequirement Required | 107 |
| Found non-transient, non-static member. Please mark as transient or provide accessors. | 107 |
| Avoid using redundant field initializer for 'interruptYield' | 108 |
| Use explicit scoping instead of the default package private level | 108 |
| fieldCommentRequirement Required | 108 |
| Found non-transient, non-static member. Please mark as transient or provide accessors. | 108 |
| Parameter 'accumulator' is not assigned and could be declared final | 110 |
| publicMethodCommentRequirement Required | 110–114 |
| Avoid prefixing parameters by in, out or inOut. Uses Javadoc to document this behavior. | 117 |
| Avoid variables with short names like in | 117 |
| publicMethodCommentRequirement Required | 117–136 |
| Parameter 'in' is not assigned and could be declared final | 117 |
| Parameter 'accumulated' is not assigned and could be declared final | 117 |
| publicMethodCommentRequirement Required | 138–141 |
| Violation | Line |
|---|---|
| Comment is too large: Too many lines | 1–15 |
| Generics names should be a one letter long and upper case. | 21 |
| fieldCommentRequirement Required | 23 |
| Found non-transient, non-static member. Please mark as transient or provide accessors. | 23 |
| Found non-transient, non-static member. Please mark as transient or provide accessors. | 24 |
| fieldCommentRequirement Required | 24 |
| fieldCommentRequirement Required | 26 |
| Avoid using redundant field initializer for 'count' | 26 |
| Use of modifier volatile is not recommended. | 26 |
| Found non-transient, non-static member. Please mark as transient or provide accessors. | 26 |
| It is a good practice to call super() in a constructor | 28–34 |
| publicMethodCommentRequirement Required | 28–34 |
| Parameter 'limit' is not assigned and could be declared final | 29 |
| Parameter 'delegate' is not assigned and could be declared final | 29 |
| publicMethodCommentRequirement Required | 37–40 |
| publicMethodCommentRequirement Required | 43–46 |
| publicMethodCommentRequirement Required | 49–52 |
| Avoid prefixing parameters by in, out or inOut. Uses Javadoc to document this behavior. | 55 |
| publicMethodCommentRequirement Required | 55–62 |
| Parameter 'in' is not assigned and could be declared final | 55 |
| Avoid variables with short names like in | 55 |
| Parameter 'accumulated' is not assigned and could be declared final | 55 |
| A method should have only one exit point, and that should be the last statement in the method | 59 |
| Violation | Line |
|---|---|
| Comment is too large: Too many lines | 1–15 |
| Generics names should be a one letter long and upper case. | 23 |
| fieldCommentRequirement Required | 25 |
| Found non-transient, non-static member. Please mark as transient or provide accessors. | 25 |
| fieldCommentRequirement Required | 26 |
| Avoid variables with short names like fn | 26 |
| Found non-transient, non-static member. Please mark as transient or provide accessors. | 26 |
| publicMethodCommentRequirement Required | 28–35 |
| Parameter 'baseSequence' is not assigned and could be declared final | 29 |
| Avoid variables with short names like fn | 30 |
| Parameter 'fn' is not assigned and could be declared final | 30 |
| publicMethodCommentRequirement Required | 38–41 |
| Parameter 'initValue' is not assigned and could be declared final | 38 |
| Parameter 'accumulator' is not assigned and could be declared final | 38 |
| publicMethodCommentRequirement Required | 44–47 |
| Parameter 'accumulator' is not assigned and could be declared final | 44 |
| Parameter 'initValue' is not assigned and could be declared final | 44 |
| Violation | Line |
|---|---|
| Comment is too large: Too many lines | 1–15 |
| Generics names should be a one letter long and upper case. | 23 |
| Generics names should be a one letter long and upper case. | 23 |
| Generics names should be a one letter long and upper case. | 23 |
| fieldCommentRequirement Required | 25 |
| Found non-transient, non-static member. Please mark as transient or provide accessors. | 25 |
| Avoid variables with short names like fn | 25 |
| fieldCommentRequirement Required | 26 |
| Found non-transient, non-static member. Please mark as transient or provide accessors. | 26 |
| publicMethodCommentRequirement Required | 28–34 |
| Parameter 'fn' is not assigned and could be declared final | 29 |
| Avoid variables with short names like fn | 29 |
| Parameter 'accumulator' is not assigned and could be declared final | 30 |
| Parameter 'accumulated' is not assigned and could be declared final | 37 |
| Parameter 'in' is not assigned and could be declared final | 37 |
| Avoid prefixing parameters by in, out or inOut. Uses Javadoc to document this behavior. | 37 |
| Avoid variables with short names like in | 37 |
| publicMethodCommentRequirement Required | 37–40 |
| Violation | Line |
|---|---|
| Comment is too large: Too many lines | 1–15 |
| Generics names should be a one letter long and upper case. | 23 |
| Generics names should be a one letter long and upper case. | 23 |
| Generics names should be a one letter long and upper case. | 23 |
| fieldCommentRequirement Required | 25 |
| Avoid variables with short names like fn | 25 |
| Found non-transient, non-static member. Please mark as transient or provide accessors. | 25 |
| Found non-transient, non-static member. Please mark as transient or provide accessors. | 26 |
| fieldCommentRequirement Required | 26 |
| It is a good practice to call super() in a constructor | 28–34 |
| publicMethodCommentRequirement Required | 28–34 |
| Avoid variables with short names like fn | 29 |
| Parameter 'fn' is not assigned and could be declared final | 29 |
| Parameter 'baseAccumulator' is not assigned and could be declared final | 30 |
| publicMethodCommentRequirement Required | 37–40 |
| publicMethodCommentRequirement Required | 43–46 |
| publicMethodCommentRequirement Required | 49–52 |
| Parameter 'in' is not assigned and could be declared final | 55 |
| Avoid variables with short names like in | 55 |
| Parameter 'accumulated' is not assigned and could be declared final | 55 |
| Avoid prefixing parameters by in, out or inOut. Uses Javadoc to document this behavior. | 55 |
| publicMethodCommentRequirement Required | 55–58 |
| Violation | Line |
|---|---|
| Comment is too large: Too many lines | 1–15 |
| Found non-transient, non-static member. Please mark as transient or provide accessors. | 29 |
| fieldCommentRequirement Required | 29 |
| fieldCommentRequirement Required | 30 |
| Found non-transient, non-static member. Please mark as transient or provide accessors. | 30 |
| publicMethodCommentRequirement Required | 32–39 |
| Parameter 'comparator' is not assigned and could be declared final | 33 |
| Parameter 'baseIterables' is not assigned and could be declared final | 34 |
| Found 'UR'-anomaly for variable 'baseIterable' (lines '42'-'46'). | 42–46 |
| publicMethodCommentRequirement Required | 42–50 |
| Local variable 'iterators' could be declared final | 44 |
| Local variable 'baseIterable' could be declared final | 45 |
| Potential violation of Law of Demeter (object not created locally) | 46 |
| Violation | Line |
|---|---|
| Comment is too large: Too many lines | 1–15 |
| fieldCommentRequirement Required | 32 |
| Found non-transient, non-static member. Please mark as transient or provide accessors. | 32 |
| publicMethodCommentRequirement Required | 34–59 |
| Parameter 'iterators' is not assigned and could be declared final | 36 |
| Parameter 'rhs' is not assigned and could be declared final | 44 |
| publicMethodCommentRequirement Required | 44–47 |
| Parameter 'lhs' is not assigned and could be declared final | 44 |
| Local variable 'iterator' could be declared final | 51 |
| publicMethodCommentRequirement Required | 62–65 |
| publicMethodCommentRequirement Required | 68–82 |
| Local variable 'retIt' could be declared final | 74 |
| Local variable 'retVal' could be declared final | 75 |
| publicMethodCommentRequirement Required | 85–88 |
| Violation | Line |
|---|---|
| Comment is too large: Too many lines | 1–15 |
| The class 'MergeSequence' has a Cyclomatic Complexity of 7 (Highest = 12). | 28–156 |
| The class 'MergeSequence' has a Modified Cyclomatic Complexity of 6 (Highest = 10). | 28–156 |
| The class 'MergeSequence' has a Standard Cyclomatic Complexity of 6 (Highest = 10). | 28–156 |
| Found non-transient, non-static member. Please mark as transient or provide accessors. | 30 |
| fieldCommentRequirement Required | 30 |
| Found non-transient, non-static member. Please mark as transient or provide accessors. | 31 |
| fieldCommentRequirement Required | 31 |
| It is a good practice to call super() in a constructor | 33–40 |
| publicMethodCommentRequirement Required | 33–40 |
| Parameter 'ordering' is not assigned and could be declared final | 34 |
| Parameter 'baseSequences' is not assigned and could be declared final | 35 |
| Parameter 'accumulator' is not assigned and could be declared final | 43 |
| Parameter 'initValue' is not assigned and could be declared final | 43 |
| publicMethodCommentRequirement Required | 43–96 |
| Found 'DU'-anomaly for variable 'pQueue' (lines '45'-'96'). | 45–96 |
| Found 'DU'-anomaly for variable 'pQueue' (lines '45'-'96'). | 45–96 |
| Found 'DU'-anomaly for variable 'pQueue' (lines '45'-'96'). | 45–96 |
| Found 'DU'-anomaly for variable 'pQueue' (lines '45'-'96'). | 45–96 |
| publicMethodCommentRequirement Required | 51–54 |
| Parameter 'input' is not assigned and could be declared final | 51 |
| Parameter 'queue' is not assigned and could be declared final | 64 |
| Parameter 'in' is not assigned and could be declared final | 64 |
| Avoid variables with short names like in | 64 |
| publicMethodCommentRequirement Required | 64–91 |
| Avoid prefixing parameters by in, out or inOut. Uses Javadoc to document this behavior. | 64 |
| Avoid prefixing parameters by in, out or inOut. Uses Javadoc to document this behavior. | 71 |
| Parameter 'in' is not assigned and could be declared final | 71 |
| Parameter 'accumulated' is not assigned and could be declared final | 71 |
| publicMethodCommentRequirement Required | 71–75 |
| Avoid variables with short names like in | 71 |
| Avoid if (x != y) ..; else ..; | 79–88 |
| The method 'makeYielder' has a Cyclomatic Complexity of 12. | 98–155 |
| The method 'makeYielder' has a Standard Cyclomatic Complexity of 10. | 98–155 |
| The method 'makeYielder' has a Modified Cyclomatic Complexity of 10. | 98–155 |
| Parameter 'initVal' is not assigned and could be declared final | 100 |
| Found 'DD'-anomaly for variable 'retVal' (lines '104'-'107'). | 104–107 |
| Found 'DD'-anomaly for variable 'retVal' (lines '104'-'107'). | 104–107 |
| Found 'DD'-anomaly for variable 'retVal' (lines '104'-'107'). | 104–107 |
| Found 'DD'-anomaly for variable 'retVal' (lines '104'-'107'). | 104–107 |
| Found 'DD'-anomaly for variable 'retVal' (lines '104'-'107'). | 104–107 |
| Potential violation of Law of Demeter (object not created locally) | 107 |
| Potential violation of Law of Demeter (object not created locally) | 108 |
| Potential violation of Law of Demeter (object not created locally) | 109 |
| Potential violation of Law of Demeter (object not created locally) | 111 |
| A method should have only one exit point, and that should be the last statement in the method | 122 |
| Found 'DU'-anomaly for variable 'yieldVal' (lines '125'-'155'). | 125–155 |
| Found 'DU'-anomaly for variable 'yieldVal' (lines '125'-'155'). | 125–155 |
| Found 'DU'-anomaly for variable 'yieldVal' (lines '125'-'155'). | 125–155 |
| Found 'DU'-anomaly for variable 'yieldVal' (lines '125'-'155'). | 125–155 |
| Found 'DU'-anomaly for variable 'yieldVal' (lines '125'-'155'). | 125–155 |
| publicMethodCommentRequirement Required | 129–132 |
| Parameter 'initValue' is not assigned and could be declared final | 135 |
| publicMethodCommentRequirement Required | 135–139 |
| publicMethodCommentRequirement Required | 142–145 |
| publicMethodCommentRequirement Required | 148–153 |
| Potential violation of Law of Demeter (method chain calls) | 151 |
| Violation | Line |
|---|---|
| Comment is too large: Too many lines | 1–15 |
| fieldCommentRequirement Required | 25 |
| Found non-transient, non-static member. Please mark as transient or provide accessors. | 25 |
| fieldCommentRequirement Required | 26 |
| Found non-transient, non-static member. Please mark as transient or provide accessors. | 26 |
| Parameter 'baseSequence' is not assigned and could be declared final | 28 |
| Parameter 'closeable' is not assigned and could be declared final | 28 |
| publicMethodCommentRequirement Required | 28–32 |
| Parameter 'initValue' is not assigned and could be declared final | 35 |
| publicMethodCommentRequirement Required | 35–43 |
| Parameter 'accumulator' is not assigned and could be declared final | 35 |
| publicMethodCommentRequirement Required | 46–60 |
| Parameter 'accumulator' is not assigned and could be declared final | 47 |
| Parameter 'initValue' is not assigned and could be declared final | 47 |
| Avoid using final local variables, turn them into fields | 50 |
| Found 'DU'-anomaly for variable 'baseYielder' (lines '52'-'60'). | 52–60 |
| Avoid catching generic exceptions such as NullPointerException, RuntimeException, Exception in try-catch block | 54 |
| Violation | Line |
|---|---|
| Comment is too large: Too many lines | 1–15 |
| Generics names should be a one letter long and upper case. | 24 |
| Found non-transient, non-static member. Please mark as transient or provide accessors. | 26 |
| fieldCommentRequirement Required | 26 |
| fieldCommentRequirement Required | 27 |
| Found non-transient, non-static member. Please mark as transient or provide accessors. | 27 |
| Parameter 'closeable' is not assigned and could be declared final | 29 |
| publicMethodCommentRequirement Required | 29–33 |
| Parameter 'baseYielder' is not assigned and could be declared final | 29 |
| publicMethodCommentRequirement Required | 36–39 |
| publicMethodCommentRequirement Required | 42–45 |
| Parameter 'initValue' is not assigned and could be declared final | 42 |
| publicMethodCommentRequirement Required | 48–51 |
| publicMethodCommentRequirement Required | 54–60 |
| Violation | Line |
|---|---|
| Comment is too large: Too many lines | 1–15 |
| Comment is too large: Too many lines | 19–31 |
| Comment is too large: Line too long | 20 |
| Comment is too large: Line too long | 21 |
| Comment is too large: Line too long | 24 |
| Comment is too large: Line too long | 25 |
| Comment is too large: Line too long | 26 |
| Comment is too large: Line too long | 28 |
| Comment is too large: Line too long | 29 |
| Comment is too large: Line too long | 30 |
| publicMethodCommentRequirement Required | 34 |
| Avoid modifiers which are implied by the context | 34 |
| Avoid modifiers which are implied by the context | 35 |
| publicMethodCommentRequirement Required | 35 |
| Violation | Line |
|---|---|
| Comment is too large: Too many lines | 1–15 |
| This class has too many methods, consider refactoring it. | 33–130 |
| All methods are static. Consider using a utility class instead. Alternatively, you could add a private constructor or make the class abstract to silence this warning. | 33–130 |
| fieldCommentRequirement Required | 35 |
| publicMethodCommentRequirement Required | 37–40 |
| publicMethodCommentRequirement Required | 43–46 |
| publicMethodCommentRequirement Required | 48–51 |
| Parameter 'sequences' is not assigned and could be declared final | 48 |
| Parameter 'sequences' is not assigned and could be declared final | 53 |
| publicMethodCommentRequirement Required | 53–56 |
| publicMethodCommentRequirement Required | 58–61 |
| Parameter 'sequences' is not assigned and could be declared final | 58 |
| Parameter 'fn' is not assigned and could be declared final | 63 |
| Avoid variables with short names like fn | 63 |
| Parameter 'sequence' is not assigned and could be declared final | 63 |
| publicMethodCommentRequirement Required | 63–66 |
| Parameter 'pred' is not assigned and could be declared final | 68 |
| publicMethodCommentRequirement Required | 68–71 |
| Parameter 'sequence' is not assigned and could be declared final | 68 |
| publicMethodCommentRequirement Required | 73–76 |
| Parameter 'baggage' is not assigned and could be declared final | 78 |
| publicMethodCommentRequirement Required | 78–81 |
| publicMethodCommentRequirement Required | 83–101 |
| To be compliant to J2EE, a webapp should not use any thread. | 83 |
| publicMethodCommentRequirement Required | 88–93 |
| Parameter 'accumulator' is not assigned and could be declared final | 88 |
| Parameter 'initValue' is not assigned and could be declared final | 88 |
| Parameter 'initValue' is not assigned and could be declared final | 96 |
| Parameter 'accumulator' is not assigned and could be declared final | 96 |
| publicMethodCommentRequirement Required | 96–99 |
| publicMethodCommentRequirement Required | 104–109 |
| Potential violation of Law of Demeter (method chain calls) | 106 |
| Potential violation of Law of Demeter (method chain calls) | 106 |
| Local variable 'seqList' could be declared final | 106 |
| Parameter 'list' is not assigned and could be declared final | 111 |
| publicMethodCommentRequirement Required | 111–114 |
| Parameter 'seq' is not assigned and could be declared final | 111 |
| Potential violation of Law of Demeter (method chain calls) | 113 |
| Potential violation of Law of Demeter (method chain calls) | 113 |
| headerCommentRequirement Required | 116–129 |
| Parameter 'accumulator' is not assigned and could be declared final | 119 |
| Parameter 'initValue' is not assigned and could be declared final | 119 |
| publicMethodCommentRequirement Required | 119–122 |
| Parameter 'initValue' is not assigned and could be declared final | 125 |
| publicMethodCommentRequirement Required | 125–128 |
| Parameter 'accumulator' is not assigned and could be declared final | 125 |
| Violation | Line |
|---|---|
| Comment is too large: Too many lines | 1–15 |
| All methods are static. Consider using a utility class instead. Alternatively, you could add a private constructor or make the class abstract to silence this warning. | 24–52 |
| Parameter 'iterable' is not assigned and could be declared final | 25 |
| publicMethodCommentRequirement Required | 25–28 |
| publicMethodCommentRequirement Required | 30–50 |
| publicMethodCommentRequirement Required | 38–41 |
| Parameter 'iterFromMake' is not assigned and could be declared final | 44 |
| publicMethodCommentRequirement Required | 44–47 |
| Document empty method body | 45–47 |
| Violation | Line |
|---|---|
| Comment is too large: Too many lines | 1–15 |
| publicMethodCommentRequirement Required | 23 |
| Avoid modifiers which are implied by the context | 23 |
| Violation | Line |
|---|---|
| Comment is too large: Too many lines | 1–15 |
| Comment is too large: Too many lines | 21–35 |
| Comment is too large: Line too long | 22 |
| Comment is too large: Line too long | 25 |
| Comment is too large: Line too long | 26 |
| Comment is too large: Line too long | 27 |
| Comment is too large: Line too long | 30 |
| Comment is too large: Line too long | 33 |
| Comment is too large: Line too long | 39 |
| Comment is too large: Line too long | 41 |
| Avoid modifiers which are implied by the context | 45 |
| Comment is too large: Too many lines | 47–56 |
| Comment is too large: Line too long | 48 |
| Comment is too large: Line too long | 51 |
| Comment is too large: Line too long | 53 |
| Avoid modifiers which are implied by the context | 57 |
| Comment is too large: Too many lines | 59–67 |
| Comment is too large: Line too long | 60 |
| Comment is too large: Line too long | 61 |
| Comment is too large: Line too long | 62 |
| Comment is too large: Line too long | 64 |
| Avoid modifiers which are implied by the context | 68 |
| Violation | Line |
|---|---|
| Comment is too large: Too many lines | 1–15 |
| All methods are static. Consider using a utility class instead. Alternatively, you could add a private constructor or make the class abstract to silence this warning. | 27–57 |
| publicMethodCommentRequirement Required | 28–56 |
| publicMethodCommentRequirement Required | 33–36 |
| Parameter 'initValue' is not assigned and could be declared final | 39 |
| publicMethodCommentRequirement Required | 39–42 |
| publicMethodCommentRequirement Required | 45–48 |
| publicMethodCommentRequirement Required | 51–54 |
| Violation | Line |
|---|---|
| Comment is too large: Too many lines | 1–15 |
| Comment is too large: Line too long | 20 |
| Comment is too large: Line too long | 23 |
| Comment is too large: Line too long | 24 |
| Each class should declare at least one constructor | 27–47 |
| Generics names should be a one letter long and upper case. | 27 |
| Abstract classes should be named AbstractXXX | 27–47 |
| Generics names should be a one letter long and upper case. | 27 |
| Avoid using redundant field initializer for 'yielded' | 29 |
| Field yielded has the same name as a method | 29 |
| fieldCommentRequirement Required | 29 |
| Found non-transient, non-static member. Please mark as transient or provide accessors. | 29 |
| publicMethodCommentRequirement Required | 31–34 |
| publicMethodCommentRequirement Required | 36–39 |
| publicMethodCommentRequirement Required | 41–44 |
| Avoid prefixing parameters by in, out or inOut. Uses Javadoc to document this behavior. | 46 |
| publicMethodCommentRequirement Required | 46 |
| Avoid variables with short names like in | 46 |
| Violation | Line |
|---|---|
| Comment is too large: Too many lines | 1–15 |
| All methods are static. Consider using a utility class instead. Alternatively, you could add a private constructor or make the class abstract to silence this warning. | 22–36 |
| publicMethodCommentRequirement Required | 23–35 |
| Parameter 'accumulated' is not assigned and could be declared final | 30 |
| Avoid variables with short names like in | 30 |
| publicMethodCommentRequirement Required | 30–33 |
| Avoid prefixing parameters by in, out or inOut. Uses Javadoc to document this behavior. | 30 |
| Parameter 'in' is not assigned and could be declared final | 30 |
| Violation | Line |
|---|---|
| Comment is too large: Too many lines | 1–15 |
| Avoid unused imports such as 'com.google.common.base.Throwables' | 19 |
| Avoid unused imports such as 'com.google.common.io.Closeables' | 20 |
| Avoid unused imports such as 'java.io.IOException' | 22 |
| Abstract classes should be named AbstractXXX | 29–43 |
| Each class should declare at least one constructor | 29–43 |
| publicMethodCommentRequirement Required | 32–42 |
| Parameter 'accumulator' is not assigned and could be declared final | 32 |