The following document contains the results of Checkstyle 6.2 with sun_checks.xml ruleset. 
| Category | Rule | Violations | Severity |
|---|---|---|---|
| blocks | EmptyBlock | 2 |  Error |
| LeftCurly | 13 | Error | |
| RightCurly | 2 | Error | |
| coding | HiddenField | 11 | Error |
| MagicNumber | 2 | Error | |
| design | DesignForExtension | 31 | Error |
| javadoc | JavadocMethod | 30 | Error |
| JavadocPackage | 2 | Error | |
| JavadocStyle | 1 | Error | |
| JavadocType | 3 | Error | |
| JavadocVariable | 12 | Error | |
| misc | FinalParameters | 33 | Error |
| NewlineAtEndOfFile | 1 | Error | |
| regexp | RegexpSingleline
|
31 | Error |
| sizes | LineLength | 8 | Error |
| whitespace | FileTabCharacter | 4 | Error |
| OperatorWrap | 2 | Error | |
| WhitespaceAfter | 1 | Error | |
| WhitespaceAround | 20 | Error |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| Error |
javadoc | JavadocType | Missing a Javadoc comment. | 6 |
| Error |
whitespace | FileTabCharacter | File contains tab characters (this is the first instance). | 7 |
| Error |
javadoc | JavadocVariable | Missing a Javadoc comment. | 7 |
| Error |
javadoc | JavadocVariable | Missing a Javadoc comment. | 8 |
| Error |
javadoc | JavadocVariable | Missing a Javadoc comment. | 9 |
| Error |
javadoc | JavadocVariable | Missing a Javadoc comment. | 10 |
| Error |
whitespace | WhitespaceAround | '=' is not preceded with whitespace. | 10 |
| Error |
whitespace | WhitespaceAround | '=' is not followed by whitespace. | 10 |
| Error |
regexp | RegexpSingleline | Line has trailing spaces. | 11 |
| Error |
javadoc | JavadocMethod | Missing a Javadoc comment. | 12 |
| Error |
whitespace | WhitespaceAround | '{' is not followed by whitespace. | 12 |
| Error |
whitespace | WhitespaceAround | '}' is not preceded with whitespace. | 12 |
| Error |
misc | FinalParameters | Parameter id should be final. | 13 |
| Error |
coding | HiddenField | 'id' hides a field. | 13 |
| Error |
blocks | LeftCurly | '{' should have line break after. | 13 |
| Error |
whitespace | WhitespaceAround | '=' is not preceded with whitespace. | 13 |
| Error |
whitespace | WhitespaceAround | '=' is not followed by whitespace. | 13 |
| Error |
regexp | RegexpSingleline | Line has trailing spaces. | 14 |
| Error |
design | DesignForExtension | Method 'getId' is not designed for extension - needs to be abstract, final or empty. | 15 |
| Error |
blocks | LeftCurly | '{' should have line break after. | 15 |
| Error |
design | DesignForExtension | Method 'getFirstName' is not designed for extension - needs to be abstract, final or empty. | 17 |
| Error |
blocks | LeftCurly | '{' should have line break after. | 17 |
| Error |
design | DesignForExtension | Method 'setFirstName' is not designed for extension - needs to be abstract, final or empty. | 18 |
| Error |
javadoc | JavadocMethod | Missing a Javadoc comment. | 18 |
| Error |
misc | FinalParameters | Parameter firstName should be final. | 18 |
| Error |
coding | HiddenField | 'firstName' hides a field. | 18 |
| Error |
regexp | RegexpSingleline | Line has trailing spaces. | 21 |
| Error |
design | DesignForExtension | Method 'getLastName' is not designed for extension - needs to be abstract, final or empty. | 22 |
| Error |
blocks | LeftCurly | '{' should have line break after. | 22 |
| Error |
design | DesignForExtension | Method 'setLastName' is not designed for extension - needs to be abstract, final or empty. | 23 |
| Error |
javadoc | JavadocMethod | Missing a Javadoc comment. | 23 |
| Error |
misc | FinalParameters | Parameter lastName should be final. | 23 |
| Error |
coding | HiddenField | 'lastName' hides a field. | 23 |
| Error |
regexp | RegexpSingleline | Line has trailing spaces. | 26 |
| Error |
design | DesignForExtension | Method 'getBooks' is not designed for extension - needs to be abstract, final or empty. | 27 |
| Error |
blocks | LeftCurly | '{' should have line break after. | 27 |
| Error |
design | DesignForExtension | Method 'setBooks' is not designed for extension - needs to be abstract, final or empty. | 28 |
| Error |
javadoc | JavadocMethod | Missing a Javadoc comment. | 28 |
| Error |
misc | FinalParameters | Parameter books should be final. | 28 |
| Error |
coding | HiddenField | 'books' hides a field. | 28 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| Error |
javadoc | JavadocPackage | Missing package-info.java file. | |
| Error |
regexp | RegexpSingleline | Line has trailing spaces. | 4 |
| Error |
regexp | RegexpSingleline | Line has trailing spaces. | 5 |
| Error |
javadoc | JavadocVariable | Missing a Javadoc comment. | 9 |
| Error |
javadoc | JavadocVariable | Missing a Javadoc comment. | 10 |
| Error |
javadoc | JavadocVariable | Missing a Javadoc comment. | 11 |
| Error |
javadoc | JavadocVariable | Missing a Javadoc comment. | 12 |
| Error |
regexp | RegexpSingleline | Line has trailing spaces. | 13 |
| Error |
javadoc | JavadocMethod | Missing a Javadoc comment. | 14 |
| Error |
whitespace | WhitespaceAround | '{' is not followed by whitespace. | 14 |
| Error |
whitespace | WhitespaceAround | '}' is not preceded with whitespace. | 14 |
| Error |
misc | FinalParameters | Parameter id should be final. | 15 |
| Error |
coding | HiddenField | 'id' hides a field. | 15 |
| Error |
blocks | LeftCurly | '{' should have line break after. | 15 |
| Error |
regexp | RegexpSingleline | Line has trailing spaces. | 16 |
| Error |
design | DesignForExtension | Method 'getId' is not designed for extension - needs to be abstract, final or empty. | 17 |
| Error |
blocks | LeftCurly | '{' should have line break after. | 17 |
| Error |
regexp | RegexpSingleline | Line has trailing spaces. | 18 |
| Error |
design | DesignForExtension | Method 'getDescription' is not designed for extension - needs to be abstract, final or empty. | 19 |
| Error |
blocks | LeftCurly | '{' should have line break after. | 19 |
| Error |
design | DesignForExtension | Method 'setDescription' is not designed for extension - needs to be abstract, final or empty. | 20 |
| Error |
javadoc | JavadocMethod | Missing a Javadoc comment. | 20 |
| Error |
misc | FinalParameters | Parameter description should be final. | 20 |
| Error |
coding | HiddenField | 'description' hides a field. | 20 |
| Error |
regexp | RegexpSingleline | Line has trailing spaces. | 23 |
| Error |
design | DesignForExtension | Method 'getPages' is not designed for extension - needs to be abstract, final or empty. | 24 |
| Error |
blocks | LeftCurly | '{' should have line break after. | 24 |
| Error |
design | DesignForExtension | Method 'setPages' is not designed for extension - needs to be abstract, final or empty. | 25 |
| Error |
javadoc | JavadocMethod | Missing a Javadoc comment. | 25 |
| Error |
misc | FinalParameters | Parameter pages should be final. | 25 |
| Error |
coding | HiddenField | 'pages' hides a field. | 25 |
| Error |
regexp | RegexpSingleline | Line has trailing spaces. | 28 |
| Error |
design | DesignForExtension | Method 'getTitle' is not designed for extension - needs to be abstract, final or empty. | 29 |
| Error |
blocks | LeftCurly | '{' should have line break after. | 29 |
| Error |
design | DesignForExtension | Method 'setTitle' is not designed for extension - needs to be abstract, final or empty. | 30 |
| Error |
javadoc | JavadocMethod | Missing a Javadoc comment. | 30 |
| Error |
misc | FinalParameters | Parameter title should be final. | 30 |
| Error |
coding | HiddenField | 'title' hides a field. | 30 |
| Error |
design | DesignForExtension | Method 'toString' is not designed for extension - needs to be abstract, final or empty. | 34 |
| Error |
javadoc | JavadocMethod | Missing a Javadoc comment. | 34 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| Error |
misc | NewlineAtEndOfFile | File does not end with a newline. | |
| Error |
regexp | RegexpSingleline | Line has trailing spaces. | 9 |
| Error |
regexp | RegexpSingleline | Line has trailing spaces. | 10 |
| Error |
javadoc | JavadocMethod | Expected an @return tag. | 16 |
| Error |
javadoc | JavadocMethod | Expected @param tag for 'book'. | 16 |
| Error |
javadoc | JavadocMethod | Expected @throws tag for 'PersistenceException'. | 16 |
| Error |
javadoc | JavadocMethod | Expected an @return tag. | 21 |
| Error |
javadoc | JavadocMethod | Expected @param tag for 'book'. | 21 |
| Error |
javadoc | JavadocMethod | Expected @throws tag for 'PersistenceException'. | 21 |
| Error |
javadoc | JavadocMethod | Expected an @return tag. | 26 |
| Error |
javadoc | JavadocMethod | Expected @param tag for 'id'. | 26 |
| Error |
javadoc | JavadocMethod | Expected @throws tag for 'PersistenceException'. | 26 |
| Error |
javadoc | JavadocMethod | Expected @param tag for 'book'. | 31 |
| Error |
javadoc | JavadocMethod | Expected @throws tag for 'PersistenceException'. | 31 |
| Error |
regexp | RegexpSingleline | Line has trailing spaces. | 32 |
| Error |
javadoc | JavadocMethod | Expected an @return tag. | 37 |
| Error |
javadoc | JavadocMethod | Expected @param tag for 'start'. | 37 |
| Error |
javadoc | JavadocMethod | Expected @param tag for 'count'. | 37 |
| Error |
javadoc | JavadocMethod | Expected @throws tag for 'PersistenceException'. | 37 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| Error |
regexp | RegexpSingleline | Line has trailing spaces. | 4 |
| Error |
regexp | RegexpSingleline | Line has trailing spaces. | 5 |
| Error |
regexp | RegexpSingleline | Line has trailing spaces. | 6 |
| Error |
regexp | RegexpSingleline | Line has trailing spaces. | 9 |
| Error |
javadoc | JavadocVariable | Missing a Javadoc comment. | 13 |
| Error |
javadoc | JavadocMethod | Missing a Javadoc comment. | 14 |
| Error |
whitespace | WhitespaceAround | '{' is not followed by whitespace. | 14 |
| Error |
whitespace | WhitespaceAround | '}' is not preceded with whitespace. | 14 |
| Error |
misc | FinalParameters | Parameter message should be final. | 15 |
| Error |
blocks | LeftCurly | '{' should have line break after. | 15 |
| Error |
regexp | RegexpSingleline | Line has trailing spaces. | 16 |
| Error |
regexp | RegexpSingleline | Line has trailing spaces. | 17 |
| Error |
regexp | RegexpSingleline | Line has trailing spaces. | 22 |
| Error |
javadoc | JavadocMethod | Missing a Javadoc comment. | 23 |
| Error |
misc | FinalParameters | Parameter message should be final. | 23 |
| Error |
misc | FinalParameters | Parameter rootCause should be final. | 23 |
| Error |
javadoc | JavadocMethod | Missing a Javadoc comment. | 26 |
| Error |
misc | FinalParameters | Parameter rootCause should be final. | 26 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| Error |
javadoc | JavadocType | Missing a Javadoc comment. | 14 |
| Error |
whitespace | FileTabCharacter | File contains tab characters (this is the first instance). | 15 |
| Error |
javadoc | JavadocVariable | Missing a Javadoc comment. | 15 |
| Error |
regexp | RegexpSingleline | Line has trailing spaces. | 16 |
| Error |
design | DesignForExtension | Method 'setConnection' is not designed for extension - needs to be abstract, final or empty. | 17 |
| Error |
javadoc | JavadocMethod | Missing a Javadoc comment. | 17 |
| Error |
misc | FinalParameters | Parameter connection should be final. | 17 |
| Error |
coding | HiddenField | 'connection' hides a field. | 17 |
| Error |
design | DesignForExtension | Method 'create' is not designed for extension - needs to be abstract, final or empty. | 21 |
| Error |
misc | FinalParameters | Parameter book should be final. | 22 |
| Error |
whitespace | WhitespaceAround | '=' is not preceded with whitespace. | 23 |
| Error |
whitespace | WhitespaceAround | '=' is not followed by whitespace. | 23 |
| Error |
whitespace | WhitespaceAround | '=' is not preceded with whitespace. | 26 |
| Error |
whitespace | WhitespaceAround | '=' is not followed by whitespace. | 26 |
| Error |
sizes | LineLength | Line is longer than 80 characters (found 92). | 27 |
| Error |
whitespace | OperatorWrap | '+' should be on a new line. | 27 |
| Error |
coding | MagicNumber | '3' is a magic number. | 31 |
| Error |
regexp | RegexpSingleline | Line has trailing spaces. | 34 |
| Error |
regexp | RegexpSingleline | Line has trailing spaces. | 42 |
| Error |
regexp | RegexpSingleline | Line has trailing spaces. | 44 |
| Error |
sizes | LineLength | Line is longer than 80 characters (found 86). | 45 |
| Error |
sizes | LineLength | Line is longer than 80 characters (found 82). | 47 |
| Error |
sizes | LineLength | Line is longer than 80 characters (found 84). | 49 |
| Error |
sizes | LineLength | Line is longer than 80 characters (found 82). | 53 |
| Error |
blocks | LeftCurly | '{' should have line break after. | 55 |
| Error |
blocks | RightCurly | '}' should have line break before. | 55 |
| Error |
whitespace | WhitespaceAround | '{' is not preceded with whitespace. | 55 |
| Error |
blocks | EmptyBlock | Must have at least one statement. | 55 |
| Error |
whitespace | WhitespaceAround | '{' is not followed by whitespace. | 55 |
| Error |
whitespace | WhitespaceAround | '}' is not preceded with whitespace. | 55 |
| Error |
blocks | LeftCurly | '{' should have line break after. | 56 |
| Error |
blocks | RightCurly | '}' should have line break before. | 56 |
| Error |
whitespace | WhitespaceAround | '{' is not preceded with whitespace. | 56 |
| Error |
blocks | EmptyBlock | Must have at least one statement. | 56 |
| Error |
whitespace | WhitespaceAround | '{' is not followed by whitespace. | 56 |
| Error |
whitespace | WhitespaceAround | '}' is not preceded with whitespace. | 56 |
| Error |
design | DesignForExtension | Method 'update' is not designed for extension - needs to be abstract, final or empty. | 60 |
| Error |
misc | FinalParameters | Parameter book should be final. | 61 |
| Error |
design | DesignForExtension | Method 'get' is not designed for extension - needs to be abstract, final or empty. | 66 |
| Error |
misc | FinalParameters | Parameter id should be final. | 67 |
| Error |
misc | FinalParameters | Parameter book should be final. | 73 |
| Error |
design | DesignForExtension | Method 'findAll' is not designed for extension - needs to be abstract, final or empty. | 78 |
| Error |
sizes | LineLength | Line is longer than 80 characters (found 85). | 79 |
| Error |
misc | FinalParameters | Parameter start should be final. | 79 |
| Error |
misc | FinalParameters | Parameter count should be final. | 79 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| Error |
javadoc | JavadocStyle | First sentence should end with a period. | 9 |
| Error |
whitespace | FileTabCharacter | File contains tab characters (this is the first instance). | 13 |
| Error |
javadoc | JavadocVariable | Missing a Javadoc comment. | 13 |
| Error |
regexp | RegexpSingleline | Line has trailing spaces. | 14 |
| Error |
design | DesignForExtension | Method 'setEntityManager' is not designed for extension - needs to be abstract, final or empty. | 15 |
| Error |
javadoc | JavadocMethod | Missing a Javadoc comment. | 15 |
| Error |
misc | FinalParameters | Parameter em should be final. | 15 |
| Error |
coding | HiddenField | 'em' hides a field. | 15 |
| Error |
design | DesignForExtension | Method 'create' is not designed for extension - needs to be abstract, final or empty. | 19 |
| Error |
misc | FinalParameters | Parameter book should be final. | 20 |
| Error |
design | DesignForExtension | Method 'update' is not designed for extension - needs to be abstract, final or empty. | 25 |
| Error |
misc | FinalParameters | Parameter book should be final. | 26 |
| Error |
design | DesignForExtension | Method 'get' is not designed for extension - needs to be abstract, final or empty. | 30 |
| Error |
misc | FinalParameters | Parameter id should be final. | 31 |
| Error |
design | DesignForExtension | Method 'remove' is not designed for extension - needs to be abstract, final or empty. | 35 |
| Error |
misc | FinalParameters | Parameter book should be final. | 36 |
| Error |
design | DesignForExtension | Method 'findAll' is not designed for extension - needs to be abstract, final or empty. | 40 |
| Error |
misc | FinalParameters | Parameter offset should be final. | 41 |
| Error |
misc | FinalParameters | Parameter limit should be final. | 41 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| Error |
javadoc | JavadocPackage | Missing package-info.java file. | |
| Error |
javadoc | JavadocType | Missing a Javadoc comment. | 11 |
| Error |
whitespace | FileTabCharacter | File contains tab characters (this is the first instance). | 12 |
| Error |
javadoc | JavadocVariable | Missing a Javadoc comment. | 12 |
| Error |
regexp | RegexpSingleline | Line has trailing spaces. | 13 |
| Error |
design | DesignForExtension | Method 'setEntityManager' is not designed for extension - needs to be abstract, final or empty. | 14 |
| Error |
javadoc | JavadocMethod | Missing a Javadoc comment. | 14 |
| Error |
misc | FinalParameters | Parameter em should be final. | 14 |
| Error |
coding | HiddenField | 'em' hides a field. | 14 |
| Error |
regexp | RegexpSingleline | Line has trailing spaces. | 17 |
| Error |
design | DesignForExtension | Method 'create' is not designed for extension - needs to be abstract, final or empty. | 18 |
| Error |
misc | FinalParameters | Parameter book should be final. | 19 |
| Error |
sizes | LineLength | Line is longer than 80 characters (found 84). | 21 |
| Error |
whitespace | OperatorWrap | '+' should be on a new line. | 21 |
| Error |
coding | MagicNumber | '3' is a magic number. | 25 |
| Error |
regexp | RegexpSingleline | Line has trailing spaces. | 27 |
| Error |
whitespace | WhitespaceAfter | 'cast' is not followed by whitespace. | 28 |
| Error |
regexp | RegexpSingleline | Line has trailing spaces. | 36 |
| Error |
regexp | RegexpSingleline | Line has trailing spaces. | 37 |
| Error |
design | DesignForExtension | Method 'update' is not designed for extension - needs to be abstract, final or empty. | 41 |
| Error |
misc | FinalParameters | Parameter book should be final. | 42 |
| Error |
design | DesignForExtension | Method 'get' is not designed for extension - needs to be abstract, final or empty. | 47 |
| Error |
misc | FinalParameters | Parameter id should be final. | 48 |
| Error |
misc | FinalParameters | Parameter book should be final. | 54 |
| Error |
design | DesignForExtension | Method 'findAll' is not designed for extension - needs to be abstract, final or empty. | 59 |
| Error |
sizes | LineLength | Line is longer than 80 characters (found 85). | 60 |
| Error |
misc | FinalParameters | Parameter start should be final. | 60 |
| Error |
misc | FinalParameters | Parameter count should be final. | 60 |
Info
Warnings