The following document contains the results of Checkstyle
.
Files | Infos ![]() |
Warnings ![]() |
Errors ![]() |
---|---|---|---|
10 | 0 | 0 | 289 |
Rules | Violations | Severity |
---|---|---|
PackageHtml | 2 | ![]() |
NewlineAtEndOfFile | 0 | ![]() |
Translation | 0 | ![]() |
JavadocMethod | 51 | ![]() |
JavadocType | 0 | ![]() |
JavadocVariable | 27 | ![]() |
JavadocStyle | 9 | ![]() |
ConstantName | 1 | ![]() |
LocalFinalVariableName | 0 | ![]() |
LocalVariableName | 0 | ![]() |
MemberName | 0 | ![]() |
MethodName | 0 | ![]() |
PackageName | 0 | ![]() |
ParameterName | 0 | ![]() |
StaticVariableName | 0 | ![]() |
TypeName | 0 | ![]() |
AvoidStarImport | 0 | ![]() |
IllegalImport | 0 | ![]() |
RedundantImport | 0 | ![]() |
UnusedImports | 0 | ![]() |
FileLength | 0 | ![]() |
LineLength | 5 | ![]() |
MethodLength | 1 | ![]() |
ParameterNumber | 0 | ![]() |
EmptyForIteratorPad | 0 | ![]() |
MethodParamPad | 0 | ![]() |
NoWhitespaceAfter | 0 | ![]() |
NoWhitespaceBefore | 0 | ![]() |
OperatorWrap | 0 | ![]() |
ParenPad | 0 | ![]() |
TypecastParenPad | 0 | ![]() |
TabCharacter | 0 | ![]() |
WhitespaceAfter | 48 | ![]() |
WhitespaceAround | 0 | ![]() |
ModifierOrder | 1 | ![]() |
RedundantModifier | 0 | ![]() |
AvoidNestedBlocks | 0 | ![]() |
EmptyBlock | 0 | ![]() |
LeftCurly | 0 | ![]() |
NeedBraces | 37 | ![]() |
RightCurly | 27 | ![]() |
AvoidInlineConditionals | 11 | ![]() |
DoubleCheckedLocking | 0 | ![]() |
EmptyStatement | 0 | ![]() |
EqualsHashCode | 0 | ![]() |
HiddenField | 2 | ![]() |
IllegalInstantiation | 0 | ![]() |
InnerAssignment | 0 | ![]() |
MagicNumber | 5 | ![]() |
MissingSwitchDefault | 0 | ![]() |
RedundantThrows | 0 | ![]() |
SimplifyBooleanExpression | 0 | ![]() |
SimplifyBooleanReturn | 0 | ![]() |
DesignForExtension | 28 | ![]() |
FinalClass | 0 | ![]() |
HideUtilityClassConstructor | 0 | ![]() |
InterfaceIsType | 0 | ![]() |
VisibilityModifier | 15 | ![]() |
ArrayTypeStyle | 0 | ![]() |
FinalParameters | 17 | ![]() |
GenericIllegalRegexp
|
2 | ![]() |
TodoComment | 0 | ![]() |
UpperEll | 0 | ![]() |
Violation | Message | Line |
---|---|---|
![]() |
Missing a Javadoc comment. | 14 |
![]() |
Line is longer than 80 characters. | 18 |
![]() |
Method 'handle' is not designed for extension - needs to be abstract, final or empty. | 20 |
![]() |
Parameter context should be final. | 20 |
![]() |
'cast' is not followed by whitespace. | 21 |
![]() |
'cast' is not followed by whitespace. | 22 |
![]() |
'if' construct must use '{}'s. | 23 |
![]() |
'if' construct must use '{}'s. | 26 |
![]() |
'cast' is not followed by whitespace. | 28 |
![]() |
'}' should be on the same line. | 32 |
![]() |
'cast' is not followed by whitespace. | 39 |
![]() |
'}' should be on the same line. | 40 |
Violation | Message | Line |
---|---|---|
![]() |
Missing a Javadoc comment. | 22 |
![]() |
Missing a Javadoc comment. | 23 |
![]() |
Missing a Javadoc comment. | 24 |
![]() |
Missing a Javadoc comment. | 25 |
![]() |
Variable 'userhours' must be private and have accessor methods. | 25 |
![]() |
Missing a Javadoc comment. | 26 |
![]() |
First sentence should end with a period. | 28 |
![]() |
Parameter db should be final. | 33 |
![]() |
Expected @param tag for 'db'. | 33 |
![]() |
Parameter rs should be final. | 33 |
![]() |
Expected @param tag for 'rs'. | 33 |
![]() |
First sentence should end with a period. | 38 |
![]() |
Parameter db should be final. | 43 |
![]() |
Expected @param tag for 'db'. | 43 |
![]() |
First sentence should end with a period. | 47 |
![]() |
Parameter db should be final. | 52 |
![]() |
Expected @param tag for 'db'. | 52 |
![]() |
Parameter record should be final. | 52 |
![]() |
Expected @param tag for 'record'. | 52 |
![]() |
Missing a Javadoc comment. | 57 |
![]() |
'if' construct must use '{}'s. | 59 |
![]() |
Missing a Javadoc comment. | 64 |
![]() |
'if' construct must use '{}'s. | 66 |
![]() |
Missing a Javadoc comment. | 71 |
![]() |
Method 'getFixedValueLeftDisplay' is not designed for extension - needs to be abstract, final or empty. | 75 |
![]() |
Missing a Javadoc comment. | 75 |
![]() |
Missing a Javadoc comment. | 79 |
![]() |
'if' construct must use '{}'s. | 81 |
![]() |
Missing a Javadoc comment. | 86 |
![]() |
'100' is a magic number. | 87 |
![]() |
Missing a Javadoc comment. | 90 |
![]() |
Method 'getSalesFeeDisplay' is not designed for extension - needs to be abstract, final or empty. | 94 |
![]() |
Missing a Javadoc comment. | 94 |
![]() |
Method 'getSalesFeeLeftDisplay' is not designed for extension - needs to be abstract, final or empty. | 98 |
![]() |
Missing a Javadoc comment. | 98 |
![]() |
Missing a Javadoc comment. | 102 |
![]() |
'if' construct must use '{}'s. | 104 |
![]() |
Missing a Javadoc comment. | 109 |
![]() |
'100' is a magic number. | 110 |
![]() |
Missing a Javadoc comment. | 113 |
![]() |
Method 'getManagementFeeDisplay' is not designed for extension - needs to be abstract, final or empty. | 117 |
![]() |
Missing a Javadoc comment. | 117 |
![]() |
Method 'getManagementFeeLeftDisplay' is not designed for extension - needs to be abstract, final or empty. | 121 |
![]() |
Missing a Javadoc comment. | 121 |
![]() |
Method 'getHoursWorkedDisplay' is not designed for extension - needs to be abstract, final or empty. | 125 |
![]() |
Missing a Javadoc comment. | 125 |
![]() |
Method 'getBudgetRemaining' is not designed for extension - needs to be abstract, final or empty. | 129 |
![]() |
Missing a Javadoc comment. | 129 |
![]() |
Method 'getBudgetRemainingPercent' is not designed for extension - needs to be abstract, final or empty. | 133 |
![]() |
Missing a Javadoc comment. | 133 |
![]() |
'if' construct must use '{}'s. | 134 |
![]() |
'100' is a magic number. | 136 |
![]() |
Missing a Javadoc comment. | 140 |
![]() |
'}' should be on the same line. | 143 |
![]() |
Method 'getHourlyRateDisplay' is not designed for extension - needs to be abstract, final or empty. | 149 |
![]() |
Missing a Javadoc comment. | 149 |
![]() |
Missing a Javadoc comment. | 153 |
![]() |
Parameter f should be final. | 153 |
![]() |
Expected an @return tag. | 160 |
![]() |
Method 'getBreakdown' is not designed for extension - needs to be abstract, final or empty. | 160 |
![]() |
Line is longer than 80 characters. | 170 |
![]() |
'}' should be on the same line. | 180 |
![]() |
'cast' is not followed by whitespace. | 182 |
![]() |
'cast' is not followed by whitespace. | 191 |
![]() |
Method 'getBudget' is not designed for extension - needs to be abstract, final or empty. | 199 |
![]() |
Missing a Javadoc comment. | 199 |
![]() |
First sentence should end with a period. | 220 |
![]() |
Line is longer than 80 characters. | 221 |
![]() |
Missing a Javadoc comment. | 224 |
![]() |
Variable 'user' must be private and have accessor methods. | 224 |
![]() |
Missing a Javadoc comment. | 225 |
![]() |
Variable 'hours' must be private and have accessor methods. | 225 |
![]() |
Missing a Javadoc comment. | 226 |
![]() |
Variable 'payment' must be private and have accessor methods. | 226 |
![]() |
Method 'getPaymentDisplay' is not designed for extension - needs to be abstract, final or empty. | 228 |
![]() |
Missing a Javadoc comment. | 228 |
Violation | Message | Line |
---|---|---|
![]() |
Missing a Javadoc comment. | 26 |
![]() |
Method 'handle' is not designed for extension - needs to be abstract, final or empty. | 28 |
![]() |
Missing a Javadoc comment. | 28 |
![]() |
Parameter context should be final. | 28 |
![]() |
'cast' is not followed by whitespace. | 29 |
![]() |
'cast' is not followed by whitespace. | 30 |
![]() |
'if' construct must use '{}'s. | 31 |
![]() |
'if' construct must use '{}'s. | 34 |
![]() |
'cast' is not followed by whitespace. | 36 |
![]() |
'cast' is not followed by whitespace. | 37 |
![]() |
'cast' is not followed by whitespace. | 38 |
![]() |
'if' construct must use '{}'s. | 39 |
![]() |
'if' construct must use '{}'s. | 41 |
![]() |
'if' construct must use '{}'s. | 60 |
![]() |
'if' construct must use '{}'s. | 62 |
![]() |
'cast' is not followed by whitespace. | 75 |
![]() |
'}' should be on the same line. | 81 |
![]() |
'}' should be on the same line. | 84 |
![]() |
'}' should be on the same line. | 104 |
![]() |
'}' should be on the same line. | 108 |
![]() |
'cast' is not followed by whitespace. | 115 |
![]() |
'}' should be on the same line. | 116 |
Violation | Message | Line |
---|---|---|
![]() |
Missing a Javadoc comment. | 14 |
![]() |
Line is longer than 80 characters. | 18 |
![]() |
Method 'handle' is not designed for extension - needs to be abstract, final or empty. | 20 |
![]() |
Parameter context should be final. | 20 |
![]() |
'cast' is not followed by whitespace. | 21 |
![]() |
'cast' is not followed by whitespace. | 22 |
![]() |
'if' construct must use '{}'s. | 23 |
![]() |
'cast' is not followed by whitespace. | 26 |
![]() |
'if' construct must use '{}'s. | 27 |
![]() |
'if' construct must use '{}'s. | 29 |
![]() |
'cast' is not followed by whitespace. | 31 |
![]() |
'}' should be on the same line. | 35 |
![]() |
'cast' is not followed by whitespace. | 42 |
![]() |
'}' should be on the same line. | 43 |
Violation | Message | Line |
---|---|---|
![]() |
Line has trailing spaces. | 21 |
![]() |
Line has trailing spaces. | 22 |
![]() |
Missing a Javadoc comment. | 28 |
![]() |
First sentence should end with a period. | 30 |
![]() |
Variable 'connMgr' must be private and have accessor methods. | 31 |
![]() |
'static' modifier out of order with the JLS suggestions. | 34 |
![]() |
Name 'debug' must match pattern '^[A-Z][A-Z0-9]*(_[A-Z0-9]+)*$'. | 34 |
![]() |
Expected an @return tag. | 51 |
![]() |
Method 'handle' is not designed for extension - needs to be abstract, final or empty. | 51 |
![]() |
Method length is 426 lines (max allowed is 150). | 51 |
![]() |
Parameter context should be final. | 51 |
![]() |
Expected @param tag for 'context'. | 51 |
![]() |
Expected @throws tag for 'HandlerException'. | 51 |
![]() |
'cast' is not followed by whitespace. | 65 |
![]() |
'if' construct must use '{}'s. | 66 |
![]() |
'cast' is not followed by whitespace. | 71 |
![]() |
'if' construct must use '{}'s. | 72 |
![]() |
'cast' is not followed by whitespace. | 77 |
![]() |
'if' construct must use '{}'s. | 108 |
![]() |
'}' should be on the same line. | 129 |
![]() |
'if' construct must use '{}'s. | 140 |
![]() |
'if' construct must use '{}'s. | 157 |
![]() |
'if' construct must use '{}'s. | 186 |
![]() |
Line is longer than 80 characters. | 188 |
![]() |
'if' construct must use '{}'s. | 203 |
![]() |
'}' should be on the same line. | 216 |
![]() |
'}' should be on the same line. | 229 |
![]() |
Avoid inline conditionals. | 249 |
![]() |
Avoid inline conditionals. | 250 |
![]() |
Avoid inline conditionals. | 255 |
![]() |
Avoid inline conditionals. | 263 |
![]() |
Avoid inline conditionals. | 269 |
![]() |
'}' should be on the same line. | 270 |
![]() |
'}' should be on the same line. | 288 |
![]() |
'cast' is not followed by whitespace. | 302 |
![]() |
Avoid inline conditionals. | 307 |
![]() |
'if' construct must use '{}'s. | 320 |
![]() |
'}' should be on the same line. | 322 |
![]() |
'cast' is not followed by whitespace. | 346 |
![]() |
'cast' is not followed by whitespace. | 353 |
![]() |
Avoid inline conditionals. | 358 |
![]() |
Avoid inline conditionals. | 359 |
![]() |
Avoid inline conditionals. | 361 |
![]() |
Avoid inline conditionals. | 364 |
![]() |
'if' construct must use '{}'s. | 376 |
![]() |
'}' should be on the same line. | 384 |
![]() |
'if' construct must use '{}'s. | 388 |
![]() |
'}' should be on the same line. | 393 |
![]() |
'cast' is not followed by whitespace. | 431 |
![]() |
'if' construct must use '{}'s. | 432 |
![]() |
Avoid inline conditionals. | 457 |
![]() |
',' is not followed by whitespace. | 461 |
![]() |
'cast' is not followed by whitespace. | 468 |
![]() |
'}' should be on the same line. | 469 |
![]() |
'}' should be on the same line. | 472 |
![]() |
First sentence should end with a period. | 478 |
![]() |
Expected an @return tag. | 482 |
![]() |
Parameter param should be final. | 482 |
![]() |
Expected @param tag for 'param'. | 482 |
![]() |
Parameter context should be final. | 482 |
![]() |
Expected @param tag for 'context'. | 482 |
![]() |
'cast' is not followed by whitespace. | 483 |
![]() |
'if' construct must use '{}'s. | 484 |
![]() |
'if' construct must use '{}'s. | 486 |
![]() |
First sentence should end with a period. | 491 |
![]() |
Missing a Javadoc comment. | 495 |
![]() |
Variable 'project' must be private and have accessor methods. | 495 |
![]() |
Missing a Javadoc comment. | 496 |
![]() |
Variable 'tasks' must be private and have accessor methods. | 496 |
![]() |
Missing a Javadoc comment. | 497 |
![]() |
Variable 'totalBudget' must be private and have accessor methods. | 497 |
![]() |
Missing a Javadoc comment. | 498 |
![]() |
Variable 'totalWorked' must be private and have accessor methods. | 498 |
![]() |
Method 'overBudget' is not designed for extension - needs to be abstract, final or empty. | 500 |
![]() |
Missing a Javadoc comment. | 500 |
![]() |
First sentence should end with a period. | 505 |
![]() |
Missing a Javadoc comment. | 509 |
![]() |
Variable 'task' must be private and have accessor methods. | 509 |
![]() |
Missing a Javadoc comment. | 510 |
![]() |
Variable 'tSheets' must be private and have accessor methods. | 510 |
![]() |
Missing a Javadoc comment. | 511 |
![]() |
Variable 'totalWorked' must be private and have accessor methods. | 511 |
![]() |
Method 'overBudget' is not designed for extension - needs to be abstract, final or empty. | 513 |
![]() |
Missing a Javadoc comment. | 513 |
![]() |
'if' construct must use '{}'s. | 515 |
Violation | Message | Line |
---|---|---|
![]() |
Missing a Javadoc comment. | 20 |
![]() |
Method 'handle' is not designed for extension - needs to be abstract, final or empty. | 22 |
![]() |
Missing a Javadoc comment. | 22 |
![]() |
Parameter context should be final. | 22 |
![]() |
'cast' is not followed by whitespace. | 23 |
![]() |
'cast' is not followed by whitespace. | 34 |
![]() |
'if' construct must use '{}'s. | 35 |
![]() |
'cast' is not followed by whitespace. | 37 |
![]() |
'cast' is not followed by whitespace. | 43 |
![]() |
'10' is a magic number. | 47 |
![]() |
'cast' is not followed by whitespace. | 51 |
![]() |
'cast' is not followed by whitespace. | 54 |
![]() |
'}' should be on the same line. | 84 |
![]() |
'}' should be on the same line. | 90 |
![]() |
'cast' is not followed by whitespace. | 97 |
![]() |
'}' should be on the same line. | 98 |
Violation | Message | Line |
---|---|---|
![]() |
Missing package documentation file. | 0 |
Violation | Message | Line |
---|---|---|
![]() |
Missing a Javadoc comment. | 20 |
![]() |
Method 'handle' is not designed for extension - needs to be abstract, final or empty. | 22 |
![]() |
Missing a Javadoc comment. | 22 |
![]() |
Parameter context should be final. | 22 |
![]() |
'cast' is not followed by whitespace. | 23 |
![]() |
'cast' is not followed by whitespace. | 38 |
![]() |
'if' construct must use '{}'s. | 39 |
![]() |
'cast' is not followed by whitespace. | 42 |
![]() |
'cast' is not followed by whitespace. | 48 |
![]() |
'cast' is not followed by whitespace. | 52 |
![]() |
'if' construct must use '{}'s. | 63 |
![]() |
'10' is a magic number. | 72 |
![]() |
'}' should be on the same line. | 79 |
![]() |
'cast' is not followed by whitespace. | 86 |
![]() |
'}' should be on the same line. | 87 |
Violation | Message | Line |
---|---|---|
![]() |
Missing a Javadoc comment. | 22 |
![]() |
Missing a Javadoc comment. | 23 |
![]() |
Missing a Javadoc comment. | 25 |
![]() |
Missing a Javadoc comment. | 26 |
![]() |
Variable 'id' must be private and have accessor methods. | 26 |
![]() |
Missing a Javadoc comment. | 27 |
![]() |
Variable 'db' must be private and have accessor methods. | 27 |
![]() |
Missing a Javadoc comment. | 28 |
![]() |
Variable 'connMgr' must be private and have accessor methods. | 28 |
![]() |
First sentence should end with a period. | 30 |
![]() |
Parameter db should be final. | 33 |
![]() |
'db' hides a field. | 33 |
![]() |
Expected @param tag for 'db'. | 33 |
![]() |
Parameter id should be final. | 33 |
![]() |
'id' hides a field. | 33 |
![]() |
Expected @param tag for 'id'. | 33 |
![]() |
Expected @throws tag for 'Exception'. | 33 |
![]() |
'if' construct must use '{}'s. | 36 |
![]() |
Missing a Javadoc comment. | 53 |
![]() |
Parameter id should be final. | 53 |
![]() |
Method 'getHeader' is not designed for extension - needs to be abstract, final or empty. | 59 |
![]() |
Missing a Javadoc comment. | 59 |
![]() |
Method 'getInvoiceLine' is not designed for extension - needs to be abstract, final or empty. | 63 |
![]() |
Missing a Javadoc comment. | 63 |
![]() |
Method 'getTotal' is not designed for extension - needs to be abstract, final or empty. | 67 |
![]() |
Missing a Javadoc comment. | 67 |
![]() |
'cast' is not followed by whitespace. | 70 |
![]() |
'if' construct must use '{}'s. | 72 |
![]() |
Method 'getTotalDisplay' is not designed for extension - needs to be abstract, final or empty. | 79 |
![]() |
Missing a Javadoc comment. | 79 |
![]() |
Method 'hasTimesheets' is not designed for extension - needs to be abstract, final or empty. | 83 |
![]() |
Missing a Javadoc comment. | 83 |
![]() |
Method 'write' is not designed for extension - needs to be abstract, final or empty. | 96 |
![]() |
Missing a Javadoc comment. | 96 |
![]() |
'cast' is not followed by whitespace. | 99 |
![]() |
'cast' is not followed by whitespace. | 100 |
![]() |
'cast' is not followed by whitespace. | 103 |
![]() |
'}' should be on the same line. | 112 |
![]() |
Method 'isOverdue' is not designed for extension - needs to be abstract, final or empty. | 119 |
![]() |
Missing a Javadoc comment. | 119 |
![]() |
'cast' is not followed by whitespace. | 121 |
![]() |
'cast' is not followed by whitespace. | 122 |
![]() |
'if' construct must use '{}'s. | 123 |
![]() |
'if' construct must use '{}'s. | 126 |
![]() |
Method 'isOutstanding' is not designed for extension - needs to be abstract, final or empty. | 132 |
![]() |
Missing a Javadoc comment. | 132 |
![]() |
'cast' is not followed by whitespace. | 133 |