Checkstyle Results
The following document contains the results of Checkstyle 9.3 with Checkstyle.xml ruleset.
Summary
| Files | |||
|---|---|---|---|
| 39 | 0 | 44 | 0 |
Files
Rules
| Category | Rule | Violations | Severity |
|---|---|---|---|
| coding | AvoidNoArgumentSuperConstructorCall | 1 | |
FinalLocalVariable
|
4 | ||
| HiddenField | 1 | ||
| MagicNumber | 2 | ||
| MissingCtor | 4 | ||
| MultipleStringLiterals | 2 | ||
| design | HideUtilityClassConstructor | 1 | |
| imports | UnusedImports | 1 | |
| indentation | CommentsIndentation | 1 | |
Indentation
|
5 | ||
| javadoc | InvalidJavadocPosition | 3 | |
| JavadocContentLocation | 3 | ||
| JavadocMethod | 6 | ||
| JavadocStyle | 2 | ||
| JavadocVariable | 3 | ||
| misc | FinalParameters | 4 | |
| UncommentedMain | 1 |
Details
uk/pallas/systems/typr/domain/entities/v1/FieldDefinitionDomain.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| coding | MagicNumber | '100' is a magic number. | 58 |
uk/pallas/systems/typr/entities/v1/validation/ValidationRuleConstants.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| coding | AvoidNoArgumentSuperConstructorCall | Unnecessary call to superclass constructor with no arguments. | 25 |
uk/pallas/systems/typr/rest/CategoriesController.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| coding | MissingCtor | Class should define a constructor. | 13 | |
| javadoc | InvalidJavadocPosition | Javadoc comment is placed in the wrong location. | 32 | |
| indentation | CommentsIndentation | Block comment has incorrect indentation level 1, expected is 2, indentation should be the same level as line 28. | 37 |
uk/pallas/systems/typr/rest/FieldDefinitionsController.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| coding | MissingCtor | Class should define a constructor. | 40 | |
| javadoc | JavadocVariable | Missing a Javadoc comment. | 54 | |
| javadoc | JavadocVariable | Missing a Javadoc comment. | 57 | |
| coding | MultipleStringLiterals | The String "No definitions stored within Typr" appears 2 times in the file. | 112 | |
| indentation | Indentation | 'annotation array initialization' child has incorrect indentation level 4, expected level should be 6. | 134 | |
| indentation | Indentation | 'annotation array initialization' child has incorrect indentation level 4, expected level should be 6. | 205 | |
| coding | MultipleStringLiterals | The String "Object had incorrect unit type" appears 2 times in the file. | 213 | |
| indentation | Indentation | 'annotation array initialization' child has incorrect indentation level 4, expected level should be 6. | 236 | |
| indentation | Indentation | 'annotation array initialization' child has incorrect indentation level 4, expected level should be 6. | 272 | |
| indentation | Indentation | 'annotation array initialization' child has incorrect indentation level 4, expected level should be 6. | 299 |
uk/pallas/systems/typr/rest/entities/v1/FieldDefinitionDTO.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| coding | MagicNumber | '100' is a magic number. | 55 | |
| coding | HiddenField | 'rules' hides a field. | 96 |
uk/pallas/systems/typr/runnable/Application.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| design | HideUtilityClassConstructor | Utility classes should not have a public or default constructor. | 16 | |
| misc | UncommentedMain | Uncommented main method found. | 34 |
uk/pallas/systems/typr/services/FieldDefinitionService.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| javadoc | JavadocMethod | Expected @param tag for 'string'. | 21 | |
| javadoc | JavadocStyle | First sentence should end with a period. | 23 | |
| javadoc | JavadocStyle | First sentence should end with a period. | 30 |
uk/pallas/systems/typr/services/FieldDefinitionServiceImpl.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| coding | MissingCtor | Class should define a constructor. | 13 | |
| javadoc | JavadocVariable | Missing a Javadoc comment. | 16 | |
| javadoc | JavadocContentLocation | Javadoc content should start from the next line after /**. | 19 | |
| javadoc | JavadocMethod | @return tag should be present and have description. | 23 | |
| javadoc | JavadocContentLocation | Javadoc content should start from the next line after /**. | 32 | |
| javadoc | JavadocMethod | @return tag should be present and have description. | 37 | |
| javadoc | JavadocContentLocation | Javadoc content should start from the next line after /**. | 46 | |
| javadoc | JavadocMethod | @return tag should be present and have description. | 51 |
uk/pallas/systems/typr/services/impl/CountryServiceImpl.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| coding | MissingCtor | Class should define a constructor. | 13 |
uk/pallas/systems/typr/services/impl/UnitsServiceImpl.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| imports | UnusedImports | Unused import - javax.measure.quantity.Length. | 9 | |
| javadoc | InvalidJavadocPosition | Javadoc comment is placed in the wrong location. | 67 | |
| javadoc | InvalidJavadocPosition | Javadoc comment is placed in the wrong location. | 73 | |
| misc | FinalParameters | Parameter value should be final. | 89 | |
| coding | FinalLocalVariable | Variable 'value' should be declared final. | 89 | |
| misc | FinalParameters | Parameter name should be final. | 89 | |
| coding | FinalLocalVariable | Variable 'name' should be declared final. | 89 | |
| misc | FinalParameters | Parameter text should be final. | 89 | |
| coding | FinalLocalVariable | Variable 'text' should be declared final. | 89 | |
| misc | FinalParameters | Parameter isLabel should be final. | 89 | |
| coding | FinalLocalVariable | Variable 'isLabel' should be declared final. | 89 | |
| javadoc | JavadocMethod | Unused @param tag for 'units'. | 112 | |
| javadoc | JavadocMethod | Expected @param tag for 'values'. | 114 |