The following document contains the results of Checkstyle 8.35 with development/dsol-checks.xml ruleset.
Files | ![]() |
![]() |
![]() |
---|---|---|---|
65 | 8 | 106 | 0 |
Category | Rule | Violations | Severity |
---|---|---|---|
blocks | NeedBraces | 11 | ![]() |
design | DesignForExtension | 7 | ![]() |
VisibilityModifier | 7 | ![]() | |
javadoc | JavadocVariable | 62 | ![]() |
misc | FinalParameters | 10 | ![]() |
TodoComment | 1 | ![]() | |
modifier | RedundantModifier | 8 | ![]() |
sizes | LineLength
|
8 | ![]() |
Severity | Category | Rule | Message | Line |
---|---|---|---|---|
![]() |
misc | TodoComment | Comment matches to-do format 'TODO:'. | 246 |
Severity | Category | Rule | Message | Line |
---|---|---|---|---|
![]() |
design | VisibilityModifier | Variable 'code' must be private and have accessor methods. | 22 |
Severity | Category | Rule | Message | Line |
---|---|---|---|---|
![]() |
javadoc | JavadocVariable | Missing a Javadoc comment. | 19 |
![]() |
javadoc | JavadocVariable | Missing a Javadoc comment. | 21 |
![]() |
sizes | LineLength | Line is longer than 128 characters (found 171). | 22 |
![]() |
javadoc | JavadocVariable | Missing a Javadoc comment. | 24 |
![]() |
sizes | LineLength | Line is longer than 128 characters (found 201). | 25 |
![]() |
javadoc | JavadocVariable | Missing a Javadoc comment. | 27 |
![]() |
sizes | LineLength | Line is longer than 128 characters (found 135). | 28 |
![]() |
javadoc | JavadocVariable | Missing a Javadoc comment. | 30 |
![]() |
sizes | LineLength | Line is longer than 128 characters (found 161). | 31 |
![]() |
javadoc | JavadocVariable | Missing a Javadoc comment. | 33 |
![]() |
javadoc | JavadocVariable | Missing a Javadoc comment. | 35 |
![]() |
modifier | RedundantModifier | Redundant 'private' modifier. | 47 |
Severity | Category | Rule | Message | Line |
---|---|---|---|---|
![]() |
javadoc | JavadocVariable | Missing a Javadoc comment. | 19 |
![]() |
javadoc | JavadocVariable | Missing a Javadoc comment. | 21 |
![]() |
javadoc | JavadocVariable | Missing a Javadoc comment. | 25 |
![]() |
javadoc | JavadocVariable | Missing a Javadoc comment. | 28 |
![]() |
javadoc | JavadocVariable | Missing a Javadoc comment. | 32 |
![]() |
javadoc | JavadocVariable | Missing a Javadoc comment. | 34 |
![]() |
javadoc | JavadocVariable | Missing a Javadoc comment. | 38 |
![]() |
javadoc | JavadocVariable | Missing a Javadoc comment. | 42 |
![]() |
javadoc | JavadocVariable | Missing a Javadoc comment. | 44 |
![]() |
javadoc | JavadocVariable | Missing a Javadoc comment. | 47 |
![]() |
modifier | RedundantModifier | Redundant 'private' modifier. | 59 |
Severity | Category | Rule | Message | Line |
---|---|---|---|---|
![]() |
javadoc | JavadocVariable | Missing a Javadoc comment. | 19 |
![]() |
javadoc | JavadocVariable | Missing a Javadoc comment. | 21 |
![]() |
javadoc | JavadocVariable | Missing a Javadoc comment. | 24 |
![]() |
javadoc | JavadocVariable | Missing a Javadoc comment. | 28 |
![]() |
javadoc | JavadocVariable | Missing a Javadoc comment. | 31 |
![]() |
javadoc | JavadocVariable | Missing a Javadoc comment. | 35 |
![]() |
javadoc | JavadocVariable | Missing a Javadoc comment. | 37 |
![]() |
modifier | RedundantModifier | Redundant 'private' modifier. | 60 |
Severity | Category | Rule | Message | Line |
---|---|---|---|---|
![]() |
javadoc | JavadocVariable | Missing a Javadoc comment. | 19 |
![]() |
javadoc | JavadocVariable | Missing a Javadoc comment. | 21 |
![]() |
sizes | LineLength | Line is longer than 128 characters (found 168). | 22 |
![]() |
javadoc | JavadocVariable | Missing a Javadoc comment. | 24 |
![]() |
sizes | LineLength | Line is longer than 128 characters (found 160). | 25 |
![]() |
javadoc | JavadocVariable | Missing a Javadoc comment. | 27 |
![]() |
javadoc | JavadocVariable | Missing a Javadoc comment. | 29 |
![]() |
sizes | LineLength | Line is longer than 128 characters (found 135). | 30 |
![]() |
javadoc | JavadocVariable | Missing a Javadoc comment. | 32 |
![]() |
sizes | LineLength | Line is longer than 128 characters (found 154). | 33 |
![]() |
javadoc | JavadocVariable | Missing a Javadoc comment. | 35 |
![]() |
modifier | RedundantModifier | Redundant 'private' modifier. | 47 |
Severity | Category | Rule | Message | Line |
---|---|---|---|---|
![]() |
modifier | RedundantModifier | Redundant 'private' modifier. | 33 |
Severity | Category | Rule | Message | Line |
---|---|---|---|---|
![]() |
modifier | RedundantModifier | Redundant 'private' modifier. | 33 |
Severity | Category | Rule | Message | Line |
---|---|---|---|---|
![]() |
modifier | RedundantModifier | Redundant 'private' modifier. | 33 |
Severity | Category | Rule | Message | Line |
---|---|---|---|---|
![]() |
javadoc | JavadocVariable | Missing a Javadoc comment. | 19 |
![]() |
javadoc | JavadocVariable | Missing a Javadoc comment. | 21 |
![]() |
javadoc | JavadocVariable | Missing a Javadoc comment. | 23 |
![]() |
javadoc | JavadocVariable | Missing a Javadoc comment. | 26 |
![]() |
javadoc | JavadocVariable | Missing a Javadoc comment. | 28 |
![]() |
javadoc | JavadocVariable | Missing a Javadoc comment. | 30 |
![]() |
javadoc | JavadocVariable | Missing a Javadoc comment. | 32 |
![]() |
javadoc | JavadocVariable | Missing a Javadoc comment. | 34 |
![]() |
javadoc | JavadocVariable | Missing a Javadoc comment. | 36 |
![]() |
javadoc | JavadocVariable | Missing a Javadoc comment. | 38 |
![]() |
javadoc | JavadocVariable | Missing a Javadoc comment. | 40 |
![]() |
javadoc | JavadocVariable | Missing a Javadoc comment. | 42 |
![]() |
javadoc | JavadocVariable | Missing a Javadoc comment. | 44 |
![]() |
javadoc | JavadocVariable | Missing a Javadoc comment. | 46 |
![]() |
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 | JavadocVariable | Missing a Javadoc comment. | 58 |
![]() |
javadoc | JavadocVariable | Missing a Javadoc comment. | 60 |
![]() |
modifier | RedundantModifier | Redundant 'private' modifier. | 72 |
Severity | Category | Rule | Message | Line |
---|---|---|---|---|
![]() |
design | VisibilityModifier | Variable 'count' must be private and have accessor methods. | 32 |
![]() |
design | VisibilityModifier | Variable 'n' must be private and have accessor methods. | 35 |
![]() |
design | VisibilityModifier | Variable 'description' must be private and have accessor methods. | 38 |
Severity | Category | Rule | Message | Line |
---|---|---|---|---|
![]() |
design | VisibilityModifier | Variable 'frequency' must be private and have accessor methods. | 34 |
![]() |
design | VisibilityModifier | Variable 'n' must be private and have accessor methods. | 37 |
![]() |
design | VisibilityModifier | Variable 'description' must be private and have accessor methods. | 40 |
![]() |
blocks | NeedBraces | 'if' construct must use '{}'s. | 91 |
![]() |
blocks | NeedBraces | 'else' construct must use '{}'s. | 93 |
Severity | Category | Rule | Message | Line |
---|---|---|---|---|
![]() |
misc | FinalParameters | Parameter description should be final. | 52 |
![]() |
misc | FinalParameters | Parameter simulator should be final. | 52 |
![]() |
blocks | NeedBraces | 'if' construct must use '{}'s. | 62 |
![]() |
blocks | NeedBraces | 'else' construct must use '{}'s. | 64 |
![]() |
blocks | NeedBraces | 'if' construct must use '{}'s. | 120 |
![]() |
blocks | NeedBraces | 'if' construct must use '{}'s. | 122 |
![]() |
blocks | NeedBraces | 'if' construct must use '{}'s. | 125 |
Severity | Category | Rule | Message | Line |
---|---|---|---|---|
![]() |
blocks | NeedBraces | 'if' construct must use '{}'s. | 91 |
![]() |
blocks | NeedBraces | 'else' construct must use '{}'s. | 93 |
![]() |
blocks | NeedBraces | 'if' construct must use '{}'s. | 262 |
![]() |
blocks | NeedBraces | 'if' construct must use '{}'s. | 264 |
Severity | Category | Rule | Message | Line |
---|---|---|---|---|
![]() |
misc | FinalParameters | Parameter description should be final. | 34 |
![]() |
misc | FinalParameters | Parameter simulator should be final. | 34 |
Severity | Category | Rule | Message | Line |
---|---|---|---|---|
![]() |
misc | FinalParameters | Parameter name should be final. | 25 |
![]() |
misc | FinalParameters | Parameter location should be final. | 25 |
Severity | Category | Rule | Message | Line |
---|---|---|---|---|
![]() |
javadoc | JavadocVariable | Missing a Javadoc comment. | 6 |
![]() |
javadoc | JavadocVariable | Missing a Javadoc comment. | 8 |
![]() |
javadoc | JavadocVariable | Missing a Javadoc comment. | 10 |
![]() |
javadoc | JavadocVariable | Missing a Javadoc comment. | 12 |
![]() |
javadoc | JavadocVariable | Missing a Javadoc comment. | 14 |
![]() |
javadoc | JavadocVariable | Missing a Javadoc comment. | 16 |
![]() |
javadoc | JavadocVariable | Missing a Javadoc comment. | 18 |
![]() |
misc | FinalParameters | Parameter feePortPerFullMove should be final. | 20 |
![]() |
misc | FinalParameters | Parameter feePortPerEmptyMove should be final. | 20 |
![]() |
misc | FinalParameters | Parameter feePortPerDWT should be final. | 20 |
![]() |
misc | FinalParameters | Parameter feeStrategy should be final. | 20 |
![]() |
design | DesignForExtension | Class 'TerminalPortTax' looks like designed for extension (can be subclassed), but the method 'getFeePortPerShipDWT' does not have javadoc that explains how to do that safely. If class is not designed for extension consider making the class 'TerminalPortTax' final or making the method 'getFeePortPerShipDWT' static/final/abstract/empty, or adding allowed annotation for the method. | 51 |
![]() |
design | DesignForExtension | Class 'TerminalPortTax' looks like designed for extension (can be subclassed), but the method 'getFeePortPerFullMove' does not have javadoc that explains how to do that safely. If class is not designed for extension consider making the class 'TerminalPortTax' final or making the method 'getFeePortPerFullMove' static/final/abstract/empty, or adding allowed annotation for the method. | 57 |
![]() |
design | DesignForExtension | Class 'TerminalPortTax' looks like designed for extension (can be subclassed), but the method 'getFeePortPerEmptyMove' does not have javadoc that explains how to do that safely. If class is not designed for extension consider making the class 'TerminalPortTax' final or making the method 'getFeePortPerEmptyMove' static/final/abstract/empty, or adding allowed annotation for the method. | 62 |
![]() |
design | DesignForExtension | Class 'TerminalPortTax' looks like designed for extension (can be subclassed), but the method 'isUsePortFeePerEmptyMove' does not have javadoc that explains how to do that safely. If class is not designed for extension consider making the class 'TerminalPortTax' final or making the method 'isUsePortFeePerEmptyMove' static/final/abstract/empty, or adding allowed annotation for the method. | 67 |
![]() |
design | DesignForExtension | Class 'TerminalPortTax' looks like designed for extension (can be subclassed), but the method 'isUsePortFeePerFullMove' does not have javadoc that explains how to do that safely. If class is not designed for extension consider making the class 'TerminalPortTax' final or making the method 'isUsePortFeePerFullMove' static/final/abstract/empty, or adding allowed annotation for the method. | 72 |
![]() |
design | DesignForExtension | Class 'TerminalPortTax' looks like designed for extension (can be subclassed), but the method 'isUsePortFeePerShipDWT' does not have javadoc that explains how to do that safely. If class is not designed for extension consider making the class 'TerminalPortTax' final or making the method 'isUsePortFeePerShipDWT' static/final/abstract/empty, or adding allowed annotation for the method. | 77 |
![]() |
design | DesignForExtension | Class 'TerminalPortTax' looks like designed for extension (can be subclassed), but the method 'getFeeStrategy' does not have javadoc that explains how to do that safely. If class is not designed for extension consider making the class 'TerminalPortTax' final or making the method 'getFeeStrategy' static/final/abstract/empty, or adding allowed annotation for the method. | 82 |
Severity | Category | Rule | Message | Line |
---|---|---|---|---|
![]() |
javadoc | JavadocVariable | Missing a Javadoc comment. | 20 |
![]() |
javadoc | JavadocVariable | Missing a Javadoc comment. | 21 |
![]() |
javadoc | JavadocVariable | Missing a Javadoc comment. | 22 |