The following document contains the results of Checkstyle 6.11.2 with sun_checks.xml ruleset. 
| Category | Rule | Violations | Severity |
|---|---|---|---|
| coding | HiddenField | 30 | |
| MagicNumber | 5 | ||
| design | DesignForExtension | 57 | |
| VisibilityModifier | 2 | ||
| javadoc | JavadocMethod | 3 | |
| JavadocPackage | 4 | ||
| JavadocStyle | 22 | ||
| JavadocVariable | 25 | ||
| misc | FinalParameters | 47 | |
| regexp | RegexpSingleline
|
48 | |
| sizes | LineLength | 12 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| regexp | RegexpSingleline | Line has trailing spaces. | 9 | |
| regexp | RegexpSingleline | Line has trailing spaces. | 11 | |
| regexp | RegexpSingleline | Line has trailing spaces. | 16 | |
| regexp | RegexpSingleline | Line has trailing spaces. | 17 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| regexp | RegexpSingleline | Line has trailing spaces. | 9 | |
| regexp | RegexpSingleline | Line has trailing spaces. | 11 | |
| regexp | RegexpSingleline | Line has trailing spaces. | 16 | |
| regexp | RegexpSingleline | Line has trailing spaces. | 17 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| regexp | RegexpSingleline | Line has trailing spaces. | 9 | |
| regexp | RegexpSingleline | Line has trailing spaces. | 11 | |
| regexp | RegexpSingleline | Line has trailing spaces. | 16 | |
| regexp | RegexpSingleline | Line has trailing spaces. | 17 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| regexp | RegexpSingleline | Line has trailing spaces. | 13 | |
| regexp | RegexpSingleline | Line has trailing spaces. | 18 | |
| javadoc | JavadocVariable | Missing a Javadoc comment. | 37 | |
| javadoc | JavadocVariable | Missing a Javadoc comment. | 38 | |
| javadoc | JavadocVariable | Missing a Javadoc comment. | 39 | |
| design | DesignForExtension | Method 'execute' is not designed for extension - needs to be abstract, final or empty. | 42 | |
| design | DesignForExtension | Method 'getPageTitle' is not designed for extension - needs to be abstract, final or empty. | 54 | |
| design | DesignForExtension | Method 'getPpd' is not designed for extension - needs to be abstract, final or empty. | 63 | |
| design | DesignForExtension | Method 'getId' is not designed for extension - needs to be abstract, final or empty. | 72 | |
| design | DesignForExtension | Method 'getStatus' is not designed for extension - needs to be abstract, final or empty. | 81 | |
| sizes | LineLength | Line is longer than 80 characters (found 91). | 83 | |
| design | DesignForExtension | Method 'getProgress' is not designed for extension - needs to be abstract, final or empty. | 91 | |
| design | DesignForExtension | Method 'getEta' is not designed for extension - needs to be abstract, final or empty. | 100 | |
| design | DesignForExtension | Method 'getCredit' is not designed for extension - needs to be abstract, final or empty. | 109 | |
| design | DesignForExtension | Method 'getPrcg' is not designed for extension - needs to be abstract, final or empty. | 118 | |
| sizes | LineLength | Line is longer than 80 characters (found 110). | 119 | |
| design | DesignForExtension | Method 'getDescription' is not designed for extension - needs to be abstract, final or empty. | 127 | |
| design | DesignForExtension | Method 'getControl' is not designed for extension - needs to be abstract, final or empty. | 136 | |
| design | DesignForExtension | Method 'setControl' is not designed for extension - needs to be abstract, final or empty. | 145 | |
| misc | FinalParameters | Parameter control should be final. | 145 | |
| coding | HiddenField | 'control' hides a field. | 145 | |
| design | DesignForExtension | Method 'setConnection' is not designed for extension - needs to be abstract, final or empty. | 149 | |
| javadoc | JavadocMethod | Missing a Javadoc comment. | 149 | |
| misc | FinalParameters | Parameter connection should be final. | 149 | |
| coding | HiddenField | 'connection' hides a field. | 149 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| regexp | RegexpSingleline | Line has trailing spaces. | 13 | |
| regexp | RegexpSingleline | Line has trailing spaces. | 18 | |
| javadoc | JavadocStyle | First sentence should end with a period. | 35 | |
| javadoc | JavadocVariable | Missing a Javadoc comment. | 38 | |
| javadoc | JavadocVariable | Missing a Javadoc comment. | 39 | |
| design | DesignForExtension | Method 'getPageTitle' is not designed for extension - needs to be abstract, final or empty. | 46 | |
| design | DesignForExtension | Method 'setPageTitle' is not designed for extension - needs to be abstract, final or empty. | 55 | |
| misc | FinalParameters | Parameter pageTitle should be final. | 55 | |
| coding | HiddenField | 'pageTitle' hides a field. | 55 | |
| design | DesignForExtension | Method 'getPage' is not designed for extension - needs to be abstract, final or empty. | 64 | |
| design | DesignForExtension | Method 'setPage' is not designed for extension - needs to be abstract, final or empty. | 73 | |
| misc | FinalParameters | Parameter page should be final. | 73 | |
| coding | HiddenField | 'page' hides a field. | 73 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| javadoc | JavadocPackage | Missing package-info.java file. | ||
| regexp | RegexpSingleline | Line has trailing spaces. | 13 | |
| regexp | RegexpSingleline | Line has trailing spaces. | 18 | |
| javadoc | JavadocVariable | Missing a Javadoc comment. | 37 | |
| javadoc | JavadocVariable | Missing a Javadoc comment. | 38 | |
| sizes | LineLength | Line is longer than 80 characters (found 86). | 43 | |
| misc | FinalParameters | Parameter teamService should be final. | 45 | |
| coding | HiddenField | 'teamService' hides a field. | 45 | |
| design | DesignForExtension | Method 'execute' is not designed for extension - needs to be abstract, final or empty. | 50 | |
| design | DesignForExtension | Method 'getTeamList' is not designed for extension - needs to be abstract, final or empty. | 67 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| regexp | RegexpSingleline | Line has trailing spaces. | 9 | |
| regexp | RegexpSingleline | Line has trailing spaces. | 11 | |
| regexp | RegexpSingleline | Line has trailing spaces. | 16 | |
| regexp | RegexpSingleline | Line has trailing spaces. | 17 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| regexp | RegexpSingleline | Line has trailing spaces. | 13 | |
| regexp | RegexpSingleline | Line has trailing spaces. | 18 | |
| javadoc | JavadocStyle | First sentence should end with a period. | 42 | |
| javadoc | JavadocVariable | Missing a Javadoc comment. | 44 | |
| javadoc | JavadocVariable | Missing a Javadoc comment. | 45 | |
| sizes | LineLength | Line is longer than 80 characters (found 86). | 50 | |
| sizes | LineLength | Line is longer than 80 characters (found 86). | 51 | |
| misc | FinalParameters | Parameter teamService should be final. | 53 | |
| coding | HiddenField | 'teamService' hides a field. | 53 | |
| misc | FinalParameters | Parameter userService should be final. | 53 | |
| coding | HiddenField | 'userService' hides a field. | 53 | |
| design | DesignForExtension | Method 'execute' is not designed for extension - needs to be abstract, final or empty. | 59 | |
| design | DesignForExtension | Method 'updateTeams' is not designed for extension - needs to be abstract, final or empty. | 72 | |
| sizes | LineLength | Line is longer than 80 characters (found 88). | 73 | |
| coding | MagicNumber | '4' is a magic number. | 83 | |
| design | DesignForExtension | Method 'updateUsers' is not designed for extension - needs to be abstract, final or empty. | 98 | |
| sizes | LineLength | Line is longer than 80 characters (found 88). | 99 | |
| coding | MagicNumber | '4' is a magic number. | 109 | |
| javadoc | JavadocMethod | Missing a Javadoc comment. | 119 | |
| misc | FinalParameters | Parameter line should be final. | 119 | |
| coding | MagicNumber | '3' is a magic number. | 125 | |
| javadoc | JavadocMethod | Missing a Javadoc comment. | 130 | |
| misc | FinalParameters | Parameter line should be final. | 130 | |
| coding | MagicNumber | '3' is a magic number. | 136 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| regexp | RegexpSingleline | Line has trailing spaces. | 13 | |
| regexp | RegexpSingleline | Line has trailing spaces. | 18 | |
| javadoc | JavadocVariable | Missing a Javadoc comment. | 37 | |
| javadoc | JavadocVariable | Missing a Javadoc comment. | 38 | |
| sizes | LineLength | Line is longer than 80 characters (found 86). | 43 | |
| misc | FinalParameters | Parameter userService should be final. | 45 | |
| coding | HiddenField | 'userService' hides a field. | 45 | |
| design | DesignForExtension | Method 'execute' is not designed for extension - needs to be abstract, final or empty. | 50 | |
| design | DesignForExtension | Method 'getUserList' is not designed for extension - needs to be abstract, final or empty. | 67 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| regexp | RegexpSingleline | Line has trailing spaces. | 9 | |
| regexp | RegexpSingleline | Line has trailing spaces. | 11 | |
| regexp | RegexpSingleline | Line has trailing spaces. | 16 | |
| regexp | RegexpSingleline | Line has trailing spaces. | 17 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| regexp | RegexpSingleline | Line has trailing spaces. | 13 | |
| regexp | RegexpSingleline | Line has trailing spaces. | 18 | |
| javadoc | JavadocVariable | Missing a Javadoc comment. | 41 | |
| design | DesignForExtension | Method 'setSessionFactory' is not designed for extension - needs to be abstract, final or empty. | 48 | |
| misc | FinalParameters | Parameter sessionFactory should be final. | 48 | |
| coding | HiddenField | 'sessionFactory' hides a field. | 48 | |
| design | DesignForExtension | Method 'addTeam' is not designed for extension - needs to be abstract, final or empty. | 57 | |
| misc | FinalParameters | Parameter team should be final. | 58 | |
| design | DesignForExtension | Method 'listTeam' is not designed for extension - needs to be abstract, final or empty. | 69 | |
| misc | FinalParameters | Parameter firstResult should be final. | 70 | |
| misc | FinalParameters | Parameter maxResults should be final. | 70 | |
| sizes | LineLength | Line is longer than 80 characters (found 102). | 71 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| javadoc | JavadocPackage | Missing package-info.java file. | ||
| regexp | RegexpSingleline | Line has trailing spaces. | 13 | |
| regexp | RegexpSingleline | Line has trailing spaces. | 18 | |
| javadoc | JavadocVariable | Missing a Javadoc comment. | 41 | |
| design | DesignForExtension | Method 'setSessionFactory' is not designed for extension - needs to be abstract, final or empty. | 48 | |
| misc | FinalParameters | Parameter sessionFactory should be final. | 48 | |
| coding | HiddenField | 'sessionFactory' hides a field. | 48 | |
| design | DesignForExtension | Method 'addUser' is not designed for extension - needs to be abstract, final or empty. | 57 | |
| misc | FinalParameters | Parameter user should be final. | 58 | |
| design | DesignForExtension | Method 'listUser' is not designed for extension - needs to be abstract, final or empty. | 69 | |
| misc | FinalParameters | Parameter firstResult should be final. | 70 | |
| misc | FinalParameters | Parameter maxResults should be final. | 70 | |
| sizes | LineLength | Line is longer than 80 characters (found 106). | 71 | |
| design | DesignForExtension | Method 'listUserForTeam' is not designed for extension - needs to be abstract, final or empty. | 85 | |
| sizes | LineLength | Line is longer than 80 characters (found 82). | 86 | |
| misc | FinalParameters | Parameter team should be final. | 86 | |
| misc | FinalParameters | Parameter firstResult should be final. | 86 | |
| misc | FinalParameters | Parameter maxResults should be final. | 86 | |
| sizes | LineLength | Line is longer than 80 characters (found 123). | 87 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| javadoc | JavadocPackage | Missing package-info.java file. | ||
| regexp | RegexpSingleline | Line has trailing spaces. | 13 | |
| regexp | RegexpSingleline | Line has trailing spaces. | 18 | |
| javadoc | JavadocVariable | Missing a Javadoc comment. | 35 | |
| javadoc | JavadocVariable | Missing a Javadoc comment. | 36 | |
| javadoc | JavadocVariable | Missing a Javadoc comment. | 37 | |
| javadoc | JavadocVariable | Missing a Javadoc comment. | 38 | |
| javadoc | JavadocVariable | Missing a Javadoc comment. | 39 | |
| misc | FinalParameters | Parameter team should be final. | 56 | |
| coding | HiddenField | 'team' hides a field. | 56 | |
| misc | FinalParameters | Parameter teamName should be final. | 56 | |
| coding | HiddenField | 'teamName' hides a field. | 56 | |
| misc | FinalParameters | Parameter score should be final. | 56 | |
| coding | HiddenField | 'score' hides a field. | 56 | |
| misc | FinalParameters | Parameter wu should be final. | 56 | |
| coding | HiddenField | 'wu' hides a field. | 56 | |
| javadoc | JavadocStyle | First sentence should end with a period. | 63 | |
| design | DesignForExtension | Method 'getUniqueId' is not designed for extension - needs to be abstract, final or empty. | 68 | |
| javadoc | JavadocStyle | First sentence should end with a period. | 72 | |
| design | DesignForExtension | Method 'setUniqueId' is not designed for extension - needs to be abstract, final or empty. | 77 | |
| misc | FinalParameters | Parameter uniqueId should be final. | 77 | |
| coding | HiddenField | 'uniqueId' hides a field. | 77 | |
| javadoc | JavadocStyle | First sentence should end with a period. | 81 | |
| design | DesignForExtension | Method 'getTeam' is not designed for extension - needs to be abstract, final or empty. | 86 | |
| javadoc | JavadocStyle | First sentence should end with a period. | 90 | |
| design | DesignForExtension | Method 'setTeam' is not designed for extension - needs to be abstract, final or empty. | 95 | |
| misc | FinalParameters | Parameter team should be final. | 95 | |
| coding | HiddenField | 'team' hides a field. | 95 | |
| javadoc | JavadocStyle | First sentence should end with a period. | 99 | |
| design | DesignForExtension | Method 'getTeamName' is not designed for extension - needs to be abstract, final or empty. | 104 | |
| javadoc | JavadocStyle | First sentence should end with a period. | 108 | |
| design | DesignForExtension | Method 'setTeamName' is not designed for extension - needs to be abstract, final or empty. | 113 | |
| misc | FinalParameters | Parameter teamName should be final. | 113 | |
| coding | HiddenField | 'teamName' hides a field. | 113 | |
| javadoc | JavadocStyle | First sentence should end with a period. | 117 | |
| design | DesignForExtension | Method 'getScore' is not designed for extension - needs to be abstract, final or empty. | 122 | |
| javadoc | JavadocStyle | First sentence should end with a period. | 126 | |
| design | DesignForExtension | Method 'setScore' is not designed for extension - needs to be abstract, final or empty. | 131 | |
| misc | FinalParameters | Parameter score should be final. | 131 | |
| coding | HiddenField | 'score' hides a field. | 131 | |
| javadoc | JavadocStyle | First sentence should end with a period. | 135 | |
| design | DesignForExtension | Method 'getWu' is not designed for extension - needs to be abstract, final or empty. | 140 | |
| javadoc | JavadocStyle | First sentence should end with a period. | 144 | |
| design | DesignForExtension | Method 'setWu' is not designed for extension - needs to be abstract, final or empty. | 149 | |
| misc | FinalParameters | Parameter wu should be final. | 149 | |
| coding | HiddenField | 'wu' hides a field. | 149 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| regexp | RegexpSingleline | Line has trailing spaces. | 13 | |
| regexp | RegexpSingleline | Line has trailing spaces. | 18 | |
| javadoc | JavadocVariable | Missing a Javadoc comment. | 33 | |
| javadoc | JavadocVariable | Missing a Javadoc comment. | 34 | |
| javadoc | JavadocVariable | Missing a Javadoc comment. | 35 | |
| javadoc | JavadocVariable | Missing a Javadoc comment. | 36 | |
| javadoc | JavadocVariable | Missing a Javadoc comment. | 37 | |
| misc | FinalParameters | Parameter name should be final. | 54 | |
| coding | HiddenField | 'name' hides a field. | 54 | |
| misc | FinalParameters | Parameter newCredit should be final. | 54 | |
| coding | HiddenField | 'newCredit' hides a field. | 54 | |
| misc | FinalParameters | Parameter sum should be final. | 54 | |
| coding | HiddenField | 'sum' hides a field. | 54 | |
| misc | FinalParameters | Parameter team should be final. | 54 | |
| coding | HiddenField | 'team' hides a field. | 54 | |
| javadoc | JavadocStyle | First sentence should end with a period. | 61 | |
| design | DesignForExtension | Method 'getUniqueId' is not designed for extension - needs to be abstract, final or empty. | 66 | |
| javadoc | JavadocStyle | First sentence should end with a period. | 70 | |
| design | DesignForExtension | Method 'setUniqueId' is not designed for extension - needs to be abstract, final or empty. | 75 | |
| misc | FinalParameters | Parameter uniqueId should be final. | 75 | |
| coding | HiddenField | 'uniqueId' hides a field. | 75 | |
| javadoc | JavadocStyle | First sentence should end with a period. | 79 | |
| design | DesignForExtension | Method 'getName' is not designed for extension - needs to be abstract, final or empty. | 84 | |
| javadoc | JavadocStyle | First sentence should end with a period. | 88 | |
| design | DesignForExtension | Method 'setName' is not designed for extension - needs to be abstract, final or empty. | 93 | |
| misc | FinalParameters | Parameter name should be final. | 93 | |
| coding | HiddenField | 'name' hides a field. | 93 | |
| javadoc | JavadocStyle | First sentence should end with a period. | 97 | |
| design | DesignForExtension | Method 'getNewCredit' is not designed for extension - needs to be abstract, final or empty. | 102 | |
| javadoc | JavadocStyle | First sentence should end with a period. | 106 | |
| design | DesignForExtension | Method 'setNewCredit' is not designed for extension - needs to be abstract, final or empty. | 111 | |
| misc | FinalParameters | Parameter newCredit should be final. | 111 | |
| coding | HiddenField | 'newCredit' hides a field. | 111 | |
| javadoc | JavadocStyle | First sentence should end with a period. | 115 | |
| design | DesignForExtension | Method 'getSum' is not designed for extension - needs to be abstract, final or empty. | 120 | |
| javadoc | JavadocStyle | First sentence should end with a period. | 124 | |
| design | DesignForExtension | Method 'setSum' is not designed for extension - needs to be abstract, final or empty. | 129 | |
| misc | FinalParameters | Parameter sum should be final. | 129 | |
| coding | HiddenField | 'sum' hides a field. | 129 | |
| javadoc | JavadocStyle | First sentence should end with a period. | 133 | |
| design | DesignForExtension | Method 'getTeam' is not designed for extension - needs to be abstract, final or empty. | 138 | |
| javadoc | JavadocStyle | First sentence should end with a period. | 142 | |
| design | DesignForExtension | Method 'setTeam' is not designed for extension - needs to be abstract, final or empty. | 147 | |
| misc | FinalParameters | Parameter team should be final. | 147 | |
| coding | HiddenField | 'team' hides a field. | 147 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| regexp | RegexpSingleline | Line has trailing spaces. | 9 | |
| regexp | RegexpSingleline | Line has trailing spaces. | 11 | |
| regexp | RegexpSingleline | Line has trailing spaces. | 16 | |
| regexp | RegexpSingleline | Line has trailing spaces. | 17 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| regexp | RegexpSingleline | Line has trailing spaces. | 13 | |
| regexp | RegexpSingleline | Line has trailing spaces. | 18 | |
| javadoc | JavadocVariable | Missing a Javadoc comment. | 37 | |
| design | VisibilityModifier | Variable 'teamDao' must be private and have accessor methods. | 37 | |
| regexp | RegexpSingleline | Line has trailing spaces. | 38 | |
| design | DesignForExtension | Method 'setTeamDao' is not designed for extension - needs to be abstract, final or empty. | 44 | |
| misc | FinalParameters | Parameter teamDao should be final. | 44 | |
| coding | HiddenField | 'teamDao' hides a field. | 44 | |
| design | DesignForExtension | Method 'addTeam' is not designed for extension - needs to be abstract, final or empty. | 53 | |
| misc | FinalParameters | Parameter team should be final. | 53 | |
| design | DesignForExtension | Method 'listTeam' is not designed for extension - needs to be abstract, final or empty. | 64 | |
| misc | FinalParameters | Parameter firstResult should be final. | 64 | |
| misc | FinalParameters | Parameter maxResults should be final. | 64 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| javadoc | JavadocPackage | Missing package-info.java file. | ||
| regexp | RegexpSingleline | Line has trailing spaces. | 13 | |
| regexp | RegexpSingleline | Line has trailing spaces. | 18 | |
| javadoc | JavadocVariable | Missing a Javadoc comment. | 37 | |
| design | VisibilityModifier | Variable 'userDao' must be private and have accessor methods. | 37 | |
| regexp | RegexpSingleline | Line has trailing spaces. | 38 | |
| design | DesignForExtension | Method 'setUserDao' is not designed for extension - needs to be abstract, final or empty. | 44 | |
| misc | FinalParameters | Parameter userDao should be final. | 44 | |
| coding | HiddenField | 'userDao' hides a field. | 44 | |
| design | DesignForExtension | Method 'addUser' is not designed for extension - needs to be abstract, final or empty. | 53 | |
| misc | FinalParameters | Parameter user should be final. | 53 | |
| design | DesignForExtension | Method 'listUser' is not designed for extension - needs to be abstract, final or empty. | 64 | |
| misc | FinalParameters | Parameter firstResult should be final. | 64 | |
| misc | FinalParameters | Parameter maxResults should be final. | 64 | |
| coding | MagicNumber | '39299' is a magic number. | 65 |