The following document contains the results of Checkstyle 7.2 with sun_checks.xml ruleset.
Category | Rule | Violations | Severity |
---|---|---|---|
blocks | LeftCurly | 1 | ![]() |
coding | AvoidInlineConditionals | 1 | ![]() |
HiddenField | 4 | ![]() | |
MagicNumber | 1 | ![]() | |
SimplifyBooleanExpression | 1 | ![]() | |
design | DesignForExtension | 20 | ![]() |
javadoc | JavadocMethod | 9 | ![]() |
JavadocPackage | 2 | ![]() | |
JavadocStyle | 3 | ![]() | |
JavadocType | 1 | ![]() | |
JavadocVariable | 16 | ![]() | |
misc | FinalParameters | 16 | ![]() |
regexp | RegexpSingleline
|
3 | ![]() |
sizes | LineLength | 12 | ![]() |
whitespace | ParenPad | 6 | ![]() |
WhitespaceAfter | 1 | ![]() |
Severity | Category | Rule | Message | Line |
---|---|---|---|---|
![]() |
javadoc | JavadocPackage | Missing package-info.java file. | |
![]() |
javadoc | JavadocType | Missing a Javadoc comment. | 19 |
![]() |
blocks | LeftCurly | '{' at column 1 should be on the previous line. | 25 |
![]() |
javadoc | JavadocStyle | First sentence should end with a period. | 27 |
![]() |
regexp | RegexpSingleline | Line has trailing spaces. | 29 |
![]() |
javadoc | JavadocVariable | Missing a Javadoc comment. | 34 |
![]() |
sizes | LineLength | Line is longer than 80 characters (found 85). | 35 |
![]() |
javadoc | JavadocStyle | First sentence should end with a period. | 37 |
![]() |
regexp | RegexpSingleline | Line has trailing spaces. | 39 |
![]() |
javadoc | JavadocStyle | First sentence should end with a period. | 49 |
![]() |
regexp | RegexpSingleline | Line has trailing spaces. | 51 |
![]() |
misc | FinalParameters | Parameter path should be final. | 57 |
![]() |
coding | HiddenField | 'path' hides a field. | 57 |
![]() |
design | DesignForExtension | Class 'FileBufferConfiguration' looks like designed for extension (can be subclassed), but the method 'withPath' does not have javadoc that explains how to do that safely. If class is not designed for extension consider making the class 'FileBufferConfiguration' final or making the method 'withPath' static/final/abstract/empty, or adding allowed annotation for the method. | 61 |
![]() |
javadoc | JavadocMethod | Missing a Javadoc comment. | 61 |
![]() |
misc | FinalParameters | Parameter path should be final. | 61 |
![]() |
coding | HiddenField | 'path' hides a field. | 61 |
![]() |
design | DesignForExtension | Class 'FileBufferConfiguration' looks like designed for extension (can be subclassed), but the method 'toString' does not have javadoc that explains how to do that safely. If class is not designed for extension consider making the class 'FileBufferConfiguration' final or making the method 'toString' static/final/abstract/empty, or adding allowed annotation for the method. | 66 |
![]() |
design | DesignForExtension | Class 'FileBufferConfiguration' looks like designed for extension (can be subclassed), but the method 'getAdditionalProperties' does not have javadoc that explains how to do that safely. If class is not designed for extension consider making the class 'FileBufferConfiguration' final or making the method 'getAdditionalProperties' static/final/abstract/empty, or adding allowed annotation for the method. | 71 |
![]() |
javadoc | JavadocMethod | Missing a Javadoc comment. | 71 |
![]() |
design | DesignForExtension | Class 'FileBufferConfiguration' looks like designed for extension (can be subclassed), but the method 'setAdditionalProperty' does not have javadoc that explains how to do that safely. If class is not designed for extension consider making the class 'FileBufferConfiguration' final or making the method 'setAdditionalProperty' static/final/abstract/empty, or adding allowed annotation for the method. | 77 |
![]() |
javadoc | JavadocMethod | Missing a Javadoc comment. | 77 |
![]() |
misc | FinalParameters | Parameter name should be final. | 79 |
![]() |
misc | FinalParameters | Parameter value should be final. | 79 |
![]() |
sizes | LineLength | Line is longer than 80 characters (found 86). | 83 |
![]() |
design | DesignForExtension | Class 'FileBufferConfiguration' looks like designed for extension (can be subclassed), but the method 'withAdditionalProperty' does not have javadoc that explains how to do that safely. If class is not designed for extension consider making the class 'FileBufferConfiguration' final or making the method 'withAdditionalProperty' static/final/abstract/empty, or adding allowed annotation for the method. | 83 |
![]() |
javadoc | JavadocMethod | Missing a Javadoc comment. | 83 |
![]() |
misc | FinalParameters | Parameter name should be final. | 83 |
![]() |
misc | FinalParameters | Parameter value should be final. | 83 |
![]() |
design | DesignForExtension | Class 'FileBufferConfiguration' looks like designed for extension (can be subclassed), but the method 'hashCode' does not have javadoc that explains how to do that safely. If class is not designed for extension consider making the class 'FileBufferConfiguration' final or making the method 'hashCode' static/final/abstract/empty, or adding allowed annotation for the method. | 88 |
![]() |
sizes | LineLength | Line is longer than 80 characters (found 92). | 90 |
![]() |
design | DesignForExtension | Class 'FileBufferConfiguration' looks like designed for extension (can be subclassed), but the method 'equals' does not have javadoc that explains how to do that safely. If class is not designed for extension consider making the class 'FileBufferConfiguration' final or making the method 'equals' static/final/abstract/empty, or adding allowed annotation for the method. | 93 |
![]() |
misc | FinalParameters | Parameter other should be final. | 94 |
![]() |
coding | SimplifyBooleanExpression | Expression can be simplified. | 98 |
![]() |
sizes | LineLength | Line is longer than 80 characters (found 124). | 102 |
Severity | Category | Rule | Message | Line |
---|---|---|---|---|
![]() |
sizes | LineLength | Line is longer than 80 characters (found 84). | 50 |
![]() |
javadoc | JavadocVariable | Missing a Javadoc comment. | 52 |
![]() |
sizes | LineLength | Line is longer than 80 characters (found 94). | 54 |
![]() |
javadoc | JavadocVariable | Missing a Javadoc comment. | 54 |
![]() |
javadoc | JavadocVariable | Missing a Javadoc comment. | 56 |
![]() |
javadoc | JavadocVariable | Missing a Javadoc comment. | 58 |
![]() |
javadoc | JavadocVariable | Missing a Javadoc comment. | 60 |
![]() |
javadoc | JavadocVariable | Missing a Javadoc comment. | 62 |
![]() |
javadoc | JavadocVariable | Missing a Javadoc comment. | 64 |
![]() |
javadoc | JavadocVariable | Missing a Javadoc comment. | 65 |
![]() |
javadoc | JavadocVariable | Missing a Javadoc comment. | 67 |
![]() |
javadoc | JavadocMethod | Missing a Javadoc comment. | 69 |
![]() |
sizes | LineLength | Line is longer than 80 characters (found 87). | 71 |
![]() |
javadoc | JavadocMethod | Missing a Javadoc comment. | 74 |
![]() |
misc | FinalParameters | Parameter config should be final. | 74 |
![]() |
coding | HiddenField | 'config' hides a field. | 74 |
![]() |
design | DesignForExtension | Class 'FileBufferPersistReader' looks like designed for extension (can be subclassed), but the method 'getId' does not have javadoc that explains how to do that safely. If class is not designed for extension consider making the class 'FileBufferPersistReader' final or making the method 'getId' static/final/abstract/empty, or adding allowed annotation for the method. | 78 |
![]() |
design | DesignForExtension | Class 'FileBufferPersistReader' looks like designed for extension (can be subclassed), but the method 'readAll' does not have javadoc that explains how to do that safely. If class is not designed for extension consider making the class 'FileBufferPersistReader' final or making the method 'readAll' static/final/abstract/empty, or adding allowed annotation for the method. | 83 |
![]() |
design | DesignForExtension | Class 'FileBufferPersistReader' looks like designed for extension (can be subclassed), but the method 'startStream' does not have javadoc that explains how to do that safely. If class is not designed for extension consider making the class 'FileBufferPersistReader' final or making the method 'startStream' static/final/abstract/empty, or adding allowed annotation for the method. | 88 |
![]() |
design | DesignForExtension | Class 'FileBufferPersistReader' looks like designed for extension (can be subclassed), but the method 'readCurrent' does not have javadoc that explains how to do that safely. If class is not designed for extension consider making the class 'FileBufferPersistReader' final or making the method 'readCurrent' static/final/abstract/empty, or adding allowed annotation for the method. | 93 |
![]() |
javadoc | JavadocMethod | Missing a Javadoc comment. | 117 |
![]() |
whitespace | ParenPad | '(' is followed by whitespace. | 117 |
![]() |
misc | FinalParameters | Parameter entry should be final. | 117 |
![]() |
whitespace | ParenPad | ')' is preceded with whitespace. | 117 |
![]() |
design | DesignForExtension | Class 'FileBufferPersistReader' looks like designed for extension (can be subclassed), but the method 'readNew' does not have javadoc that explains how to do that safely. If class is not designed for extension consider making the class 'FileBufferPersistReader' final or making the method 'readNew' static/final/abstract/empty, or adding allowed annotation for the method. | 121 |
![]() |
misc | FinalParameters | Parameter bigInteger should be final. | 122 |
![]() |
design | DesignForExtension | Class 'FileBufferPersistReader' looks like designed for extension (can be subclassed), but the method 'readRange' does not have javadoc that explains how to do that safely. If class is not designed for extension consider making the class 'FileBufferPersistReader' final or making the method 'readRange' static/final/abstract/empty, or adding allowed annotation for the method. | 126 |
![]() |
misc | FinalParameters | Parameter dateTime should be final. | 127 |
![]() |
misc | FinalParameters | Parameter dateTime2 should be final. | 127 |
![]() |
design | DesignForExtension | Class 'FileBufferPersistReader' looks like designed for extension (can be subclassed), but the method 'isRunning' does not have javadoc that explains how to do that safely. If class is not designed for extension consider making the class 'FileBufferPersistReader' final or making the method 'isRunning' static/final/abstract/empty, or adding allowed annotation for the method. | 131 |
![]() |
design | DesignForExtension | Class 'FileBufferPersistReader' looks like designed for extension (can be subclassed), but the method 'prepare' does not have javadoc that explains how to do that safely. If class is not designed for extension consider making the class 'FileBufferPersistReader' final or making the method 'prepare' static/final/abstract/empty, or adding allowed annotation for the method. | 136 |
![]() |
misc | FinalParameters | Parameter configurationObject should be final. | 137 |
![]() |
coding | MagicNumber | '1000' is a magic number. | 140 |
![]() |
whitespace | ParenPad | '(' is followed by whitespace. | 147 |
![]() |
whitespace | ParenPad | '(' is followed by whitespace. | 149 |
![]() |
whitespace | ParenPad | ')' is preceded with whitespace. | 149 |
![]() |
design | DesignForExtension | Class 'FileBufferPersistReader' looks like designed for extension (can be subclassed), but the method 'cleanUp' does not have javadoc that explains how to do that safely. If class is not designed for extension consider making the class 'FileBufferPersistReader' final or making the method 'cleanUp' static/final/abstract/empty, or adding allowed annotation for the method. | 172 |
Severity | Category | Rule | Message | Line |
---|---|---|---|---|
![]() |
javadoc | JavadocPackage | Missing package-info.java file. | |
![]() |
sizes | LineLength | Line is longer than 80 characters (found 84). | 44 |
![]() |
javadoc | JavadocVariable | Missing a Javadoc comment. | 46 |
![]() |
sizes | LineLength | Line is longer than 80 characters (found 94). | 48 |
![]() |
javadoc | JavadocVariable | Missing a Javadoc comment. | 48 |
![]() |
javadoc | JavadocVariable | Missing a Javadoc comment. | 50 |
![]() |
javadoc | JavadocVariable | Missing a Javadoc comment. | 52 |
![]() |
javadoc | JavadocVariable | Missing a Javadoc comment. | 54 |
![]() |
javadoc | JavadocVariable | Missing a Javadoc comment. | 56 |
![]() |
javadoc | JavadocMethod | Missing a Javadoc comment. | 58 |
![]() |
sizes | LineLength | Line is longer than 80 characters (found 87). | 60 |
![]() |
javadoc | JavadocMethod | Missing a Javadoc comment. | 63 |
![]() |
misc | FinalParameters | Parameter config should be final. | 63 |
![]() |
coding | HiddenField | 'config' hides a field. | 63 |
![]() |
design | DesignForExtension | Class 'FileBufferPersistWriter' looks like designed for extension (can be subclassed), but the method 'getId' does not have javadoc that explains how to do that safely. If class is not designed for extension consider making the class 'FileBufferPersistWriter' final or making the method 'getId' static/final/abstract/empty, or adding allowed annotation for the method. | 67 |
![]() |
design | DesignForExtension | Class 'FileBufferPersistWriter' looks like designed for extension (can be subclassed), but the method 'write' does not have javadoc that explains how to do that safely. If class is not designed for extension consider making the class 'FileBufferPersistWriter' final or making the method 'write' static/final/abstract/empty, or adding allowed annotation for the method. | 72 |
![]() |
misc | FinalParameters | Parameter entry should be final. | 73 |
![]() |
sizes | LineLength | Line is longer than 80 characters (found 94). | 75 |
![]() |
coding | AvoidInlineConditionals | Avoid inline conditionals. | 75 |
![]() |
sizes | LineLength | Line is longer than 80 characters (found 86). | 79 |
![]() |
whitespace | WhitespaceAfter | 'typecast' is not followed by whitespace. | 81 |
![]() |
design | DesignForExtension | Class 'FileBufferPersistWriter' looks like designed for extension (can be subclassed), but the method 'prepare' does not have javadoc that explains how to do that safely. If class is not designed for extension consider making the class 'FileBufferPersistWriter' final or making the method 'prepare' static/final/abstract/empty, or adding allowed annotation for the method. | 89 |
![]() |
misc | FinalParameters | Parameter configurationObject should be final. | 90 |
![]() |
whitespace | ParenPad | '(' is followed by whitespace. | 94 |
![]() |
design | DesignForExtension | Class 'FileBufferPersistWriter' looks like designed for extension (can be subclassed), but the method 'cleanUp' does not have javadoc that explains how to do that safely. If class is not designed for extension consider making the class 'FileBufferPersistWriter' final or making the method 'cleanUp' static/final/abstract/empty, or adding allowed annotation for the method. | 111 |