The following document contains the results of Checkstyle 7.2 with sun_checks.xml ruleset.
Category | Rule | Violations | Severity |
---|---|---|---|
coding | HiddenField | 5 | ![]() |
MagicNumber | 1 | ![]() | |
design | DesignForExtension | 10 | ![]() |
VisibilityModifier | 4 | ![]() | |
javadoc | JavadocMethod | 8 | ![]() |
JavadocPackage | 1 | ![]() | |
JavadocStyle | 1 | ![]() | |
JavadocType | 1 | ![]() | |
JavadocVariable | 14 | ![]() | |
misc | FinalParameters | 25 | ![]() |
NewlineAtEndOfFile | 1 | ![]() | |
naming | ConstantName | 1 | ![]() |
MethodName | 2 | ![]() | |
sizes | LineLength | 17 | ![]() |
whitespace | ParenPad | 5 | ![]() |
WhitespaceAfter | 1 | ![]() |
Severity | Category | Rule | Message | Line |
---|---|---|---|---|
![]() |
misc | NewlineAtEndOfFile | File does not end with a newline. | |
![]() |
javadoc | JavadocVariable | Missing a Javadoc comment. | 63 |
![]() |
javadoc | JavadocMethod | Missing a Javadoc comment. | 65 |
![]() |
javadoc | JavadocVariable | Missing a Javadoc comment. | 68 |
![]() |
naming | ConstantName | Name 'log' must match pattern '^[A-Z][A-Z0-9]*(_[A-Z0-9]+)*$'. | 68 |
![]() |
javadoc | JavadocVariable | Missing a Javadoc comment. | 71 |
![]() |
javadoc | JavadocVariable | Missing a Javadoc comment. | 73 |
![]() |
design | VisibilityModifier | Variable 'providerQueue' must be private and have accessor methods. | 73 |
![]() |
javadoc | JavadocVariable | Missing a Javadoc comment. | 75 |
![]() |
design | VisibilityModifier | Variable 'lock' must be private and have accessor methods. | 75 |
![]() |
sizes | LineLength | Line is longer than 80 characters (found 94). | 77 |
![]() |
javadoc | JavadocVariable | Missing a Javadoc comment. | 77 |
![]() |
design | DesignForExtension | Class 'GenericWebhookResource' 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 'GenericWebhookResource' final or making the method 'getId' static/final/abstract/empty, or adding allowed annotation for the method. | 79 |
![]() |
misc | FinalParameters | Parameter headers should be final. | 92 |
![]() |
misc | FinalParameters | Parameter body should be final. | 93 |
![]() |
naming | MethodName | Name 'json_new_line' must match pattern '^[a-z][a-zA-Z0-9]*$'. | 134 |
![]() |
misc | FinalParameters | Parameter headers should be final. | 134 |
![]() |
misc | FinalParameters | Parameter body should be final. | 135 |
![]() |
naming | MethodName | Name 'json_meta' must match pattern '^[a-z][a-zA-Z0-9]*$'. | 190 |
![]() |
misc | FinalParameters | Parameter headers should be final. | 190 |
![]() |
misc | FinalParameters | Parameter body should be final. | 191 |
![]() |
sizes | LineLength | Line is longer than 80 characters (found 90). | 208 |
![]() |
whitespace | ParenPad | '(' is followed by whitespace. | 210 |
![]() |
design | DesignForExtension | Class 'GenericWebhookResource' 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 'GenericWebhookResource' final or making the method 'readCurrent' static/final/abstract/empty, or adding allowed annotation for the method. | 241 |
![]() |
design | DesignForExtension | Class 'GenericWebhookResource' 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 'GenericWebhookResource' final or making the method 'readNew' static/final/abstract/empty, or adding allowed annotation for the method. | 255 |
![]() |
misc | FinalParameters | Parameter sequence should be final. | 256 |
![]() |
design | DesignForExtension | Class 'GenericWebhookResource' 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 'GenericWebhookResource' final or making the method 'readRange' static/final/abstract/empty, or adding allowed annotation for the method. | 260 |
![]() |
misc | FinalParameters | Parameter start should be final. | 261 |
![]() |
misc | FinalParameters | Parameter end should be final. | 261 |
![]() |
design | DesignForExtension | Class 'GenericWebhookResource' 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 'GenericWebhookResource' final or making the method 'isRunning' static/final/abstract/empty, or adding allowed annotation for the method. | 265 |
![]() |
misc | FinalParameters | Parameter configurationObject should be final. | 271 |
Severity | Category | Rule | Message | Line |
---|---|---|---|---|
![]() |
javadoc | JavadocPackage | Missing package-info.java file. | |
![]() |
sizes | LineLength | Line is longer than 80 characters (found 82). | 32 |
![]() |
sizes | LineLength | Line is longer than 80 characters (found 81). | 35 |
![]() |
sizes | LineLength | Line is longer than 80 characters (found 90). | 37 |
![]() |
javadoc | JavadocMethod | Missing a Javadoc comment. | 39 |
![]() |
javadoc | JavadocMethod | Missing a Javadoc comment. | 43 |
![]() |
misc | FinalParameters | Parameter streamConfig should be final. | 43 |
![]() |
sizes | LineLength | Line is longer than 80 characters (found 90). | 44 |
![]() |
javadoc | JavadocMethod | Missing a Javadoc comment. | 47 |
![]() |
misc | FinalParameters | Parameter streamConfig should be final. | 47 |
![]() |
javadoc | JavadocMethod | Missing a Javadoc comment. | 51 |
![]() |
misc | FinalParameters | Parameter maxQueueCapacity should be final. | 51 |
![]() |
sizes | LineLength | Line is longer than 80 characters (found 90). | 55 |
![]() |
javadoc | JavadocMethod | Missing a Javadoc comment. | 55 |
![]() |
misc | FinalParameters | Parameter maxQueueCapacity should be final. | 55 |
![]() |
misc | FinalParameters | Parameter streamConfig should be final. | 55 |
![]() |
design | DesignForExtension | Class 'StreamDropwizardBuilder' looks like designed for extension (can be subclassed), but the method 'newPerpetualStream' does not have javadoc that explains how to do that safely. If class is not designed for extension consider making the class 'StreamDropwizardBuilder' final or making the method 'newPerpetualStream' static/final/abstract/empty, or adding allowed annotation for the method. | 59 |
![]() |
sizes | LineLength | Line is longer than 80 characters (found 86). | 60 |
![]() |
misc | FinalParameters | Parameter streamId should be final. | 60 |
![]() |
misc | FinalParameters | Parameter provider should be final. | 60 |
Severity | Category | Rule | Message | Line |
---|---|---|---|---|
![]() |
sizes | LineLength | Line is longer than 80 characters (found 85). | 54 |
![]() |
javadoc | JavadocVariable | Missing a Javadoc comment. | 56 |
![]() |
javadoc | JavadocVariable | Missing a Javadoc comment. | 59 |
![]() |
design | VisibilityModifier | Variable 'mapper' must be private and have accessor methods. | 59 |
![]() |
javadoc | JavadocVariable | Missing a Javadoc comment. | 61 |
![]() |
design | VisibilityModifier | Variable 'builder' must be private and have accessor methods. | 61 |
![]() |
javadoc | JavadocVariable | Missing a Javadoc comment. | 63 |
![]() |
sizes | LineLength | Line is longer than 80 characters (found 128). | 68 |
![]() |
javadoc | JavadocVariable | Missing a Javadoc comment. | 68 |
![]() |
javadoc | JavadocVariable | Missing a Javadoc comment. | 70 |
![]() |
design | DesignForExtension | Class 'StreamsApplication' looks like designed for extension (can be subclassed), but the method 'initialize' does not have javadoc that explains how to do that safely. If class is not designed for extension consider making the class 'StreamsApplication' final or making the method 'initialize' static/final/abstract/empty, or adding allowed annotation for the method. | 78 |
![]() |
misc | FinalParameters | Parameter bootstrap should be final. | 79 |
![]() |
design | DesignForExtension | Class 'StreamsApplication' 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 'StreamsApplication' final or making the method 'run' static/final/abstract/empty, or adding allowed annotation for the method. | 94 |
![]() |
sizes | LineLength | Line is longer than 80 characters (found 124). | 95 |
![]() |
misc | FinalParameters | Parameter streamsDropwizardConfiguration should be final. | 95 |
![]() |
misc | FinalParameters | Parameter environment should be final. | 95 |
![]() |
sizes | LineLength | Line is longer than 80 characters (found 112). | 99 |
![]() |
whitespace | ParenPad | '(' is followed by whitespace. | 99 |
![]() |
whitespace | ParenPad | ')' is preceded with whitespace. | 99 |
![]() |
sizes | LineLength | Line is longer than 80 characters (found 86). | 100 |
![]() |
whitespace | WhitespaceAfter | 'typecast' is not followed by whitespace. | 100 |
![]() |
whitespace | ParenPad | '(' is followed by whitespace. | 101 |
![]() |
sizes | LineLength | Line is longer than 80 characters (found 107). | 110 |
![]() |
sizes | LineLength | Line is longer than 80 characters (found 81). | 114 |
![]() |
coding | MagicNumber | '10000' is a magic number. | 117 |
![]() |
sizes | LineLength | Line is longer than 80 characters (found 113). | 132 |
![]() |
misc | FinalParameters | Parameter streamsConfiguration should be final. | 132 |
![]() |
coding | HiddenField | 'streamsConfiguration' hides a field. | 132 |
![]() |
misc | FinalParameters | Parameter resourceProviders should be final. | 132 |
![]() |
coding | HiddenField | 'resourceProviders' hides a field. | 132 |
![]() |
coding | HiddenField | 'builder' hides a field. | 134 |
![]() |
whitespace | ParenPad | '(' is followed by whitespace. | 137 |
![]() |
javadoc | JavadocType | Missing a Javadoc comment. | 146 |
![]() |
javadoc | JavadocVariable | Missing a Javadoc comment. | 148 |
![]() |
javadoc | JavadocVariable | Missing a Javadoc comment. | 150 |
![]() |
sizes | LineLength | Line is longer than 80 characters (found 105). | 152 |
![]() |
javadoc | JavadocMethod | Missing a Javadoc comment. | 152 |
![]() |
misc | FinalParameters | Parameter builder should be final. | 152 |
![]() |
coding | HiddenField | 'builder' hides a field. | 152 |
![]() |
misc | FinalParameters | Parameter streamsConfiguration should be final. | 152 |
![]() |
coding | HiddenField | 'streamsConfiguration' hides a field. | 152 |
![]() |
javadoc | JavadocStyle | First sentence should end with a period. | 165 |
![]() |
misc | FinalParameters | Parameter args should be final. | 174 |
Severity | Category | Rule | Message | Line |
---|---|---|---|---|
![]() |
design | DesignForExtension | Class 'StreamsDropwizardModule' looks like designed for extension (can be subclassed), but the method 'configure' does not have javadoc that explains how to do that safely. If class is not designed for extension consider making the class 'StreamsDropwizardModule' final or making the method 'configure' static/final/abstract/empty, or adding allowed annotation for the method. | 37 |
![]() |
design | DesignForExtension | Class 'StreamsDropwizardModule' looks like designed for extension (can be subclassed), but the method 'providesStreamsConfiguration' does not have javadoc that explains how to do that safely. If class is not designed for extension consider making the class 'StreamsDropwizardModule' final or making the method 'providesStreamsConfiguration' static/final/abstract/empty, or adding allowed annotation for the method. | 42 |
![]() |
javadoc | JavadocMethod | Missing a Javadoc comment. | 42 |