The following document contains the results of Checkstyle 6.2 with sun_checks.xml ruleset. 
| Category | Rule | Violations | Severity |
|---|---|---|---|
| coding | HiddenField | 26 | |
| MagicNumber | 1 | ||
| design | DesignForExtension | 42 | |
| imports | AvoidStarImport | 4 | |
| javadoc | JavadocMethod | 47 | |
| JavadocPackage | 2 | ||
| JavadocType | 5 | ||
| JavadocVariable | 23 | ||
| misc | FinalParameters | 26 | |
| regexp | RegexpSingleline
|
16 | |
| whitespace | OperatorWrap | 2 | |
| WhitespaceAround | 59 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| javadoc | JavadocPackage | Missing package-info.java file. | ||
| javadoc | JavadocType | Missing a Javadoc comment. | 7 | |
| javadoc | JavadocVariable | Missing a Javadoc comment. | 8 | |
| javadoc | JavadocVariable | Missing a Javadoc comment. | 9 | |
| javadoc | JavadocVariable | Missing a Javadoc comment. | 10 | |
| javadoc | JavadocVariable | Missing a Javadoc comment. | 11 | |
| javadoc | JavadocVariable | Missing a Javadoc comment. | 12 | |
| regexp | RegexpSingleline | Line has trailing spaces. | 13 | |
| javadoc | JavadocMethod | Missing a Javadoc comment. | 14 | |
| misc | FinalParameters | Parameter saleId should be final. | 14 | |
| coding | HiddenField | 'saleId' hides a field. | 14 | |
| misc | FinalParameters | Parameter customerId should be final. | 14 | |
| coding | HiddenField | 'customerId' hides a field. | 14 | |
| misc | FinalParameters | Parameter date should be final. | 14 | |
| coding | HiddenField | 'date' hides a field. | 14 | |
| misc | FinalParameters | Parameter amount should be final. | 14 | |
| coding | HiddenField | 'amount' hides a field. | 14 | |
| javadoc | JavadocMethod | Missing a Javadoc comment. | 17 | |
| misc | FinalParameters | Parameter saleId should be final. | 17 | |
| coding | HiddenField | 'saleId' hides a field. | 17 | |
| misc | FinalParameters | Parameter customerId should be final. | 17 | |
| coding | HiddenField | 'customerId' hides a field. | 17 | |
| misc | FinalParameters | Parameter date should be final. | 17 | |
| coding | HiddenField | 'date' hides a field. | 17 | |
| misc | FinalParameters | Parameter amount should be final. | 17 | |
| coding | HiddenField | 'amount' hides a field. | 17 | |
| design | DesignForExtension | Method 'toString' is not designed for extension - needs to be abstract, final or empty. | 23 | |
| javadoc | JavadocMethod | Missing a Javadoc comment. | 23 | |
| design | DesignForExtension | Method 'getAmount' is not designed for extension - needs to be abstract, final or empty. | 31 | |
| javadoc | JavadocMethod | Missing a Javadoc comment. | 31 | |
| design | DesignForExtension | Method 'getCustomerId' is not designed for extension - needs to be abstract, final or empty. | 34 | |
| javadoc | JavadocMethod | Missing a Javadoc comment. | 34 | |
| design | DesignForExtension | Method 'getDate' is not designed for extension - needs to be abstract, final or empty. | 37 | |
| javadoc | JavadocMethod | Missing a Javadoc comment. | 37 | |
| design | DesignForExtension | Method 'getSaleId' is not designed for extension - needs to be abstract, final or empty. | 40 | |
| javadoc | JavadocMethod | Missing a Javadoc comment. | 40 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| javadoc | JavadocPackage | Missing package-info.java file. | ||
| imports | AvoidStarImport | Using the '.*' form of import should be avoided - javax.persistence.*. | 7 | |
| javadoc | JavadocType | Missing a Javadoc comment. | 10 | |
| whitespace | WhitespaceAround | '=' is not preceded with whitespace. | 10 | |
| whitespace | WhitespaceAround | '=' is not followed by whitespace. | 10 | |
| javadoc | JavadocVariable | Missing a Javadoc comment. | 12 | |
| javadoc | JavadocVariable | Missing a Javadoc comment. | 13 | |
| javadoc | JavadocVariable | Missing a Javadoc comment. | 14 | |
| javadoc | JavadocVariable | Missing a Javadoc comment. | 15 | |
| javadoc | JavadocVariable | Missing a Javadoc comment. | 16 | |
| javadoc | JavadocVariable | Missing a Javadoc comment. | 17 | |
| design | DesignForExtension | Method 'getId' is not designed for extension - needs to be abstract, final or empty. | 19 | |
| javadoc | JavadocMethod | Missing a Javadoc comment. | 19 | |
| whitespace | WhitespaceAround | '=' is not preceded with whitespace. | 19 | |
| whitespace | WhitespaceAround | '=' is not followed by whitespace. | 19 | |
| javadoc | JavadocMethod | Missing a Javadoc comment. | 23 | |
| misc | FinalParameters | Parameter id should be final. | 24 | |
| coding | HiddenField | 'id' hides a field. | 24 | |
| design | DesignForExtension | Method 'getSales' is not designed for extension - needs to be abstract, final or empty. | 27 | |
| javadoc | JavadocMethod | Missing a Javadoc comment. | 27 | |
| whitespace | WhitespaceAround | '=' is not preceded with whitespace. | 27 | |
| whitespace | WhitespaceAround | '=' is not followed by whitespace. | 27 | |
| whitespace | WhitespaceAround | '=' is not preceded with whitespace. | 27 | |
| whitespace | WhitespaceAround | '=' is not followed by whitespace. | 27 | |
| design | DesignForExtension | Method 'setSales' is not designed for extension - needs to be abstract, final or empty. | 31 | |
| javadoc | JavadocMethod | Missing a Javadoc comment. | 31 | |
| misc | FinalParameters | Parameter sales should be final. | 31 | |
| coding | HiddenField | 'sales' hides a field. | 31 | |
| regexp | RegexpSingleline | Line has trailing spaces. | 34 | |
| design | DesignForExtension | Method 'getHireDate' is not designed for extension - needs to be abstract, final or empty. | 35 | |
| javadoc | JavadocMethod | Missing a Javadoc comment. | 35 | |
| design | DesignForExtension | Method 'setHireDate' is not designed for extension - needs to be abstract, final or empty. | 39 | |
| javadoc | JavadocMethod | Missing a Javadoc comment. | 39 | |
| misc | FinalParameters | Parameter hireDate should be final. | 39 | |
| coding | HiddenField | 'hireDate' hides a field. | 39 | |
| design | DesignForExtension | Method 'getTermDate' is not designed for extension - needs to be abstract, final or empty. | 42 | |
| javadoc | JavadocMethod | Missing a Javadoc comment. | 42 | |
| design | DesignForExtension | Method 'setTermDate' is not designed for extension - needs to be abstract, final or empty. | 46 | |
| javadoc | JavadocMethod | Missing a Javadoc comment. | 46 | |
| misc | FinalParameters | Parameter termDate should be final. | 46 | |
| coding | HiddenField | 'termDate' hides a field. | 46 | |
| regexp | RegexpSingleline | Line has trailing spaces. | 48 | |
| design | DesignForExtension | Method 'getFirstName' is not designed for extension - needs to be abstract, final or empty. | 50 | |
| javadoc | JavadocMethod | Missing a Javadoc comment. | 50 | |
| design | DesignForExtension | Method 'setFirstName' is not designed for extension - needs to be abstract, final or empty. | 53 | |
| javadoc | JavadocMethod | Missing a Javadoc comment. | 53 | |
| misc | FinalParameters | Parameter firstName should be final. | 53 | |
| coding | HiddenField | 'firstName' hides a field. | 53 | |
| regexp | RegexpSingleline | Line has trailing spaces. | 55 | |
| design | DesignForExtension | Method 'getLastName' is not designed for extension - needs to be abstract, final or empty. | 56 | |
| javadoc | JavadocMethod | Missing a Javadoc comment. | 56 | |
| design | DesignForExtension | Method 'setLastName' is not designed for extension - needs to be abstract, final or empty. | 59 | |
| javadoc | JavadocMethod | Missing a Javadoc comment. | 59 | |
| misc | FinalParameters | Parameter lastName should be final. | 59 | |
| coding | HiddenField | 'lastName' hides a field. | 59 | |
| design | DesignForExtension | Method 'toString' is not designed for extension - needs to be abstract, final or empty. | 63 | |
| javadoc | JavadocMethod | Missing a Javadoc comment. | 63 | |
| whitespace | WhitespaceAround | 'for' is not followed by whitespace. | 71 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| imports | AvoidStarImport | Using the '.*' form of import should be avoided - javax.persistence.*. | 3 | |
| javadoc | JavadocType | Missing a Javadoc comment. | 5 | |
| whitespace | WhitespaceAround | '=' is not preceded with whitespace. | 5 | |
| whitespace | WhitespaceAround | '=' is not followed by whitespace. | 5 | |
| whitespace | WhitespaceAround | '=' is not preceded with whitespace. | 7 | |
| whitespace | WhitespaceAround | '=' is not followed by whitespace. | 7 | |
| whitespace | WhitespaceAround | '=' is not preceded with whitespace. | 8 | |
| whitespace | WhitespaceAround | '=' is not followed by whitespace. | 8 | |
| whitespace | OperatorWrap | '+' should be on a new line. | 8 | |
| whitespace | WhitespaceAround | '=' is not preceded with whitespace. | 10 | |
| whitespace | WhitespaceAround | '=' is not followed by whitespace. | 10 | |
| whitespace | WhitespaceAround | '=' is not preceded with whitespace. | 11 | |
| whitespace | WhitespaceAround | '=' is not followed by whitespace. | 11 | |
| whitespace | OperatorWrap | '+' should be on a new line. | 11 | |
| javadoc | JavadocVariable | Missing a Javadoc comment. | 15 | |
| javadoc | JavadocVariable | Missing a Javadoc comment. | 16 | |
| javadoc | JavadocVariable | Missing a Javadoc comment. | 17 | |
| regexp | RegexpSingleline | Line has trailing spaces. | 18 | |
| design | DesignForExtension | Method 'getId' is not designed for extension - needs to be abstract, final or empty. | 19 | |
| javadoc | JavadocMethod | Missing a Javadoc comment. | 19 | |
| whitespace | WhitespaceAround | '=' is not preceded with whitespace. | 19 | |
| whitespace | WhitespaceAround | '=' is not followed by whitespace. | 19 | |
| javadoc | JavadocMethod | Missing a Javadoc comment. | 23 | |
| misc | FinalParameters | Parameter id should be final. | 24 | |
| coding | HiddenField | 'id' hides a field. | 24 | |
| design | DesignForExtension | Method 'getFirstName' is not designed for extension - needs to be abstract, final or empty. | 28 | |
| javadoc | JavadocMethod | Missing a Javadoc comment. | 28 | |
| design | DesignForExtension | Method 'setFirstName' is not designed for extension - needs to be abstract, final or empty. | 31 | |
| javadoc | JavadocMethod | Missing a Javadoc comment. | 31 | |
| misc | FinalParameters | Parameter firstName should be final. | 31 | |
| coding | HiddenField | 'firstName' hides a field. | 31 | |
| design | DesignForExtension | Method 'getLastName' is not designed for extension - needs to be abstract, final or empty. | 34 | |
| javadoc | JavadocMethod | Missing a Javadoc comment. | 34 | |
| design | DesignForExtension | Method 'setLastName' is not designed for extension - needs to be abstract, final or empty. | 37 | |
| javadoc | JavadocMethod | Missing a Javadoc comment. | 37 | |
| misc | FinalParameters | Parameter lastName should be final. | 37 | |
| coding | HiddenField | 'lastName' hides a field. | 37 | |
| regexp | RegexpSingleline | Line has trailing spaces. | 40 | |
| design | DesignForExtension | Method 'toString' is not designed for extension - needs to be abstract, final or empty. | 41 | |
| javadoc | JavadocMethod | Missing a Javadoc comment. | 41 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| imports | AvoidStarImport | Using the '.*' form of import should be avoided - javax.persistence.*. | 8 | |
| javadoc | JavadocType | Missing a Javadoc comment. | 10 | |
| whitespace | WhitespaceAround | '=' is not preceded with whitespace. | 11 | |
| whitespace | WhitespaceAround | '=' is not followed by whitespace. | 11 | |
| javadoc | JavadocVariable | Missing a Javadoc comment. | 13 | |
| javadoc | JavadocVariable | Missing a Javadoc comment. | 14 | |
| javadoc | JavadocVariable | Missing a Javadoc comment. | 15 | |
| javadoc | JavadocVariable | Missing a Javadoc comment. | 16 | |
| javadoc | JavadocVariable | Missing a Javadoc comment. | 17 | |
| javadoc | JavadocVariable | Missing a Javadoc comment. | 18 | |
| regexp | RegexpSingleline | Line has trailing spaces. | 19 | |
| design | DesignForExtension | Method 'getId' is not designed for extension - needs to be abstract, final or empty. | 20 | |
| javadoc | JavadocMethod | Missing a Javadoc comment. | 20 | |
| whitespace | WhitespaceAround | '=' is not preceded with whitespace. | 20 | |
| whitespace | WhitespaceAround | '=' is not followed by whitespace. | 20 | |
| design | DesignForExtension | Method 'setId' is not designed for extension - needs to be abstract, final or empty. | 24 | |
| javadoc | JavadocMethod | Missing a Javadoc comment. | 24 | |
| misc | FinalParameters | Parameter id should be final. | 24 | |
| coding | HiddenField | 'id' hides a field. | 24 | |
| design | DesignForExtension | Method 'getClerks' is not designed for extension - needs to be abstract, final or empty. | 27 | |
| javadoc | JavadocMethod | Missing a Javadoc comment. | 27 | |
| whitespace | WhitespaceAround | '=' is not preceded with whitespace. | 27 | |
| whitespace | WhitespaceAround | '=' is not followed by whitespace. | 27 | |
| whitespace | WhitespaceAround | '=' is not preceded with whitespace. | 28 | |
| whitespace | WhitespaceAround | '=' is not followed by whitespace. | 28 | |
| whitespace | WhitespaceAround | '=' is not preceded with whitespace. | 29 | |
| whitespace | WhitespaceAround | '=' is not followed by whitespace. | 29 | |
| whitespace | WhitespaceAround | '=' is not preceded with whitespace. | 29 | |
| whitespace | WhitespaceAround | '=' is not followed by whitespace. | 29 | |
| whitespace | WhitespaceAround | '=' is not preceded with whitespace. | 30 | |
| whitespace | WhitespaceAround | '=' is not followed by whitespace. | 30 | |
| whitespace | WhitespaceAround | '=' is not preceded with whitespace. | 30 | |
| whitespace | WhitespaceAround | '=' is not followed by whitespace. | 30 | |
| design | DesignForExtension | Method 'setClerks' is not designed for extension - needs to be abstract, final or empty. | 35 | |
| javadoc | JavadocMethod | Missing a Javadoc comment. | 35 | |
| misc | FinalParameters | Parameter clerks should be final. | 35 | |
| coding | HiddenField | 'clerks' hides a field. | 35 | |
| design | DesignForExtension | Method 'getStore' is not designed for extension - needs to be abstract, final or empty. | 38 | |
| javadoc | JavadocMethod | Missing a Javadoc comment. | 38 | |
| whitespace | WhitespaceAround | '=' is not preceded with whitespace. | 38 | |
| whitespace | WhitespaceAround | '=' is not followed by whitespace. | 38 | |
| whitespace | WhitespaceAround | '=' is not preceded with whitespace. | 39 | |
| whitespace | WhitespaceAround | '=' is not followed by whitespace. | 39 | |
| design | DesignForExtension | Method 'setStore' is not designed for extension - needs to be abstract, final or empty. | 43 | |
| javadoc | JavadocMethod | Missing a Javadoc comment. | 43 | |
| misc | FinalParameters | Parameter store should be final. | 43 | |
| coding | HiddenField | 'store' hides a field. | 43 | |
| regexp | RegexpSingleline | Line has trailing spaces. | 45 | |
| design | DesignForExtension | Method 'getAmount' is not designed for extension - needs to be abstract, final or empty. | 47 | |
| javadoc | JavadocMethod | Missing a Javadoc comment. | 47 | |
| whitespace | WhitespaceAround | '=' is not preceded with whitespace. | 47 | |
| coding | MagicNumber | '5' is a magic number. | 47 | |
| whitespace | WhitespaceAround | '=' is not followed by whitespace. | 47 | |
| whitespace | WhitespaceAround | '=' is not preceded with whitespace. | 47 | |
| whitespace | WhitespaceAround | '=' is not followed by whitespace. | 47 | |
| design | DesignForExtension | Method 'setAmount' is not designed for extension - needs to be abstract, final or empty. | 51 | |
| javadoc | JavadocMethod | Missing a Javadoc comment. | 51 | |
| misc | FinalParameters | Parameter amount should be final. | 51 | |
| coding | HiddenField | 'amount' hides a field. | 51 | |
| design | DesignForExtension | Method 'getBuyerId' is not designed for extension - needs to be abstract, final or empty. | 54 | |
| javadoc | JavadocMethod | Missing a Javadoc comment. | 54 | |
| whitespace | WhitespaceAround | '=' is not preceded with whitespace. | 54 | |
| whitespace | WhitespaceAround | '=' is not followed by whitespace. | 54 | |
| design | DesignForExtension | Method 'setBuyerId' is not designed for extension - needs to be abstract, final or empty. | 58 | |
| javadoc | JavadocMethod | Missing a Javadoc comment. | 58 | |
| misc | FinalParameters | Parameter buyerId should be final. | 58 | |
| coding | HiddenField | 'buyerId' hides a field. | 58 | |
| regexp | RegexpSingleline | Line has trailing spaces. | 61 | |
| design | DesignForExtension | Method 'getDate' is not designed for extension - needs to be abstract, final or empty. | 62 | |
| javadoc | JavadocMethod | Missing a Javadoc comment. | 62 | |
| design | DesignForExtension | Method 'setDate' is not designed for extension - needs to be abstract, final or empty. | 66 | |
| javadoc | JavadocMethod | Missing a Javadoc comment. | 66 | |
| misc | FinalParameters | Parameter date should be final. | 66 | |
| coding | HiddenField | 'date' hides a field. | 66 | |
| regexp | RegexpSingleline | Line has trailing spaces. | 69 | |
| design | DesignForExtension | Method 'toString' is not designed for extension - needs to be abstract, final or empty. | 70 | |
| javadoc | JavadocMethod | Missing a Javadoc comment. | 70 | |
| whitespace | WhitespaceAround | 'for' is not followed by whitespace. | 77 | |
| regexp | RegexpSingleline | Line has trailing spaces. | 79 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| imports | AvoidStarImport | Using the '.*' form of import should be avoided - javax.persistence.*. | 6 | |
| javadoc | JavadocType | Missing a Javadoc comment. | 8 | |
| whitespace | WhitespaceAround | '=' is not preceded with whitespace. | 8 | |
| whitespace | WhitespaceAround | '=' is not followed by whitespace. | 8 | |
| javadoc | JavadocVariable | Missing a Javadoc comment. | 10 | |
| javadoc | JavadocVariable | Missing a Javadoc comment. | 11 | |
| javadoc | JavadocVariable | Missing a Javadoc comment. | 12 | |
| design | DesignForExtension | Method 'getId' is not designed for extension - needs to be abstract, final or empty. | 14 | |
| javadoc | JavadocMethod | Missing a Javadoc comment. | 14 | |
| whitespace | WhitespaceAround | '=' is not preceded with whitespace. | 14 | |
| whitespace | WhitespaceAround | '=' is not followed by whitespace. | 14 | |
| javadoc | JavadocMethod | Missing a Javadoc comment. | 18 | |
| misc | FinalParameters | Parameter id should be final. | 19 | |
| coding | HiddenField | 'id' hides a field. | 19 | |
| regexp | RegexpSingleline | Line has trailing spaces. | 23 | |
| design | DesignForExtension | Method 'getSales' is not designed for extension - needs to be abstract, final or empty. | 23 | |
| javadoc | JavadocMethod | Missing a Javadoc comment. | 23 | |
| whitespace | WhitespaceAround | '=' is not preceded with whitespace. | 23 | |
| whitespace | WhitespaceAround | '=' is not followed by whitespace. | 23 | |
| regexp | RegexpSingleline | Line has trailing spaces. | 24 | |
| whitespace | WhitespaceAround | '=' is not preceded with whitespace. | 24 | |
| whitespace | WhitespaceAround | '=' is not followed by whitespace. | 24 | |
| whitespace | WhitespaceAround | '=' is not preceded with whitespace. | 25 | |
| whitespace | WhitespaceAround | '=' is not followed by whitespace. | 25 | |
| design | DesignForExtension | Method 'setSales' is not designed for extension - needs to be abstract, final or empty. | 29 | |
| javadoc | JavadocMethod | Missing a Javadoc comment. | 29 | |
| misc | FinalParameters | Parameter sales should be final. | 29 | |
| coding | HiddenField | 'sales' hides a field. | 29 | |
| regexp | RegexpSingleline | Line has trailing spaces. | 31 | |
| regexp | RegexpSingleline | Line has trailing spaces. | 32 | |
| design | DesignForExtension | Method 'getName' is not designed for extension - needs to be abstract, final or empty. | 33 | |
| javadoc | JavadocMethod | Missing a Javadoc comment. | 33 | |
| design | DesignForExtension | Method 'setName' is not designed for extension - needs to be abstract, final or empty. | 36 | |
| javadoc | JavadocMethod | Missing a Javadoc comment. | 36 | |
| misc | FinalParameters | Parameter name should be final. | 36 | |
| coding | HiddenField | 'name' hides a field. | 36 | |
| regexp | RegexpSingleline | Line has trailing spaces. | 39 | |
| design | DesignForExtension | Method 'toString' is not designed for extension - needs to be abstract, final or empty. | 40 | |
| javadoc | JavadocMethod | Missing a Javadoc comment. | 40 | |
| whitespace | WhitespaceAround | 'for' is not followed by whitespace. | 45 |