The following document contains the results of Checkstyle 6.2 with sun_checks.xml ruleset. 
| Category | Rule | Violations | Severity |
|---|---|---|---|
| blocks | LeftCurly | 21 |  Error |
| RightCurly | 3 | Error | |
| coding | AvoidInlineConditionals | 2 | Error |
| HiddenField | 22 | Error | |
| MagicNumber | 6 | Error | |
| design | DesignForExtension | 44 | Error |
| VisibilityModifier | 18 | Error | |
| imports | AvoidStarImport | 4 | Error |
| javadoc | JavadocMethod | 28 | Error |
| JavadocPackage | 2 | Error | |
| JavadocStyle | 1 | Error | |
| JavadocType | 9 | Error | |
| JavadocVariable | 40 | Error | |
| misc | FinalParameters | 25 | Error |
| naming | TypeName | 4 | Error |
| regexp | RegexpSingleline
|
38 | Error |
| sizes | LineLength | 2 | Error |
| whitespace | FileTabCharacter | 9 | Error |
| OperatorWrap | 2 | Error | |
| ParenPad | 1 | Error | |
| WhitespaceAround | 107 | Error |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| Error |
imports | AvoidStarImport | Using the '.*' form of import should be avoided - javax.persistence.*. | 7 |
| Error |
javadoc | JavadocType | Missing a Javadoc comment. | 9 |
| Error |
whitespace | WhitespaceAround | '=' is not preceded with whitespace. | 9 |
| Error |
whitespace | WhitespaceAround | '=' is not followed by whitespace. | 9 |
| Error |
regexp | RegexpSingleline | Line has trailing spaces. | 11 |
| Error |
whitespace | FileTabCharacter | File contains tab characters (this is the first instance). | 11 |
| Error |
regexp | RegexpSingleline | Line has trailing spaces. | 17 |
| Error |
sizes | LineLength | Line is longer than 80 characters (found 105). | 19 |
| Error |
sizes | LineLength | Line is longer than 80 characters (found 92). | 20 |
| Error |
whitespace | WhitespaceAround | '=' is not preceded with whitespace. | 27 |
| Error |
whitespace | WhitespaceAround | '=' is not followed by whitespace. | 27 |
| Error |
regexp | RegexpSingleline | Line has trailing spaces. | 30 |
| Error |
whitespace | WhitespaceAround | '=' is not preceded with whitespace. | 31 |
| Error |
whitespace | WhitespaceAround | '=' is not followed by whitespace. | 31 |
| Error |
whitespace | ParenPad | ')' is preceded with whitespace. | 31 |
| Error |
whitespace | WhitespaceAround | '=' is not preceded with whitespace. | 32 |
| Error |
whitespace | WhitespaceAround | '=' is not followed by whitespace. | 32 |
| Error |
javadoc | JavadocVariable | Missing a Javadoc comment. | 36 |
| Error |
whitespace | WhitespaceAround | '=' is not preceded with whitespace. | 36 |
| Error |
whitespace | WhitespaceAround | '=' is not followed by whitespace. | 36 |
| Error |
javadoc | JavadocVariable | Missing a Javadoc comment. | 39 |
| Error |
whitespace | WhitespaceAround | '=' is not preceded with whitespace. | 39 |
| Error |
whitespace | WhitespaceAround | '=' is not followed by whitespace. | 39 |
| Error |
whitespace | WhitespaceAround | '=' is not preceded with whitespace. | 39 |
| Error |
coding | MagicNumber | '16' is a magic number. | 39 |
| Error |
whitespace | WhitespaceAround | '=' is not followed by whitespace. | 39 |
| Error |
whitespace | WhitespaceAround | '=' is not preceded with whitespace. | 39 |
| Error |
whitespace | WhitespaceAround | '=' is not followed by whitespace. | 39 |
| Error |
regexp | RegexpSingleline | Line has trailing spaces. | 41 |
| Error |
javadoc | JavadocVariable | Missing a Javadoc comment. | 42 |
| Error |
whitespace | WhitespaceAround | '=' is not preceded with whitespace. | 42 |
| Error |
whitespace | WhitespaceAround | '=' is not followed by whitespace. | 42 |
| Error |
whitespace | WhitespaceAround | '=' is not preceded with whitespace. | 42 |
| Error |
coding | MagicNumber | '16' is a magic number. | 42 |
| Error |
whitespace | WhitespaceAround | '=' is not followed by whitespace. | 42 |
| Error |
whitespace | WhitespaceAround | '=' is not preceded with whitespace. | 42 |
| Error |
whitespace | WhitespaceAround | '=' is not followed by whitespace. | 42 |
| Error |
regexp | RegexpSingleline | Line has trailing spaces. | 44 |
| Error |
javadoc | JavadocVariable | Missing a Javadoc comment. | 45 |
| Error |
whitespace | WhitespaceAround | '=' is not preceded with whitespace. | 46 |
| Error |
whitespace | WhitespaceAround | '=' is not followed by whitespace. | 46 |
| Error |
whitespace | WhitespaceAround | '=' is not preceded with whitespace. | 46 |
| Error |
whitespace | WhitespaceAround | '=' is not followed by whitespace. | 46 |
| Error |
regexp | RegexpSingleline | Line has trailing spaces. | 48 |
| Error |
javadoc | JavadocVariable | Missing a Javadoc comment. | 49 |
| Error |
whitespace | WhitespaceAround | '=' is not preceded with whitespace. | 50 |
| Error |
whitespace | WhitespaceAround | '=' is not followed by whitespace. | 50 |
| Error |
whitespace | WhitespaceAround | '=' is not preceded with whitespace. | 50 |
| Error |
whitespace | WhitespaceAround | '=' is not followed by whitespace. | 50 |
| Error |
javadoc | JavadocVariable | Missing a Javadoc comment. | 53 |
| Error |
whitespace | WhitespaceAround | '=' is not preceded with whitespace. | 53 |
| Error |
whitespace | WhitespaceAround | '=' is not followed by whitespace. | 53 |
| Error |
whitespace | WhitespaceAround | '=' is not preceded with whitespace. | 53 |
| Error |
whitespace | WhitespaceAround | '=' is not followed by whitespace. | 53 |
| Error |
design | DesignForExtension | Method 'getId' is not designed for extension - needs to be abstract, final or empty. | 56 |
| Error |
blocks | LeftCurly | '{' should have line break after. | 56 |
| Error |
design | DesignForExtension | Method 'getSales' is not designed for extension - needs to be abstract, final or empty. | 58 |
| Error |
blocks | LeftCurly | '{' should have line break after. | 58 |
| Error |
design | DesignForExtension | Method 'setSales' is not designed for extension - needs to be abstract, final or empty. | 59 |
| Error |
javadoc | JavadocMethod | Missing a Javadoc comment. | 59 |
| Error |
misc | FinalParameters | Parameter sales should be final. | 59 |
| Error |
coding | HiddenField | 'sales' hides a field. | 59 |
| Error |
design | DesignForExtension | Method 'addSale' is not designed for extension - needs to be abstract, final or empty. | 63 |
| Error |
javadoc | JavadocMethod | Missing a Javadoc comment. | 63 |
| Error |
misc | FinalParameters | Parameter sale should be final. | 63 |
| Error |
blocks | LeftCurly | '{' should have line break after. | 66 |
| Error |
blocks | RightCurly | '}' should have line break before. | 66 |
| Error |
regexp | RegexpSingleline | Line has trailing spaces. | 71 |
| Error |
design | DesignForExtension | Method 'getHireDate' is not designed for extension - needs to be abstract, final or empty. | 72 |
| Error |
blocks | LeftCurly | '{' should have line break after. | 72 |
| Error |
design | DesignForExtension | Method 'setHireDate' is not designed for extension - needs to be abstract, final or empty. | 73 |
| Error |
javadoc | JavadocMethod | Missing a Javadoc comment. | 73 |
| Error |
misc | FinalParameters | Parameter hireDate should be final. | 73 |
| Error |
coding | HiddenField | 'hireDate' hides a field. | 73 |
| Error |
regexp | RegexpSingleline | Line has trailing spaces. | 77 |
| Error |
design | DesignForExtension | Method 'getTermDate' is not designed for extension - needs to be abstract, final or empty. | 78 |
| Error |
blocks | LeftCurly | '{' should have line break after. | 78 |
| Error |
design | DesignForExtension | Method 'setTermDate' is not designed for extension - needs to be abstract, final or empty. | 79 |
| Error |
javadoc | JavadocMethod | Missing a Javadoc comment. | 79 |
| Error |
misc | FinalParameters | Parameter termDate should be final. | 79 |
| Error |
coding | HiddenField | 'termDate' hides a field. | 79 |
| Error |
regexp | RegexpSingleline | Line has trailing spaces. | 82 |
| Error |
design | DesignForExtension | Method 'getFirstName' is not designed for extension - needs to be abstract, final or empty. | 84 |
| Error |
blocks | LeftCurly | '{' should have line break after. | 84 |
| Error |
design | DesignForExtension | Method 'setFirstName' is not designed for extension - needs to be abstract, final or empty. | 85 |
| Error |
javadoc | JavadocMethod | Missing a Javadoc comment. | 85 |
| Error |
misc | FinalParameters | Parameter firstName should be final. | 85 |
| Error |
coding | HiddenField | 'firstName' hides a field. | 85 |
| Error |
regexp | RegexpSingleline | Line has trailing spaces. | 88 |
| Error |
regexp | RegexpSingleline | Line has trailing spaces. | 89 |
| Error |
design | DesignForExtension | Method 'getLastName' is not designed for extension - needs to be abstract, final or empty. | 90 |
| Error |
blocks | LeftCurly | '{' should have line break after. | 90 |
| Error |
design | DesignForExtension | Method 'setLastName' is not designed for extension - needs to be abstract, final or empty. | 91 |
| Error |
javadoc | JavadocMethod | Missing a Javadoc comment. | 91 |
| Error |
misc | FinalParameters | Parameter lastName should be final. | 91 |
| Error |
coding | HiddenField | 'lastName' hides a field. | 91 |
| Error |
design | DesignForExtension | Method 'toString' is not designed for extension - needs to be abstract, final or empty. | 96 |
| Error |
javadoc | JavadocMethod | Missing a Javadoc comment. | 96 |
| Error |
whitespace | WhitespaceAround | 'for' is not followed by whitespace. | 103 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| Error |
javadoc | JavadocPackage | Missing package-info.java file. | |
| Error |
javadoc | JavadocType | Missing a Javadoc comment. | 9 |
| Error |
naming | TypeName | Name 'Clerk_' must match pattern '^[A-Z][a-zA-Z0-9]*$'. | 11 |
| Error |
whitespace | FileTabCharacter | File contains tab characters (this is the first instance). | 13 |
| Error |
javadoc | JavadocVariable | Missing a Javadoc comment. | 13 |
| Error |
design | VisibilityModifier | Variable 'firstName' must be private and have accessor methods. | 13 |
| Error |
javadoc | JavadocVariable | Missing a Javadoc comment. | 14 |
| Error |
design | VisibilityModifier | Variable 'lastName' must be private and have accessor methods. | 14 |
| Error |
javadoc | JavadocVariable | Missing a Javadoc comment. | 15 |
| Error |
design | VisibilityModifier | Variable 'hireDate' must be private and have accessor methods. | 15 |
| Error |
javadoc | JavadocVariable | Missing a Javadoc comment. | 16 |
| Error |
design | VisibilityModifier | Variable 'id' must be private and have accessor methods. | 16 |
| Error |
javadoc | JavadocVariable | Missing a Javadoc comment. | 17 |
| Error |
design | VisibilityModifier | Variable 'termDate' must be private and have accessor methods. | 17 |
| Error |
javadoc | JavadocVariable | Missing a Javadoc comment. | 18 |
| Error |
design | VisibilityModifier | Variable 'sales' must be private and have accessor methods. | 18 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| Error |
imports | AvoidStarImport | Using the '.*' form of import should be avoided - javax.persistence.*. | 3 |
| Error |
regexp | RegexpSingleline | Line has trailing spaces. | 5 |
| Error |
javadoc | JavadocType | Missing a Javadoc comment. | 5 |
| Error |
whitespace | WhitespaceAround | '=' is not preceded with whitespace. | 6 |
| Error |
whitespace | WhitespaceAround | '=' is not followed by whitespace. | 6 |
| Error |
whitespace | WhitespaceAround | '=' is not preceded with whitespace. | 8 |
| Error |
whitespace | WhitespaceAround | '=' is not followed by whitespace. | 8 |
| Error |
whitespace | WhitespaceAround | '=' is not preceded with whitespace. | 9 |
| Error |
whitespace | WhitespaceAround | '=' is not followed by whitespace. | 9 |
| Error |
whitespace | OperatorWrap | '+' should be on a new line. | 9 |
| Error |
whitespace | WhitespaceAround | '=' is not preceded with whitespace. | 11 |
| Error |
whitespace | WhitespaceAround | '=' is not followed by whitespace. | 11 |
| Error |
whitespace | WhitespaceAround | '=' is not preceded with whitespace. | 12 |
| Error |
whitespace | WhitespaceAround | '=' is not followed by whitespace. | 12 |
| Error |
whitespace | OperatorWrap | '+' should be on a new line. | 12 |
| Error |
regexp | RegexpSingleline | Line has trailing spaces. | 17 |
| Error |
whitespace | FileTabCharacter | File contains tab characters (this is the first instance). | 17 |
| Error |
regexp | RegexpSingleline | Line has trailing spaces. | 23 |
| Error |
javadoc | JavadocVariable | Missing a Javadoc comment. | 23 |
| Error |
regexp | RegexpSingleline | Line has trailing spaces. | 24 |
| Error |
whitespace | WhitespaceAround | '=' is not preceded with whitespace. | 24 |
| Error |
whitespace | WhitespaceAround | '=' is not followed by whitespace. | 24 |
| Error |
regexp | RegexpSingleline | Line has trailing spaces. | 26 |
| Error |
javadoc | JavadocVariable | Missing a Javadoc comment. | 27 |
| Error |
whitespace | WhitespaceAround | '=' is not preceded with whitespace. | 27 |
| Error |
whitespace | WhitespaceAround | '=' is not followed by whitespace. | 27 |
| Error |
whitespace | WhitespaceAround | '=' is not preceded with whitespace. | 27 |
| Error |
coding | MagicNumber | '16' is a magic number. | 27 |
| Error |
whitespace | WhitespaceAround | '=' is not followed by whitespace. | 27 |
| Error |
whitespace | WhitespaceAround | '=' is not preceded with whitespace. | 27 |
| Error |
whitespace | WhitespaceAround | '=' is not followed by whitespace. | 27 |
| Error |
regexp | RegexpSingleline | Line has trailing spaces. | 29 |
| Error |
javadoc | JavadocVariable | Missing a Javadoc comment. | 30 |
| Error |
whitespace | WhitespaceAround | '=' is not preceded with whitespace. | 30 |
| Error |
whitespace | WhitespaceAround | '=' is not followed by whitespace. | 30 |
| Error |
whitespace | WhitespaceAround | '=' is not preceded with whitespace. | 30 |
| Error |
coding | MagicNumber | '16' is a magic number. | 30 |
| Error |
whitespace | WhitespaceAround | '=' is not followed by whitespace. | 30 |
| Error |
whitespace | WhitespaceAround | '=' is not preceded with whitespace. | 30 |
| Error |
whitespace | WhitespaceAround | '=' is not followed by whitespace. | 30 |
| Error |
regexp | RegexpSingleline | Line has trailing spaces. | 32 |
| Error |
design | DesignForExtension | Method 'getId' is not designed for extension - needs to be abstract, final or empty. | 33 |
| Error |
blocks | LeftCurly | '{' should have line break after. | 33 |
| Error |
design | DesignForExtension | Method 'getFirstName' is not designed for extension - needs to be abstract, final or empty. | 35 |
| Error |
blocks | LeftCurly | '{' should have line break after. | 35 |
| Error |
design | DesignForExtension | Method 'setFirstName' is not designed for extension - needs to be abstract, final or empty. | 36 |
| Error |
javadoc | JavadocMethod | Missing a Javadoc comment. | 36 |
| Error |
misc | FinalParameters | Parameter firstName should be final. | 36 |
| Error |
coding | HiddenField | 'firstName' hides a field. | 36 |
| Error |
regexp | RegexpSingleline | Line has trailing spaces. | 40 |
| Error |
design | DesignForExtension | Method 'getLastName' is not designed for extension - needs to be abstract, final or empty. | 41 |
| Error |
blocks | LeftCurly | '{' should have line break after. | 41 |
| Error |
design | DesignForExtension | Method 'setLastName' is not designed for extension - needs to be abstract, final or empty. | 42 |
| Error |
javadoc | JavadocMethod | Missing a Javadoc comment. | 42 |
| Error |
misc | FinalParameters | Parameter lastName should be final. | 42 |
| Error |
coding | HiddenField | 'lastName' hides a field. | 42 |
| Error |
regexp | RegexpSingleline | Line has trailing spaces. | 46 |
| Error |
design | DesignForExtension | Method 'toString' is not designed for extension - needs to be abstract, final or empty. | 47 |
| Error |
javadoc | JavadocMethod | Missing a Javadoc comment. | 47 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| Error |
javadoc | JavadocType | Missing a Javadoc comment. | 7 |
| Error |
naming | TypeName | Name 'Customer_' must match pattern '^[A-Z][a-zA-Z0-9]*$'. | 9 |
| Error |
whitespace | FileTabCharacter | File contains tab characters (this is the first instance). | 11 |
| Error |
javadoc | JavadocVariable | Missing a Javadoc comment. | 11 |
| Error |
design | VisibilityModifier | Variable 'firstName' must be private and have accessor methods. | 11 |
| Error |
javadoc | JavadocVariable | Missing a Javadoc comment. | 12 |
| Error |
design | VisibilityModifier | Variable 'lastName' must be private and have accessor methods. | 12 |
| Error |
javadoc | JavadocVariable | Missing a Javadoc comment. | 13 |
| Error |
design | VisibilityModifier | Variable 'id' must be private and have accessor methods. | 13 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| Error |
javadoc | JavadocType | Missing a Javadoc comment. | 12 |
| Error |
javadoc | JavadocVariable | Missing a Javadoc comment. | 13 |
| Error |
javadoc | JavadocVariable | Missing a Javadoc comment. | 14 |
| Error |
javadoc | JavadocVariable | Missing a Javadoc comment. | 15 |
| Error |
javadoc | JavadocVariable | Missing a Javadoc comment. | 16 |
| Error |
javadoc | JavadocVariable | Missing a Javadoc comment. | 17 |
| Error |
regexp | RegexpSingleline | Line has trailing spaces. | 18 |
| Error |
javadoc | JavadocMethod | Missing a Javadoc comment. | 19 |
| Error |
misc | FinalParameters | Parameter saleId should be final. | 19 |
| Error |
coding | HiddenField | 'saleId' hides a field. | 19 |
| Error |
misc | FinalParameters | Parameter customerId should be final. | 19 |
| Error |
coding | HiddenField | 'customerId' hides a field. | 19 |
| Error |
misc | FinalParameters | Parameter date should be final. | 19 |
| Error |
coding | HiddenField | 'date' hides a field. | 19 |
| Error |
misc | FinalParameters | Parameter amount should be final. | 19 |
| Error |
coding | HiddenField | 'amount' hides a field. | 19 |
| Error |
javadoc | JavadocMethod | Missing a Javadoc comment. | 22 |
| Error |
misc | FinalParameters | Parameter saleId should be final. | 22 |
| Error |
coding | HiddenField | 'saleId' hides a field. | 22 |
| Error |
misc | FinalParameters | Parameter customerId should be final. | 22 |
| Error |
coding | HiddenField | 'customerId' hides a field. | 22 |
| Error |
misc | FinalParameters | Parameter date should be final. | 22 |
| Error |
coding | HiddenField | 'date' hides a field. | 22 |
| Error |
misc | FinalParameters | Parameter amount should be final. | 22 |
| Error |
coding | HiddenField | 'amount' hides a field. | 22 |
| Error |
design | DesignForExtension | Method 'toString' is not designed for extension - needs to be abstract, final or empty. | 28 |
| Error |
javadoc | JavadocMethod | Missing a Javadoc comment. | 28 |
| Error |
whitespace | FileTabCharacter | File contains tab characters (this is the first instance). | 29 |
| Error |
whitespace | WhitespaceAround | '==' is not preceded with whitespace. | 34 |
| Error |
whitespace | WhitespaceAround | '==' is not followed by whitespace. | 34 |
| Error |
coding | AvoidInlineConditionals | Avoid inline conditionals. | 34 |
| Error |
design | DesignForExtension | Method 'getAmount' is not designed for extension - needs to be abstract, final or empty. | 38 |
| Error |
javadoc | JavadocMethod | Missing a Javadoc comment. | 38 |
| Error |
design | DesignForExtension | Method 'getCustomerId' is not designed for extension - needs to be abstract, final or empty. | 41 |
| Error |
javadoc | JavadocMethod | Missing a Javadoc comment. | 41 |
| Error |
design | DesignForExtension | Method 'getDate' is not designed for extension - needs to be abstract, final or empty. | 44 |
| Error |
javadoc | JavadocMethod | Missing a Javadoc comment. | 44 |
| Error |
design | DesignForExtension | Method 'getSaleId' is not designed for extension - needs to be abstract, final or empty. | 47 |
| Error |
javadoc | JavadocMethod | Missing a Javadoc comment. | 47 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| Error |
javadoc | JavadocPackage | Missing package-info.java file. | |
| Error |
imports | AvoidStarImport | Using the '.*' form of import should be avoided - javax.persistence.*. | 13 |
| Error |
javadoc | JavadocType | Missing a Javadoc comment. | 15 |
| Error |
whitespace | WhitespaceAround | '=' is not preceded with whitespace. | 16 |
| Error |
whitespace | WhitespaceAround | '=' is not followed by whitespace. | 16 |
| Error |
regexp | RegexpSingleline | Line has trailing spaces. | 18 |
| Error |
javadoc | JavadocVariable | Missing a Javadoc comment. | 18 |
| Error |
whitespace | WhitespaceAround | '=' is not preceded with whitespace. | 19 |
| Error |
whitespace | WhitespaceAround | '=' is not followed by whitespace. | 19 |
| Error |
javadoc | JavadocVariable | Missing a Javadoc comment. | 22 |
| Error |
javadoc | JavadocVariable | Missing a Javadoc comment. | 25 |
| Error |
whitespace | WhitespaceAround | '=' is not preceded with whitespace. | 25 |
| Error |
coding | MagicNumber | '5' is a magic number. | 25 |
| Error |
whitespace | WhitespaceAround | '=' is not followed by whitespace. | 25 |
| Error |
whitespace | WhitespaceAround | '=' is not preceded with whitespace. | 25 |
| Error |
whitespace | WhitespaceAround | '=' is not followed by whitespace. | 25 |
| Error |
regexp | RegexpSingleline | Line has trailing spaces. | 27 |
| Error |
javadoc | JavadocStyle | First sentence should end with a period. | 28 |
| Error |
whitespace | WhitespaceAround | '=' is not preceded with whitespace. | 32 |
| Error |
whitespace | WhitespaceAround | '=' is not followed by whitespace. | 32 |
| Error |
whitespace | WhitespaceAround | '=' is not preceded with whitespace. | 32 |
| Error |
whitespace | WhitespaceAround | '=' is not followed by whitespace. | 32 |
| Error |
javadoc | JavadocVariable | Missing a Javadoc comment. | 35 |
| Error |
whitespace | WhitespaceAround | '=' is not preceded with whitespace. | 35 |
| Error |
whitespace | WhitespaceAround | '=' is not followed by whitespace. | 35 |
| Error |
whitespace | WhitespaceAround | '=' is not preceded with whitespace. | 36 |
| Error |
whitespace | WhitespaceAround | '=' is not followed by whitespace. | 36 |
| Error |
javadoc | JavadocVariable | Missing a Javadoc comment. | 39 |
| Error |
whitespace | WhitespaceAround | '=' is not preceded with whitespace. | 40 |
| Error |
whitespace | WhitespaceAround | '=' is not followed by whitespace. | 40 |
| Error |
whitespace | WhitespaceAround | '=' is not preceded with whitespace. | 41 |
| Error |
whitespace | WhitespaceAround | '=' is not followed by whitespace. | 41 |
| Error |
whitespace | WhitespaceAround | '=' is not preceded with whitespace. | 41 |
| Error |
whitespace | WhitespaceAround | '=' is not followed by whitespace. | 41 |
| Error |
whitespace | WhitespaceAround | '=' is not preceded with whitespace. | 42 |
| Error |
whitespace | WhitespaceAround | '=' is not followed by whitespace. | 42 |
| Error |
whitespace | WhitespaceAround | '=' is not preceded with whitespace. | 42 |
| Error |
whitespace | WhitespaceAround | '=' is not followed by whitespace. | 42 |
| Error |
regexp | RegexpSingleline | Line has trailing spaces. | 45 |
| Error |
design | DesignForExtension | Method 'getId' is not designed for extension - needs to be abstract, final or empty. | 46 |
| Error |
blocks | LeftCurly | '{' should have line break after. | 46 |
| Error |
regexp | RegexpSingleline | Line has trailing spaces. | 47 |
| Error |
design | DesignForExtension | Method 'getClerks' is not designed for extension - needs to be abstract, final or empty. | 48 |
| Error |
blocks | LeftCurly | '{' should have line break after. | 48 |
| Error |
design | DesignForExtension | Method 'setClerks' is not designed for extension - needs to be abstract, final or empty. | 49 |
| Error |
javadoc | JavadocMethod | Missing a Javadoc comment. | 49 |
| Error |
misc | FinalParameters | Parameter clerks should be final. | 49 |
| Error |
coding | HiddenField | 'clerks' hides a field. | 49 |
| Error |
design | DesignForExtension | Method 'addClerk' is not designed for extension - needs to be abstract, final or empty. | 53 |
| Error |
javadoc | JavadocMethod | Missing a Javadoc comment. | 53 |
| Error |
misc | FinalParameters | Parameter clerk should be final. | 53 |
| Error |
whitespace | FileTabCharacter | File contains tab characters (this is the first instance). | 54 |
| Error |
whitespace | WhitespaceAround | '!=' is not preceded with whitespace. | 54 |
| Error |
whitespace | WhitespaceAround | '!=' is not followed by whitespace. | 54 |
| Error |
blocks | LeftCurly | '{' should have line break after. | 56 |
| Error |
blocks | RightCurly | '}' should have line break before. | 56 |
| Error |
regexp | RegexpSingleline | Line has trailing spaces. | 61 |
| Error |
design | DesignForExtension | Method 'getStore' is not designed for extension - needs to be abstract, final or empty. | 62 |
| Error |
blocks | LeftCurly | '{' should have line break after. | 62 |
| Error |
design | DesignForExtension | Method 'setStore' is not designed for extension - needs to be abstract, final or empty. | 63 |
| Error |
javadoc | JavadocMethod | Missing a Javadoc comment. | 63 |
| Error |
misc | FinalParameters | Parameter store should be final. | 63 |
| Error |
coding | HiddenField | 'store' hides a field. | 63 |
| Error |
regexp | RegexpSingleline | Line has trailing spaces. | 66 |
| Error |
design | DesignForExtension | Method 'getAmount' is not designed for extension - needs to be abstract, final or empty. | 68 |
| Error |
blocks | LeftCurly | '{' should have line break after. | 68 |
| Error |
design | DesignForExtension | Method 'setAmount' is not designed for extension - needs to be abstract, final or empty. | 69 |
| Error |
javadoc | JavadocMethod | Missing a Javadoc comment. | 69 |
| Error |
misc | FinalParameters | Parameter amount should be final. | 69 |
| Error |
coding | HiddenField | 'amount' hides a field. | 69 |
| Error |
regexp | RegexpSingleline | Line has trailing spaces. | 73 |
| Error |
design | DesignForExtension | Method 'getBuyerId' is not designed for extension - needs to be abstract, final or empty. | 74 |
| Error |
blocks | LeftCurly | '{' should have line break after. | 74 |
| Error |
design | DesignForExtension | Method 'setBuyerId' is not designed for extension - needs to be abstract, final or empty. | 75 |
| Error |
javadoc | JavadocMethod | Missing a Javadoc comment. | 75 |
| Error |
misc | FinalParameters | Parameter buyerId should be final. | 75 |
| Error |
coding | HiddenField | 'buyerId' hides a field. | 75 |
| Error |
regexp | RegexpSingleline | Line has trailing spaces. | 79 |
| Error |
design | DesignForExtension | Method 'getDate' is not designed for extension - needs to be abstract, final or empty. | 80 |
| Error |
blocks | LeftCurly | '{' should have line break after. | 80 |
| Error |
design | DesignForExtension | Method 'setDate' is not designed for extension - needs to be abstract, final or empty. | 81 |
| Error |
javadoc | JavadocMethod | Missing a Javadoc comment. | 81 |
| Error |
misc | FinalParameters | Parameter date should be final. | 81 |
| Error |
coding | HiddenField | 'date' hides a field. | 81 |
| Error |
regexp | RegexpSingleline | Line has trailing spaces. | 85 |
| Error |
design | DesignForExtension | Method 'toString' is not designed for extension - needs to be abstract, final or empty. | 86 |
| Error |
javadoc | JavadocMethod | Missing a Javadoc comment. | 86 |
| Error |
whitespace | WhitespaceAround | '==' is not preceded with whitespace. | 90 |
| Error |
whitespace | WhitespaceAround | '==' is not followed by whitespace. | 90 |
| Error |
coding | AvoidInlineConditionals | Avoid inline conditionals. | 90 |
| Error |
whitespace | WhitespaceAround | 'for' is not followed by whitespace. | 94 |
| Error |
regexp | RegexpSingleline | Line has trailing spaces. | 96 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| Error |
javadoc | JavadocType | Missing a Javadoc comment. | 10 |
| Error |
naming | TypeName | Name 'Sale_' must match pattern '^[A-Z][a-zA-Z0-9]*$'. | 12 |
| Error |
whitespace | FileTabCharacter | File contains tab characters (this is the first instance). | 14 |
| Error |
javadoc | JavadocVariable | Missing a Javadoc comment. | 14 |
| Error |
design | VisibilityModifier | Variable 'date' must be private and have accessor methods. | 14 |
| Error |
javadoc | JavadocVariable | Missing a Javadoc comment. | 15 |
| Error |
design | VisibilityModifier | Variable 'amount' must be private and have accessor methods. | 15 |
| Error |
javadoc | JavadocVariable | Missing a Javadoc comment. | 16 |
| Error |
design | VisibilityModifier | Variable 'id' must be private and have accessor methods. | 16 |
| Error |
javadoc | JavadocVariable | Missing a Javadoc comment. | 17 |
| Error |
design | VisibilityModifier | Variable 'store' must be private and have accessor methods. | 17 |
| Error |
javadoc | JavadocVariable | Missing a Javadoc comment. | 18 |
| Error |
design | VisibilityModifier | Variable 'buyerId' must be private and have accessor methods. | 18 |
| Error |
javadoc | JavadocVariable | Missing a Javadoc comment. | 19 |
| Error |
design | VisibilityModifier | Variable 'clerks' must be private and have accessor methods. | 19 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| Error |
imports | AvoidStarImport | Using the '.*' form of import should be avoided - javax.persistence.*. | 7 |
| Error |
javadoc | JavadocType | Missing a Javadoc comment. | 9 |
| Error |
whitespace | WhitespaceAround | '=' is not preceded with whitespace. | 9 |
| Error |
whitespace | WhitespaceAround | '=' is not followed by whitespace. | 9 |
| Error |
regexp | RegexpSingleline | Line has trailing spaces. | 11 |
| Error |
javadoc | JavadocVariable | Missing a Javadoc comment. | 11 |
| Error |
whitespace | WhitespaceAround | '=' is not preceded with whitespace. | 12 |
| Error |
whitespace | WhitespaceAround | '=' is not followed by whitespace. | 12 |
| Error |
regexp | RegexpSingleline | Line has trailing spaces. | 14 |
| Error |
javadoc | JavadocVariable | Missing a Javadoc comment. | 15 |
| Error |
whitespace | WhitespaceAround | '=' is not preceded with whitespace. | 15 |
| Error |
coding | MagicNumber | '20' is a magic number. | 15 |
| Error |
whitespace | WhitespaceAround | '=' is not followed by whitespace. | 15 |
| Error |
whitespace | WhitespaceAround | '=' is not preceded with whitespace. | 15 |
| Error |
whitespace | WhitespaceAround | '=' is not followed by whitespace. | 15 |
| Error |
regexp | RegexpSingleline | Line has trailing spaces. | 18 |
| Error |
javadoc | JavadocVariable | Missing a Javadoc comment. | 18 |
| Error |
whitespace | WhitespaceAround | '=' is not preceded with whitespace. | 18 |
| Error |
whitespace | WhitespaceAround | '=' is not followed by whitespace. | 18 |
| Error |
regexp | RegexpSingleline | Line has trailing spaces. | 19 |
| Error |
whitespace | WhitespaceAround | '=' is not preceded with whitespace. | 19 |
| Error |
whitespace | WhitespaceAround | '=' is not followed by whitespace. | 19 |
| Error |
whitespace | WhitespaceAround | '=' is not preceded with whitespace. | 20 |
| Error |
whitespace | WhitespaceAround | '=' is not followed by whitespace. | 20 |
| Error |
design | DesignForExtension | Method 'getId' is not designed for extension - needs to be abstract, final or empty. | 23 |
| Error |
blocks | LeftCurly | '{' should have line break after. | 23 |
| Error |
regexp | RegexpSingleline | Line has trailing spaces. | 24 |
| Error |
design | DesignForExtension | Method 'getSales' is not designed for extension - needs to be abstract, final or empty. | 25 |
| Error |
blocks | LeftCurly | '{' should have line break after. | 25 |
| Error |
design | DesignForExtension | Method 'setSales' is not designed for extension - needs to be abstract, final or empty. | 26 |
| Error |
javadoc | JavadocMethod | Missing a Javadoc comment. | 26 |
| Error |
misc | FinalParameters | Parameter sales should be final. | 26 |
| Error |
coding | HiddenField | 'sales' hides a field. | 26 |
| Error |
design | DesignForExtension | Method 'addSale' 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 sale should be final. | 30 |
| Error |
whitespace | FileTabCharacter | File contains tab characters (this is the first instance). | 31 |
| Error |
blocks | LeftCurly | '{' should have line break after. | 33 |
| Error |
blocks | RightCurly | '}' should have line break before. | 33 |
| Error |
regexp | RegexpSingleline | Line has trailing spaces. | 38 |
| Error |
design | DesignForExtension | Method 'getName' is not designed for extension - needs to be abstract, final or empty. | 39 |
| Error |
blocks | LeftCurly | '{' should have line break after. | 39 |
| Error |
design | DesignForExtension | Method 'setName' is not designed for extension - needs to be abstract, final or empty. | 40 |
| Error |
javadoc | JavadocMethod | Missing a Javadoc comment. | 40 |
| Error |
misc | FinalParameters | Parameter name should be final. | 40 |
| Error |
coding | HiddenField | 'name' hides a field. | 40 |
| Error |
regexp | RegexpSingleline | Line has trailing spaces. | 44 |
| Error |
design | DesignForExtension | Method 'toString' is not designed for extension - needs to be abstract, final or empty. | 45 |
| Error |
javadoc | JavadocMethod | Missing a Javadoc comment. | 45 |
| Error |
whitespace | WhitespaceAround | 'for' is not followed by whitespace. | 49 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| Error |
javadoc | JavadocType | Missing a Javadoc comment. | 8 |
| Error |
naming | TypeName | Name 'Store_' must match pattern '^[A-Z][a-zA-Z0-9]*$'. | 10 |
| Error |
whitespace | FileTabCharacter | File contains tab characters (this is the first instance). | 12 |
| Error |
javadoc | JavadocVariable | Missing a Javadoc comment. | 12 |
| Error |
design | VisibilityModifier | Variable 'name' must be private and have accessor methods. | 12 |
| Error |
javadoc | JavadocVariable | Missing a Javadoc comment. | 13 |
| Error |
design | VisibilityModifier | Variable 'id' must be private and have accessor methods. | 13 |
| Error |
javadoc | JavadocVariable | Missing a Javadoc comment. | 14 |
| Error |
design | VisibilityModifier | Variable 'sales' must be private and have accessor methods. | 14 |
Info
Warnings