The following document contains the results of PMD 6.13.0.
| Rule | Violation | Line |
|---|---|---|
| FormalParameterNamingConventions | The method parameter name '_labelName' doesn't match '[a-z][a-zA-Z0-9]*' | 48 |
| FormalParameterNamingConventions | The method parameter name '_nSize' doesn't match '[a-z][a-zA-Z0-9]*' | 48 |
| FormalParameterNamingConventions | The method parameter name '_labelName' doesn't match '[a-z][a-zA-Z0-9]*' | 59 |
| FormalParameterNamingConventions | The method parameter name '_nSize' doesn't match '[a-z][a-zA-Z0-9]*' | 69 |
| Rule | Violation | Line |
|---|---|---|
| FormalParameterNamingConventions | The method parameter name '_nId' doesn't match '[a-z][a-zA-Z0-9]*' | 49 |
| FormalParameterNamingConventions | The method parameter name '_strTitle' doesn't match '[a-z][a-zA-Z0-9]*' | 49 |
| FormalParameterNamingConventions | The method parameter name '_nId' doesn't match '[a-z][a-zA-Z0-9]*' | 60 |
| FormalParameterNamingConventions | The method parameter name '_strTitle' doesn't match '[a-z][a-zA-Z0-9]*' | 70 |
| FormalParameterNamingConventions | The method parameter name '_type' doesn't match '[a-z][a-zA-Z0-9]*' | 80 |
| Rule | Violation | Line |
|---|---|---|
| CommentRequired | Header comments are required | 36–74 |
| CommentRequired | Public method and constructor comments are required | 44–46 |
| UncommentedEmptyConstructor | Document empty constructor | 44–46 |
| CommentRequired | Public method and constructor comments are required | 48–52 |
| CommentRequired | Public method and constructor comments are required | 64–67 |
| CommentRequired | Public method and constructor comments are required | 69–72 |
| Rule | Violation | Line |
|---|---|---|
| CommentRequired | Public method and constructor comments are required | 60–64 |
| Rule | Violation | Line |
|---|---|---|
| UseDiamondOperator | Explicit type arguments can be replaced by Diamond Operator | 235 |
| MissingOverride | The method 'loadByQuestionId(int, int, Plugin)' is missing an @Override annotation. | 261–283 |
| Rule | Violation | Line |
|---|---|---|
| CommentRequired | Header comments are required | 36–95 |
| UncommentedEmptyConstructor | Document empty constructor | 44–47 |
| CommentRequired | Public method and constructor comments are required | 44–47 |
| CommentRequired | Public method and constructor comments are required | 49–53 |
| CommentRequired | Public method and constructor comments are required | 55–58 |
| CommentRequired | Public method and constructor comments are required | 60–63 |
| CommentRequired | Public method and constructor comments are required | 65–68 |
| CommentRequired | Public method and constructor comments are required | 70–73 |
| CommentRequired | Public method and constructor comments are required | 75–78 |
| CommentRequired | Public method and constructor comments are required | 80–83 |
| CommentRequired | Public method and constructor comments are required | 85–88 |
| CommentRequired | Public method and constructor comments are required | 90–93 |
| Rule | Violation | Line |
|---|---|---|
| CommentRequired | Header comments are required | 56–116 |
| UseUtilityClass | All methods are static. Consider using a utility class instead. Alternatively, you could add a private constructor or make the class abstract to silence this warning. | 57–116 |
| CommentRequired | Public method and constructor comments are required | 59–114 |
| UseConcurrentHashMap | If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation | 62 |
| SimplifyBooleanExpressions | Avoid unnecessary comparisons in boolean expressions | 71 |
| UseDiamondOperator | Explicit type arguments can be replaced by Diamond Operator | 87 |
| Rule | Violation | Line |
|---|---|---|
| UnusedPrivateField | Avoid unused private fields such as 'INFO_POLLFORM_QUESTION_REMOVED'. | 134 |
| NullAssignment | Assigning an Object to null is a code smell. Consider refactoring. | 149 |
| ConfusingTernary | Avoid if (x != y) ..; else ..; | 165 |
| ConfusingTernary | Avoid if (x != y) ..; else ..; | 330–337 |
| LinguisticNaming | Linguistics Antipattern - The variable 'isToolBox' indicates linguistically it is a boolean, but it is 'String' | 387 |
| ConfusingTernary | Avoid if (x != y) ..; else ..; | 391–398 |
| Rule | Violation | Line |
|---|---|---|
| BooleanGetMethodName | A getX() method which returns a boolean should be named isX() | 65–68 |
| BooleanGetMethodName | A getX() method which returns a boolean should be named isX() | 191–194 |
| Rule | Violation | Line |
|---|---|---|
| BooleanGetMethodName | A getX() method which returns a boolean should be named isX() | 134–137 |
| BooleanGetMethodName | A getX() method which returns a boolean should be named isX() | 155–158 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| CommentRequired | Header comments are required | 3 | 36–74 |
| CommentRequired | Public method and constructor comments are required | 3 | 44–46 |
| UncommentedEmptyConstructor | Document empty constructor | 3 | 44–46 |
| FormalParameterNamingConventions | The method parameter name '_labelName' doesn't match '[a-z][a-zA-Z0-9]*' | 1 | 48 |
| FormalParameterNamingConventions | The method parameter name '_nSize' doesn't match '[a-z][a-zA-Z0-9]*' | 1 | 48 |
| CommentRequired | Public method and constructor comments are required | 3 | 48–52 |
| FormalParameterNamingConventions | The method parameter name '_labelName' doesn't match '[a-z][a-zA-Z0-9]*' | 1 | 59 |
| CommentRequired | Public method and constructor comments are required | 3 | 64–67 |
| FormalParameterNamingConventions | The method parameter name '_nSize' doesn't match '[a-z][a-zA-Z0-9]*' | 1 | 69 |
| CommentRequired | Public method and constructor comments are required | 3 | 69–72 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| BooleanGetMethodName | A getX() method which returns a boolean should be named isX() | 4 | 65–68 |
| BooleanGetMethodName | A getX() method which returns a boolean should be named isX() | 4 | 191–194 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| CommentRequired | Public method and constructor comments are required | 3 | 60–64 |
| BooleanGetMethodName | A getX() method which returns a boolean should be named isX() | 4 | 134–137 |
| BooleanGetMethodName | A getX() method which returns a boolean should be named isX() | 4 | 155–158 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| UseDiamondOperator | Explicit type arguments can be replaced by Diamond Operator | 3 | 235 |
| MissingOverride | The method 'loadByQuestionId(int, int, Plugin)' is missing an @Override annotation. | 3 | 261–283 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| CommentRequired | Header comments are required | 3 | 36–95 |
| UncommentedEmptyConstructor | Document empty constructor | 3 | 44–47 |
| CommentRequired | Public method and constructor comments are required | 3 | 44–47 |
| FormalParameterNamingConventions | The method parameter name '_nId' doesn't match '[a-z][a-zA-Z0-9]*' | 1 | 49 |
| CommentRequired | Public method and constructor comments are required | 3 | 49–53 |
| FormalParameterNamingConventions | The method parameter name '_strTitle' doesn't match '[a-z][a-zA-Z0-9]*' | 1 | 49 |
| CommentRequired | Public method and constructor comments are required | 3 | 55–58 |
| FormalParameterNamingConventions | The method parameter name '_nId' doesn't match '[a-z][a-zA-Z0-9]*' | 1 | 60 |
| CommentRequired | Public method and constructor comments are required | 3 | 60–63 |
| CommentRequired | Public method and constructor comments are required | 3 | 65–68 |
| FormalParameterNamingConventions | The method parameter name '_strTitle' doesn't match '[a-z][a-zA-Z0-9]*' | 1 | 70 |
| CommentRequired | Public method and constructor comments are required | 3 | 70–73 |
| CommentRequired | Public method and constructor comments are required | 3 | 75–78 |
| FormalParameterNamingConventions | The method parameter name '_type' doesn't match '[a-z][a-zA-Z0-9]*' | 1 | 80 |
| CommentRequired | Public method and constructor comments are required | 3 | 80–83 |
| CommentRequired | Public method and constructor comments are required | 3 | 85–88 |
| CommentRequired | Public method and constructor comments are required | 3 | 90–93 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| CommentRequired | Header comments are required | 3 | 56–116 |
| UseUtilityClass | All methods are static. Consider using a utility class instead. Alternatively, you could add a private constructor or make the class abstract to silence this warning. | 3 | 57–116 |
| CommentRequired | Public method and constructor comments are required | 3 | 59–114 |
| UseConcurrentHashMap | If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation | 3 | 62 |
| SimplifyBooleanExpressions | Avoid unnecessary comparisons in boolean expressions | 3 | 71 |
| UseDiamondOperator | Explicit type arguments can be replaced by Diamond Operator | 3 | 87 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| UnusedPrivateField | Avoid unused private fields such as 'INFO_POLLFORM_QUESTION_REMOVED'. | 3 | 134 |
| NullAssignment | Assigning an Object to null is a code smell. Consider refactoring. | 3 | 149 |
| ConfusingTernary | Avoid if (x != y) ..; else ..; | 3 | 165 |
| ConfusingTernary | Avoid if (x != y) ..; else ..; | 3 | 330–337 |
| LinguisticNaming | Linguistics Antipattern - The variable 'isToolBox' indicates linguistically it is a boolean, but it is 'String' | 3 | 387 |
| ConfusingTernary | Avoid if (x != y) ..; else ..; | 3 | 391–398 |
