Checkstyle Results
The following document contains the results of Checkstyle 6.1.1. 
Summary
| Files | Info | Warnings | Errors |
|---|---|---|---|
| 8 | 0 | 73 | 0 |
Files
| File | I | W | E |
|---|---|---|---|
| org/nicepop/domain/App.java | 0 | 14 | 0 |
| org/nicepop/domain/AppTest.java | 0 | 7 | 0 |
| org/nicepop/domain/Dinner.java | 0 | 2 | 0 |
| org/nicepop/domain/Meal.java | 0 | 6 | 0 |
| org/nicepop/domain/Product.java | 0 | 14 | 0 |
| org/nicepop/domain/Recipe.java | 0 | 12 | 0 |
| org/nicepop/domain/RecipeTest.java | 0 | 14 | 0 |
| org/nicepop/domain/StatusTestIT.java | 0 | 4 | 0 |
Rules
| Category | Rule | Violations | Severity |
|---|---|---|---|
| blocks | AvoidNestedBlocks | 1 | Warning |
| LeftCurly | 4 | Warning | |
| design | DesignForExtension | 12 | Warning |
| HideUtilityClassConstructor | 1 | Warning | |
| javadoc | JavadocStyle | 1 | Warning |
| JavadocType | 6 | Warning | |
| misc | FinalParameters | 10 | Warning |
TodoComment
|
1 | Warning | |
| regexp | RegexpSingleline
|
27 | Error |
| whitespace | ParenPad | 8 | Warning |
WhitespaceAround
|
2 | Warning |
Details
org/nicepop/domain/App.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| Warning | regexp | RegexpSingleline | Line has trailing spaces. | 12 |
| Warning | regexp | RegexpSingleline | Line has trailing spaces. | 14 |
| Warning | regexp | RegexpSingleline | Line has trailing spaces. | 27 |
| Warning | design | HideUtilityClassConstructor | Utility classes should not have a public or default constructor. | 27 |
| Warning | blocks | LeftCurly | '{' should be on the previous line. | 29 |
| Warning | whitespace | ParenPad | '(' is followed by whitespace. | 30 |
| Warning | misc | FinalParameters | Parameter args should be final. | 30 |
| Warning | whitespace | ParenPad | ')' is preceded with whitespace. | 30 |
| Warning | blocks | LeftCurly | '{' should be on the previous line. | 31 |
| Warning | whitespace | ParenPad | '(' is followed by whitespace. | 32 |
| Warning | whitespace | ParenPad | ')' is preceded with whitespace. | 32 |
| Warning | whitespace | ParenPad | '(' is followed by whitespace. | 33 |
| Warning | whitespace | ParenPad | ')' is preceded with whitespace. | 33 |
| Warning | blocks | LeftCurly | '{' should be on the previous line. | 34 |
org/nicepop/domain/AppTest.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| Warning | regexp | RegexpSingleline | Line has trailing spaces. | 12 |
| Warning | regexp | RegexpSingleline | Line has trailing spaces. | 14 |
| Warning | javadoc | JavadocStyle | First sentence should end with a period. | 31 |
| Warning | design | DesignForExtension | Method 'testApp' is not designed for extension - needs to be abstract, final or empty. | 34 |
| Warning | blocks | LeftCurly | '{' should be on the previous line. | 36 |
| Warning | whitespace | ParenPad | '(' is followed by whitespace. | 37 |
| Warning | whitespace | ParenPad | ')' is preceded with whitespace. | 37 |
org/nicepop/domain/Dinner.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| Warning | javadoc | JavadocType | Missing a Javadoc comment. | 3 |
| Warning | misc | FinalParameters | Parameter recipe should be final. | 5 |
org/nicepop/domain/Meal.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| Warning | javadoc | JavadocType | Missing a Javadoc comment. | 3 |
| Warning | regexp | RegexpSingleline | Line has trailing spaces. | 6 |
| Warning | misc | FinalParameters | Parameter recipe should be final. | 7 |
| Warning | design | DesignForExtension | Method 'getRecipe' is not designed for extension - needs to be abstract, final or empty. | 11 |
| Warning | design | DesignForExtension | Method 'setRecipe' is not designed for extension - needs to be abstract, final or empty. | 15 |
| Warning | misc | FinalParameters | Parameter recipe should be final. | 15 |
org/nicepop/domain/Product.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| Warning | regexp | RegexpSingleline | Line has trailing spaces. | 12 |
| Warning | regexp | RegexpSingleline | Line has trailing spaces. | 14 |
| Warning | javadoc | JavadocType | Missing a Javadoc comment. | 23 |
| Warning | regexp | RegexpSingleline | Line has trailing spaces. | 24 |
| Warning | misc | TodoComment | Comment matches to-do format 'TODO:|todo:'. | 25 |
| Warning | regexp | RegexpSingleline | Line has trailing spaces. | 28 |
| Warning | misc | FinalParameters | Parameter name should be final. | 29 |
| Warning | blocks | AvoidNestedBlocks | Avoid nested blocks. | 31 |
| Warning | whitespace | WhitespaceAround | '{' is not followed by whitespace. | 31 |
| Warning | whitespace | WhitespaceAround | '}' is not preceded with whitespace. | 31 |
| Warning | design | DesignForExtension | Method 'getName' is not designed for extension - needs to be abstract, final or empty. | 35 |
| Warning | design | DesignForExtension | Method 'setName' is not designed for extension - needs to be abstract, final or empty. | 39 |
| Warning | misc | FinalParameters | Parameter name should be final. | 39 |
| Warning | regexp | RegexpSingleline | Line has trailing spaces. | 42 |
org/nicepop/domain/Recipe.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| Warning | regexp | RegexpSingleline | Line has trailing spaces. | 12 |
| Warning | regexp | RegexpSingleline | Line has trailing spaces. | 14 |
| Warning | javadoc | JavadocType | Missing a Javadoc comment. | 26 |
| Warning | regexp | RegexpSingleline | Line has trailing spaces. | 27 |
| Warning | misc | FinalParameters | Parameter version should be final. | 31 |
| Warning | misc | FinalParameters | Parameter products should be final. | 31 |
| Warning | design | DesignForExtension | Method 'getProducts' is not designed for extension - needs to be abstract, final or empty. | 37 |
| Warning | design | DesignForExtension | Method 'setProducts' is not designed for extension - needs to be abstract, final or empty. | 41 |
| Warning | misc | FinalParameters | Parameter products should be final. | 41 |
| Warning | design | DesignForExtension | Method 'getVersion' is not designed for extension - needs to be abstract, final or empty. | 45 |
| Warning | design | DesignForExtension | Method 'setVersion' is not designed for extension - needs to be abstract, final or empty. | 49 |
| Warning | misc | FinalParameters | Parameter version should be final. | 49 |
org/nicepop/domain/RecipeTest.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| Warning | regexp | RegexpSingleline | Line has trailing spaces. | 12 |
| Warning | regexp | RegexpSingleline | Line has trailing spaces. | 14 |
| Warning | javadoc | JavadocType | Missing a Javadoc comment. | 30 |
| Warning | regexp | RegexpSingleline | Line has trailing spaces. | 32 |
| Warning | regexp | RegexpSingleline | Line has trailing spaces. | 33 |
| Warning | regexp | RegexpSingleline | Line has trailing spaces. | 34 |
| Warning | regexp | RegexpSingleline | Line has trailing spaces. | 35 |
| Warning | design | DesignForExtension | Method 'changeProductTest' is not designed for extension - needs to be abstract, final or empty. | 36 |
| Warning | regexp | RegexpSingleline | Line has trailing spaces. | 40 |
| Warning | regexp | RegexpSingleline | Line has trailing spaces. | 42 |
| Warning | regexp | RegexpSingleline | Line has trailing spaces. | 45 |
| Warning | regexp | RegexpSingleline | Line has trailing spaces. | 47 |
| Warning | regexp | RegexpSingleline | Line has trailing spaces. | 50 |
| Warning | design | DesignForExtension | Method 'failingTest' is not designed for extension - needs to be abstract, final or empty. | 50 |
org/nicepop/domain/StatusTestIT.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| Warning | regexp | RegexpSingleline | Line has trailing spaces. | 12 |
| Warning | regexp | RegexpSingleline | Line has trailing spaces. | 14 |
| Warning | javadoc | JavadocType | Missing a Javadoc comment. | 27 |
| Warning | design | DesignForExtension | Method 'shouldFail' is not designed for extension - needs to be abstract, final or empty. | 29 |