The following document contains the results of Checkstyle
.
| Files | Infos |
Warnings |
Errors |
|---|---|---|---|
| 102 | 0 | 0 | 3542 |
| Rules | Violations | Severity |
|---|---|---|
| PackageHtml | 13 | |
| NewlineAtEndOfFile | 34 | |
| Translation | 0 | |
| JavadocMethod | 491 | |
| JavadocType | 42 | |
| JavadocVariable | 192 | |
| JavadocStyle | 59 | |
| ConstantName | 12 | |
| LocalFinalVariableName | 0 | |
| LocalVariableName | 0 | |
| MemberName | 0 | |
| MethodName | 0 | |
| PackageName | 0 | |
| ParameterName | 0 | |
| StaticVariableName | 2 | |
| TypeName | 0 | |
| AvoidStarImport | 5 | |
| IllegalImport | 0 | |
| RedundantImport | 0 | |
| UnusedImports | 6 | |
| FileLength | 0 | |
| LineLength | 722 | |
| MethodLength | 1 | |
| ParameterNumber | 0 | |
| EmptyForIteratorPad | 0 | |
| MethodParamPad | 3 | |
| NoWhitespaceAfter | 4 | |
| NoWhitespaceBefore | 0 | |
| OperatorWrap | 11 | |
| ParenPad | 1 | |
| TypecastParenPad | 0 | |
| TabCharacter | 128 | |
| WhitespaceAfter | 13 | |
| WhitespaceAround | 25 | |
| ModifierOrder | 3 | |
| RedundantModifier | 37 | |
| AvoidNestedBlocks | 0 | |
| EmptyBlock | 0 | |
| LeftCurly | 762 | |
| NeedBraces | 66 | |
| RightCurly | 84 | |
| AvoidInlineConditionals | 23 | |
| DoubleCheckedLocking | 0 | |
| EmptyStatement | 0 | |
| EqualsHashCode | 0 | |
| HiddenField | 88 | |
| IllegalInstantiation | 0 | |
| InnerAssignment | 2 | |
| MagicNumber | 58 | |
| MissingSwitchDefault | 3 | |
| RedundantThrows | 4 | |
| SimplifyBooleanExpression | 0 | |
| SimplifyBooleanReturn | 1 | |
| DesignForExtension | 146 | |
| FinalClass | 2 | |
| HideUtilityClassConstructor | 9 | |
| InterfaceIsType | 0 | |
| VisibilityModifier | 25 | |
| ArrayTypeStyle | 1 | |
| FinalParameters | 400 | |
GenericIllegalRegexp
|
59 | |
| TodoComment | 1 | |
| UpperEll | 0 |
| Violation | Message | Line |
|---|---|---|
| Using the '.*' form of import should be avoided - java.util.*. | 17 | |
| Line is longer than 80 characters. | 23 | |
| Line contains a tab character. | 29 | |
| Line contains a tab character. | 30 | |
| Line contains a tab character. | 31 | |
| Line contains a tab character. | 32 | |
| Line contains a tab character. | 33 | |
| Line contains a tab character. | 34 | |
| Line contains a tab character. | 35 | |
| Line contains a tab character. | 36 | |
| '{' should be on the previous line. | 42 | |
| Missing a Javadoc comment. | 43 | |
| Name 'log' must match pattern '^[A-Z][A-Z0-9]*(_[A-Z0-9]+)*$'. | 43 | |
| Missing a Javadoc comment. | 44 | |
| Name 'CONFIG_FILE_PARAM_KEY' must match pattern '^[a-z][a-zA-Z0-9]*$'. | 44 | |
| Variable 'CONFIG_FILE_PARAM_KEY' must be private and have accessor methods. | 44 | |
| Missing a Javadoc comment. | 46 | |
| Variable 'configFileLocation' must be private and have accessor methods. | 46 | |
| Line is longer than 80 characters. | 49 | |
| Missing a Javadoc comment. | 49 | |
| '10' is a magic number. | 49 | |
| '10' is a magic number. | 49 | |
| Missing a Javadoc comment. | 52 | |
| First sentence should end with a period. | 55 | |
| Method 'init' is not designed for extension - needs to be abstract, final or empty. | 58 | |
| Parameter params should be final. | 58 | |
| Expected @param tag for 'params'. | 58 | |
| Parameter config should be final. | 58 | |
| Expected @param tag for 'config'. | 58 | |
| '{' should be on the previous line. | 59 | |
| '{' should be on the previous line. | 61 | |
| '{' should be on the previous line. | 63 | |
| '}' should be on the same line. | 68 | |
| '{' should be on the previous line. | 70 | |
| First sentence should end with a period. | 76 | |
| '{' should be on the previous line. | 80 | |
| '{' should be on the previous line. | 82 | |
| Line is longer than 80 characters. | 83 | |
| Line is longer than 80 characters. | 84 | |
| 'if' construct must use '{}'s. | 86 | |
| Line is longer than 80 characters. | 87 | |
| 'if' construct must use '{}'s. | 90 | |
| Line is longer than 80 characters. | 94 | |
| '{' should be on the previous line. | 102 | |
| Line is longer than 80 characters. | 107 | |
| '{' should be on the previous line. | 110 | |
| '}' should be on the same line. | 116 | |
| '{' should be on the previous line. | 118 | |
| Method 'parseRoles' is not designed for extension - needs to be abstract, final or empty. | 123 | |
| Missing a Javadoc comment. | 123 | |
| Parameter roleNames should be final. | 123 | |
| Missing a Javadoc comment. | 134 | |
| '{' should be on the previous line. | 135 | |
| Method 'getRequiredRoles' is not designed for extension - needs to be abstract, final or empty. | 138 | |
| Missing a Javadoc comment. | 138 | |
| Parameter request should be final. | 138 | |
| '{' should be on the previous line. | 139 | |
| Method 'getRequiredRoles' is not designed for extension - needs to be abstract, final or empty. | 144 | |
| Missing a Javadoc comment. | 144 | |
| Parameter servletPath should be final. | 144 | |
| '{' should be on the previous line. | 145 | |
| 'if' construct must use '{}'s. | 151 | |
| Line is longer than 80 characters. | 151 | |
| '+' is not preceded with whitespace. | 151 | |
| '+' is not followed by whitespace. | 151 | |
| Line is longer than 80 characters. | 152 | |
| '{' should be on the previous line. | 158 | |
| Line is longer than 80 characters. | 159 |
| Violation | Message | Line |
|---|---|---|
| Line is longer than 80 characters. | 8 | |
| Line is longer than 80 characters. | 10 | |
| '{' should be on the previous line. | 13 | |
| Line is longer than 80 characters. | 14 | |
| Missing a Javadoc comment. | 14 | |
| Line is longer than 80 characters. | 16 | |
| Missing a Javadoc comment. | 16 |
| Violation | Message | Line |
|---|---|---|
| File does not end with a newline. | 0 | |
| Line is longer than 80 characters. | 4 | |
| '{' should be on the previous line. | 8 | |
| Line has trailing spaces. | 11 | |
| Expected an @return tag. | 14 | |
| Expected @param tag for 'ipAddress'. | 14 | |
| Redundant throws: 'IPAddressFormatException' is unchecked exception. | 14 |
| Violation | Message | Line |
|---|---|---|
| Missing package documentation file. | 0 |
| Violation | Message | Line |
|---|---|---|
| Line is longer than 80 characters. | 3 | |
| First sentence should end with a period. | 5 | |
| Line is longer than 80 characters. | 6 | |
| '{' should be on the previous line. | 9 | |
| Line is longer than 80 characters. | 10 | |
| Method 'getApplicationCertificate' is not designed for extension - needs to be abstract, final or empty. | 10 | |
| Missing a Javadoc comment. | 10 | |
| Parameter baseUrl should be final. | 10 | |
| Method 'generateUID' is not designed for extension - needs to be abstract, final or empty. | 16 | |
| Missing a Javadoc comment. | 16 | |
| '{' should be on the previous line. | 17 |
| Violation | Message | Line |
|---|---|---|
| Missing a Javadoc comment. | 10 | |
| '{' should be on the previous line. | 11 | |
| Line is longer than 80 characters. | 12 | |
| Missing a Javadoc comment. | 12 | |
| Name 'log' must match pattern '^[A-Z][A-Z0-9]*(_[A-Z0-9]+)*$'. | 12 | |
| Line is longer than 80 characters. | 14 | |
| Method 'invalidateCookie' is not designed for extension - needs to be abstract, final or empty. | 14 | |
| Missing a Javadoc comment. | 14 | |
| Parameter request should be final. | 14 | |
| Parameter response should be final. | 14 | |
| Parameter cookieName should be final. | 14 | |
| Parameter path should be final. | 15 | |
| '{' should be on the previous line. | 16 | |
| '{' should be on the previous line. | 18 | |
| Method 'getCookie' is not designed for extension - needs to be abstract, final or empty. | 24 | |
| Missing a Javadoc comment. | 24 | |
| Parameter request should be final. | 24 | |
| Parameter name should be final. | 24 | |
| '{' should be on the previous line. | 25 | |
| Array brackets at illegal position. | 30 | |
| '{' should be on the previous line. | 32 | |
| '{' should be on the previous line. | 34 | |
| '{' should be on the previous line. | 43 | |
| '{' should be on the previous line. | 45 | |
| '{' should be on the previous line. | 50 | |
| Line is longer than 80 characters. | 56 | |
| Method 'setCookie' is not designed for extension - needs to be abstract, final or empty. | 56 | |
| Missing a Javadoc comment. | 56 | |
| Parameter request should be final. | 56 | |
| Parameter response should be final. | 56 | |
| Parameter name should be final. | 56 | |
| Parameter value should be final. | 56 | |
| Parameter maxAge should be final. | 57 | |
| Parameter path should be final. | 57 | |
| '{' should be on the previous line. | 58 | |
| Line is longer than 80 characters. | 59 | |
| '{' should be on the previous line. | 61 | |
| Line is longer than 80 characters. | 62 | |
| Avoid inline conditionals. | 62 | |
| '{' should be on the previous line. | 68 | |
| Method 'getCookieValue' is not designed for extension - needs to be abstract, final or empty. | 77 | |
| Missing a Javadoc comment. | 77 | |
| Parameter request should be final. | 77 | |
| Parameter name should be final. | 77 | |
| '{' should be on the previous line. | 78 | |
| '{' should be on the previous line. | 81 |
| Violation | Message | Line |
|---|---|---|
| File does not end with a newline. | 0 | |
| Missing a Javadoc comment. | 3 | |
| '{' should be on the previous line. | 4 | |
| Missing a Javadoc comment. | 5 | |
| Line has trailing spaces. | 6 | |
| Missing a Javadoc comment. | 7 | |
| Parameter error should be final. | 7 | |
| 'error' hides a field. | 7 | |
| '{' should be on the previous line. | 8 | |
| Missing a Javadoc comment. | 14 | |
| Parameter error should be final. | 14 | |
| 'error' hides a field. | 14 | |
| Parameter exception should be final. | 14 | |
| '{' should be on the previous line. | 15 | |
| Missing a Javadoc comment. | 21 | |
| '{' should be on the previous line. | 22 |
| Violation | Message | Line |
|---|---|---|
| File does not end with a newline. | 0 | |
| Missing a Javadoc comment. | 3 | |
| '{' should be on the previous line. | 4 | |
| Line is longer than 80 characters. | 5 | |
| Missing a Javadoc comment. | 5 | |
| Parameter error should be final. | 5 | |
| Parameter cause should be final. | 5 | |
| Missing a Javadoc comment. | 10 | |
| Parameter error should be final. | 10 | |
| '{' should be on the previous line. | 11 | |
| Missing a Javadoc comment. | 15 | |
| Parameter exception should be final. | 15 | |
| '{' should be on the previous line. | 16 | |
| Method 'getMessage' is not designed for extension - needs to be abstract, final or empty. | 20 | |
| Missing a Javadoc comment. | 20 | |
| '{' should be on the previous line. | 21 | |
| '{' should be on the previous line. | 24 |
| Violation | Message | Line |
|---|---|---|
| Line is longer than 80 characters. | 8 | |
| Line is longer than 80 characters. | 10 | |
| '{' should be on the previous line. | 14 | |
| Missing a Javadoc comment. | 15 | |
| Redundant 'public' modifier. | 15 | |
| Missing a Javadoc comment. | 17 | |
| Redundant 'public' modifier. | 17 |
| Violation | Message | Line |
|---|---|---|
| File does not end with a newline. | 0 | |
| First sentence should end with a period. | 5 | |
| Line is longer than 80 characters. | 6 | |
| Missing a Javadoc comment. | 10 | |
| Parameter code should be final. | 10 | |
| Parameter address should be final. | 10 | |
| '{' should be on the previous line. | 11 | |
| Line is longer than 80 characters. | 12 |
| Violation | Message | Line |
|---|---|---|
| Utility classes should not have a public or default constructor. | 10 | |
| '{' should be on the previous line. | 11 | |
| Line is longer than 80 characters. | 13 | |
| Expected an @return tag. | 16 | |
| Parameter parent should be final. | 16 | |
| Expected @param tag for 'parent'. | 16 | |
| Parameter childTagName should be final. | 16 | |
| Expected @param tag for 'childTagName'. | 16 | |
| '{' should be on the previous line. | 17 | |
| '{' should be on the previous line. | 19 | |
| Line is longer than 80 characters. | 20 | |
| '}' should be on the same line. | 23 | |
| '{' should be on the previous line. | 25 |
| Violation | Message | Line |
|---|---|---|
| Line is longer than 80 characters. | 10 | |
| '{' should be on the previous line. | 13 | |
| Line is longer than 80 characters. | 14 | |
| Missing a Javadoc comment. | 14 | |
| Redundant 'public' modifier. | 14 | |
| Missing a Javadoc comment. | 16 | |
| Redundant 'public' modifier. | 16 | |
| Missing a Javadoc comment. | 18 | |
| Redundant 'public' modifier. | 18 | |
| Missing a Javadoc comment. | 20 | |
| Redundant 'public' modifier. | 20 | |
| Line is longer than 80 characters. | 22 | |
| Missing a Javadoc comment. | 22 | |
| Redundant 'public' modifier. | 22 | |
| Expected an @return tag. | 25 | |
| Redundant 'public' modifier. | 25 | |
| Expected @param tag for 'request'. | 25 | |
| Expected @param tag for 'role'. | 25 | |
| Line is longer than 80 characters. | 27 | |
| Missing a Javadoc comment. | 27 | |
| Redundant 'public' modifier. | 27 | |
| Line is longer than 80 characters. | 29 | |
| Missing a Javadoc comment. | 29 | |
| Redundant 'public' modifier. | 29 | |
| Line is longer than 80 characters. | 31 | |
| Missing a Javadoc comment. | 31 | |
| Redundant 'public' modifier. | 31 |
| Violation | Message | Line |
|---|---|---|
| Missing package documentation file. | 0 |
| Violation | Message | Line |
|---|---|---|
| File does not end with a newline. | 0 | |
| Line is longer than 80 characters. | 4 | |
| Line is longer than 80 characters. | 5 | |
| Line is longer than 80 characters. | 6 | |
| '{' should be on the previous line. | 10 | |
| First sentence should end with a period. | 11 | |
| Expected an @return tag. | 14 | |
| Line is longer than 80 characters. | 19 | |
| Expected an @return tag. | 21 | |
| First sentence should end with a period. | 23 | |
| Expected an @return tag. | 28 | |
| Expected an @return tag. | 35 | |
| Line is longer than 80 characters. | 40 | |
| Expected an @return tag. | 42 |
| Violation | Message | Line |
|---|---|---|
| File does not end with a newline. | 0 | |
| '{' should be on the previous line. | 7 | |
| First sentence should end with a period. | 8 | |
| Expected an @return tag. | 11 | |
| Expected @param tag for 'urlPath'. | 11 |
| Violation | Message | Line |
|---|---|---|
| First sentence should end with a period. | 6 | |
| Line is longer than 80 characters. | 7 | |
| '{' should be on the previous line. | 10 | |
| Line contains a tab character. | 11 | |
| Line is longer than 80 characters. | 12 | |
| Line contains a tab character. | 12 | |
| Line has trailing spaces. | 13 | |
| Line contains a tab character. | 13 | |
| Line has trailing spaces. | 14 | |
| Line contains a tab character. | 14 | |
| Line contains a tab character. | 15 | |
| Line contains a tab character. | 16 | |
| Line contains a tab character. | 17 | |
| Redundant 'public' modifier. | 17 |
| Violation | Message | Line |
|---|---|---|
| Missing package documentation file. | 0 |
| Violation | Message | Line |
|---|---|---|
| Line is longer than 80 characters. | 3 | |
| Line is longer than 80 characters. | 13 | |
| Line is longer than 80 characters. | 14 | |
| Line is longer than 80 characters. | 17 | |
| '{' should be on the previous line. | 20 | |
| Line is longer than 80 characters. | 22 | |
| Comment matches to-do format 'TODO:'. | 23 | |
| Line is longer than 80 characters. | 23 | |
| Line has trailing spaces. | 24 | |
| Line is longer than 80 characters. | 29 | |
| Unused @throws tag for 'RuntimeException'. | 31 | |
| Line is longer than 80 characters. | 32 | |
| Line is longer than 80 characters. | 34 | |
| First sentence should end with a period. | 36 | |
| Line has trailing spaces. | 38 | |
| Line is longer than 80 characters. | 45 | |
| Line is longer than 80 characters. | 48 | |
| Line is longer than 80 characters. | 49 | |
| Line has trailing spaces. | 50 | |
| First sentence should end with a period. | 55 | |
| Line has trailing spaces. | 57 | |
| Line is longer than 80 characters. | 68 | |
| First sentence should end with a period. | 70 | |
| Line is longer than 80 characters. | 71 | |
| Line has trailing spaces. | 72 | |
| Expected an @return tag. | 81 | |
| Line is longer than 80 characters. | 81 | |
| First sentence should end with a period. | 83 | |
| Line has trailing spaces. | 85 | |
| Line is longer than 80 characters. | 96 | |
| First sentence should end with a period. | 98 | |
| Line has trailing spaces. | 100 | |
| Line is longer than 80 characters. | 111 |
| Violation | Message | Line |
|---|---|---|
| Missing package documentation file. | 0 |
| Violation | Message | Line |
|---|---|---|
| Unused import - javax.servlet.http.HttpSession. | 12 | |
| Line is longer than 80 characters. | 18 | |
| Line is longer than 80 characters. | 21 | |
| Unclosed HTML tag found: <code>, this servlet | 21 | |
| Line is longer than 80 characters. | 22 | |
| Line is longer than 80 characters. | 23 | |
| '{' should be on the previous line. | 28 | |
| Missing a Javadoc comment. | 29 | |
| Method 'init' is not designed for extension - needs to be abstract, final or empty. | 31 | |
| Missing a Javadoc comment. | 31 | |
| '{' should be on the previous line. | 32 | |
| Method 'init' is not designed for extension - needs to be abstract, final or empty. | 37 | |
| Missing a Javadoc comment. | 37 | |
| Parameter servletConfig should be final. | 37 | |
| '{' should be on the previous line. | 38 | |
| Line is longer than 80 characters. | 40 | |
| Line is longer than 80 characters. | 43 | |
| Method 'service' is not designed for extension - needs to be abstract, final or empty. | 43 | |
| Missing a Javadoc comment. | 43 | |
| Parameter request should be final. | 43 | |
| Parameter response should be final. | 43 | |
| '{' should be on the previous line. | 46 | |
| Line is longer than 80 characters. | 47 | |
| Line is longer than 80 characters. | 48 | |
| '}' should be on the same line. | 49 | |
| '{' should be on the previous line. | 51 | |
| Line is longer than 80 characters. | 52 | |
| '{' should be on the previous line. | 54 | |
| '}' should be on the same line. | 57 | |
| '{' should be on the previous line. | 59 | |
| Line is longer than 80 characters. | 60 | |
| Missing a Javadoc comment. | 66 | |
| '{' should be on the previous line. | 67 | |
| Method 'getSecurityConfig' is not designed for extension - needs to be abstract, final or empty. | 71 | |
| Missing a Javadoc comment. | 71 | |
| Method 'getAuthenticator' is not designed for extension - needs to be abstract, final or empty. | 75 | |
| Missing a Javadoc comment. | 75 |
| Violation | Message | Line |
|---|---|---|
| First sentence should end with a period. | 3 | |
| '{' should be on the previous line. | 7 | |
| Missing a Javadoc comment. | 8 |
| Violation | Message | Line |
|---|---|---|
| Utility classes should not have a public or default constructor. | 14 | |
| '{' should be on the previous line. | 15 | |
| Line is longer than 80 characters. | 17 | |
| Line is longer than 80 characters. | 19 | |
| Line is longer than 80 characters. | 20 | |
| Line is longer than 80 characters. | 23 | |
| Line is longer than 80 characters. | 24 | |
| Line is longer than 80 characters. | 25 | |
| Line is longer than 80 characters. | 27 | |
| Line is longer than 80 characters. | 28 | |
| Line is longer than 80 characters. | 29 | |
| Line is longer than 80 characters. | 31 | |
| Line is longer than 80 characters. | 35 | |
| Parameter request should be final. | 37 | |
| '{' should be on the previous line. | 38 | |
| Line is longer than 80 characters. | 45 | |
| Line is longer than 80 characters. | 46 | |
| Line is longer than 80 characters. | 47 | |
| Line is longer than 80 characters. | 48 | |
| Line is longer than 80 characters. | 50 | |
| Expected an @return tag. | 52 | |
| Parameter request should be final. | 52 | |
| Expected @param tag for 'request'. | 52 | |
| '{' should be on the previous line. | 53 | |
| Line is longer than 80 characters. | 60 | |
| Missing a Javadoc comment. | 60 | |
| Parameter loginURL should be final. | 60 | |
| Parameter request should be final. | 60 | |
| '{' should be on the previous line. | 65 | |
| '}' should be on the same line. | 67 | |
| '{' should be on the previous line. | 69 | |
| Missing a Javadoc comment. | 74 | |
| Parameter loginURL should be final. | 74 | |
| '{' should be on the previous line. | 75 | |
| Expected an @return tag. | 82 | |
| Line is longer than 80 characters. | 82 | |
| Expected @param tag for 'loginURL'. | 82 | |
| Expected @param tag for 'request'. | 82 | |
| Parameter external should be final. | 82 | |
| Expected @param tag for 'external'. | 82 | |
| '{' should be on the previous line. | 86 | |
| Line is longer than 80 characters. | 88 | |
| Line is longer than 80 characters. | 89 | |
| Avoid inline conditionals. | 89 | |
| ')' is preceded with whitespace. | 89 | |
| '+' is not preceded with whitespace. | 89 | |
| '}' should be on the same line. | 90 | |
| 'else' construct must use '{}'s. | 91 | |
| Expected an @return tag. | 98 | |
| Line is longer than 80 characters. | 98 | |
| Parameter request should be final. | 98 | |
| Expected @param tag for 'request'. | 98 | |
| Parameter external should be final. | 98 | |
| Expected @param tag for 'external'. | 98 | |
| Line is longer than 80 characters. | 100 | |
| '{' should be on the previous line. | 102 | |
| 'if' construct must use '{}'s. | 103 | |
| 'else' construct must use '{}'s. | 105 | |
| 'if' construct must use '{}'s. | 109 | |
| Line is longer than 80 characters. | 110 | |
| Avoid inline conditionals. | 110 | |
| 'else' construct must use '{}'s. | 111 | |
| '+' should be on a new line. | 112 | |
| Avoid inline conditionals. | 113 | |
| '+' should be on a new line. | 113 | |
| Line is longer than 80 characters. | 114 | |
| Avoid inline conditionals. | 114 | |
| Expected an @return tag. | 121 | |
| Parameter request should be final. | 121 | |
| Expected @param tag for 'request'. | 121 | |
| '{' should be on the previous line. | 122 | |
| Line is longer than 80 characters. | 127 | |
| '!' is followed by whitespace. | 127 | |
| '80' is a magic number. | 127 | |
| '443' is a magic number. | 127 | |
| First sentence should end with a period. | 135 | |
| Line is longer than 80 characters. | 136 | |
| Expected an @return tag. | 138 | |
| Line is longer than 80 characters. | 138 | |
| Parameter request should be final. | 138 | |
| Expected @param tag for 'request'. | 138 | |
| Parameter basicAuth should be final. | 138 | |
| Expected @param tag for 'basicAuth'. | 138 | |
| 'if' construct must use '{}'s. | 141 | |
| 'if' construct must use '{}'s. | 145 | |
| Conditiona |