Checkstyle Results
The following document contains the results of Checkstyle. 
Files
Rules
| Rules | Violations | Severity |
|---|---|---|
JavadocPackage
| 0 | |
| NewlineAtEndOfFile | 0 | |
| Translation | 0 | |
| FileLength | 0 | |
FileTabCharacter
| 0 | |
RegexpSingleline
| 0 | |
| JavadocMethod | 16 | |
| JavadocType | 0 | |
| JavadocVariable | 3 | |
| JavadocStyle | 2 | |
| 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 | |
| LineLength | 41 | |
| MethodLength | 0 | |
| ParameterNumber | 0 | |
| EmptyForIteratorPad | 0 | |
| MethodParamPad | 0 | |
| NoWhitespaceAfter | 2 | |
| NoWhitespaceBefore | 0 | |
| OperatorWrap | 9 | |
| ParenPad | 22 | |
| TypecastParenPad | 0 | |
| WhitespaceAfter | 2 | |
| WhitespaceAround | 25 | |
| ModifierOrder | 0 | |
| RedundantModifier | 0 | |
| AvoidNestedBlocks | 0 | |
| EmptyBlock | 0 | |
| LeftCurly | 0 | |
| NeedBraces | 17 | |
| RightCurly | 0 | |
| AvoidInlineConditionals | 1 | |
| DoubleCheckedLocking | 0 | |
| EmptyStatement | 0 | |
| EqualsHashCode | 0 | |
| HiddenField | 5 | |
| IllegalInstantiation | 0 | |
| InnerAssignment | 0 | |
| MagicNumber | 1 | |
| MissingSwitchDefault | 0 | |
| RedundantThrows | 0 | |
| SimplifyBooleanExpression | 0 | |
| SimplifyBooleanReturn | 0 | |
| DesignForExtension | 19 | |
| FinalClass | 1 | |
| HideUtilityClassConstructor | 0 | |
| InterfaceIsType | 0 | |
| VisibilityModifier | 1 | |
| ArrayTypeStyle | 0 | |
| FinalParameters | 25 | |
| TodoComment | 1 | |
| UpperEll | 0 |
Details
org/apache/any23/cli/Crawler.java
| Violation | Message | Line |
|---|---|---|
| Missing a Javadoc comment. | 44 | |
| Missing a Javadoc comment. | 46 | |
| Parameter args should be final. | 46 | |
| '(' is followed by whitespace. | 48 | |
| ')' is preceded with whitespace. | 48 | |
| Method 'run' is not designed for extension - needs to be abstract, final or empty. | 54 | |
| Parameter args should be final. | 55 | |
| 'if' construct must use '{}'s. | 58 | |
| Line is longer than 80 characters. | 58 | |
| 'if' is not followed by whitespace. | 58 | |
| Line is longer than 80 characters. | 63 | |
| '(' is followed by whitespace. | 63 | |
| ')' is preceded with whitespace. | 63 | |
| Line is longer than 80 characters. | 66 | |
| Avoid inline conditionals. | 66 | |
| 'if' is not followed by whitespace. | 68 | |
| Line is longer than 80 characters. | 69 | |
| '(' is followed by whitespace. | 69 | |
| ')' is preceded with whitespace. | 69 | |
| 'if' is not followed by whitespace. | 71 | |
| 'if' is not followed by whitespace. | 74 | |
| Line is longer than 80 characters. | 78 | |
| 'if' construct must use '{}'s. | 79 | |
| Line is longer than 80 characters. | 79 | |
| 'if' is not followed by whitespace. | 79 | |
| Parameter page should be final. | 84 | |
| Line is longer than 80 characters. | 86 | |
| '(' is followed by whitespace. | 86 | |
| ')' is preceded with whitespace. | 86 | |
| Line is longer than 80 characters. | 99 | |
| '(' is followed by whitespace. | 105 | |
| '(' is followed by whitespace. | 109 | |
| ')' is preceded with whitespace. | 109 | |
| Comment matches to-do format 'TODO:'. | 110 | |
| 'if' construct must use '{}'s. | 119 | |
| 'if' is not followed by whitespace. | 119 | |
| 'if' is not followed by whitespace. | 120 | |
| Method 'createOptions' is not designed for extension - needs to be abstract, final or empty. | 127 | |
| Missing a Javadoc comment. | 134 | |
| Parameter options should be final. | 134 | |
| Line is longer than 80 characters. | 136 | |
| Line is longer than 80 characters. | 139 | |
| Line is longer than 80 characters. | 142 | |
| Line is longer than 80 characters. | 145 | |
| Line is longer than 80 characters. | 148 | |
| Line is longer than 80 characters. | 151 | |
| Missing a Javadoc comment. | 155 | |
| Parameter commandLine should be final. | 155 | |
| 'if' is not followed by whitespace. | 156 | |
| Line is longer than 80 characters. | 158 | |
| '(' is followed by whitespace. | 158 | |
| ')' is preceded with whitespace. | 158 | |
| Line is longer than 80 characters. | 160 | |
| '6' is a magic number. | 160 | |
| Missing a Javadoc comment. | 166 | |
| Parameter commandLine should be final. | 166 | |
| 'if' is not followed by whitespace. | 167 | |
| Line is longer than 80 characters. | 168 | |
| '(' is followed by whitespace. | 168 | |
| ')' is preceded with whitespace. | 168 | |
| 'if' construct must use '{}'s. | 169 | |
| 'if' is not followed by whitespace. | 169 | |
| Line is longer than 80 characters. | 170 | |
| Line is longer than 80 characters. | 173 | |
| Missing a Javadoc comment. | 179 | |
| Parameter cl should be final. | 179 | |
| Parameter option should be final. | 179 | |
| Line is longer than 80 characters. | 184 |
org/apache/any23/cli/package-info.java
| Violation | Message | Line |
|---|---|---|
| First sentence should end with a period. | 18 |
org/apache/any23/plugin/crawler/DefaultWebCrawler.java
| Violation | Message | Line |
|---|---|---|
| Line is longer than 80 characters. | 35 | |
| Missing a Javadoc comment. | 35 | |
| Name 'logger' must match pattern '^[A-Z][A-Z0-9]*(_[A-Z0-9]+)*$'. | 35 | |
| Line is longer than 80 characters. | 48 | |
| Expected an @return tag. | 50 | |
| Method 'shouldVisit' is not designed for extension - needs to be abstract, final or empty. | 50 | |
| Parameter url should be final. | 51 | |
| Expected @param tag for 'url'. | 51 | |
| 'if' construct must use '{}'s. | 52 | |
| 'if' construct must use '{}'s. | 54 | |
| 'if' is not followed by whitespace. | 54 | |
| '(' is followed by whitespace. | 54 | |
| '!' is followed by whitespace. | 54 | |
| '(' is followed by whitespace. | 54 | |
| ')' is preceded with whitespace. | 54 | |
| ')' is preceded with whitespace. | 54 | |
| '!' is followed by whitespace. | 55 | |
| Method 'visit' is not designed for extension - needs to be abstract, final or empty. | 61 | |
| Parameter page should be final. | 62 | |
| Expected @param tag for 'page'. | 62 |
org/apache/any23/plugin/crawler/SharedData.java
| Violation | Message | Line |
|---|---|---|
| Class SharedData should be declared as final. | 32 | |
| 'if' construct must use '{}'s. | 63 | |
| Line is longer than 80 characters. | 63 | |
| 'if' is not followed by whitespace. | 63 | |
| Line is longer than 80 characters. | 74 | |
| Parameter seed should be final. | 74 | |
| Parameter regex should be final. | 74 | |
| Parameter listeners should be final. | 74 | |
| Line is longer than 80 characters. | 85 | |
| Parameter seed should be final. | 85 | |
| 'seed' hides a field. | 85 | |
| Expected @param tag for 'seed'. | 85 | |
| Parameter pattern should be final. | 85 | |
| 'pattern' hides a field. | 85 | |
| Expected @param tag for 'pattern'. | 85 | |
| Parameter listeners should be final. | 85 | |
| 'listeners' hides a field. | 85 | |
| Expected @param tag for 'listeners'. | 85 | |
| 'if' construct must use '{}'s. | 86 | |
| 'if' is not followed by whitespace. | 86 | |
| Parameter page should be final. | 115 | |
| 'for' is not followed by whitespace. | 116 |
org/apache/any23/plugin/crawler/SiteCrawler.java
| Violation | Message | Line |
|---|---|---|
| Missing a Javadoc comment. | 40 | |
| '+' should be on a new line. | 41 | |
| '+' should be on a new line. | 42 | |
| '+' should be on a new line. | 43 | |
| '+' should be on a new line. | 44 | |
| '+' should be on a new line. | 45 | |
| '+' should be on a new line. | 46 | |
| '+' should be on a new line. | 47 | |
| '+' should be on a new line. | 48 | |
| '+' should be on a new line. | 49 | |
| Line is longer than 80 characters. | 60 | |
| Line is longer than 80 characters. | 65 | |
| Variable 'defaultFilters' must be private and have accessor methods. | 65 | |
| Line is longer than 80 characters. | 75 | |
| Line is longer than 80 characters. | 110 | |
| Parameter storageFolder should be final. | 112 | |
| '(' is followed by whitespace. | 114 | |
| ')' is preceded with whitespace. | 114 | |
| Line is longer than 80 characters. | 116 | |
| Method 'getNumOfCrawlers' is not designed for extension - needs to be abstract, final or empty. | 123 | |
| Method 'setNumOfCrawlers' is not designed for extension - needs to be abstract, final or empty. | 132 | |
| Parameter n should be final. | 132 | |
| 'if' construct must use '{}'s. | 133 | |
| Line is longer than 80 characters. | 133 | |
| 'if' is not followed by whitespace. | 133 | |
| '<=' is not followed by whitespace. | 133 | |
| Method 'getWebCrawler' is not designed for extension - needs to be abstract, final or empty. | 137 | |
| Missing a Javadoc comment. | 137 | |
| Method 'setWebCrawler' is not designed for extension - needs to be abstract, final or empty. | 146 | |
| Parameter c should be final. | 146 | |
| 'if' construct must use '{}'s. | 147 | |
| 'if' is not followed by whitespace. | 147 | |
| Method 'getMaxDepth' is not designed for extension - needs to be abstract, final or empty. | 154 | |
| Method 'setMaxDepth' is not designed for extension - needs to be abstract, final or empty. | 163 | |
| Parameter maxDepth should be final. | 163 | |
| 'maxDepth' hides a field. | 163 | |
| 'if' construct must use '{}'s. | 164 | |
| Line is longer than 80 characters. | 164 | |
| 'if' is not followed by whitespace. | 164 | |
| 'if' construct must use '{}'s. | 165 | |
| 'if' is not followed by whitespace. | 165 | |
| Line is longer than 80 characters. | 168 | |
| Method 'getMaxPages' is not designed for extension - needs to be abstract, final or empty. | 176 | |
| Method 'setMaxPages' is not designed for extension - needs to be abstract, final or empty. | 185 | |
| Parameter maxPages should be final. | 185 | |
| 'maxPages' hides a field. | 185 | |
| 'if' construct must use '{}'s. | 186 | |
| Line is longer than 80 characters. | 186 | |
| 'if' is not followed by whitespace. | 186 | |
| 'if' construct must use '{}'s. | 187 | |
| 'if' is not followed by whitespace. | 187 | |
| Method 'getPolitenessDelay' is not designed for extension - needs to be abstract, final or empty. | 194 | |
| Method 'setPolitenessDelay' is not designed for extension - needs to be abstract, final or empty. | 203 | |
| Parameter millis should be final. | 203 | |
| 'if' construct must use '{}'s. | 204 | |
| Line is longer than 80 characters. | 204 | |
| 'if' is not followed by whitespace. | 204 | |
| 'if' construct must use '{}'s. | 205 | |
| 'if' is not followed by whitespace. | 205 | |
| Method 'addListener' is not designed for extension - needs to be abstract, final or empty. | 214 | |
| Parameter listener should be final. | 214 | |
| Expected @param tag for 'listener'. | 214 | |
| Method 'removeListener' is not designed for extension - needs to be abstract, final or empty. | 223 | |
| Parameter listener should be final. | 223 | |
| Expected @param tag for 'listener'. | 223 | |
| Line is longer than 80 characters. | 231 | |
| Line is longer than 80 characters. | 232 | |
| Method 'start' is not designed for extension - needs to be abstract, final or empty. | 235 | |
| Expected @throws tag for 'Exception'. | 237 | |
| Line is longer than 80 characters. | 238 | |
| ')' is preceded with whitespace. | 238 | |
| 'if' is not followed by whitespace. | 246 | |
| 'if' construct must use '{}'s. | 249 | |
| Line is longer than 80 characters. | 249 | |
| 'if' is not followed by whitespace. | 249 | |
| Line is longer than 80 characters. | 259 | |
| Method 'start' is not designed for extension - needs to be abstract, final or empty. | 262 | |
| Expected @throws tag for 'Exception'. | 262 | |
| Line is longer than 80 characters. | 267 | |
| Method 'stop' is not designed for extension - needs to be abstract, final or empty. | 269 |
org/apache/any23/plugin/crawler/package-info.java
| Violation | Message | Line |
|---|---|---|
| First sentence should end with a period. | 18 |