Le document suivant contient les résultats de PMD 6.13.0.
| Rule | Violation | Ligne |
|---|---|---|
| LocalVariableNamingConventions | The local variable name 'MenusList' doesn't match '[a-z][a-zA-Z0-9]*' | 182 |
| LocalVariableNamingConventions | The local variable name 'MenusList' doesn't match '[a-z][a-zA-Z0-9]*' | 262 |
| Rule | Violation | Ligne |
|---|---|---|
| FormalParameterNamingConventions | The method parameter name '_plugin' doesn't match '[a-z][a-zA-Z0-9]*' | 105 |
| FormalParameterNamingConventions | The method parameter name '_plugin' doesn't match '[a-z][a-zA-Z0-9]*' | 114 |
| FormalParameterNamingConventions | The method parameter name '_plugin' doesn't match '[a-z][a-zA-Z0-9]*' | 126 |
| Rule | Violation | Ligne |
|---|---|---|
| FormalParameterNamingConventions | The method parameter name '_plugin' doesn't match '[a-z][a-zA-Z0-9]*' | 141 |
| Rule | Violation | Ligne |
|---|---|---|
| FieldNamingConventions | The final field name '_listItems' doesn't match '[a-z][a-zA-Z0-9]*' | 49 |
| Rule | Violation | Ligne |
|---|---|---|
| FieldNamingConventions | The field name 'isActive' doesn't match '_[a-z][a-zA-Z0-9]*' | 47 |
| Rule | Violation | Ligne |
|---|---|---|
| FieldNamingConventions | The final field name 'PROPERTY_SUFFIX_DEFAULT_LIST_ITEM_PER_PAGE' doesn't match '[a-z][a-zA-Z0-9]*' | 57 |
| Rule | Violation | Ligne |
|---|---|---|
| FieldNamingConventions | The final field name 'MESSAGE_TYPE_NOT_EMPTY' doesn't match '[a-z][a-zA-Z0-9]*' | 60 |
| FieldNamingConventions | The final field name 'MESSAGE_TYPE_NOT_VALID' doesn't match '[a-z][a-zA-Z0-9]*' | 61 |
| FieldNamingConventions | The final field name 'MESSAGE_PARENT_MENU_NOT_VALID' doesn't match '[a-z][a-zA-Z0-9]*' | 62 |
| FieldNamingConventions | The final field name 'MESSAGE_SUBMENU_NOT_EMPTY' doesn't match '[a-z][a-zA-Z0-9]*' | 63 |
| FieldNamingConventions | The final field name 'MESSAGE_URL_NOT_EMPTY' doesn't match '[a-z][a-zA-Z0-9]*' | 64 |
| FieldNamingConventions | The final field name 'MESSAGE_LABEL_NOT_EMPTY' doesn't match '[a-z][a-zA-Z0-9]*' | 65 |
| FieldNamingConventions | The final field name 'MESSAGE_DYNAMIC_PAGE_LABEL_NOT_EMPTY' doesn't match '[a-z][a-zA-Z0-9]*' | 66 |
| Rule | Violation | Ligne |
|---|---|---|
| UseDiamondOperator | Explicit type arguments can be replaced by Diamond Operator | 78 |
| Rule | Violation | Ligne |
|---|---|---|
| ForLoopCanBeForeach | This for loop can be replaced by a foreach loop | 188–191 |
| UnnecessaryModifier | Unnecessary modifier 'final' on resource specification 'daoUtil': resource specifications are implicitly final | 196 |
| UseDiamondOperator | Explicit type arguments can be replaced by Diamond Operator | 240 |
| UnnecessaryModifier | Unnecessary modifier 'final' on resource specification 'daoUtil': resource specifications are implicitly final | 268 |
| ConsecutiveAppendsShouldReuse | StringBuffer (or StringBuilder).append is called consecutively without reusing the target variable. | 292 |
| ConsecutiveAppendsShouldReuse | StringBuffer (or StringBuilder).append is called consecutively without reusing the target variable. | 293 |
| UnusedLocalVariable | Avoid unused local variables such as 'criteria'. | 296 |
| Rule | Violation | Ligne |
|---|---|---|
| ConfusingTernary | Avoid if (x != y) ..; else ..; | 211–222 |
| UseIndexOfChar | String.indexOf(char) is faster than String.indexOf(String). | 213 |
| Rule | Violation | Ligne |
|---|---|---|
| UseDiamondOperator | Explicit type arguments can be replaced by Diamond Operator | 49 |
| Rule | Violation | Ligne |
|---|---|---|
| UseDiamondOperator | Explicit type arguments can be replaced by Diamond Operator | 192 |
| Rule | Violation | Ligne |
|---|---|---|
| CommentDefaultAccessModifier | To avoid mistakes add a comment at the beginning of the PageInfo constructor if you want a default access modifier | 57–63 |
| ShortVariable | Avoid variables with short names like id | 77 |
| Rule | Violation | Ligne |
|---|---|---|
| CommentRequired | Header comments are required | 49–227 |
| ConfusingTernary | Avoid if (x != y) ..; else ..; | 126 |
| CyclomaticComplexity | The method 'traverseItem(MenuItem, ReferenceList, String)' has a cyclomatic complexity of 10. | 186–210 |
| ConfusingTernary | Avoid if (x != y) ..; else ..; | 195 |
| ConfusingTernary | Avoid if (x != y) ..; else ..; | 196 |
| CommentRequired | Public method and constructor comments are required | 212–226 |
| Rule | Violation | Ligne |
|---|---|---|
| AvoidLiteralsInIfCondition | Avoid using Literals in Conditional Statements | 119 |
| Rule | Violation | Ligne |
|---|---|---|
| UseDiamondOperator | Explicit type arguments can be replaced by Diamond Operator | 112 |
| Rule | Violation | Ligne |
|---|---|---|
| CyclomaticComplexity | The method 'fillTemplate(Map, PageData, int, HttpServletRequest)' has a cyclomatic complexity of 11. | 102–183 |
| AvoidDeeplyNestedIfStmts | Deeply nested if..then statements are hard to read | 139–167 |
| AvoidDeeplyNestedIfStmts | Deeply nested if..then statements are hard to read | 169–173 |
| AvoidDeeplyNestedIfStmts | Deeply nested if..then statements are hard to read | 175–179 |
| ShortVariable | Avoid variables with short names like cm | 196 |
| UnusedFormalParameter | Avoid unused method parameters such as 'nCurrentPageId'. | 196 |
| UseDiamondOperator | Explicit type arguments can be replaced by Diamond Operator | 199 |
| AvoidCatchingGenericException | Avoid catching generic exceptions such as NullPointerException, RuntimeException, Exception in try-catch block | 267 |
| Rule | Violation | Ligne |
|---|---|---|
| TooManyMethods | This class has too many methods, consider refactoring it. | 71–1052 |
| NullAssignment | Assigning an Object to null is a code smell. Consider refactoring. | 217 |
| NullAssignment | Assigning an Object to null is a code smell. Consider refactoring. | 218 |
| NullAssignment | Assigning an Object to null is a code smell. Consider refactoring. | 394 |
| NullAssignment | Assigning an Object to null is a code smell. Consider refactoring. | 481 |
| NullAssignment | Assigning an Object to null is a code smell. Consider refactoring. | 512 |
| NullAssignment | Assigning an Object to null is a code smell. Consider refactoring. | 557 |
| NullAssignment | Assigning an Object to null is a code smell. Consider refactoring. | 587 |
| NullAssignment | Assigning an Object to null is a code smell. Consider refactoring. | 682 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 712 |
| DefaultPackage | Use explicit scoping instead of the default package private level | 817–821 |
| CommentDefaultAccessModifier | To avoid mistakes add a comment at the beginning of the getItemsFromIds method if you want a default access modifier | 818–821 |
| ShortVariable | Avoid variables with short names like cs | 853 |
| AvoidCatchingGenericException | Avoid catching generic exceptions such as NullPointerException, RuntimeException, Exception in try-catch block | 971 |
| ControlStatementBraces | This statement should have braces | 989 |
| ControlStatementBraces | This statement should have braces | 991 |
| ControlStatementBraces | This statement should have braces | 993 |
| Rule | Violation | Ligne |
|---|---|---|
| UnusedPrivateField | Avoid unused private fields such as 'MARK_MENU_NAME'. | 69 |
| UseDiamondOperator | Explicit type arguments can be replaced by Diamond Operator | 131 |
| UseDiamondOperator | Explicit type arguments can be replaced by Diamond Operator | 168 |
| Rule | Violation | Ligne |
|---|---|---|
| UseDiamondOperator | Explicit type arguments can be replaced by Diamond Operator | 130 |
| UseDiamondOperator | Explicit type arguments can be replaced by Diamond Operator | 167 |
| Rule | Violation | Ligne |
|---|---|---|
| UseDiamondOperator | Explicit type arguments can be replaced by Diamond Operator | 82 |
| UseConcurrentHashMap | If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation | 82 |
| Rule | Violation | Ligne |
|---|---|---|
| UseConcurrentHashMap | If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation | 82 |
| UseDiamondOperator | Explicit type arguments can be replaced by Diamond Operator | 82 |
| Rule | Violation | Ligne |
|---|---|---|
| FinalFieldCouldBeStatic | This final field could be made static | 57 |
| DefaultPackage | Use explicit scoping instead of the default package private level | 164 |
| CommentDefaultAccessModifier | To avoid mistakes add a comment at the beginning of the getItemsFromIds method if you want a default access modifier | 164 |
| Rule | Violation | Ligne |
|---|---|---|
| UseConcurrentHashMap | If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation | 211 |
| UseDiamondOperator | Explicit type arguments can be replaced by Diamond Operator | 211 |
| Rule | Violation | Ligne |
|---|---|---|
| UseDiamondOperator | Explicit type arguments can be replaced by Diamond Operator | 109 |
| UseDiamondOperator | Explicit type arguments can be replaced by Diamond Operator | 110 |
| Rule | Violation | Ligne |
|---|---|---|
| AvoidCatchingGenericException | Avoid catching generic exceptions such as NullPointerException, RuntimeException, Exception in try-catch block | 114 |
| AvoidCatchingGenericException | Avoid catching generic exceptions such as NullPointerException, RuntimeException, Exception in try-catch block | 147 |
| AvoidCatchingGenericException | Avoid catching generic exceptions such as NullPointerException, RuntimeException, Exception in try-catch block | 173 |
| AvoidCatchingGenericException | Avoid catching generic exceptions such as NullPointerException, RuntimeException, Exception in try-catch block | 207 |
| AvoidCatchingGenericException | Avoid catching generic exceptions such as NullPointerException, RuntimeException, Exception in try-catch block | 236 |
| Rule | Violation | Ligne |
|---|---|---|
| FinalFieldCouldBeStatic | This final field could be made static | 60 |
| FinalFieldCouldBeStatic | This final field could be made static | 61 |
| FinalFieldCouldBeStatic | This final field could be made static | 62 |
| FinalFieldCouldBeStatic | This final field could be made static | 63 |
| FinalFieldCouldBeStatic | This final field could be made static | 64 |
| FinalFieldCouldBeStatic | This final field could be made static | 65 |
| FinalFieldCouldBeStatic | This final field could be made static | 66 |
| ImmutableField | Private field '_listErrors' could be made final; it is only initialized in the declaration or constructor. | 68 |
| CyclomaticComplexity | The method 'isValid(CustomMenuItem, Locale)' has a cyclomatic complexity of 14. | 91–155 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 95 |
| UnusedFormalParameter | Avoid unused method parameters such as 'locale'. | 162 |
| UnusedFormalParameter | Avoid unused method parameters such as 'locale'. | 167 |
| UnusedFormalParameter | Avoid unused method parameters such as 'locale'. | 172 |
| SimplifyBooleanReturns | Avoid unnecessary if..then..else statements when returning booleans | 175–182 |
| Rule | Violation | Ligne |
|---|---|---|
| UnusedImports | Avoid unused imports such as 'java.util.Collection' | 36 |
| UnusedImports | Avoid unused imports such as 'java.util.List' | 37 |
| Rule | Violation | Priority | Ligne |
|---|---|---|---|
| UseDiamondOperator | Explicit type arguments can be replaced by Diamond Operator | 3 | 78 |
| Rule | Violation | Priority | Ligne |
|---|---|---|---|
| LocalVariableNamingConventions | The local variable name 'MenusList' doesn't match '[a-z][a-zA-Z0-9]*' | 1 | 182 |
| ForLoopCanBeForeach | This for loop can be replaced by a foreach loop | 3 | 188–191 |
| UnnecessaryModifier | Unnecessary modifier 'final' on resource specification 'daoUtil': resource specifications are implicitly final | 3 | 196 |
| UseDiamondOperator | Explicit type arguments can be replaced by Diamond Operator | 3 | 240 |
| LocalVariableNamingConventions | The local variable name 'MenusList' doesn't match '[a-z][a-zA-Z0-9]*' | 1 | 262 |
| UnnecessaryModifier | Unnecessary modifier 'final' on resource specification 'daoUtil': resource specifications are implicitly final | 3 | 268 |
| ConsecutiveAppendsShouldReuse | StringBuffer (or StringBuilder).append is called consecutively without reusing the target variable. | 3 | 292 |
| ConsecutiveAppendsShouldReuse | StringBuffer (or StringBuilder).append is called consecutively without reusing the target variable. | 3 | 293 |
| UnusedLocalVariable | Avoid unused local variables such as 'criteria'. | 3 | 296 |
| Rule | Violation | Priority | Ligne |
|---|---|---|---|
| ConfusingTernary | Avoid if (x != y) ..; else ..; | 3 | 211–222 |
| UseIndexOfChar | String.indexOf(char) is faster than String.indexOf(String). | 3 | 213 |
| Rule | Violation | Priority | Ligne |
|---|---|---|---|
| FormalParameterNamingConventions | The method parameter name '_plugin' doesn't match '[a-z][a-zA-Z0-9]*' | 1 | 105 |
| FormalParameterNamingConventions | The method parameter name '_plugin' doesn't match '[a-z][a-zA-Z0-9]*' | 1 | 114 |
| FormalParameterNamingConventions | The method parameter name '_plugin' doesn't match '[a-z][a-zA-Z0-9]*' | 1 | 126 |
| Rule | Violation | Priority | Ligne |
|---|---|---|---|
| FormalParameterNamingConventions | The method parameter name '_plugin' doesn't match '[a-z][a-zA-Z0-9]*' | 1 | 141 |
| Rule | Violation | Priority | Ligne |
|---|---|---|---|
| UseDiamondOperator | Explicit type arguments can be replaced by Diamond Operator | 3 | 49 |
| FieldNamingConventions | The final field name '_listItems' doesn't match '[a-z][a-zA-Z0-9]*' | 1 | 49 |
| Rule | Violation | Priority | Ligne |
|---|---|---|---|
| FieldNamingConventions | The field name 'isActive' doesn't match '_[a-z][a-zA-Z0-9]*' | 1 | 47 |
| Rule | Violation | Priority | Ligne |
|---|---|---|---|
| UseDiamondOperator | Explicit type arguments can be replaced by Diamond Operator | 3 | 192 |
| Rule | Violation | Priority | Ligne |
|---|---|---|---|
| CommentDefaultAccessModifier | To avoid mistakes add a comment at the beginning of the PageInfo constructor if you want a default access modifier | 3 | 57–63 |
| ShortVariable | Avoid variables with short names like id | 3 | 77 |
| Rule | Violation | Priority | Ligne |
|---|---|---|---|
| CommentRequired | Header comments are required | 3 | 49–227 |
| ConfusingTernary | Avoid if (x != y) ..; else ..; | 3 | 126 |
| CyclomaticComplexity | The method 'traverseItem(MenuItem, ReferenceList, String)' has a cyclomatic complexity of 10. | 3 | 186–210 |
| ConfusingTernary | Avoid if (x != y) ..; else ..; | 3 | 195 |
| ConfusingTernary | Avoid if (x != y) ..; else ..; | 3 | 196 |
| CommentRequired | Public method and constructor comments are required | 3 | 212–226 |
| Rule | Violation | Priority | Ligne |
|---|---|---|---|
| AvoidLiteralsInIfCondition | Avoid using Literals in Conditional Statements | 3 | 119 |
| Rule | Violation | Priority | Ligne |
|---|---|---|---|
| UseDiamondOperator | Explicit type arguments can be replaced by Diamond Operator | 3 | 112 |
| Rule | Violation | Priority | Ligne |
|---|---|---|---|
| CyclomaticComplexity | The method 'fillTemplate(Map, PageData, int, HttpServletRequest)' has a cyclomatic complexity of 11. | 3 | 102–183 |
| AvoidDeeplyNestedIfStmts | Deeply nested if..then statements are hard to read | 3 | 139–167 |
| AvoidDeeplyNestedIfStmts | Deeply nested if..then statements are hard to read | 3 | 169–173 |
| AvoidDeeplyNestedIfStmts | Deeply nested if..then statements are hard to read | 3 | 175–179 |
| ShortVariable | Avoid variables with short names like cm | 3 | 196 |
| UnusedFormalParameter | Avoid unused method parameters such as 'nCurrentPageId'. | 3 | 196 |
| UseDiamondOperator | Explicit type arguments can be replaced by Diamond Operator | 3 | 199 |
| AvoidCatchingGenericException | Avoid catching generic exceptions such as NullPointerException, RuntimeException, Exception in try-catch block | 3 | 267 |
| Rule | Violation | Priority | Ligne |
|---|---|---|---|
| TooManyMethods | This class has too many methods, consider refactoring it. | 3 | 71–1052 |
| NullAssignment | Assigning an Object to null is a code smell. Consider refactoring. | 3 | 217 |
| NullAssignment | Assigning an Object to null is a code smell. Consider refactoring. | 3 | 218 |
| NullAssignment | Assigning an Object to null is a code smell. Consider refactoring. | 3 | 394 |
| NullAssignment | Assigning an Object to null is a code smell. Consider refactoring. | 3 | 481 |
| NullAssignment | Assigning an Object to null is a code smell. Consider refactoring. | 3 | 512 |
| NullAssignment | Assigning an Object to null is a code smell. Consider refactoring. | 3 | 557 |
| NullAssignment | Assigning an Object to null is a code smell. Consider refactoring. | 3 | 587 |
| NullAssignment | Assigning an Object to null is a code smell. Consider refactoring. | 3 | 682 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 3 | 712 |
| DefaultPackage | Use explicit scoping instead of the default package private level | 3 | 817–821 |
| CommentDefaultAccessModifier | To avoid mistakes add a comment at the beginning of the getItemsFromIds method if you want a default access modifier | 3 | 818–821 |
| ShortVariable | Avoid variables with short names like cs | 3 | 853 |
| AvoidCatchingGenericException | Avoid catching generic exceptions such as NullPointerException, RuntimeException, Exception in try-catch block | 3 | 971 |
| ControlStatementBraces | This statement should have braces | 3 | 989 |
| ControlStatementBraces | This statement should have braces | 3 | 991 |
| ControlStatementBraces | This statement should have braces | 3 | 993 |
| Rule | Violation | Priority | Ligne |
|---|---|---|---|
| UnusedPrivateField | Avoid unused private fields such as 'MARK_MENU_NAME'. | 3 | 69 |
| UseDiamondOperator | Explicit type arguments can be replaced by Diamond Operator | 3 | 131 |
| UseDiamondOperator | Explicit type arguments can be replaced by Diamond Operator | 3 | 168 |
| Rule | Violation | Priority | Ligne |
|---|---|---|---|
| UseDiamondOperator | Explicit type arguments can be replaced by Diamond Operator | 3 | 130 |
| UseDiamondOperator | Explicit type arguments can be replaced by Diamond Operator | 3 | 167 |
| Rule | Violation | Priority | Ligne |
|---|---|---|---|
| UseDiamondOperator | Explicit type arguments can be replaced by Diamond Operator | 3 | 82 |
| UseConcurrentHashMap | If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation | 3 | 82 |
| Rule | Violation | Priority | Ligne |
|---|---|---|---|
| UseConcurrentHashMap | If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation | 3 | 82 |
| UseDiamondOperator | Explicit type arguments can be replaced by Diamond Operator | 3 | 82 |
| Rule | Violation | Priority | Ligne |
|---|---|---|---|
| FinalFieldCouldBeStatic | This final field could be made static | 3 | 57 |
| FieldNamingConventions | The final field name 'PROPERTY_SUFFIX_DEFAULT_LIST_ITEM_PER_PAGE' doesn't match '[a-z][a-zA-Z0-9]*' | 1 | 57 |
| DefaultPackage | Use explicit scoping instead of the default package private level | 3 | 164 |
| CommentDefaultAccessModifier | To avoid mistakes add a comment at the beginning of the getItemsFromIds method if you want a default access modifier | 3 | 164 |
| Rule | Violation | Priority | Ligne |
|---|---|---|---|
| UseConcurrentHashMap | If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation | 3 | 211 |
| UseDiamondOperator | Explicit type arguments can be replaced by Diamond Operator | 3 | 211 |
| Rule | Violation | Priority | Ligne |
|---|---|---|---|
| UseDiamondOperator | Explicit type arguments can be replaced by Diamond Operator | 3 | 109 |
| UseDiamondOperator | Explicit type arguments can be replaced by Diamond Operator | 3 | 110 |
| Rule | Violation | Priority | Ligne |
|---|---|---|---|
| UnusedImports | Avoid unused imports such as 'java.util.Collection' | 4 | 36 |
| UnusedImports | Avoid unused imports such as 'java.util.List' | 4 | 37 |
| AvoidCatchingGenericException | Avoid catching generic exceptions such as NullPointerException, RuntimeException, Exception in try-catch block | 3 | 114 |
| AvoidCatchingGenericException | Avoid catching generic exceptions such as NullPointerException, RuntimeException, Exception in try-catch block | 3 | 147 |
| AvoidCatchingGenericException | Avoid catching generic exceptions such as NullPointerException, RuntimeException, Exception in try-catch block | 3 | 173 |
| AvoidCatchingGenericException | Avoid catching generic exceptions such as NullPointerException, RuntimeException, Exception in try-catch block | 3 | 207 |
| AvoidCatchingGenericException | Avoid catching generic exceptions such as NullPointerException, RuntimeException, Exception in try-catch block | 3 | 236 |
| Rule | Violation | Priority | Ligne |
|---|---|---|---|
| FinalFieldCouldBeStatic | This final field could be made static | 3 | 60 |
| FieldNamingConventions | The final field name 'MESSAGE_TYPE_NOT_EMPTY' doesn't match '[a-z][a-zA-Z0-9]*' | 1 | 60 |
| FinalFieldCouldBeStatic | This final field could be made static | 3 | 61 |
| FieldNamingConventions | The final field name 'MESSAGE_TYPE_NOT_VALID' doesn't match '[a-z][a-zA-Z0-9]*' | 1 | 61 |
| FinalFieldCouldBeStatic | This final field could be made static | 3 | 62 |
| FieldNamingConventions | The final field name 'MESSAGE_PARENT_MENU_NOT_VALID' doesn't match '[a-z][a-zA-Z0-9]*' | 1 | 62 |
| FieldNamingConventions | The final field name 'MESSAGE_SUBMENU_NOT_EMPTY' doesn't match '[a-z][a-zA-Z0-9]*' | 1 | 63 |
| FinalFieldCouldBeStatic | This final field could be made static | 3 | 63 |
| FieldNamingConventions | The final field name 'MESSAGE_URL_NOT_EMPTY' doesn't match '[a-z][a-zA-Z0-9]*' | 1 | 64 |
| FinalFieldCouldBeStatic | This final field could be made static | 3 | 64 |
| FinalFieldCouldBeStatic | This final field could be made static | 3 | 65 |
| FieldNamingConventions | The final field name 'MESSAGE_LABEL_NOT_EMPTY' doesn't match '[a-z][a-zA-Z0-9]*' | 1 | 65 |
| FinalFieldCouldBeStatic | This final field could be made static | 3 | 66 |
| FieldNamingConventions | The final field name 'MESSAGE_DYNAMIC_PAGE_LABEL_NOT_EMPTY' doesn't match '[a-z][a-zA-Z0-9]*' | 1 | 66 |
| ImmutableField | Private field '_listErrors' could be made final; it is only initialized in the declaration or constructor. | 3 | 68 |
| CyclomaticComplexity | The method 'isValid(CustomMenuItem, Locale)' has a cyclomatic complexity of 14. | 3 | 91–155 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 3 | 95 |
| UnusedFormalParameter | Avoid unused method parameters such as 'locale'. | 3 | 162 |
| UnusedFormalParameter | Avoid unused method parameters such as 'locale'. | 3 | 167 |
| UnusedFormalParameter | Avoid unused method parameters such as 'locale'. | 3 | 172 |
| SimplifyBooleanReturns | Avoid unnecessary if..then..else statements when returning booleans | 3 | 175–182 |
