The following document contains the results of Checkstyle 6.2 with sun_checks.xml ruleset.
Category | Rule | Violations | Severity |
---|---|---|---|
blocks | RightCurly | 2 | ![]() |
coding | MagicNumber | 1 | ![]() |
design | DesignForExtension | 19 | ![]() |
javadoc | JavadocMethod | 20 | ![]() |
JavadocPackage | 1 | ![]() | |
JavadocType | 4 | ![]() | |
JavadocVariable | 10 | ![]() | |
misc | FinalParameters | 16 | ![]() |
naming | ConstantName | 2 | ![]() |
regexp | RegexpSingleline
|
39 | ![]() |
whitespace | FileTabCharacter | 2 | ![]() |
OperatorWrap | 1 | ![]() | |
WhitespaceAfter | 3 | ![]() | |
WhitespaceAround | 10 | ![]() |
Severity | Category | Rule | Message | Line |
---|---|---|---|---|
![]() |
regexp | RegexpSingleline | Line has trailing spaces. | 26 |
![]() |
javadoc | JavadocVariable | Missing a Javadoc comment. | 34 |
![]() |
regexp | RegexpSingleline | Line has trailing spaces. | 35 |
![]() |
javadoc | JavadocVariable | Missing a Javadoc comment. | 36 |
![]() |
whitespace | WhitespaceAround | '=' is not preceded with whitespace. | 36 |
![]() |
whitespace | WhitespaceAround | '=' is not followed by whitespace. | 36 |
![]() |
regexp | RegexpSingleline | Line has trailing spaces. | 38 |
![]() |
javadoc | JavadocVariable | Missing a Javadoc comment. | 39 |
![]() |
javadoc | JavadocVariable | Missing a Javadoc comment. | 43 |
![]() |
regexp | RegexpSingleline | Line has trailing spaces. | 44 |
![]() |
regexp | RegexpSingleline | Line has trailing spaces. | 46 |
![]() |
design | DesignForExtension | Method 'init' is not designed for extension - needs to be abstract, final or empty. | 51 |
![]() |
regexp | RegexpSingleline | Line has trailing spaces. | 56 |
![]() |
whitespace | WhitespaceAfter | 'cast' is not followed by whitespace. | 58 |
![]() |
whitespace | WhitespaceAfter | 'cast' is not followed by whitespace. | 60 |
![]() |
regexp | RegexpSingleline | Line has trailing spaces. | 62 |
![]() |
regexp | RegexpSingleline | Line has trailing spaces. | 65 |
![]() |
design | DesignForExtension | Method 'close' is not designed for extension - needs to be abstract, final or empty. | 68 |
![]() |
whitespace | WhitespaceAround | '=' is not preceded with whitespace. | 70 |
![]() |
whitespace | WhitespaceAround | '=' is not followed by whitespace. | 70 |
![]() |
regexp | RegexpSingleline | Line has trailing spaces. | 71 |
![]() |
design | DesignForExtension | Method 'cancelReservation' is not designed for extension - needs to be abstract, final or empty. | 77 |
![]() |
misc | FinalParameters | Parameter reservation should be final. | 77 |
![]() |
javadoc | JavadocMethod | Expected @param tag for 'reservation'. | 77 |
![]() |
javadoc | JavadocMethod | Expected @throws tag for 'HotelReservationException'. | 78 |
![]() |
blocks | RightCurly | '}' should be on the same line. | 81 |
![]() |
javadoc | JavadocMethod | Expected an @return tag. | 93 |
![]() |
design | DesignForExtension | Method 'commitReservation' is not designed for extension - needs to be abstract, final or empty. | 93 |
![]() |
misc | FinalParameters | Parameter reservation should be final. | 93 |
![]() |
javadoc | JavadocMethod | Expected @param tag for 'reservation'. | 93 |
![]() |
javadoc | JavadocMethod | Expected @throws tag for 'HotelReservationException'. | 94 |
![]() |
javadoc | JavadocMethod | Expected an @return tag. | 102 |
![]() |
design | DesignForExtension | Method 'createReservation' is not designed for extension - needs to be abstract, final or empty. | 102 |
![]() |
misc | FinalParameters | Parameter person should be final. | 102 |
![]() |
javadoc | JavadocMethod | Expected @param tag for 'person'. | 102 |
![]() |
misc | FinalParameters | Parameter startDate should be final. | 102 |
![]() |
javadoc | JavadocMethod | Expected @param tag for 'startDate'. | 102 |
![]() |
misc | FinalParameters | Parameter endDate should be final. | 103 |
![]() |
javadoc | JavadocMethod | Expected @param tag for 'endDate'. | 103 |
![]() |
javadoc | JavadocMethod | Expected @throws tag for 'HotelReservationException'. | 103 |
![]() |
blocks | RightCurly | '}' should be on the same line. | 106 |
![]() |
design | DesignForExtension | Method 'getReservations' is not designed for extension - needs to be abstract, final or empty. | 115 |
![]() |
javadoc | JavadocMethod | Missing a Javadoc comment. | 115 |
![]() |
misc | FinalParameters | Parameter index should be final. | 115 |
![]() |
misc | FinalParameters | Parameter count should be final. | 115 |
![]() |
design | DesignForExtension | Method 'getReservationsForPerson' is not designed for extension - needs to be abstract, final or empty. | 120 |
![]() |
javadoc | JavadocMethod | Missing a Javadoc comment. | 120 |
![]() |
misc | FinalParameters | Parameter person should be final. | 120 |
![]() |
misc | FinalParameters | Parameter index should be final. | 120 |
![]() |
misc | FinalParameters | Parameter count should be final. | 121 |
![]() |
regexp | RegexpSingleline | Line has trailing spaces. | 125 |
![]() |
design | DesignForExtension | Method 'getReservationByConfirmation' is not designed for extension - needs to be abstract, final or empty. | 125 |
![]() |
javadoc | JavadocMethod | Missing a Javadoc comment. | 125 |
![]() |
misc | FinalParameters | Parameter confirmation should be final. | 125 |
![]() |
regexp | RegexpSingleline | Line has trailing spaces. | 130 |
![]() |
design | DesignForExtension | Method 'cleanupReservation' is not designed for extension - needs to be abstract, final or empty. | 130 |
![]() |
javadoc | JavadocMethod | Missing a Javadoc comment. | 130 |
![]() |
misc | FinalParameters | Parameter confirmation should be final. | 130 |
Severity | Category | Rule | Message | Line |
---|---|---|---|---|
![]() |
javadoc | JavadocType | Missing a Javadoc comment. | 7 |
Severity | Category | Rule | Message | Line |
---|---|---|---|---|
![]() |
javadoc | JavadocType | Missing a Javadoc comment. | 7 |
Severity | Category | Rule | Message | Line |
---|---|---|---|---|
![]() |
regexp | RegexpSingleline | Line has trailing spaces. | 29 |
![]() |
regexp | RegexpSingleline | Line has trailing spaces. | 32 |
![]() |
regexp | RegexpSingleline | Line has trailing spaces. | 34 |
![]() |
regexp | RegexpSingleline | Line has trailing spaces. | 35 |
![]() |
regexp | RegexpSingleline | Line has trailing spaces. | 42 |
![]() |
regexp | RegexpSingleline | Line has trailing spaces. | 43 |
![]() |
regexp | RegexpSingleline | Line has trailing spaces. | 44 |
![]() |
regexp | RegexpSingleline | Line has trailing spaces. | 50 |
![]() |
regexp | RegexpSingleline | Line has trailing spaces. | 51 |
![]() |
regexp | RegexpSingleline | Line has trailing spaces. | 53 |
![]() |
whitespace | WhitespaceAround | '=' is not preceded with whitespace. | 60 |
![]() |
coding | MagicNumber | '3' is a magic number. | 60 |
![]() |
whitespace | WhitespaceAround | '=' is not followed by whitespace. | 60 |
![]() |
whitespace | WhitespaceAround | '=' is not preceded with whitespace. | 60 |
![]() |
whitespace | WhitespaceAround | '=' is not followed by whitespace. | 60 |
![]() |
regexp | RegexpSingleline | Line has trailing spaces. | 64 |
![]() |
javadoc | JavadocVariable | Missing a Javadoc comment. | 64 |
![]() |
naming | ConstantName | Name 'log' must match pattern '^[A-Z][A-Z0-9]*(_[A-Z0-9]+)*$'. | 64 |
![]() |
javadoc | JavadocVariable | Missing a Javadoc comment. | 66 |
![]() |
regexp | RegexpSingleline | Line has trailing spaces. | 67 |
![]() |
javadoc | JavadocVariable | Missing a Javadoc comment. | 68 |
![]() |
regexp | RegexpSingleline | Line has trailing spaces. | 70 |
![]() |
javadoc | JavadocVariable | Missing a Javadoc comment. | 72 |
![]() |
regexp | RegexpSingleline | Line has trailing spaces. | 73 |
![]() |
regexp | RegexpSingleline | Line has trailing spaces. | 77 |
![]() |
design | DesignForExtension | Method 'init' is not designed for extension - needs to be abstract, final or empty. | 81 |
![]() |
regexp | RegexpSingleline | Line has trailing spaces. | 84 |
![]() |
regexp | RegexpSingleline | Line has trailing spaces. | 88 |
![]() |
whitespace | WhitespaceAfter | 'cast' is not followed by whitespace. | 91 |
![]() |
regexp | RegexpSingleline | Line has trailing spaces. | 93 |
![]() |
regexp | RegexpSingleline | Line has trailing spaces. | 96 |
![]() |
design | DesignForExtension | Method 'closing' is not designed for extension - needs to be abstract, final or empty. | 98 |
![]() |
design | DesignForExtension | Method 'close' is not designed for extension - needs to be abstract, final or empty. | 106 |
![]() |
design | DesignForExtension | Method 'createReservation' is not designed for extension - needs to be abstract, final or empty. | 112 |
![]() |
javadoc | JavadocMethod | Missing a Javadoc comment. | 112 |
![]() |
misc | FinalParameters | Parameter person should be final. | 112 |
![]() |
misc | FinalParameters | Parameter startDate should be final. | 112 |
![]() |
misc | FinalParameters | Parameter endDate should be final. | 112 |
![]() |
regexp | RegexpSingleline | Line has trailing spaces. | 116 |
![]() |
design | DesignForExtension | Method 'cancelReservations' is not designed for extension - needs to be abstract, final or empty. | 117 |
![]() |
javadoc | JavadocMethod | Missing a Javadoc comment. | 117 |
![]() |
design | DesignForExtension | Method 'commit' is not designed for extension - needs to be abstract, final or empty. | 121 |
![]() |
javadoc | JavadocMethod | Missing a Javadoc comment. | 121 |
![]() |
regexp | RegexpSingleline | Line has trailing spaces. | 126 |
![]() |
design | DesignForExtension | Method 'afterBegin' is not designed for extension - needs to be abstract, final or empty. | 129 |
![]() |
regexp | RegexpSingleline | Line has trailing spaces. | 130 |
![]() |
regexp | RegexpSingleline | Line has trailing spaces. | 133 |
![]() |
regexp | RegexpSingleline | Line has trailing spaces. | 134 |
![]() |
design | DesignForExtension | Method 'beforeCompletion' is not designed for extension - needs to be abstract, final or empty. | 136 |
![]() |
javadoc | JavadocMethod | Expected @throws tag for 'EJBException'. | 136 |
![]() |
javadoc | JavadocMethod | Expected @throws tag for 'RemoteException'. | 136 |
![]() |
regexp | RegexpSingleline | Line has trailing spaces. | 137 |
![]() |
whitespace | OperatorWrap | '+' should be on a new line. | 137 |
![]() |
regexp | RegexpSingleline | Line has trailing spaces. | 139 |
![]() |
regexp | RegexpSingleline | Line has trailing spaces. | 141 |
![]() |
design | DesignForExtension | Method 'afterCompletion' is not designed for extension - needs to be abstract, final or empty. | 143 |
![]() |
misc | FinalParameters | Parameter status should be final. | 143 |
![]() |
javadoc | JavadocMethod | Expected @param tag for 'status'. | 143 |
![]() |
regexp | RegexpSingleline | Line has trailing spaces. | 144 |
Severity | Category | Rule | Message | Line |
---|---|---|---|---|
![]() |
javadoc | JavadocType | Missing a Javadoc comment. | 7 |
Severity | Category | Rule | Message | Line |
---|---|---|---|---|
![]() |
javadoc | JavadocType | Missing a Javadoc comment. | 7 |
Severity | Category | Rule | Message | Line |
---|---|---|---|---|
![]() |
javadoc | JavadocPackage | Missing package-info.java file. | |
![]() |
whitespace | FileTabCharacter | File contains tab characters (this is the first instance). | 15 |
![]() |
javadoc | JavadocVariable | Missing a Javadoc comment. | 15 |
![]() |
naming | ConstantName | Name 'log' must match pattern '^[A-Z][A-Z0-9]*(_[A-Z0-9]+)*$'. | 15 |
![]() |
javadoc | JavadocVariable | Missing a Javadoc comment. | 17 |
![]() |
whitespace | WhitespaceAround | '=' is not preceded with whitespace. | 17 |
![]() |
whitespace | WhitespaceAround | '=' is not followed by whitespace. | 17 |
![]() |
design | DesignForExtension | Method 'reset' is not designed for extension - needs to be abstract, final or empty. | 20 |
Severity | Category | Rule | Message | Line |
---|---|---|---|---|
![]() |
whitespace | FileTabCharacter | File contains tab characters (this is the first instance). | 10 |