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 |