The following document contains the results of Checkstyle 7.2 with sun_checks.xml ruleset.
File | ![]() |
![]() |
![]() |
---|---|---|---|
org/apache/streams/console/ConsolePersistReader.java | 0 | 0 | 25 |
org/apache/streams/console/ConsolePersistWriter.java | 0 | 0 | 19 |
org/apache/streams/console/ConsolePersistWriterTask.java | 0 | 0 | 11 |
Category | Rule | Violations | Severity |
---|---|---|---|
coding | HiddenField | 3 | ![]() |
MagicNumber | 1 | ![]() | |
design | DesignForExtension | 10 | ![]() |
VisibilityModifier | 3 | ![]() | |
javadoc | JavadocMethod | 9 | ![]() |
JavadocPackage | 1 | ![]() | |
JavadocStyle | 1 | ![]() | |
JavadocVariable | 11 | ![]() | |
misc | FinalParameters | 9 | ![]() |
sizes | LineLength | 3 | ![]() |
whitespace | ParenPad | 4 | ![]() |
Severity | Category | Rule | Message | Line |
---|---|---|---|---|
![]() |
javadoc | JavadocVariable | Missing a Javadoc comment. | 40 |
![]() |
sizes | LineLength | Line is longer than 80 characters (found 91). | 42 |
![]() |
javadoc | JavadocVariable | Missing a Javadoc comment. | 42 |
![]() |
javadoc | JavadocVariable | Missing a Javadoc comment. | 44 |
![]() |
design | VisibilityModifier | Variable 'persistQueue' must be private and have accessor methods. | 44 |
![]() |
javadoc | JavadocVariable | Missing a Javadoc comment. | 46 |
![]() |
design | VisibilityModifier | Variable 'inputStream' must be private and have accessor methods. | 46 |
![]() |
javadoc | JavadocMethod | Missing a Javadoc comment. | 48 |
![]() |
javadoc | JavadocMethod | Missing a Javadoc comment. | 52 |
![]() |
misc | FinalParameters | Parameter inputStream should be final. | 52 |
![]() |
coding | HiddenField | 'inputStream' hides a field. | 52 |
![]() |
design | DesignForExtension | Class 'ConsolePersistReader' 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 'ConsolePersistReader' final or making the method 'getId' static/final/abstract/empty, or adding allowed annotation for the method. | 57 |
![]() |
javadoc | JavadocMethod | Missing a Javadoc comment. | 62 |
![]() |
misc | FinalParameters | Parameter configuration should be final. | 62 |
![]() |
javadoc | JavadocMethod | Missing a Javadoc comment. | 66 |
![]() |
design | DesignForExtension | Class 'ConsolePersistReader' 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 'ConsolePersistReader' final or making the method 'readAll' static/final/abstract/empty, or adding allowed annotation for the method. | 75 |
![]() |
design | DesignForExtension | Class 'ConsolePersistReader' 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 'ConsolePersistReader' final or making the method 'readCurrent' static/final/abstract/empty, or adding allowed annotation for the method. | 80 |
![]() |
whitespace | ParenPad | '(' is followed by whitespace. | 87 |
![]() |
whitespace | ParenPad | ')' is preceded with whitespace. | 87 |
![]() |
design | DesignForExtension | Class 'ConsolePersistReader' 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 'ConsolePersistReader' final or making the method 'readNew' static/final/abstract/empty, or adding allowed annotation for the method. | 103 |
![]() |
misc | FinalParameters | Parameter sequence should be final. | 104 |
![]() |
design | DesignForExtension | Class 'ConsolePersistReader' 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 'ConsolePersistReader' final or making the method 'readRange' static/final/abstract/empty, or adding allowed annotation for the method. | 108 |
![]() |
misc | FinalParameters | Parameter start should be final. | 109 |
![]() |
misc | FinalParameters | Parameter end should be final. | 109 |
![]() |
design | DesignForExtension | Class 'ConsolePersistReader' 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 'ConsolePersistReader' final or making the method 'isRunning' static/final/abstract/empty, or adding allowed annotation for the method. | 113 |
Severity | Category | Rule | Message | Line |
---|---|---|---|---|
![]() |
javadoc | JavadocPackage | Missing package-info.java file. | |
![]() |
javadoc | JavadocVariable | Missing a Javadoc comment. | 40 |
![]() |
sizes | LineLength | Line is longer than 80 characters (found 91). | 42 |
![]() |
javadoc | JavadocVariable | Missing a Javadoc comment. | 42 |
![]() |
javadoc | JavadocVariable | Missing a Javadoc comment. | 44 |
![]() |
javadoc | JavadocVariable | Missing a Javadoc comment. | 46 |
![]() |
design | VisibilityModifier | Variable 'persistQueue' must be private and have accessor methods. | 46 |
![]() |
javadoc | JavadocVariable | Missing a Javadoc comment. | 48 |
![]() |
javadoc | JavadocMethod | Missing a Javadoc comment. | 50 |
![]() |
javadoc | JavadocMethod | Missing a Javadoc comment. | 54 |
![]() |
misc | FinalParameters | Parameter printStream should be final. | 54 |
![]() |
coding | HiddenField | 'printStream' hides a field. | 54 |
![]() |
design | DesignForExtension | Class 'ConsolePersistWriter' 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 'ConsolePersistWriter' final or making the method 'getId' static/final/abstract/empty, or adding allowed annotation for the method. | 59 |
![]() |
design | DesignForExtension | Class 'ConsolePersistWriter' 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 'ConsolePersistWriter' final or making the method 'prepare' static/final/abstract/empty, or adding allowed annotation for the method. | 64 |
![]() |
javadoc | JavadocMethod | Missing a Javadoc comment. | 64 |
![]() |
misc | FinalParameters | Parameter configuration should be final. | 64 |
![]() |
javadoc | JavadocMethod | Missing a Javadoc comment. | 68 |
![]() |
design | DesignForExtension | Class 'ConsolePersistWriter' 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 'ConsolePersistWriter' 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 |
Severity | Category | Rule | Message | Line |
---|---|---|---|---|
![]() |
javadoc | JavadocStyle | First sentence should end with a period. | 28 |
![]() |
sizes | LineLength | Line is longer than 80 characters (found 95). | 34 |
![]() |
javadoc | JavadocVariable | Missing a Javadoc comment. | 34 |
![]() |
javadoc | JavadocVariable | Missing a Javadoc comment. | 36 |
![]() |
javadoc | JavadocMethod | Missing a Javadoc comment. | 38 |
![]() |
misc | FinalParameters | Parameter writer should be final. | 38 |
![]() |
coding | HiddenField | 'writer' hides a field. | 38 |
![]() |
design | DesignForExtension | Class 'ConsolePersistWriterTask' looks like designed for extension (can be subclassed), but the method 'run' does not have javadoc that explains how to do that safely. If class is not designed for extension consider making the class 'ConsolePersistWriterTask' final or making the method 'run' static/final/abstract/empty, or adding allowed annotation for the method. | 42 |
![]() |
whitespace | ParenPad | '(' is followed by whitespace. | 45 |
![]() |
whitespace | ParenPad | ')' is preceded with whitespace. | 45 |
![]() |
coding | MagicNumber | '100' is a magic number. | 54 |