The following document contains the results of Checkstyle. 
| Files | Infos | Warnings | Errors |
|---|---|---|---|
| 13 | 0 | 0 | 383 |
| Rules | Violations | Severity |
|---|---|---|
| PackageHtml | 4 | |
| NewlineAtEndOfFile | 0 | |
| Translation | 0 | |
| JavadocMethod | 62 | |
| JavadocType | 3 | |
| JavadocVariable | 20 | |
| JavadocStyle | 4 | |
| ConstantName | 3 | |
| LocalFinalVariableName | 0 | |
| LocalVariableName | 0 | |
| MemberName | 0 | |
| MethodName | 0 | |
| PackageName | 0 | |
| ParameterName | 0 | |
| StaticVariableName | 0 | |
| TypeName | 0 | |
| AvoidStarImport | 2 | |
| IllegalImport | 0 | |
| RedundantImport | 0 | |
| UnusedImports | 0 | |
| FileLength | 0 | |
| LineLength | 64 | |
| MethodLength | 0 | |
| ParameterNumber | 0 | |
| EmptyForIteratorPad | 0 | |
| MethodParamPad | 0 | |
| NoWhitespaceAfter | 0 | |
| NoWhitespaceBefore | 0 | |
| OperatorWrap | 0 | |
| ParenPad | 0 | |
| TypecastParenPad | 0 | |
| TabCharacter | 6 | |
| WhitespaceAfter | 1 | |
| WhitespaceAround | 2 | |
| ModifierOrder | 0 | |
| RedundantModifier | 0 | |
| AvoidNestedBlocks | 0 | |
| EmptyBlock | 0 | |
| LeftCurly | 83 | |
| NeedBraces | 16 | |
| RightCurly | 16 | |
| AvoidInlineConditionals | 3 | |
| DoubleCheckedLocking | 0 | |
| EmptyStatement | 0 | |
| EqualsHashCode | 0 | |
| HiddenField | 6 | |
| IllegalInstantiation | 0 | |
| InnerAssignment | 0 | |
| MagicNumber | 2 | |
| MissingSwitchDefault | 0 | |
| RedundantThrows | 1 | |
| SimplifyBooleanExpression | 0 | |
| SimplifyBooleanReturn | 0 | |
| DesignForExtension | 19 | |
| FinalClass | 0 | |
| HideUtilityClassConstructor | 3 | |
| InterfaceIsType | 0 | |
| VisibilityModifier | 9 | |
| ArrayTypeStyle | 0 | |
| FinalParameters | 44 | |
GenericIllegalRegexp
| 10 | |
| TodoComment | 0 | |
| UpperEll | 0 |
| Violation | Message | Line |
|---|---|---|
| '{' should be on the previous line. | 11 | |
| Missing a Javadoc comment. | 12 |
| Violation | Message | Line |
|---|---|---|
| Missing a Javadoc comment. | 7 | |
| Missing a Javadoc comment. | 8 | |
| Variable 'target' must be private and have accessor methods. | 8 | |
| Missing a Javadoc comment. | 10 | |
| Parameter target should be final. | 10 | |
| 'target' hides a field. | 10 | |
| 'if' construct must use '{}'s. | 11 | |
| Line is longer than 80 characters. | 12 | |
| Line is longer than 80 characters. | 16 | |
| Method 'invoke' is not designed for extension - needs to be abstract, final or empty. | 16 | |
| Missing a Javadoc comment. | 16 | |
| Parameter proxy should be final. | 16 | |
| Parameter method should be final. | 16 | |
| Parameter args should be final. | 16 |
| Violation | Message | Line |
|---|---|---|
| Missing package documentation file. | 0 |
| Violation | Message | Line |
|---|---|---|
| Utility classes should not have a public or default constructor. | 35 | |
| '{' should be on the previous line. | 36 | |
| Line is longer than 80 characters. | 37 | |
| Missing a Javadoc comment. | 37 | |
| Name 'log' must match pattern '^[A-Z][A-Z0-9]*(_[A-Z0-9]+)*$'. | 37 | |
| Missing a Javadoc comment. | 40 | |
| Line is longer than 80 characters. | 43 | |
| Line is longer than 80 characters. | 49 | |
| Line is longer than 80 characters. | 52 | |
| First sentence should end with a period. | 54 | |
| Line is longer than 80 characters. | 55 | |
| Line is longer than 80 characters. | 60 | |
| Line is longer than 80 characters. | 63 | |
| Missing a Javadoc comment. | 65 | |
| Missing a Javadoc comment. | 66 | |
| Line has trailing spaces. | 67 | |
| Missing a Javadoc comment. | 68 | |
| Missing a Javadoc comment. | 69 | |
| Line has trailing spaces. | 70 | |
| Line is longer than 80 characters. | 71 | |
| Line has trailing spaces. | 73 | |
| Line is longer than 80 characters. | 75 | |
| Line contains a tab character. | 75 | |
| Line is longer than 80 characters. | 76 | |
| Line contains a tab character. | 76 | |
| Missing a Javadoc comment. | 79 | |
| Parameter name should be final. | 79 | |
| '{' should be on the previous line. | 80 | |
| 'if' construct must use '{}'s. | 81 | |
| Line has trailing spaces. | 87 | |
| 'if' construct must use '{}'s. | 88 | |
| '{' should be on the previous line. | 94 | |
| Missing a Javadoc comment. | 102 | |
| Parameter name should be final. | 102 | |
| '{' should be on the previous line. | 103 | |
| 'if' construct must use '{}'s. | 104 | |
| 'if' construct must use '{}'s. | 109 | |
| Line has trailing spaces. | 111 | |
| Line is longer than 80 characters. | 113 | |
| '{' should be on the previous line. | 119 | |
| 'if' construct must use '{}'s. | 120 | |
| '}' should be on the same line. | 124 | |
| '{' should be on the previous line. | 126 | |
| '}' should be on the same line. | 129 | |
| '{' should be on the previous line. | 131 | |
| Line is longer than 80 characters. | 132 | |
| '{' should be on the previous line. | 134 | |
| Line is longer than 80 characters. | 137 | |
| Missing a Javadoc comment. | 144 | |
| Parameter currentTimer should be final. | 144 | |
| '{' should be on the previous line. | 145 | |
| '{' should be on the previous line. | 148 | |
| Missing a Javadoc comment. | 153 | |
| '{' should be on the previous line. | 154 | |
| Line is longer than 80 characters. | 155 | |
| Avoid inline conditionals. | 155 | |
| Missing a Javadoc comment. | 158 | |
| '{' should be on the previous line. | 159 | |
| Line is longer than 80 characters. | 160 | |
| Avoid inline conditionals. | 160 | |
| Missing a Javadoc comment. | 163 | |
| '{' should be on the previous line. | 164 | |
| Line contains a tab character. | 165 | |
| Missing a Javadoc comment. | 168 | |
| Parameter minTime should be final. | 168 | |
| '{' should be on the previous line. | 169 | |
| Missing a Javadoc comment. | 173 | |
| Parameter minTotalTime should be final. | 173 | |
| '{' should be on the previous line. | 174 | |
| Line has trailing spaces. | 177 | |
| Missing a Javadoc comment. | 178 | |
| '{' should be on the previous line. | 179 | |
| Line contains a tab character. | 180 | |
| Missing a Javadoc comment. | 183 | |
| Parameter active should be final. | 183 | |
| '{' should be on the previous line. | 184 | |
| Line contains a tab character. | 185 | |
| Missing a Javadoc comment. | 188 | |
| Parameter active should be final. | 188 | |
| '{' should be on the previous line. | 189 | |
| Line contains a tab character. | 190 | |
| Missing a Javadoc comment. | 193 | |
| Parameter s should be final. | 195 | |
| '{' should be on the previous line. | 196 | |
| Missing a Javadoc comment. | 201 | |
| Parameter logger should be final. | 201 | |
| 'logger' hides a field. | 201 | |
| '{' should be on the previous line. | 202 |
| Violation | Message | Line |
|---|---|---|
| Missing a Javadoc comment. | 3 | |
| Utility classes should not have a public or default constructor. | 3 | |
| '{' should be on the previous line. | 4 | |
| First sentence should end with a period. | 5 | |
| Expected an @return tag. | 8 | |
| Parameter clazz should be final. | 8 | |
| Expected @param tag for 'clazz'. | 8 | |
| '{' should be on the previous line. | 9 | |
| First sentence should end with a period. | 13 | |
| Expected an @return tag. | 16 | |
| Parameter name should be final. | 16 | |
| Expected @param tag for 'name'. | 16 | |
| '{' should be on the previous line. | 17 | |
| 'if' construct must use '{}'s. | 18 |
| Violation | Message | Line |
|---|---|---|
| Missing a Javadoc comment. | 8 | |
| '{' should be on the previous line. | 9 | |
| Line is longer than 80 characters. | 10 | |
| Missing a Javadoc comment. | 10 | |
| Name 'log' must match pattern '^[A-Z][A-Z0-9]*(_[A-Z0-9]+)*$'. | 10 | |
| Line is longer than 80 characters. | 12 | |
| Method 'logSQL' is not designed for extension - needs to be abstract, final or empty. | 12 | |
| Missing a Javadoc comment. | 12 | |
| Parameter now should be final. | 12 | |
| Parameter elapsed should be final. | 12 | |
| Parameter connectionId should be final. | 12 | |
| Parameter category should be final. | 12 | |
| Parameter prepared should be final. | 12 | |
| Parameter sql should be final. | 12 | |
| '{' should be on the previous line. | 15 | |
| Line is longer than 80 characters. | 16 | |
| Avoid inline conditionals. | 16 | |
| Method 'logException' is not designed for extension - needs to be abstract, final or empty. | 22 | |
| Missing a Javadoc comment. | 22 | |
| Parameter e should be final. | 22 | |
| '{' should be on the previous line. | 23 | |
| Method 'logText' is not designed for extension - needs to be abstract, final or empty. | 27 | |
| Missing a Javadoc comment. | 27 | |
| Parameter s should be final. | 27 | |
| '{' should be on the previous line. | 28 | |
| Method 'getLastEntry' is not designed for extension - needs to be abstract, final or empty. | 32 | |
| Missing a Javadoc comment. | 32 | |
| '{' should be on the previous line. | 33 |
| Violation | Message | Line |
|---|---|---|
| Using the '.*' form of import should be avoided - javax.servlet.*. | 7 | |
| Line is longer than 80 characters. | 11 | |
| Line is longer than 80 characters. | 18 | |
| Line is longer than 80 characters. | 35 | |
| Line is longer than 80 characters. | 40 | |
| Line is longer than 80 characters. | 41 | |
| '{' should be on the previous line. | 48 | |
| Line is longer than 80 characters. | 49 | |
| Missing a Javadoc comment. | 49 | |
| Name 'log' must match pattern '^[A-Z][A-Z0-9]*(_[A-Z0-9]+)*$'. | 49 | |
| Line is longer than 80 characters. | 52 | |
| Line is longer than 80 characters. | 57 | |
| Line is longer than 80 characters. | 68 | |
| Line is longer than 80 characters. | 73 | |
| Line is longer than 80 characters. | 101 | |
| Missing a Javadoc comment. | 105 | |
| Missing a Javadoc comment. | 107 | |
| Line is longer than 80 characters. | 111 | |
| Line is longer than 80 characters. | 112 | |
| Line is longer than 80 characters. | 113 | |
| Line is longer than 80 characters. | 116 | |
| Method 'doFilter' is not designed for extension - needs to be abstract, final or empty. | 116 | |
| Parameter request should be final. | 116 | |
| Expected @param tag for 'request'. | 116 | |
| Parameter response should be final. | 116 | |
| Expected @param tag for 'response'. | 116 | |
| Parameter chain should be final. | 116 | |
| Expected @param tag for 'chain'. | 116 | |
| Expected @throws tag for 'java.io.IOException'. | 116 | |
| Expected @throws tag for 'javax.servlet.ServletException'. | 116 | |
| Line is longer than 80 characters. | 120 | |
| 'cast' is not followed by whitespace. | 123 | |
| '{' should be on the previous line. | 125 | |
| '{' should be on the previous line. | 128 | |
| Line is longer than 80 characters. | 129 | |
| '+' is not preceded with whitespace. | 129 | |
| '+' is not followed by whitespace. | 129 | |
| 'if' construct must use '{}'s. | 131 | |
| Line has trailing spaces. | 131 | |
| Line has trailing spaces. | 134 | |
| 'if' construct must use '{}'s. | 136 | |
| '{' should be on the previous line. | 141 | |
| '{' should be on the previous line. | 150 | |
| '}' should be on the same line. | 153 | |
| '{' should be on the previous line. | 155 | |
| Missing a Javadoc comment. | 161 | |
| Parameter paramValue should be final. | 161 | |
| '{' should be on the previous line. | 162 | |
| '{' should be on the previous line. | 164 | |
| '}' should be on the same line. | 166 | |
| '{' should be on the previous line. | 168 | |
| '}' should be on the same line. | 170 | |
| Line is longer than 80 characters. | 171 | |
| '{' should be on the previous line. | 174 | |
| '}' should be on the same line. | 177 | |
| '{' should be on the previous line. | 179 | |
| Missing a Javadoc comment. | 185 | |
| Parameter request should be final. | 185 | |
| '{' should be on the previous line. | 186 | |
| 'if' construct must use '{}'s. | 188 | |
| Line is longer than 80 characters. | 189 | |
| 'else' construct must use '{}'s. | 190 | |
| Method 'setFilterConfig' is not designed for extension - needs to be abstract, final or empty. | 194 | |
| Missing a Javadoc comment. | 194 | |
| Parameter filterConfig should be final. | 194 | |
| 'filterConfig' hides a field. | 194 | |
| '{' should be on the previous line. | 195 | |
| '{' should be on the previous line. | 198 | |
| Line is longer than 80 characters. | 199 | |
| '{' should be on the previous line. | 204 | |
| Line is longer than 80 characters. | 205 | |
| '}' should be on the same line. | 207 | |
| Line is longer than 80 characters. | 208 | |
| Line is longer than 80 characters. | 210 | |
| Method 'init' is not designed for extension - needs to be abstract, final or empty. | 215 | |
| Missing a Javadoc comment. | 215 | |
| Parameter filterConfig should be final. | 215 | |
| 'filterConfig' hides a field. | 215 | |
| '{' should be on the previous line. | 216 | |
| 'if' construct must use '{}'s. | 218 | |
| Missing a Javadoc comment. | 222 | |
| '{' should be on the previous line. | 223 | |
| Missing a Javadoc comment. | 227 | |
| '{' should be on the previous line. | 228 | |
| Missing a Javadoc comment. | 232 | |
| '{' should be on the previous line. | 233 | |
| Line is longer than 80 characters. | 234 | |
| Missing a Javadoc comment. | 238 | |
| Parameter minTotalTime should be final. | 238 | |
| '{' should be on the previous line. | 239 | |
| Line is longer than 80 characters. | 240 | |
| Missing a Javadoc comment. | 245 | |
| '{' should be on the previous line. | 246 | |
| Line is longer than 80 characters. | 247 |
| Violation | Message | Line |
|---|---|---|
| Missing package documentation file. | 0 |
| Violation | Message | Line |
|---|---|---|
| Utility classes should not have a public or default constructor. | 13 | |
| '{' should be on the previous line. | 14 | |
| Line is longer than 80 characters. | 17 | |
| Line is longer than 80 characters. | 30 | |
| Line is longer than 80 characters. | 34 | |
| Line is longer than 80 characters. | 38 | |
| Parameter interfaceClazz should be final. | 40 | |
| Parameter o should be final. | 40 | |
| '{' should be on the previous line. | 41 | |
| 'if' construct must use '{}'s. | 43 | |
| Line is longer than 80 characters. | 46 | |
| '{' should be on the previous line. | 48 | |
| '}' should be on the same line. | 52 | |
| '{' should be on the previous line. | 54 | |
| Line is longer than 80 characters. | 60 | |
| Expected an @return tag. | 63 | |
| Line is longer than 80 characters. | 63 | |
| Parameter target should be final. | 63 | |
| Expected @param tag for 'target'. | 63 | |
| Parameter value should be final. | 63 | |
| Expected @param tag for 'value'. | 63 | |
| Parameter args should be final. | 63 | |
| Expected @param tag for 'args'. | 63 | |
| Expected @throws tag for 'Throwable'. | 63 | |
| '{' should be on the previous line. | 67 | |
| '{' should be on the previous line. | 69 | |
| '}' should be on the same line. | 71 | |
| '{' should be on the previous line. | 73 | |
| 'if' construct must use '{}'s. | 75 | |
| 'else' construct must use '{}'s. | 77 | |
| Line is longer than 80 characters. | 82 | |
| '{' should be on the previous line. | 86 | |
| '{' should be on the previous line. | 91 | |
| Line is longer than 80 characters. | 92 | |
| Line is longer than 80 characters. | 93 | |
| '}' should be on the same line. | 95 | |
| '{' should be on the previous line. | 97 | |
| '}' should be on the same line. | 100 | |
| '{' should be on the previous line. | 102 | |
| '}' should be on the same line. | 104 | |
| '{' should be on the previous line. | 106 | |
| Expected an @return tag. | 114 | |
| Parameter method should be final. | 114 | |
| Expected @param tag for 'method'. | 114 | |
| '{' should be on the previous line. | 115 | |
| Line has trailing spaces. | 119 | |
| Line is longer than 80 characters. | 121 | |
| Line is longer than 80 characters. | 124 | |
| Expected an @return tag. | 131 | |
| Line is longer than 80 characters. | 131 | |
| Expected @param tag for 'caption'. | 131 | |
| Parameter profilable should be final. | 131 | |
| Expected @param tag for 'profilable'. | 131 | |
| Expected @throws tag for 'RuntimeException'. | 131 | |
| Redundant throws: 'RuntimeException' is unchecked exception. | 131 | |
| '{' should be on the previous line. | 135 | |
| '}' should be on the same line. | 138 | |
| '{' should be on the previous line. | 140 | |
| '}' should be on the same line. | 142 | |
| '{' should be on the previous line. | 144 | |
| '}' should be on the same line. | 146 | |
| '{' should be on the previous line. | 148 |
| Violation | Message | Line |
|---|---|---|
| Using the '.*' form of import should be avoided - java.util.*. | 3 | |
| First sentence should end with a period. | 5 | |
| '{' should be on the previous line. | 12 | |
| Missing a Javadoc comment. | 13 | |
| Variable 'children' must be private and have accessor methods. | 13 | |
| Missing a Javadoc comment. | 14 | |
| Variable 'parent' must be private and have accessor methods. | 14 | |
| Missing a Javadoc comment. | 16 | |
| Variable 'resource' must be private and have accessor methods. | 16 | |
| Missing a Javadoc comment. | 18 | |
| Variable 'startTime' must be private and have accessor methods. | 18 | |
| Missing a Javadoc comment. | 19 | |
| Variable 'totalTime' must be private and have accessor methods. | 19 | |
| Missing a Javadoc comment. | 21 | |
| Variable 'startMem' must be private and have accessor methods. | 21 | |
| Missing a Javadoc comment. | 22 | |
| Variable 'totalMem' must be private and have accessor methods. | 22 | |
| Missing a Javadoc comment. | 23 | |
| Variable 'hasMem' must be private and have accessor methods. | 23 | |
| Missing a Javadoc comment. | 25 | |
| Parameter resource should be final. | 25 | |
| 'resource' hides a field. | 25 | |
| '{' should be on the previous line. | 26 | |
| Method 'addParent' is not designed for extension - needs to be abstract, final or empty. | 30 | |
| Missing a Javadoc comment. | 30 | |
| Parameter parent should be final. | 30 | |
| 'parent' hides a field. | 30 | |
| '{' should be on the previous line. | 31 | |
| Method 'getParent' is not designed for extension - needs to be abstract, final or empty. | 35 | |
| Missing a Javadoc comment. | 35 | |
| '{' should be on the previous line. | 36 | |
| Method 'addChild' is not designed for extension - needs to be abstract, final or empty. | 41 | |
| Missing a Javadoc comment. | 41 | |
| Parameter child should be final. | 41 | |
| '{' should be on the previous line. | 42 | |
| Method 'setStartTime' is not designed for extension - needs to be abstract, final or empty. | 48 | |
| Missing a Javadoc comment. | 48 | |
| '{' should be on the previous line. | 49 | |
| Method 'setEndTime' is not designed for extension - needs to be abstract, final or empty. | 53 | |
| Missing a Javadoc comment. | 53 | |
| '{' should be on the previous line. | 54 | |
| Line has trailing spaces. | 57 | |
| Method 'setStartMem' is not designed for extension - needs to be abstract, final or empty. | 58 | |
| Missing a Javadoc comment. | 58 | |
| '{' should be on the previous line. | 59 | |
| Line is longer than 80 characters. | 60 | |
| Method 'setEndMem' is not designed for extension - needs to be abstract, final or empty. | 64 | |
| Missing a Javadoc comment. | 64 | |
| '{' should be on the previous line. | 65 | |
| Line is longer than 80 characters. | 66 | |
| Method 'getResource' is not designed for extension - needs to be abstract, final or empty. | 69 | |
| Missing a Javadoc comment. | 69 | |
| '{' should be on the previous line. | 70 | |
| Line is longer than 80 characters. | 75 | |
| Expected an @return tag. | 78 | |
| Method 'getPrintable' is not designed for extension - needs to be abstract, final or empty. | 78 | |
| Parameter minTime should be final. | 78 | |
| Expected @param tag for 'minTime'. | 78 | |
| '{' should be on the previous line. | 79 | |
| Method 'getPrintable' is not designed for extension - needs to be abstract, final or empty. | 83 | |
| Missing a Javadoc comment. | 83 | |
| Parameter indent should be final. | 83 | |
| Parameter minTime should be final. | 83 | |
| '{' should be on the previous line. | 84 | |
| '{' should be on the previous line. | 87 | |
| '{' should be on the previous line. | 92 | |
| '1024' is a magic number. | 93 | |
| Line is longer than 80 characters. | 94 | |
| '1024' is a magic number. | 94 | |
| '{' should be on the previous line. | 102 | |
| Line is longer than 80 characters. | 103 | |
| '}' should be on the same line. | 107 | |
| 'else' construct must use '{}'s. | 108 | |
| Method 'getTotalTime' is not designed for extension - needs to be abstract, final or empty. | 112 | |
| Missing a Javadoc comment. | 112 | |
| '{' should be on the previous line. | 113 |