The following document contains the results of PMD 6.13.0.
| Rule | Violation | Line |
|---|---|---|
| FieldNamingConventions | The field name 'id' doesn't match '_[a-z][a-zA-Z0-9]*' | 44 |
| FieldNamingConventions | The field name 'metaDataId' doesn't match '_[a-z][a-zA-Z0-9]*' | 46 |
| FieldNamingConventions | The field name 'value' doesn't match '_[a-z][a-zA-Z0-9]*' | 48 |
| FieldNamingConventions | The field name '__equalsCalc' doesn't match '_[a-z][a-zA-Z0-9]*' | 122 |
| LocalVariableNamingConventions | The local variable name '_equals' doesn't match '[a-z][a-zA-Z0-9]*' | 132 |
| FieldNamingConventions | The field name '__hashCodeCalc' doesn't match '_[a-z][a-zA-Z0-9]*' | 147 |
| LocalVariableNamingConventions | The local variable name '_hashCode' doesn't match '[a-z][a-zA-Z0-9]*' | 153 |
| FieldNamingConventions | The static field name 'typeDesc' doesn't match '_[a-z][a-zA-Z0-9]*' | 168 |
| FormalParameterNamingConventions | The method parameter name '_javaType' doesn't match '[a-z][a-zA-Z0-9]*' | 205 |
| FormalParameterNamingConventions | The method parameter name '_xmlType' doesn't match '[a-z][a-zA-Z0-9]*' | 206 |
| FormalParameterNamingConventions | The method parameter name '_javaType' doesn't match '[a-z][a-zA-Z0-9]*' | 217 |
| FormalParameterNamingConventions | The method parameter name '_xmlType' doesn't match '[a-z][a-zA-Z0-9]*' | 218 |
| Rule | Violation | Line |
|---|---|---|
| FieldNamingConventions | The field name '__equalsCalc' doesn't match '_[a-z][a-zA-Z0-9]*' | 53 |
| LocalVariableNamingConventions | The local variable name '_equals' doesn't match '[a-z][a-zA-Z0-9]*' | 63 |
| FieldNamingConventions | The field name '__hashCodeCalc' doesn't match '_[a-z][a-zA-Z0-9]*' | 69 |
| LocalVariableNamingConventions | The local variable name '_hashCode' doesn't match '[a-z][a-zA-Z0-9]*' | 75 |
| FieldNamingConventions | The static field name 'typeDesc' doesn't match '_[a-z][a-zA-Z0-9]*' | 81 |
| FormalParameterNamingConventions | The method parameter name '_javaType' doesn't match '[a-z][a-zA-Z0-9]*' | 100 |
| FormalParameterNamingConventions | The method parameter name '_xmlType' doesn't match '[a-z][a-zA-Z0-9]*' | 101 |
| FormalParameterNamingConventions | The method parameter name '_javaType' doesn't match '[a-z][a-zA-Z0-9]*' | 112 |
| FormalParameterNamingConventions | The method parameter name '_xmlType' doesn't match '[a-z][a-zA-Z0-9]*' | 113 |
| Rule | Violation | Line |
|---|---|---|
| FieldNamingConventions | The field name 'SearchAssetService_address' doesn't match '_[a-z][a-zA-Z0-9]*' | 58 |
| FieldNamingConventions | The field name 'SearchAssetServiceWSDDServiceName' doesn't match '_[a-z][a-zA-Z0-9]*' | 65 |
| LocalVariableNamingConventions | The local variable name '_stub' doesn't match '[a-z][a-zA-Z0-9]*' | 88 |
| LocalVariableNamingConventions | The local variable name '_stub' doesn't match '[a-z][a-zA-Z0-9]*' | 109 |
| LocalVariableNamingConventions | The local variable name '_stub' doesn't match '[a-z][a-zA-Z0-9]*' | 134 |
| FieldNamingConventions | The field name 'ports' doesn't match '_[a-z][a-zA-Z0-9]*' | 144 |
| Rule | Violation | Line |
|---|---|---|
| FieldNamingConventions | The field name 'cachedSerClasses' doesn't match '_[a-z][a-zA-Z0-9]*' | 44 |
| FieldNamingConventions | The field name 'cachedSerQNames' doesn't match '_[a-z][a-zA-Z0-9]*' | 45 |
| FieldNamingConventions | The field name 'cachedSerFactories' doesn't match '_[a-z][a-zA-Z0-9]*' | 46 |
| FieldNamingConventions | The field name 'cachedDeserFactories' doesn't match '_[a-z][a-zA-Z0-9]*' | 47 |
| MethodNamingConventions | The static method name '_initOperationDesc1' doesn't match '[a-z][a-zA-Z0-9]*' | 56–122 |
| LocalVariableNamingConventions | The local variable name '_call' doesn't match '[a-z][a-zA-Z0-9]*' | 219 |
| LocalVariableNamingConventions | The exception block parameter name '_t' doesn't match '[a-z][a-zA-Z0-9]*' | 277 |
| LocalVariableNamingConventions | The local variable name '_call' doesn't match '[a-z][a-zA-Z0-9]*' | 286 |
| LocalVariableNamingConventions | The local variable name '_resp' doesn't match '[a-z][a-zA-Z0-9]*' | 295 |
| LocalVariableNamingConventions | The exception block parameter name '_exception' doesn't match '[a-z][a-zA-Z0-9]*' | 304 |
| LocalVariableNamingConventions | The local variable name '_call' doesn't match '[a-z][a-zA-Z0-9]*' | 325 |
| LocalVariableNamingConventions | The local variable name '_resp' doesn't match '[a-z][a-zA-Z0-9]*' | 334 |
| LocalVariableNamingConventions | The exception block parameter name '_exception' doesn't match '[a-z][a-zA-Z0-9]*' | 343 |
| LocalVariableNamingConventions | The local variable name '_call' doesn't match '[a-z][a-zA-Z0-9]*' | 364 |
| LocalVariableNamingConventions | The local variable name '_resp' doesn't match '[a-z][a-zA-Z0-9]*' | 373 |
| LocalVariableNamingConventions | The exception block parameter name '_exception' doesn't match '[a-z][a-zA-Z0-9]*' | 382 |
| Rule | Violation | Line |
|---|---|---|
| FieldNamingConventions | The field name 'assetId' doesn't match '_[a-z][a-zA-Z0-9]*' | 49 |
| FieldNamingConventions | The field name 'creationDate' doesn't match '_[a-z][a-zA-Z0-9]*' | 51 |
| FieldNamingConventions | The field name 'description' doesn't match '_[a-z][a-zA-Z0-9]*' | 53 |
| FieldNamingConventions | The field name 'formats' doesn't match '_[a-z][a-zA-Z0-9]*' | 55 |
| FieldNamingConventions | The field name 'metaValues' doesn't match '_[a-z][a-zA-Z0-9]*' | 57 |
| FieldNamingConventions | The field name 'previewURL' doesn't match '_[a-z][a-zA-Z0-9]*' | 59 |
| FieldNamingConventions | The field name 'thumbnail' doesn't match '_[a-z][a-zA-Z0-9]*' | 61 |
| FieldNamingConventions | The field name 'title' doesn't match '_[a-z][a-zA-Z0-9]*' | 63 |
| FieldNamingConventions | The field name 'videoURL' doesn't match '_[a-z][a-zA-Z0-9]*' | 65 |
| FieldNamingConventions | The field name '__equalsCalc' doesn't match '_[a-z][a-zA-Z0-9]*' | 270 |
| LocalVariableNamingConventions | The local variable name '_equals' doesn't match '[a-z][a-zA-Z0-9]*' | 280 |
| FieldNamingConventions | The field name '__hashCodeCalc' doesn't match '_[a-z][a-zA-Z0-9]*' | 311 |
| LocalVariableNamingConventions | The local variable name '_hashCode' doesn't match '[a-z][a-zA-Z0-9]*' | 317 |
| FieldNamingConventions | The static field name 'typeDesc' doesn't match '_[a-z][a-zA-Z0-9]*' | 364 |
| FormalParameterNamingConventions | The method parameter name '_javaType' doesn't match '[a-z][a-zA-Z0-9]*' | 437 |
| FormalParameterNamingConventions | The method parameter name '_xmlType' doesn't match '[a-z][a-zA-Z0-9]*' | 438 |
| FormalParameterNamingConventions | The method parameter name '_javaType' doesn't match '[a-z][a-zA-Z0-9]*' | 449 |
| FormalParameterNamingConventions | The method parameter name '_xmlType' doesn't match '[a-z][a-zA-Z0-9]*' | 450 |
| Rule | Violation | Line |
|---|---|---|
| FieldNamingConventions | The field name 'htmlEmbed' doesn't match '_[a-z][a-zA-Z0-9]*' | 44 |
| FieldNamingConventions | The field name 'thumbnail' doesn't match '_[a-z][a-zA-Z0-9]*' | 46 |
| FieldNamingConventions | The field name '__equalsCalc' doesn't match '_[a-z][a-zA-Z0-9]*' | 98 |
| LocalVariableNamingConventions | The local variable name '_equals' doesn't match '[a-z][a-zA-Z0-9]*' | 108 |
| FieldNamingConventions | The field name '__hashCodeCalc' doesn't match '_[a-z][a-zA-Z0-9]*' | 120 |
| LocalVariableNamingConventions | The local variable name '_hashCode' doesn't match '[a-z][a-zA-Z0-9]*' | 126 |
| FieldNamingConventions | The static field name 'typeDesc' doesn't match '_[a-z][a-zA-Z0-9]*' | 138 |
| FormalParameterNamingConventions | The method parameter name '_javaType' doesn't match '[a-z][a-zA-Z0-9]*' | 169 |
| FormalParameterNamingConventions | The method parameter name '_xmlType' doesn't match '[a-z][a-zA-Z0-9]*' | 170 |
| FormalParameterNamingConventions | The method parameter name '_javaType' doesn't match '[a-z][a-zA-Z0-9]*' | 181 |
| FormalParameterNamingConventions | The method parameter name '_xmlType' doesn't match '[a-z][a-zA-Z0-9]*' | 182 |
| Rule | Violation | Line |
|---|---|---|
| FieldNamingConventions | The field name '__equalsCalc' doesn't match '_[a-z][a-zA-Z0-9]*' | 53 |
| LocalVariableNamingConventions | The local variable name '_equals' doesn't match '[a-z][a-zA-Z0-9]*' | 63 |
| FieldNamingConventions | The field name '__hashCodeCalc' doesn't match '_[a-z][a-zA-Z0-9]*' | 69 |
| LocalVariableNamingConventions | The local variable name '_hashCode' doesn't match '[a-z][a-zA-Z0-9]*' | 75 |
| FieldNamingConventions | The static field name 'typeDesc' doesn't match '_[a-z][a-zA-Z0-9]*' | 81 |
| FormalParameterNamingConventions | The method parameter name '_javaType' doesn't match '[a-z][a-zA-Z0-9]*' | 100 |
| FormalParameterNamingConventions | The method parameter name '_xmlType' doesn't match '[a-z][a-zA-Z0-9]*' | 101 |
| FormalParameterNamingConventions | The method parameter name '_javaType' doesn't match '[a-z][a-zA-Z0-9]*' | 112 |
| FormalParameterNamingConventions | The method parameter name '_xmlType' doesn't match '[a-z][a-zA-Z0-9]*' | 113 |
| Rule | Violation | Line |
|---|---|---|
| FieldNamingConventions | The field name 'UserService_address' doesn't match '_[a-z][a-zA-Z0-9]*' | 58 |
| FieldNamingConventions | The field name 'UserServiceWSDDServiceName' doesn't match '_[a-z][a-zA-Z0-9]*' | 65 |
| LocalVariableNamingConventions | The local variable name '_stub' doesn't match '[a-z][a-zA-Z0-9]*' | 88 |
| LocalVariableNamingConventions | The local variable name '_stub' doesn't match '[a-z][a-zA-Z0-9]*' | 109 |
| LocalVariableNamingConventions | The local variable name '_stub' doesn't match '[a-z][a-zA-Z0-9]*' | 134 |
| FieldNamingConventions | The field name 'ports' doesn't match '_[a-z][a-zA-Z0-9]*' | 144 |
| Rule | Violation | Line |
|---|---|---|
| FieldNamingConventions | The field name 'cachedSerClasses' doesn't match '_[a-z][a-zA-Z0-9]*' | 44 |
| FieldNamingConventions | The field name 'cachedSerQNames' doesn't match '_[a-z][a-zA-Z0-9]*' | 45 |
| FieldNamingConventions | The field name 'cachedSerFactories' doesn't match '_[a-z][a-zA-Z0-9]*' | 46 |
| FieldNamingConventions | The field name 'cachedDeserFactories' doesn't match '_[a-z][a-zA-Z0-9]*' | 47 |
| MethodNamingConventions | The static method name '_initOperationDesc1' doesn't match '[a-z][a-zA-Z0-9]*' | 56–119 |
| LocalVariableNamingConventions | The local variable name '_call' doesn't match '[a-z][a-zA-Z0-9]*' | 239 |
| LocalVariableNamingConventions | The exception block parameter name '_t' doesn't match '[a-z][a-zA-Z0-9]*' | 297 |
| LocalVariableNamingConventions | The local variable name '_call' doesn't match '[a-z][a-zA-Z0-9]*' | 306 |
| LocalVariableNamingConventions | The local variable name '_resp' doesn't match '[a-z][a-zA-Z0-9]*' | 315 |
| LocalVariableNamingConventions | The local variable name '_call' doesn't match '[a-z][a-zA-Z0-9]*' | 330 |
| LocalVariableNamingConventions | The local variable name '_resp' doesn't match '[a-z][a-zA-Z0-9]*' | 339 |
| LocalVariableNamingConventions | The exception block parameter name '_exception' doesn't match '[a-z][a-zA-Z0-9]*' | 348 |
| LocalVariableNamingConventions | The local variable name '_call' doesn't match '[a-z][a-zA-Z0-9]*' | 369 |
| LocalVariableNamingConventions | The local variable name '_resp' doesn't match '[a-z][a-zA-Z0-9]*' | 378 |
| LocalVariableNamingConventions | The exception block parameter name '_exception' doesn't match '[a-z][a-zA-Z0-9]*' | 387 |
| LocalVariableNamingConventions | The local variable name '_call' doesn't match '[a-z][a-zA-Z0-9]*' | 408 |
| LocalVariableNamingConventions | The local variable name '_resp' doesn't match '[a-z][a-zA-Z0-9]*' | 417 |
| LocalVariableNamingConventions | The exception block parameter name '_exception' doesn't match '[a-z][a-zA-Z0-9]*' | 426 |
| Rule | Violation | Line |
|---|---|---|
| FieldNamingConventions | The field name 'metadataId' doesn't match '_[a-z][a-zA-Z0-9]*' | 44 |
| FieldNamingConventions | The field name 'name' doesn't match '_[a-z][a-zA-Z0-9]*' | 46 |
| FieldNamingConventions | The field name 'type' doesn't match '_[a-z][a-zA-Z0-9]*' | 48 |
| FieldNamingConventions | The field name 'valueList' doesn't match '_[a-z][a-zA-Z0-9]*' | 50 |
| FieldNamingConventions | The field name '__equalsCalc' doesn't match '_[a-z][a-zA-Z0-9]*' | 146 |
| LocalVariableNamingConventions | The local variable name '_equals' doesn't match '[a-z][a-zA-Z0-9]*' | 156 |
| FieldNamingConventions | The field name '__hashCodeCalc' doesn't match '_[a-z][a-zA-Z0-9]*' | 170 |
| LocalVariableNamingConventions | The local variable name '_hashCode' doesn't match '[a-z][a-zA-Z0-9]*' | 176 |
| FieldNamingConventions | The static field name 'typeDesc' doesn't match '_[a-z][a-zA-Z0-9]*' | 198 |
| FormalParameterNamingConventions | The method parameter name '_javaType' doesn't match '[a-z][a-zA-Z0-9]*' | 241 |
| FormalParameterNamingConventions | The method parameter name '_xmlType' doesn't match '[a-z][a-zA-Z0-9]*' | 242 |
| FormalParameterNamingConventions | The method parameter name '_javaType' doesn't match '[a-z][a-zA-Z0-9]*' | 253 |
| FormalParameterNamingConventions | The method parameter name '_xmlType' doesn't match '[a-z][a-zA-Z0-9]*' | 254 |
| Rule | Violation | Line |
|---|---|---|
| FieldNamingConventions | The field name 'id' doesn't match '_[a-z][a-zA-Z0-9]*' | 44 |
| FieldNamingConventions | The field name 'name' doesn't match '_[a-z][a-zA-Z0-9]*' | 46 |
| FieldNamingConventions | The field name '__equalsCalc' doesn't match '_[a-z][a-zA-Z0-9]*' | 98 |
| LocalVariableNamingConventions | The local variable name '_equals' doesn't match '[a-z][a-zA-Z0-9]*' | 108 |
| FieldNamingConventions | The field name '__hashCodeCalc' doesn't match '_[a-z][a-zA-Z0-9]*' | 120 |
| LocalVariableNamingConventions | The local variable name '_hashCode' doesn't match '[a-z][a-zA-Z0-9]*' | 126 |
| FieldNamingConventions | The static field name 'typeDesc' doesn't match '_[a-z][a-zA-Z0-9]*' | 138 |
| FormalParameterNamingConventions | The method parameter name '_javaType' doesn't match '[a-z][a-zA-Z0-9]*' | 169 |
| FormalParameterNamingConventions | The method parameter name '_xmlType' doesn't match '[a-z][a-zA-Z0-9]*' | 170 |
| FormalParameterNamingConventions | The method parameter name '_javaType' doesn't match '[a-z][a-zA-Z0-9]*' | 181 |
| FormalParameterNamingConventions | The method parameter name '_xmlType' doesn't match '[a-z][a-zA-Z0-9]*' | 182 |
| Rule | Violation | Line |
|---|---|---|
| FieldNamingConventions | The field name 'description' doesn't match '_[a-z][a-zA-Z0-9]*' | 44 |
| FieldNamingConventions | The field name 'groupId' doesn't match '_[a-z][a-zA-Z0-9]*' | 46 |
| FieldNamingConventions | The field name 'name' doesn't match '_[a-z][a-zA-Z0-9]*' | 48 |
| FieldNamingConventions | The field name '__equalsCalc' doesn't match '_[a-z][a-zA-Z0-9]*' | 122 |
| LocalVariableNamingConventions | The local variable name '_equals' doesn't match '[a-z][a-zA-Z0-9]*' | 132 |
| FieldNamingConventions | The field name '__hashCodeCalc' doesn't match '_[a-z][a-zA-Z0-9]*' | 147 |
| LocalVariableNamingConventions | The local variable name '_hashCode' doesn't match '[a-z][a-zA-Z0-9]*' | 153 |
| FieldNamingConventions | The static field name 'typeDesc' doesn't match '_[a-z][a-zA-Z0-9]*' | 168 |
| FormalParameterNamingConventions | The method parameter name '_javaType' doesn't match '[a-z][a-zA-Z0-9]*' | 205 |
| FormalParameterNamingConventions | The method parameter name '_xmlType' doesn't match '[a-z][a-zA-Z0-9]*' | 206 |
| FormalParameterNamingConventions | The method parameter name '_javaType' doesn't match '[a-z][a-zA-Z0-9]*' | 217 |
| FormalParameterNamingConventions | The method parameter name '_xmlType' doesn't match '[a-z][a-zA-Z0-9]*' | 218 |
| Rule | Violation | Line |
|---|---|---|
| FieldNamingConventions | The field name 'metaDataList' doesn't match '_[a-z][a-zA-Z0-9]*' | 44 |
| FieldNamingConventions | The field name 'userGroupList' doesn't match '_[a-z][a-zA-Z0-9]*' | 46 |
| FieldNamingConventions | The field name '__equalsCalc' doesn't match '_[a-z][a-zA-Z0-9]*' | 98 |
| LocalVariableNamingConventions | The local variable name '_equals' doesn't match '[a-z][a-zA-Z0-9]*' | 108 |
| FieldNamingConventions | The field name '__hashCodeCalc' doesn't match '_[a-z][a-zA-Z0-9]*' | 120 |
| LocalVariableNamingConventions | The local variable name '_hashCode' doesn't match '[a-z][a-zA-Z0-9]*' | 126 |
| FieldNamingConventions | The static field name 'typeDesc' doesn't match '_[a-z][a-zA-Z0-9]*' | 154 |
| FormalParameterNamingConventions | The method parameter name '_javaType' doesn't match '[a-z][a-zA-Z0-9]*' | 185 |
| FormalParameterNamingConventions | The method parameter name '_xmlType' doesn't match '[a-z][a-zA-Z0-9]*' | 186 |
| FormalParameterNamingConventions | The method parameter name '_javaType' doesn't match '[a-z][a-zA-Z0-9]*' | 197 |
| FormalParameterNamingConventions | The method parameter name '_xmlType' doesn't match '[a-z][a-zA-Z0-9]*' | 198 |
| Rule | Violation | Line |
|---|---|---|
| FieldNamingConventions | The field name 'email' doesn't match '_[a-z][a-zA-Z0-9]*' | 44 |
| FieldNamingConventions | The field name 'firstName' doesn't match '_[a-z][a-zA-Z0-9]*' | 46 |
| FieldNamingConventions | The field name 'lastName' doesn't match '_[a-z][a-zA-Z0-9]*' | 48 |
| FieldNamingConventions | The field name 'login' doesn't match '_[a-z][a-zA-Z0-9]*' | 50 |
| FieldNamingConventions | The field name 'userId' doesn't match '_[a-z][a-zA-Z0-9]*' | 52 |
| FieldNamingConventions | The field name '__equalsCalc' doesn't match '_[a-z][a-zA-Z0-9]*' | 170 |
| LocalVariableNamingConventions | The local variable name '_equals' doesn't match '[a-z][a-zA-Z0-9]*' | 180 |
| FieldNamingConventions | The field name '__hashCodeCalc' doesn't match '_[a-z][a-zA-Z0-9]*' | 201 |
| LocalVariableNamingConventions | The local variable name '_hashCode' doesn't match '[a-z][a-zA-Z0-9]*' | 207 |
| FieldNamingConventions | The static field name 'typeDesc' doesn't match '_[a-z][a-zA-Z0-9]*' | 228 |
| FormalParameterNamingConventions | The method parameter name '_javaType' doesn't match '[a-z][a-zA-Z0-9]*' | 277 |
| FormalParameterNamingConventions | The method parameter name '_xmlType' doesn't match '[a-z][a-zA-Z0-9]*' | 278 |
| FormalParameterNamingConventions | The method parameter name '_javaType' doesn't match '[a-z][a-zA-Z0-9]*' | 289 |
| FormalParameterNamingConventions | The method parameter name '_xmlType' doesn't match '[a-z][a-zA-Z0-9]*' | 290 |
| Rule | Violation | Line |
|---|---|---|
| FieldNamingConventions | The field name 'error' doesn't match '_[a-z][a-zA-Z0-9]*' | 44 |
| FieldNamingConventions | The field name '__equalsCalc' doesn't match '_[a-z][a-zA-Z0-9]*' | 74 |
| LocalVariableNamingConventions | The local variable name '_equals' doesn't match '[a-z][a-zA-Z0-9]*' | 84 |
| FieldNamingConventions | The field name '__hashCodeCalc' doesn't match '_[a-z][a-zA-Z0-9]*' | 93 |
| LocalVariableNamingConventions | The local variable name '_hashCode' doesn't match '[a-z][a-zA-Z0-9]*' | 99 |
| FieldNamingConventions | The static field name 'typeDesc' doesn't match '_[a-z][a-zA-Z0-9]*' | 116 |
| FormalParameterNamingConventions | The method parameter name '_javaType' doesn't match '[a-z][a-zA-Z0-9]*' | 141 |
| FormalParameterNamingConventions | The method parameter name '_xmlType' doesn't match '[a-z][a-zA-Z0-9]*' | 142 |
| FormalParameterNamingConventions | The method parameter name '_javaType' doesn't match '[a-z][a-zA-Z0-9]*' | 153 |
| FormalParameterNamingConventions | The method parameter name '_xmlType' doesn't match '[a-z][a-zA-Z0-9]*' | 154 |
| Rule | Violation | Line |
|---|---|---|
| FormalParameterNamingConventions | The method parameter name 'ParisVideo' doesn't match '[a-z][a-zA-Z0-9]*' | 51 |
| Rule | Violation | Line |
|---|---|---|
| FieldNamingConventions | The static field name 'uniqueCookie1' doesn't match '_[a-z][a-zA-Z0-9]*' | 45 |
| FieldNamingConventions | The static field name 'uniqueCookie2' doesn't match '_[a-z][a-zA-Z0-9]*' | 46 |
| Rule | Violation | Line |
|---|---|---|
| LocalVariableNamingConventions | The local variable name '_collec' doesn't match '[a-z][a-zA-Z0-9]*' | 99 |
| LocalVariableNamingConventions | The local variable name '_strAction' doesn't match '[a-z][a-zA-Z0-9]*' | 116 |
| LocalVariableNamingConventions | The local variable name '_strName' doesn't match '[a-z][a-zA-Z0-9]*' | 117 |
| LocalVariableNamingConventions | The local variable name '_strLogin' doesn't match '[a-z][a-zA-Z0-9]*' | 118 |
| LocalVariableNamingConventions | The local variable name '_strPassword' doesn't match '[a-z][a-zA-Z0-9]*' | 119 |
| Rule | Violation | Line |
|---|---|---|
| LocalVariableNamingConventions | The local variable name '_collec' doesn't match '[a-z][a-zA-Z0-9]*' | 122 |
| Rule | Violation | Line |
|---|---|---|
| IntegerInstantiation | Avoid instantiating Integer objects. Call Integer.valueOf() instead. | 295 |
| IntegerInstantiation | Avoid instantiating Integer objects. Call Integer.valueOf() instead. | 295 |
| IntegerInstantiation | Avoid instantiating Integer objects. Call Integer.valueOf() instead. | 334 |
| IntegerInstantiation | Avoid instantiating Integer objects. Call Integer.valueOf() instead. | 373 |
| Rule | Violation | Line |
|---|---|---|
| IntegerInstantiation | Avoid instantiating Integer objects. Call Integer.valueOf() instead. | 378 |
| IntegerInstantiation | Avoid instantiating Integer objects. Call Integer.valueOf() instead. | 417 |
| Rule | Violation | Line |
|---|---|---|
| MissingSerialVersionUID | Classes implementing Serializable should set a serialVersionUID | 43–224 |
| ShortVariable | Avoid variables with short names like id | 44 |
| CommentRequired | Public method and constructor comments are required | 50–51 |
| UncommentedEmptyConstructor | Document empty constructor | 50–51 |
| CommentRequired | Public method and constructor comments are required | 53–60 |
| ShortVariable | Avoid variables with short names like id | 54 |
| ShortVariable | Avoid variables with short names like id | 78 |
| RedundantFieldInitializer | Avoid using redundant field initializer for '__equalsCalc' | 122 |
| FieldDeclarationsShouldBeAtStartOfClass | Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. | 122 |
| CommentRequired | Public method and constructor comments are required | 123–145 |
| AvoidSynchronizedAtMethodLevel | Use block level rather than method level synchronization | 123–145 |
| MissingOverride | The method 'equals(java.lang.Object)' is missing an @Override annotation. | 123–145 |
| ControlStatementBraces | This statement should have braces | 124 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 125 |
| ControlStatementBraces | This statement should have braces | 126 |
| ControlStatementBraces | This statement should have braces | 127 |
| CompareObjectsWithEquals | Use equals() to compare object references. | 129 |
| NullAssignment | Assigning an Object to null is a code smell. Consider refactoring. | 143 |
| FieldDeclarationsShouldBeAtStartOfClass | Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. | 147 |
| RedundantFieldInitializer | Avoid using redundant field initializer for '__hashCodeCalc' | 147 |
| CommentRequired | Public method and constructor comments are required | 148–165 |
| AvoidSynchronizedAtMethodLevel | Use block level rather than method level synchronization | 148–165 |
| MissingOverride | The method 'hashCode()' is missing an @Override annotation. | 148–165 |
| FieldDeclarationsShouldBeAtStartOfClass | Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. | 168–169 |
| Rule | Violation | Line |
|---|---|---|
| UnnecessaryModifier | Unnecessary modifier 'public' on method 'getAsset': the method is declared in an interface type | 44 |
| CommentRequired | Public method and constructor comments are required | 44 |
| ShortVariable | Avoid variables with short names like id | 44 |
| UnnecessaryModifier | Unnecessary modifier 'public' on method 'getAssetsList': the method is declared in an interface type | 45 |
| CommentRequired | Public method and constructor comments are required | 45 |
| UnnecessaryModifier | Unnecessary modifier 'public' on method 'getAssetPlayer': the method is declared in an interface type | 46 |
| ShortVariable | Avoid variables with short names like id | 46 |
| CommentRequired | Public method and constructor comments are required | 46 |
| Rule | Violation | Line |
|---|---|---|
| MissingSerialVersionUID | Classes implementing Serializable should set a serialVersionUID | 43–126 |
| CallSuperInConstructor | It is a good practice to call super() in a constructor | 44–45 |
| CommentRequired | Public method and constructor comments are required | 44–45 |
| UncommentedEmptyConstructor | Document empty constructor | 44–45 |
| CommentRequired | Public method and constructor comments are required | 47–51 |
| RedundantFieldInitializer | Avoid using redundant field initializer for '__equalsCalc' | 53 |
| FieldDeclarationsShouldBeAtStartOfClass | Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. | 53 |
| CommentRequired | Public method and constructor comments are required | 54–67 |
| MissingOverride | The method 'equals(java.lang.Object)' is missing an @Override annotation. | 54–67 |
| AvoidSynchronizedAtMethodLevel | Use block level rather than method level synchronization | 54–67 |
| ControlStatementBraces | This statement should have braces | 55 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 56 |
| UnusedLocalVariable | Avoid unused local variables such as 'other'. | 56 |
| ControlStatementBraces | This statement should have braces | 57 |
| ControlStatementBraces | This statement should have braces | 58 |
| CompareObjectsWithEquals | Use equals() to compare object references. | 60 |
| NullAssignment | Assigning an Object to null is a code smell. Consider refactoring. | 65 |
| FieldDeclarationsShouldBeAtStartOfClass | Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. | 69 |
| RedundantFieldInitializer | Avoid using redundant field initializer for '__hashCodeCalc' | 69 |
| AvoidSynchronizedAtMethodLevel | Use block level rather than method level synchronization | 70–78 |
| MissingOverride | The method 'hashCode()' is missing an @Override annotation. | 70–78 |
| CommentRequired | Public method and constructor comments are required | 70–78 |
| FieldDeclarationsShouldBeAtStartOfClass | Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. | 81–82 |
| MissingOverride | The method 'writeDetails(javax.xml.namespace.QName, org.apache.axis.encoding.SerializationContext)' is missing an @Override annotation. | 123–125 |
| Rule | Violation | Line |
|---|---|---|
| UnnecessaryModifier | Unnecessary modifier 'public' on method 'getSearchAssetServiceAddress': the method is declared in an interface type | 44 |
| CommentRequired | Public method and constructor comments are required | 44 |
| CommentRequired | Public method and constructor comments are required | 46 |
| UnnecessaryModifier | Unnecessary modifier 'public' on method 'getSearchAssetService': the method is declared in an interface type | 46 |
| UnnecessaryModifier | Unnecessary modifier 'public' on method 'getSearchAssetService': the method is declared in an interface type | 48 |
| CommentRequired | Public method and constructor comments are required | 48 |
| Rule | Violation | Line |
|---|---|---|
| MissingSerialVersionUID | Classes implementing Serializable should set a serialVersionUID | 43–175 |
| CommentRequired | Public method and constructor comments are required | 45–46 |
| UncommentedEmptyConstructor | Document empty constructor | 45–46 |
| CallSuperInConstructor | It is a good practice to call super() in a constructor | 45–46 |
| CommentRequired | Public method and constructor comments are required | 49–51 |
| CommentRequired | Public method and constructor comments are required | 53–55 |
| FieldDeclarationsShouldBeAtStartOfClass | Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. | 58 |
| MissingOverride | The method 'getSearchAssetServiceAddress()' is missing an @Override annotation. | 60–62 |
| CommentRequired | Public method and constructor comments are required | 60–62 |
| FieldDeclarationsShouldBeAtStartOfClass | Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. | 65 |
| AvoidDuplicateLiterals | The String literal "SearchAssetService" appears 4 times in this file; the first occurrence is on line 65 | 65 |
| CommentRequired | Public method and constructor comments are required | 75–84 |
| MissingOverride | The method 'getSearchAssetService()' is missing an @Override annotation. | 75–84 |
| CommentRequired | Public method and constructor comments are required | 86–95 |
| MissingOverride | The method 'getSearchAssetService(java.net.URL)' is missing an @Override annotation. | 86–95 |
| CommentRequired | Public method and constructor comments are required | 97–99 |
| MissingOverride | The method 'getPort(Class)' is missing an @Override annotation. | 106–118 |
| MissingOverride | The method 'getPort(javax.xml.namespace.QName, Class)' is missing an @Override annotation. | 125–138 |
| CommentRequired | Public method and constructor comments are required | 140–142 |
| MissingOverride | The method 'getServiceName()' is missing an @Override annotation. | 140–142 |
| RedundantFieldInitializer | Avoid using redundant field initializer for 'ports' | 144 |
| LooseCoupling | Avoid using implementation types like 'java.util.HashSet'; use the interface instead | 144 |
| FieldDeclarationsShouldBeAtStartOfClass | Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. | 144 |
| MissingOverride | The method 'getPorts()' is missing an @Override annotation. | 146–152 |
| Rule | Violation | Line |
|---|---|---|
| ImmutableField | Private field 'cachedSerClasses' could be made final; it is only initialized in the declaration or constructor. | 44 |
| LooseCoupling | Avoid using implementation types like 'java.util.Vector'; use the interface instead | 44 |
| UseArrayListInsteadOfVector | Use ArrayList instead of Vector | 44 |
| LooseCoupling | Avoid using implementation types like 'java.util.Vector'; use the interface instead | 45 |
| UseArrayListInsteadOfVector | Use ArrayList instead of Vector | 45 |
| ImmutableField | Private field 'cachedSerQNames' could be made final; it is only initialized in the declaration or constructor. | 45 |
| ImmutableField | Private field 'cachedSerFactories' could be made final; it is only initialized in the declaration or constructor. | 46 |
| UseArrayListInsteadOfVector | Use ArrayList instead of Vector | 46 |
| LooseCoupling | Avoid using implementation types like 'java.util.Vector'; use the interface instead | 46 |
| ImmutableField | Private field 'cachedDeserFactories' could be made final; it is only initialized in the declaration or constructor. | 47 |
| LooseCoupling | Avoid using implementation types like 'java.util.Vector'; use the interface instead | 47 |
| UseArrayListInsteadOfVector | Use ArrayList instead of Vector | 47 |
| DefaultPackage | Use explicit scoping instead of the default package private level | 49 |
| CommentDefaultAccessModifier | To avoid mistakes add a comment at the beginning of the _operations field if you want a default access modifier | 49 |
| AvoidDuplicateLiterals | The String literal "int" appears 4 times in this file; the first occurrence is on line 61 | 61 |
| AvoidDuplicateLiterals | The String literal "http://www.w3.org/2001/XMLSchema" appears 9 times in this file; the first occurrence is on line 61 | 61 |
| AvoidDuplicateLiterals | The String literal "http://common.service.searchengine.server.tvnavig.com" appears 5 times in this file; the first occurrence is on line 65 | 65 |
| AvoidDuplicateLiterals | The String literal "http://searchengine.server.tvnavig.com" appears 15 times in this file; the first occurrence is on line 71 | 71 |
| AvoidDuplicateLiterals | The String literal "SearchAssetServiceException" appears 4 times in this file; the first occurrence is on line 73 | 73 |
| CommentRequired | Public method and constructor comments are required | 124–126 |
| CommentRequired | Public method and constructor comments are required | 128–131 |
| NcssCount | The constructor 'SearchAssetServiceSoapBindingStub(javax.xml.rpc.Service)' has a NCSS line count of 73. | 133–215 |
| CallSuperInConstructor | It is a good practice to call super() in a constructor | 133–215 |
| CommentRequired | Public method and constructor comments are required | 133–215 |
| UnusedLocalVariable | Avoid unused local variables such as 'enumsf'. | 145 |
| UnusedLocalVariable | Avoid unused local variables such as 'enumdf'. | 146 |
| UnusedLocalVariable | Avoid unused local variables such as 'arraysf'. | 147 |
| UnusedLocalVariable | Avoid unused local variables such as 'arraydf'. | 148 |
| UnusedLocalVariable | Avoid unused local variables such as 'simplesf'. | 149 |
| UnusedLocalVariable | Avoid unused local variables such as 'simpledf'. | 150 |
| UnusedLocalVariable | Avoid unused local variables such as 'simplelistsf'. | 151 |
| UnusedLocalVariable | Avoid unused local variables such as 'simplelistdf'. | 152 |
| NullAssignment | Assigning an Object to null is a code smell. Consider refactoring. | 179 |
| NullAssignment | Assigning an Object to null is a code smell. Consider refactoring. | 188 |
| NullAssignment | Assigning an Object to null is a code smell. Consider refactoring. | 197 |
| CyclomaticComplexity | The method 'createCall()' has a cyclomatic complexity of 14. | 217–280 |
| CommentRequired | Protected method constructor comments are required | 217–280 |
| NPathComplexity | The method 'createCall()' has an NPath complexity of 641 | 217–280 |
| ShortVariable | Avoid variables with short names like x | 257 |
| ShortVariable | Avoid variables with short names like sf | 259 |
| ShortVariable | Avoid variables with short names like df | 261 |
| ShortVariable | Avoid variables with short names like sf | 266 |
| ShortVariable | Avoid variables with short names like df | 268 |
| CommentRequired | Public method and constructor comments are required | 282–319 |
| ShortVariable | Avoid variables with short names like id | 282 |
| CyclomaticComplexity | The method 'getAsset(int, int)' has a cyclomatic complexity of 13. | 282–319 |
| MissingOverride | The method 'getAsset(int, int)' is missing an @Override annotation. | 282–319 |
| MissingOverride | The method 'getAssetsList(int, java.lang.String, com.tvnavig.server.asset.service.common.MetaValueResult, java.util.Calendar, java.util.Calendar, java.lang.String)' is missing an @Override annotation. | 321–358 |
| CommentRequired | Public method and constructor comments are required | 321–358 |
| CyclomaticComplexity | The method 'getAssetsList(int, java.lang.String, com.tvnavig.server.asset.service.common.MetaValueResult, java.util.Calendar, java.util.Calendar, java.lang.String)' has a cyclomatic complexity of 13. | 321–358 |
| MissingOverride | The method 'getAssetPlayer(int)' is missing an @Override annotation. | 360–397 |
| CommentRequired | Public method and constructor comments are required | 360–397 |
| CyclomaticComplexity | The method 'getAssetPlayer(int)' has a cyclomatic complexity of 13. | 360–397 |
| ShortVariable | Avoid variables with short names like id | 360 |
| Rule | Violation | Line |
|---|---|---|
| DataClass | The class 'AssetOGC' is suspected to be a Data Class (WOC=23.077%, NOPA=0, NOAM=18, WMC=45) | 45–461 |
| UncommentedEmptyConstructor | Document empty constructor | 67–68 |
| CommentRequired | Public method and constructor comments are required | 67–68 |
| CommentRequired | Public method and constructor comments are required | 70–89 |
| ArrayIsStoredDirectly | The user-supplied array 'formats' is stored directly. | 74 |
| ArrayIsStoredDirectly | The user-supplied array 'metaValues' is stored directly. | 75 |
| MethodReturnsInternalArray | Returning 'formats' may expose an internal array. | 157 |
| ArrayIsStoredDirectly | The user-supplied array 'formats' is stored directly. | 166 |
| MethodReturnsInternalArray | Returning 'metaValues' may expose an internal array. | 177 |
| ArrayIsStoredDirectly | The user-supplied array 'metaValues' is stored directly. | 186 |
| RedundantFieldInitializer | Avoid using redundant field initializer for '__equalsCalc' | 270 |
| FieldDeclarationsShouldBeAtStartOfClass | Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. | 270 |
| MissingOverride | The method 'equals(java.lang.Object)' is missing an @Override annotation. | 271–309 |
| AvoidSynchronizedAtMethodLevel | Use block level rather than method level synchronization | 271–309 |
| CommentRequired | Public method and constructor comments are required | 271–309 |
| ControlStatementBraces | This statement should have braces | 272 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 273 |
| ControlStatementBraces | This statement should have braces | 274 |
| ControlStatementBraces | This statement should have braces | 275 |
| CompareObjectsWithEquals | Use equals() to compare object references. | 277 |
| NullAssignment | Assigning an Object to null is a code smell. Consider refactoring. | 307 |
| RedundantFieldInitializer | Avoid using redundant field initializer for '__hashCodeCalc' | 311 |
| FieldDeclarationsShouldBeAtStartOfClass | Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. | 311 |
| AvoidSynchronizedAtMethodLevel | Use block level rather than method level synchronization | 312–361 |
| CyclomaticComplexity | The method 'hashCode()' has a cyclomatic complexity of 16. | 312–361 |
| MissingOverride | The method 'hashCode()' is missing an @Override annotation. | 312–361 |
| CommentRequired | Public method and constructor comments are required | 312–361 |
| NPathComplexity | The method 'hashCode()' has an NPath complexity of 3200 | 312–361 |
| FieldDeclarationsShouldBeAtStartOfClass | Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. | 364–365 |
| AvoidDuplicateLiterals | The String literal "http://www.w3.org/2001/XMLSchema" appears 8 times in this file; the first occurrence is on line 372 | 372 |
| AvoidDuplicateLiterals | The String literal "string" appears 6 times in this file; the first occurrence is on line 384 | 384 |
| CommentRequired | Public method and constructor comments are required | 456–459 |
| MissingOverride | The method 'compareTo(AssetOGC)' is missing an @Override annotation. | 456–459 |
| Rule | Violation | Line |
|---|---|---|
| MissingSerialVersionUID | Classes implementing Serializable should set a serialVersionUID | 43–188 |
| UncommentedEmptyConstructor | Document empty constructor | 48–49 |
| CommentRequired | Public method and constructor comments are required | 48–49 |
| CommentRequired | Public method and constructor comments are required | 51–56 |
| RedundantFieldInitializer | Avoid using redundant field initializer for '__equalsCalc' | 98 |
| FieldDeclarationsShouldBeAtStartOfClass | Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. | 98 |
| CommentRequired | Public method and constructor comments are required | 99–118 |
| MissingOverride | The method 'equals(java.lang.Object)' is missing an @Override annotation. | 99–118 |
| AvoidSynchronizedAtMethodLevel | Use block level rather than method level synchronization | 99–118 |
| ControlStatementBraces | This statement should have braces | 100 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 101 |
| ControlStatementBraces | This statement should have braces | 102 |
| ControlStatementBraces | This statement should have braces | 103 |
| CompareObjectsWithEquals | Use equals() to compare object references. | 105 |
| NullAssignment | Assigning an Object to null is a code smell. Consider refactoring. | 116 |
| RedundantFieldInitializer | Avoid using redundant field initializer for '__hashCodeCalc' | 120 |
| FieldDeclarationsShouldBeAtStartOfClass | Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. | 120 |
| AvoidSynchronizedAtMethodLevel | Use block level rather than method level synchronization | 121–135 |
| CommentRequired | Public method and constructor comments are required | 121–135 |
| MissingOverride | The method 'hashCode()' is missing an @Override annotation. | 121–135 |
| FieldDeclarationsShouldBeAtStartOfClass | Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. | 138–139 |
| Rule | Violation | Line |
|---|---|---|
| UnnecessaryModifier | Unnecessary modifier 'public' on method 'isAlive': the method is declared in an interface type | 44 |
| CommentRequired | Public method and constructor comments are required | 44 |
| UnnecessaryModifier | Unnecessary modifier 'public' on method 'authenticate': the method is declared in an interface type | 45 |
| CommentRequired | Public method and constructor comments are required | 45 |
| CommentRequired | Public method and constructor comments are required | 46 |
| UnnecessaryModifier | Unnecessary modifier 'public' on method 'getDomainsFromUser': the method is declared in an interface type | 46 |
| CommentRequired | Public method and constructor comments are required | 47 |
| UnnecessaryModifier | Unnecessary modifier 'public' on method 'getGroupsandMetaDatasFromDomain': the method is declared in an interface type | 47 |
| Rule | Violation | Line |
|---|---|---|
| MissingSerialVersionUID | Classes implementing Serializable should set a serialVersionUID | 43–126 |
| CallSuperInConstructor | It is a good practice to call super() in a constructor | 44–45 |
| CommentRequired | Public method and constructor comments are required | 44–45 |
| UncommentedEmptyConstructor | Document empty constructor | 44–45 |
| CommentRequired | Public method and constructor comments are required | 47–51 |
| FieldDeclarationsShouldBeAtStartOfClass | Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. | 53 |
| RedundantFieldInitializer | Avoid using redundant field initializer for '__equalsCalc' | 53 |
| AvoidSynchronizedAtMethodLevel | Use block level rather than method level synchronization | 54–67 |
| CommentRequired | Public method and constructor comments are required | 54–67 |
| MissingOverride | The method 'equals(java.lang.Object)' is missing an @Override annotation. | 54–67 |
| ControlStatementBraces | This statement should have braces | 55 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 56 |
| UnusedLocalVariable | Avoid unused local variables such as 'other'. | 56 |
| ControlStatementBraces | This statement should have braces | 57 |
| ControlStatementBraces | This statement should have braces | 58 |
| CompareObjectsWithEquals | Use equals() to compare object references. | 60 |
| NullAssignment | Assigning an Object to null is a code smell. Consider refactoring. | 65 |
| RedundantFieldInitializer | Avoid using redundant field initializer for '__hashCodeCalc' | 69 |
| FieldDeclarationsShouldBeAtStartOfClass | Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. | 69 |
| MissingOverride | The method 'hashCode()' is missing an @Override annotation. | 70–78 |
| CommentRequired | Public method and constructor comments are required | 70–78 |
| AvoidSynchronizedAtMethodLevel | Use block level rather than method level synchronization | 70–78 |
| FieldDeclarationsShouldBeAtStartOfClass | Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. | 81–82 |
| MissingOverride | The method 'writeDetails(javax.xml.namespace.QName, org.apache.axis.encoding.SerializationContext)' is missing an @Override annotation. | 123–125 |
| Rule | Violation | Line |
|---|---|---|
| CommentRequired | Public method and constructor comments are required | 44 |
| UnnecessaryModifier | Unnecessary modifier 'public' on method 'getUserServiceAddress': the method is declared in an interface type | 44 |
| CommentRequired | Public method and constructor comments are required | 46 |
| UnnecessaryModifier | Unnecessary modifier 'public' on method 'getUserService': the method is declared in an interface type | 46 |
| CommentRequired | Public method and constructor comments are required | 48 |
| UnnecessaryModifier | Unnecessary modifier 'public' on method 'getUserService': the method is declared in an interface type | 48 |
| Rule | Violation | Line |
|---|---|---|
| MissingSerialVersionUID | Classes implementing Serializable should set a serialVersionUID | 43–175 |
| CommentRequired | Public method and constructor comments are required | 45–46 |
| UncommentedEmptyConstructor | Document empty constructor | 45–46 |
| CallSuperInConstructor | It is a good practice to call super() in a constructor | 45–46 |
| CommentRequired | Public method and constructor comments are required | 49–51 |
| CommentRequired | Public method and constructor comments are required | 53–55 |
| FieldDeclarationsShouldBeAtStartOfClass | Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. | 58 |
| MissingOverride | The method 'getUserServiceAddress()' is missing an @Override annotation. | 60–62 |
| CommentRequired | Public method and constructor comments are required | 60–62 |
| FieldDeclarationsShouldBeAtStartOfClass | Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. | 65 |
| AvoidDuplicateLiterals | The String literal "UserService" appears 4 times in this file; the first occurrence is on line 65 | 65 |
| CommentRequired | Public method and constructor comments are required | 75–84 |
| MissingOverride | The method 'getUserService()' is missing an @Override annotation. | 75–84 |
| CommentRequired | Public method and constructor comments are required | 86–95 |
| MissingOverride | The method 'getUserService(java.net.URL)' is missing an @Override annotation. | 86–95 |
| CommentRequired | Public method and constructor comments are required | 97–99 |
| MissingOverride | The method 'getPort(Class)' is missing an @Override annotation. | 106–118 |
| MissingOverride | The method 'getPort(javax.xml.namespace.QName, Class)' is missing an @Override annotation. | 125–138 |
| CommentRequired | Public method and constructor comments are required | 140–142 |
| MissingOverride | The method 'getServiceName()' is missing an @Override annotation. | 140–142 |
| FieldDeclarationsShouldBeAtStartOfClass | Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. | 144 |
| RedundantFieldInitializer | Avoid using redundant field initializer for 'ports' | 144 |
| LooseCoupling | Avoid using implementation types like 'java.util.HashSet'; use the interface instead | 144 |
| MissingOverride | The method 'getPorts()' is missing an @Override annotation. | 146–152 |
| Rule | Violation | Line |
|---|---|---|
| UseArrayListInsteadOfVector | Use ArrayList instead of Vector | 44 |
| ImmutableField | Private field 'cachedSerClasses' could be made final; it is only initialized in the declaration or constructor. | 44 |
| LooseCoupling | Avoid using implementation types like 'java.util.Vector'; use the interface instead | 44 |
| UseArrayListInsteadOfVector | Use ArrayList instead of Vector | 45 |
| ImmutableField | Private field 'cachedSerQNames' could be made final; it is only initialized in the declaration or constructor. | 45 |
| LooseCoupling | Avoid using implementation types like 'java.util.Vector'; use the interface instead | 45 |
| UseArrayListInsteadOfVector | Use ArrayList instead of Vector | 46 |
| ImmutableField | Private field 'cachedSerFactories' could be made final; it is only initialized in the declaration or constructor. | 46 |
| LooseCoupling | Avoid using implementation types like 'java.util.Vector'; use the interface instead | 46 |
| ImmutableField | Private field 'cachedDeserFactories' could be made final; it is only initialized in the declaration or constructor. | 47 |
| UseArrayListInsteadOfVector | Use ArrayList instead of Vector | 47 |
| LooseCoupling | Avoid using implementation types like 'java.util.Vector'; use the interface instead | 47 |
| CommentDefaultAccessModifier | To avoid mistakes add a comment at the beginning of the _operations field if you want a default access modifier | 49 |
| DefaultPackage | Use explicit scoping instead of the default package private level | 49 |
| AvoidDuplicateLiterals | The String literal "http://www.w3.org/2001/XMLSchema" appears 5 times in this file; the first occurrence is on line 68 | 68 |
| AvoidDuplicateLiterals | The String literal "http://common.service.user.server.tvnavig.com" appears 10 times in this file; the first occurrence is on line 72 | 72 |
| AvoidDuplicateLiterals | The String literal "http://user.server.tvnavig.com" appears 16 times in this file; the first occurrence is on line 78 | 78 |
| AvoidDuplicateLiterals | The String literal "UserServiceException" appears 4 times in this file; the first occurrence is on line 80 | 80 |
| CommentRequired | Public method and constructor comments are required | 121–123 |
| CommentRequired | Public method and constructor comments are required | 125–128 |
| CallSuperInConstructor | It is a good practice to call super() in a constructor | 130–235 |
| CommentRequired | Public method and constructor comments are required | 130–235 |
| ExcessiveMethodLength | Avoid really long methods. | 130–235 |
| NcssCount | The constructor 'UserServiceSoapBindingStub(javax.xml.rpc.Service)' has a NCSS line count of 93. | 130–235 |
| UnusedLocalVariable | Avoid unused local variables such as 'enumsf'. | 142 |
| UnusedLocalVariable | Avoid unused local variables such as 'enumdf'. | 143 |
| UnusedLocalVariable | Avoid unused local variables such as 'arraysf'. | 144 |
| UnusedLocalVariable | Avoid unused local variables such as 'arraydf'. | 145 |
| UnusedLocalVariable | Avoid unused local variables such as 'simplesf'. | 146 |
| UnusedLocalVariable | Avoid unused local variables such as 'simpledf'. | 147 |
| UnusedLocalVariable | Avoid unused local variables such as 'simplelistsf'. | 148 |
| UnusedLocalVariable | Avoid unused local variables such as 'simplelistdf'. | 149 |
| NullAssignment | Assigning an Object to null is a code smell. Consider refactoring. | 190 |
| NullAssignment | Assigning an Object to null is a code smell. Consider refactoring. | 199 |
| NullAssignment | Assigning an Object to null is a code smell. Consider refactoring. | 208 |
| NullAssignment | Assigning an Object to null is a code smell. Consider refactoring. | 217 |
| CommentRequired | Protected method constructor comments are required | 237–300 |
| CyclomaticComplexity | The method 'createCall()' has a cyclomatic complexity of 14. | 237–300 |
| NPathComplexity | The method 'createCall()' has an NPath complexity of 641 | 237–300 |
| ShortVariable | Avoid variables with short names like x | 277 |
| ShortVariable | Avoid variables with short names like sf | 279 |
| ShortVariable | Avoid variables with short names like df | 281 |
| ShortVariable | Avoid variables with short names like sf | 286 |
| ShortVariable | Avoid variables with short names like df | 288 |
| CommentRequired | Public method and constructor comments are required | 302–324 |
| MissingOverride | The method 'isAlive()' is missing an @Override annotation. | 302–324 |
| AvoidRethrowingException | A catch statement that catches an exception only to rethrow it should be avoided. | 321–323 |
| CommentRequired | Public method and constructor comments are required | 326–363 |
| CyclomaticComplexity | The method 'authenticate(java.lang.String, java.lang.String)' has a cyclomatic complexity of 13. | 326–363 |
| MissingOverride | The method 'authenticate(java.lang.String, java.lang.String)' is missing an @Override annotation. | 326–363 |
| CommentRequired | Public method and constructor comments are required | 365–402 |
| CyclomaticComplexity | The method 'getDomainsFromUser(int)' has a cyclomatic complexity of 13. | 365–402 |
| MissingOverride | The method 'getDomainsFromUser(int)' is missing an @Override annotation. | 365–402 |
| CyclomaticComplexity | The method 'getGroupsandMetaDatasFromDomain(int)' has a cyclomatic complexity of 13. | 404–441 |
| CommentRequired | Public method and constructor comments are required | 404–441 |
| MissingOverride | The method 'getGroupsandMetaDatasFromDomain(int)' is missing an @Override annotation. | 404–441 |
| Rule | Violation | Line |
|---|---|---|
| MissingSerialVersionUID | Classes implementing Serializable should set a serialVersionUID | 43–260 |
| CommentRequired | Public method and constructor comments are required | 52–53 |
| UncommentedEmptyConstructor | Document empty constructor | 52–53 |
| CommentRequired | Public method and constructor comments are required | 55–64 |
| ArrayIsStoredDirectly | The user-supplied array 'valueList' is stored directly. | 59 |
| MethodReturnsInternalArray | Returning 'valueList' may expose an internal array. | 133 |
| ArrayIsStoredDirectly | The user-supplied array 'valueList' is stored directly. | 142 |
| RedundantFieldInitializer | Avoid using redundant field initializer for '__equalsCalc' | 146 |
| FieldDeclarationsShouldBeAtStartOfClass | Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. | 146 |
| AvoidSynchronizedAtMethodLevel | Use block level rather than method level synchronization | 147–168 |
| MissingOverride | The method 'equals(java.lang.Object)' is missing an @Override annotation. | 147–168 |
| CommentRequired | Public method and constructor comments are required | 147–168 |
| ControlStatementBraces | This statement should have braces | 148 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 149 |
| ControlStatementBraces | This statement should have braces | 150 |
| ControlStatementBraces | This statement should have braces | 151 |
| CompareObjectsWithEquals | Use equals() to compare object references. | 153 |
| NullAssignment | Assigning an Object to null is a code smell. Consider refactoring. | 166 |
| RedundantFieldInitializer | Avoid using redundant field initializer for '__hashCodeCalc' | 170 |
| FieldDeclarationsShouldBeAtStartOfClass | Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. | 170 |
| MissingOverride | The method 'hashCode()' is missing an @Override annotation. | 171–195 |
| AvoidSynchronizedAtMethodLevel | Use block level rather than method level synchronization | 171–195 |
| CommentRequired | Public method and constructor comments are required | 171–195 |
| FieldDeclarationsShouldBeAtStartOfClass | Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. | 198–199 |
| AvoidDuplicateLiterals | The String literal "http://www.w3.org/2001/XMLSchema" appears 4 times in this file; the first occurrence is on line 206 | 206 |
| Rule | Violation | Line |
|---|---|---|
| MissingSerialVersionUID | Classes implementing Serializable should set a serialVersionUID | 43–188 |
| ShortVariable | Avoid variables with short names like id | 44 |
| CommentRequired | Public method and constructor comments are required | 48–49 |
| UncommentedEmptyConstructor | Document empty constructor | 48–49 |
| CommentRequired | Public method and constructor comments are required | 51–56 |
| ShortVariable | Avoid variables with short names like id | 52 |
| ShortVariable | Avoid variables with short names like id | 74 |
| RedundantFieldInitializer | Avoid using redundant field initializer for '__equalsCalc' | 98 |
| FieldDeclarationsShouldBeAtStartOfClass | Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. | 98 |
| CommentRequired | Public method and constructor comments are required | 99–118 |
| MissingOverride | The method 'equals(java.lang.Object)' is missing an @Override annotation. | 99–118 |
| AvoidSynchronizedAtMethodLevel | Use block level rather than method level synchronization | 99–118 |
| ControlStatementBraces | This statement should have braces | 100 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 101 |
| ControlStatementBraces | This statement should have braces | 102 |
| ControlStatementBraces | This statement should have braces | 103 |
| CompareObjectsWithEquals | Use equals() to compare object references. | 105 |
| NullAssignment | Assigning an Object to null is a code smell. Consider refactoring. | 116 |
| RedundantFieldInitializer | Avoid using redundant field initializer for '__hashCodeCalc' | 120 |
| FieldDeclarationsShouldBeAtStartOfClass | Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. | 120 |
| AvoidSynchronizedAtMethodLevel | Use block level rather than method level synchronization | 121–135 |
| MissingOverride | The method 'hashCode()' is missing an @Override annotation. | 121–135 |
| CommentRequired | Public method and constructor comments are required | 121–135 |
| FieldDeclarationsShouldBeAtStartOfClass | Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. | 138–139 |
| Rule | Violation | Line |
|---|---|---|
| MissingSerialVersionUID | Classes implementing Serializable should set a serialVersionUID | 43–224 |
| CommentRequired | Public method and constructor comments are required | 50–51 |
| UncommentedEmptyConstructor | Document empty constructor | 50–51 |
| CommentRequired | Public method and constructor comments are required | 53–60 |
| RedundantFieldInitializer | Avoid using redundant field initializer for '__equalsCalc' | 122 |
| FieldDeclarationsShouldBeAtStartOfClass | Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. | 122 |
| CommentRequired | Public method and constructor comments are required | 123–145 |
| MissingOverride | The method 'equals(java.lang.Object)' is missing an @Override annotation. | 123–145 |
| AvoidSynchronizedAtMethodLevel | Use block level rather than method level synchronization | 123–145 |
| ControlStatementBraces | This statement should have braces | 124 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 125 |
| ControlStatementBraces | This statement should have braces | 126 |
| ControlStatementBraces | This statement should have braces | 127 |
| CompareObjectsWithEquals | Use equals() to compare object references. | 129 |
| NullAssignment | Assigning an Object to null is a code smell. Consider refactoring. | 143 |
| FieldDeclarationsShouldBeAtStartOfClass | Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. | 147 |
| RedundantFieldInitializer | Avoid using redundant field initializer for '__hashCodeCalc' | 147 |
| AvoidSynchronizedAtMethodLevel | Use block level rather than method level synchronization | 148–165 |
| CommentRequired | Public method and constructor comments are required | 148–165 |
| MissingOverride | The method 'hashCode()' is missing an @Override annotation. | 148–165 |
| FieldDeclarationsShouldBeAtStartOfClass | Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. | 168–169 |
| Rule | Violation | Line |
|---|---|---|
| MissingSerialVersionUID | Classes implementing Serializable should set a serialVersionUID | 43–204 |
| UncommentedEmptyConstructor | Document empty constructor | 48–49 |
| CommentRequired | Public method and constructor comments are required | 48–49 |
| CommentRequired | Public method and constructor comments are required | 51–56 |
| ArrayIsStoredDirectly | The user-supplied array 'metaDataList' is stored directly. | 52 |
| ArrayIsStoredDirectly | The user-supplied array 'userGroupList' is stored directly. | 53 |
| MethodReturnsInternalArray | Returning 'metaDataList' may expose an internal array. | 65 |
| ArrayIsStoredDirectly | The user-supplied array 'metaDataList' is stored directly. | 74 |
| MethodReturnsInternalArray | Returning 'userGroupList' may expose an internal array. | 85 |
| ArrayIsStoredDirectly | The user-supplied array 'userGroupList' is stored directly. | 94 |
| RedundantFieldInitializer | Avoid using redundant field initializer for '__equalsCalc' | 98 |
| FieldDeclarationsShouldBeAtStartOfClass | Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. | 98 |
| AvoidSynchronizedAtMethodLevel | Use block level rather than method level synchronization | 99–118 |
| MissingOverride | The method 'equals(java.lang.Object)' is missing an @Override annotation. | 99–118 |
| CommentRequired | Public method and constructor comments are required | 99–118 |
| ControlStatementBraces | This statement should have braces | 100 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 101 |
| ControlStatementBraces | This statement should have braces | 102 |
| ControlStatementBraces | This statement should have braces | 103 |
| CompareObjectsWithEquals | Use equals() to compare object references. | 105 |
| NullAssignment | Assigning an Object to null is a code smell. Consider refactoring. | 116 |
| RedundantFieldInitializer | Avoid using redundant field initializer for '__hashCodeCalc' | 120 |
| FieldDeclarationsShouldBeAtStartOfClass | Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. | 120 |
| CommentRequired | Public method and constructor comments are required | 121–151 |
| MissingOverride | The method 'hashCode()' is missing an @Override annotation. | 121–151 |
| AvoidSynchronizedAtMethodLevel | Use block level rather than method level synchronization | 121–151 |
| CyclomaticComplexity | The method 'hashCode()' has a cyclomatic complexity of 10. | 121–151 |
| FieldDeclarationsShouldBeAtStartOfClass | Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. | 154–155 |
| Rule | Violation | Line |
|---|---|---|
| DataClass | The class 'UserServiceResult' is suspected to be a Data Class (WOC=29.412%, NOPA=0, NOAM=10, WMC=27) | 43–296 |
| MissingSerialVersionUID | Classes implementing Serializable should set a serialVersionUID | 43–296 |
| CommentRequired | Public method and constructor comments are required | 54–55 |
| UncommentedEmptyConstructor | Document empty constructor | 54–55 |
| CommentRequired | Public method and constructor comments are required | 57–68 |
| FieldDeclarationsShouldBeAtStartOfClass | Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. | 170 |
| RedundantFieldInitializer | Avoid using redundant field initializer for '__equalsCalc' | 170 |
| AvoidSynchronizedAtMethodLevel | Use block level rather than method level synchronization | 171–199 |
| MissingOverride | The method 'equals(java.lang.Object)' is missing an @Override annotation. | 171–199 |
| CommentRequired | Public method and constructor comments are required | 171–199 |
| ControlStatementBraces | This statement should have braces | 172 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 173 |
| ControlStatementBraces | This statement should have braces | 174 |
| ControlStatementBraces | This statement should have braces | 175 |
| CompareObjectsWithEquals | Use equals() to compare object references. | 177 |
| NullAssignment | Assigning an Object to null is a code smell. Consider refactoring. | 197 |
| RedundantFieldInitializer | Avoid using redundant field initializer for '__hashCodeCalc' | 201 |
| FieldDeclarationsShouldBeAtStartOfClass | Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. | 201 |
| MissingOverride | The method 'hashCode()' is missing an @Override annotation. | 202–225 |
| CommentRequired | Public method and constructor comments are required | 202–225 |
| AvoidSynchronizedAtMethodLevel | Use block level rather than method level synchronization | 202–225 |
| FieldDeclarationsShouldBeAtStartOfClass | Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. | 228–229 |
| AvoidDuplicateLiterals | The String literal "string" appears 4 times in this file; the first occurrence is on line 236 | 236 |
| AvoidDuplicateLiterals | The String literal "http://www.w3.org/2001/XMLSchema" appears 5 times in this file; the first occurrence is on line 236 | 236 |
| Rule | Violation | Line |
|---|---|---|
| MissingSerialVersionUID | Classes implementing Serializable should set a serialVersionUID | 43–167 |
| UncommentedEmptyConstructor | Document empty constructor | 46–47 |
| CallSuperInConstructor | It is a good practice to call super() in a constructor | 46–47 |
| CommentRequired | Public method and constructor comments are required | 46–47 |
| CommentRequired | Public method and constructor comments are required | 49–52 |
| CallSuperInConstructor | It is a good practice to call super() in a constructor | 49–52 |
| ArrayIsStoredDirectly | The user-supplied array 'error' is stored directly. | 50 |
| MethodReturnsInternalArray | Returning 'error' may expose an internal array. | 61 |
| ArrayIsStoredDirectly | The user-supplied array 'error' is stored directly. | 70 |
| FieldDeclarationsShouldBeAtStartOfClass | Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. | 74 |
| RedundantFieldInitializer | Avoid using redundant field initializer for '__equalsCalc' | 74 |
| AvoidSynchronizedAtMethodLevel | Use block level rather than method level synchronization | 75–91 |
| CommentRequired | Public method and constructor comments are required | 75–91 |
| MissingOverride | The method 'equals(java.lang.Object)' is missing an @Override annotation. | 75–91 |
| ControlStatementBraces | This statement should have braces | 76 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 77 |
| ControlStatementBraces | This statement should have braces | 78 |
| ControlStatementBraces | This statement should have braces | 79 |
| CompareObjectsWithEquals | Use equals() to compare object references. | 81 |
| NullAssignment | Assigning an Object to null is a code smell. Consider refactoring. | 89 |
| RedundantFieldInitializer | Avoid using redundant field initializer for '__hashCodeCalc' | 93 |
| FieldDeclarationsShouldBeAtStartOfClass | Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. | 93 |
| MissingOverride | The method 'hashCode()' is missing an @Override annotation. | 94–113 |
| AvoidSynchronizedAtMethodLevel | Use block level rather than method level synchronization | 94–113 |
| CommentRequired | Public method and constructor comments are required | 94–113 |
| FieldDeclarationsShouldBeAtStartOfClass | Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. | 116–117 |
| MissingOverride | The method 'writeDetails(javax.xml.namespace.QName, org.apache.axis.encoding.SerializationContext)' is missing an @Override annotation. | 164–166 |
| Rule | Violation | Line |
|---|---|---|
| CommentRequired | Public method and constructor comments are required | 53 |
| Rule | Violation | Line |
|---|---|---|
| CommentRequired | Public method and constructor comments are required | 49–52 |
| ShortVariable | Avoid variables with short names like id | 54 |
| CommentRequired | Public method and constructor comments are required | 54–57 |
| CommentRequired | Public method and constructor comments are required | 59–65 |
| ControlStatementBraces | This statement should have braces | 62 |
| ControlStatementBraces | This statement should have braces | 64 |
| CommentRequired | Public method and constructor comments are required | 67–70 |
| CommentRequired | Public method and constructor comments are required | 72–78 |
| ControlStatementBraces | This statement should have braces | 75 |
| ControlStatementBraces | This statement should have braces | 77 |
| CommentRequired | Public method and constructor comments are required | 80–83 |
| CommentRequired | Public method and constructor comments are required | 85–91 |
| ControlStatementBraces | This statement should have braces | 88 |
| ControlStatementBraces | This statement should have braces | 90 |
| CommentRequired | Public method and constructor comments are required | 93–96 |
| Rule | Violation | Line |
|---|---|---|
| MissingOverride | The method 'insert(ParisVideoAccount, Plugin)' is missing an @Override annotation. | 86–98 |
| ShortVariable | Avoid variables with short names like id | 89 |
| MissingOverride | The method 'selectAll(Plugin)' is missing an @Override annotation. | 106–126 |
| UseDiamondOperator | Explicit type arguments can be replaced by Diamond Operator | 108 |
| MissingOverride | The method 'deleteAll(Plugin)' is missing an @Override annotation. | 128–133 |
| CommentRequired | Public method and constructor comments are required | 128–133 |
| Rule | Violation | Line |
|---|---|---|
| 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. | 45–65 |
| CommentRequired | Public method and constructor comments are required | 49–54 |
| Rule | Violation | Line |
|---|---|---|
| SignatureDeclareThrowsException | A method/constructor should not explicitly throw java.lang.Exception | 70 |
| UnnecessaryLocalBeforeReturn | Consider simply returning the value vs storing it in local variable 'udrArray' | 80 |
| SignatureDeclareThrowsException | A method/constructor should not explicitly throw java.lang.Exception | 95 |
| ShortVariable | Avoid variables with short names like as | 97 |
| SignatureDeclareThrowsException | A method/constructor should not explicitly throw java.lang.Exception | 110 |
| ShortVariable | Avoid variables with short names like as | 112 |
| ShortVariable | Avoid variables with short names like ap | 116 |
| UnnecessaryLocalBeforeReturn | Consider simply returning the value vs storing it in local variable 'ap' | 117 |
| Rule | Violation | Line |
|---|---|---|
| CommentRequired | Header comments are required | 43–67 |
| RedundantFieldInitializer | Avoid using redundant field initializer for 'uniqueCookie1' | 45 |
| RedundantFieldInitializer | Avoid using redundant field initializer for 'uniqueCookie2' | 46 |
| ShortVariable | Avoid variables with short names like mc | 49 |
| ControlStatementBraces | This statement should have braces | 53 |
| ControlStatementBraces | This statement should have braces | 55 |
| Rule | Violation | Line |
|---|---|---|
| UseConcurrentHashMap | If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation | 87 |
| UseDiamondOperator | Explicit type arguments can be replaced by Diamond Operator | 87 |
| UseCollectionIsEmpty | Substitute calls to size() == 0 (or size() != 0, size() > 0, size() < 1) with calls to isEmpty() | 101 |
| Rule | Violation | Line |
|---|---|---|
| UnusedPrivateField | Avoid unused private fields such as 'TEMPLATE_AUT_1'. | 86 |
| ImmutableField | Private field '_nDefaultItemsPerPage' could be made final; it is only initialized in the declaration or constructor. | 94 |
| RedundantFieldInitializer | Avoid using redundant field initializer for '_plugin' | 100 |
| MissingOverride | The method 'getInsertServiceSelectorUI(HttpServletRequest)' is missing an @Override annotation. | 113–153 |
| UseDiamondOperator | Explicit type arguments can be replaced by Diamond Operator | 117 |
| UseConcurrentHashMap | If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation | 117 |
| UseCollectionIsEmpty | Substitute calls to size() == 0 (or size() != 0, size() > 0, size() < 1) with calls to isEmpty() | 124 |
| UseDiamondOperator | Explicit type arguments can be replaced by Diamond Operator | 136 |
| EmptyCatchBlock | Avoid empty catch blocks | 143–145 |
| AvoidCatchingGenericException | Avoid catching generic exceptions such as NullPointerException, RuntimeException, Exception in try-catch block | 143 |
| SignatureDeclareThrowsException | A method/constructor should not explicitly throw java.lang.Exception | 158 |
| UseDiamondOperator | Explicit type arguments can be replaced by Diamond Operator | 160 |
| UseConcurrentHashMap | If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation | 160 |
| NullAssignment | Assigning an Object to null is a code smell. Consider refactoring. | 180 |
| ControlStatementBraces | This statement should have braces | 180 |
| AddEmptyString | Do not add empty strings | 191 |
| SignatureDeclareThrowsException | A method/constructor should not explicitly throw java.lang.Exception | 227 |
| UseConcurrentHashMap | If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation | 229 |
| UseDiamondOperator | Explicit type arguments can be replaced by Diamond Operator | 229 |
| ControlStatementBraces | This statement should have braces | 232 |
| ShortVariable | Avoid variables with short names like ap | 241 |
| SignatureDeclareThrowsException | A method/constructor should not explicitly throw java.lang.Exception | 251 |
| ShortVariable | Avoid variables with short names like ap | 257 |
| UseConcurrentHashMap | If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation | 259 |
| UseDiamondOperator | Explicit type arguments can be replaced by Diamond Operator | 259 |
| Rule | Violation | Line |
|---|---|---|
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.Integer' due to existing implicit import 'java.lang.*' | 44 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.Integer' due to existing implicit import 'java.lang.*' | 46 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.String' due to existing implicit import 'java.lang.*' | 48 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.Integer' due to existing implicit import 'java.lang.*' | 54 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.Integer' due to existing implicit import 'java.lang.*' | 55 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.String' due to existing implicit import 'java.lang.*' | 56 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.Integer' due to existing implicit import 'java.lang.*' | 68 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.Integer' due to existing implicit import 'java.lang.*' | 78 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.Integer' due to existing implicit import 'java.lang.*' | 88 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.Integer' due to existing implicit import 'java.lang.*' | 98 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.String' due to existing implicit import 'java.lang.*' | 108 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.String' due to existing implicit import 'java.lang.*' | 118 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.Object' due to existing implicit import 'java.lang.*' | 122 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.Object' due to existing implicit import 'java.lang.*' | 123 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.String' due to existing implicit import 'java.lang.*' | 204 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.Class' due to existing implicit import 'java.lang.*' | 205 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.String' due to existing implicit import 'java.lang.*' | 216 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.Class' due to existing implicit import 'java.lang.*' | 217 |
| Rule | Violation | Line |
|---|---|---|
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'com.tvnavig.server.searchengine.SearchAssetServiceException' due to existing same package import 'com.tvnavig.server.searchengine.*' | 44 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.String' due to existing implicit import 'java.lang.*' | 45 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.String' due to existing implicit import 'java.lang.*' | 45 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'com.tvnavig.server.searchengine.SearchAssetServiceException' due to existing same package import 'com.tvnavig.server.searchengine.*' | 45 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'com.tvnavig.server.searchengine.SearchAssetServiceException' due to existing same package import 'com.tvnavig.server.searchengine.*' | 46 |
| Rule | Violation | Line |
|---|---|---|
| UseVarargs | Consider using varargs for methods or constructors which take an array the last parameter. | 48 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.String' due to existing implicit import 'java.lang.*' | 48 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.Object' due to existing implicit import 'java.lang.*' | 53 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.Object' due to existing implicit import 'java.lang.*' | 54 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.String' due to existing implicit import 'java.lang.*' | 99 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.Class' due to existing implicit import 'java.lang.*' | 100 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.String' due to existing implicit import 'java.lang.*' | 111 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.Class' due to existing implicit import 'java.lang.*' | 112 |
| Rule | Violation | Line |
|---|---|---|
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.String' due to existing implicit import 'java.lang.*' | 44 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'com.tvnavig.server.searchengine.SearchAssetService' due to existing same package import 'com.tvnavig.server.searchengine.*' | 46 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'com.tvnavig.server.searchengine.SearchAssetService' due to existing same package import 'com.tvnavig.server.searchengine.*' | 48 |
| Rule | Violation | Line |
|---|---|---|
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'com.tvnavig.server.searchengine.SearchAssetServiceService' due to existing same package import 'com.tvnavig.server.searchengine.*' | 43 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.String' due to existing implicit import 'java.lang.*' | 53 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.String' due to existing implicit import 'java.lang.*' | 58 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.String' due to existing implicit import 'java.lang.*' | 60 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.String' due to existing implicit import 'java.lang.*' | 65 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.String' due to existing implicit import 'java.lang.*' | 67 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.String' due to existing implicit import 'java.lang.*' | 71 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'com.tvnavig.server.searchengine.SearchAssetService' due to existing same package import 'com.tvnavig.server.searchengine.*' | 75 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'com.tvnavig.server.searchengine.SearchAssetService' due to existing same package import 'com.tvnavig.server.searchengine.*' | 86 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'com.tvnavig.server.searchengine.SearchAssetServiceSoapBindingStub' due to existing same package import 'com.tvnavig.server.searchengine.*' | 88 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'com.tvnavig.server.searchengine.SearchAssetServiceSoapBindingStub' due to existing same package import 'com.tvnavig.server.searchengine.*' | 88 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.String' due to existing implicit import 'java.lang.*' | 97 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'com.tvnavig.server.searchengine.SearchAssetService' due to existing same package import 'com.tvnavig.server.searchengine.*' | 108 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'com.tvnavig.server.searchengine.SearchAssetServiceSoapBindingStub' due to existing same package import 'com.tvnavig.server.searchengine.*' | 109 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'com.tvnavig.server.searchengine.SearchAssetServiceSoapBindingStub' due to existing same package import 'com.tvnavig.server.searchengine.*' | 109 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.Throwable' due to existing implicit import 'java.lang.*' | 114 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.String' due to existing implicit import 'java.lang.*' | 129 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.String' due to existing implicit import 'java.lang.*' | 157 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.String' due to existing implicit import 'java.lang.*' | 157 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.String' due to existing implicit import 'java.lang.*' | 171 |
| Rule | Violation | Line |
|---|---|---|
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'com.tvnavig.server.searchengine.SearchAssetService' due to existing same package import 'com.tvnavig.server.searchengine.*' | 43 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.String' due to existing implicit import 'java.lang.*' | 82 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.String' due to existing implicit import 'java.lang.*' | 90 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.Class' due to existing implicit import 'java.lang.*' | 140 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.Class' due to existing implicit import 'java.lang.*' | 143 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.Class' due to existing implicit import 'java.lang.*' | 144 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.Class' due to existing implicit import 'java.lang.*' | 145 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.Class' due to existing implicit import 'java.lang.*' | 146 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.Class' due to existing implicit import 'java.lang.*' | 147 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.Class' due to existing implicit import 'java.lang.*' | 148 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.Class' due to existing implicit import 'java.lang.*' | 149 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.Class' due to existing implicit import 'java.lang.*' | 150 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.Class' due to existing implicit import 'java.lang.*' | 151 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.Class' due to existing implicit import 'java.lang.*' | 152 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.String' due to existing implicit import 'java.lang.*' | 194 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'com.tvnavig.server.searchengine.SearchAssetServiceException' due to existing same package import 'com.tvnavig.server.searchengine.*' | 203 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.String' due to existing implicit import 'java.lang.*' | 240 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.String' due to existing implicit import 'java.lang.*' | 240 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.Class' due to existing implicit import 'java.lang.*' | 254 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.Class' due to existing implicit import 'java.lang.*' | 254 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.Object' due to existing implicit import 'java.lang.*' | 257 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.Class' due to existing implicit import 'java.lang.*' | 259 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.Class' due to existing implicit import 'java.lang.*' | 259 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.Class' due to existing implicit import 'java.lang.*' | 261 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.Class' due to existing implicit import 'java.lang.*' | 261 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.Throwable' due to existing implicit import 'java.lang.*' | 277 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'com.tvnavig.server.searchengine.SearchAssetServiceException' due to existing same package import 'com.tvnavig.server.searchengine.*' | 282 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.Integer' due to existing implicit import 'java.lang.*' | 295 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.Object' due to existing implicit import 'java.lang.*' | 295 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.Object' due to existing implicit import 'java.lang.*' | 295 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.Integer' due to existing implicit import 'java.lang.*' | 295 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.Exception' due to existing implicit import 'java.lang.*' | 304 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'com.tvnavig.server.searchengine.SearchAssetServiceException' due to existing same package import 'com.tvnavig.server.searchengine.*' | 313 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'com.tvnavig.server.searchengine.SearchAssetServiceException' due to existing same package import 'com.tvnavig.server.searchengine.*' | 314 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.String' due to existing implicit import 'java.lang.*' | 321 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'com.tvnavig.server.searchengine.SearchAssetServiceException' due to existing same package import 'com.tvnavig.server.searchengine.*' | 321 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.String' due to existing implicit import 'java.lang.*' | 321 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.Object' due to existing implicit import 'java.lang.*' | 334 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.Object' due to existing implicit import 'java.lang.*' | 334 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.Integer' due to existing implicit import 'java.lang.*' | 334 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.Exception' due to existing implicit import 'java.lang.*' | 343 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'com.tvnavig.server.searchengine.SearchAssetServiceException' due to existing same package import 'com.tvnavig.server.searchengine.*' | 352 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'com.tvnavig.server.searchengine.SearchAssetServiceException' due to existing same package import 'com.tvnavig.server.searchengine.*' | 353 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'com.tvnavig.server.searchengine.SearchAssetServiceException' due to existing same package import 'com.tvnavig.server.searchengine.*' | 360 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.Object' due to existing implicit import 'java.lang.*' | 373 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.Integer' due to existing implicit import 'java.lang.*' | 373 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.Object' due to existing implicit import 'java.lang.*' | 373 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.Exception' due to existing implicit import 'java.lang.*' | 382 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'com.tvnavig.server.searchengine.SearchAssetServiceException' due to existing same package import 'com.tvnavig.server.searchengine.*' | 391 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'com.tvnavig.server.searchengine.SearchAssetServiceException' due to existing same package import 'com.tvnavig.server.searchengine.*' | 392 |
| Rule | Violation | Line |
|---|---|---|
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.String' due to existing implicit import 'java.lang.*' | 53 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.String' due to existing implicit import 'java.lang.*' | 55 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.String' due to existing implicit import 'java.lang.*' | 59 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.String' due to existing implicit import 'java.lang.*' | 61 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.String' due to existing implicit import 'java.lang.*' | 63 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.String' due to existing implicit import 'java.lang.*' | 65 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.String' due to existing implicit import 'java.lang.*' | 73 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.String' due to existing implicit import 'java.lang.*' | 74 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.String' due to existing implicit import 'java.lang.*' | 76 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.String' due to existing implicit import 'java.lang.*' | 77 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.String' due to existing implicit import 'java.lang.*' | 78 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.String' due to existing implicit import 'java.lang.*' | 79 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.String' due to existing implicit import 'java.lang.*' | 136 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.String' due to existing implicit import 'java.lang.*' | 146 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.String' due to existing implicit import 'java.lang.*' | 156 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.String' due to existing implicit import 'java.lang.*' | 166 |
| UseVarargs | Consider using varargs for methods or constructors which take an array the last parameter. | 166 |
| UseVarargs | Consider using varargs for methods or constructors which take an array the last parameter. | 186 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.String' due to existing implicit import 'java.lang.*' | 196 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.String' due to existing implicit import 'java.lang.*' | 206 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.String' due to existing implicit import 'java.lang.*' | 216 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.String' due to existing implicit import 'java.lang.*' | 226 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.String' due to existing implicit import 'java.lang.*' | 236 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.String' due to existing implicit import 'java.lang.*' | 246 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.String' due to existing implicit import 'java.lang.*' | 256 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.String' due to existing implicit import 'java.lang.*' | 266 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.Object' due to existing implicit import 'java.lang.*' | 270 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.Object' due to existing implicit import 'java.lang.*' | 271 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.Object' due to existing implicit import 'java.lang.*' | 329 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.reflect.Array.get' due to existing implicit import 'java.lang.*' | 329 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.reflect.Array.get' due to existing implicit import 'java.lang.*' | 340 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.Object' due to existing implicit import 'java.lang.*' | 340 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.String' due to existing implicit import 'java.lang.*' | 436 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.Class' due to existing implicit import 'java.lang.*' | 437 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.String' due to existing implicit import 'java.lang.*' | 448 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.Class' due to existing implicit import 'java.lang.*' | 449 |
| Rule | Violation | Line |
|---|---|---|
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.String' due to existing implicit import 'java.lang.*' | 44 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.String' due to existing implicit import 'java.lang.*' | 46 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.String' due to existing implicit import 'java.lang.*' | 52 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.String' due to existing implicit import 'java.lang.*' | 53 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.String' due to existing implicit import 'java.lang.*' | 64 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.String' due to existing implicit import 'java.lang.*' | 74 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.String' due to existing implicit import 'java.lang.*' | 84 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.String' due to existing implicit import 'java.lang.*' | 94 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.Object' due to existing implicit import 'java.lang.*' | 98 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.Object' due to existing implicit import 'java.lang.*' | 99 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.String' due to existing implicit import 'java.lang.*' | 168 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.Class' due to existing implicit import 'java.lang.*' | 169 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.String' due to existing implicit import 'java.lang.*' | 180 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.Class' due to existing implicit import 'java.lang.*' | 181 |
| Rule | Violation | Line |
|---|---|---|
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.String' due to existing implicit import 'java.lang.*' | 45 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.String' due to existing implicit import 'java.lang.*' | 45 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'com.tvnavig.server.user.UserServiceException' due to existing same package import 'com.tvnavig.server.user.*' | 45 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'com.tvnavig.server.user.UserServiceException' due to existing same package import 'com.tvnavig.server.user.*' | 46 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'com.tvnavig.server.user.UserServiceException' due to existing same package import 'com.tvnavig.server.user.*' | 47 |
| Rule | Violation | Line |
|---|---|---|
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.String' due to existing implicit import 'java.lang.*' | 48 |
| UseVarargs | Consider using varargs for methods or constructors which take an array the last parameter. | 48 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.Object' due to existing implicit import 'java.lang.*' | 53 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.Object' due to existing implicit import 'java.lang.*' | 54 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.String' due to existing implicit import 'java.lang.*' | 99 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.Class' due to existing implicit import 'java.lang.*' | 100 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.String' due to existing implicit import 'java.lang.*' | 111 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.Class' due to existing implicit import 'java.lang.*' | 112 |
| Rule | Violation | Line |
|---|---|---|
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.String' due to existing implicit import 'java.lang.*' | 44 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'com.tvnavig.server.user.UserService' due to existing same package import 'com.tvnavig.server.user.*' | 46 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'com.tvnavig.server.user.UserService' due to existing same package import 'com.tvnavig.server.user.*' | 48 |
| Rule | Violation | Line |
|---|---|---|
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'com.tvnavig.server.user.UserServiceService' due to existing same package import 'com.tvnavig.server.user.*' | 43 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.String' due to existing implicit import 'java.lang.*' | 53 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.String' due to existing implicit import 'java.lang.*' | 58 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.String' due to existing implicit import 'java.lang.*' | 60 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.String' due to existing implicit import 'java.lang.*' | 65 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.String' due to existing implicit import 'java.lang.*' | 67 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.String' due to existing implicit import 'java.lang.*' | 71 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'com.tvnavig.server.user.UserService' due to existing same package import 'com.tvnavig.server.user.*' | 75 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'com.tvnavig.server.user.UserService' due to existing same package import 'com.tvnavig.server.user.*' | 86 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'com.tvnavig.server.user.UserServiceSoapBindingStub' due to existing same package import 'com.tvnavig.server.user.*' | 88 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'com.tvnavig.server.user.UserServiceSoapBindingStub' due to existing same package import 'com.tvnavig.server.user.*' | 88 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.String' due to existing implicit import 'java.lang.*' | 97 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'com.tvnavig.server.user.UserService' due to existing same package import 'com.tvnavig.server.user.*' | 108 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'com.tvnavig.server.user.UserServiceSoapBindingStub' due to existing same package import 'com.tvnavig.server.user.*' | 109 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'com.tvnavig.server.user.UserServiceSoapBindingStub' due to existing same package import 'com.tvnavig.server.user.*' | 109 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.Throwable' due to existing implicit import 'java.lang.*' | 114 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.String' due to existing implicit import 'java.lang.*' | 129 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.String' due to existing implicit import 'java.lang.*' | 157 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.String' due to existing implicit import 'java.lang.*' | 157 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.String' due to existing implicit import 'java.lang.*' | 171 |
| Rule | Violation | Line |
|---|---|---|
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'com.tvnavig.server.user.UserService' due to existing same package import 'com.tvnavig.server.user.*' | 43 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.String' due to existing implicit import 'java.lang.*' | 68 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.String' due to existing implicit import 'java.lang.*' | 70 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.Class' due to existing implicit import 'java.lang.*' | 137 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.Class' due to existing implicit import 'java.lang.*' | 140 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.Class' due to existing implicit import 'java.lang.*' | 141 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.Class' due to existing implicit import 'java.lang.*' | 142 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.Class' due to existing implicit import 'java.lang.*' | 143 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.Class' due to existing implicit import 'java.lang.*' | 144 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.Class' due to existing implicit import 'java.lang.*' | 145 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.Class' due to existing implicit import 'java.lang.*' | 146 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.Class' due to existing implicit import 'java.lang.*' | 147 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.Class' due to existing implicit import 'java.lang.*' | 148 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.Class' due to existing implicit import 'java.lang.*' | 149 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.String' due to existing implicit import 'java.lang.*' | 214 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'com.tvnavig.server.user.UserServiceException' due to existing same package import 'com.tvnavig.server.user.*' | 223 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.String' due to existing implicit import 'java.lang.*' | 260 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.String' due to existing implicit import 'java.lang.*' | 260 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.Class' due to existing implicit import 'java.lang.*' | 274 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.Class' due to existing implicit import 'java.lang.*' | 274 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.Object' due to existing implicit import 'java.lang.*' | 277 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.Class' due to existing implicit import 'java.lang.*' | 279 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.Class' due to existing implicit import 'java.lang.*' | 279 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.Class' due to existing implicit import 'java.lang.*' | 281 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.Class' due to existing implicit import 'java.lang.*' | 281 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.Throwable' due to existing implicit import 'java.lang.*' | 297 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.Object' due to existing implicit import 'java.lang.*' | 315 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.Object' due to existing implicit import 'java.lang.*' | 315 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.String' due to existing implicit import 'java.lang.*' | 326 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.String' due to existing implicit import 'java.lang.*' | 326 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'com.tvnavig.server.user.UserServiceException' due to existing same package import 'com.tvnavig.server.user.*' | 326 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.Object' due to existing implicit import 'java.lang.*' | 339 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.Object' due to existing implicit import 'java.lang.*' | 339 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.Exception' due to existing implicit import 'java.lang.*' | 348 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'com.tvnavig.server.user.UserServiceException' due to existing same package import 'com.tvnavig.server.user.*' | 357 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'com.tvnavig.server.user.UserServiceException' due to existing same package import 'com.tvnavig.server.user.*' | 358 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'com.tvnavig.server.user.UserServiceException' due to existing same package import 'com.tvnavig.server.user.*' | 365 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.Integer' due to existing implicit import 'java.lang.*' | 378 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.Object' due to existing implicit import 'java.lang.*' | 378 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.Object' due to existing implicit import 'java.lang.*' | 378 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.Exception' due to existing implicit import 'java.lang.*' | 387 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'com.tvnavig.server.user.UserServiceException' due to existing same package import 'com.tvnavig.server.user.*' | 396 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'com.tvnavig.server.user.UserServiceException' due to existing same package import 'com.tvnavig.server.user.*' | 397 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'com.tvnavig.server.user.UserServiceException' due to existing same package import 'com.tvnavig.server.user.*' | 404 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.Object' due to existing implicit import 'java.lang.*' | 417 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.Object' due to existing implicit import 'java.lang.*' | 417 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.Integer' due to existing implicit import 'java.lang.*' | 417 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.Exception' due to existing implicit import 'java.lang.*' | 426 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'com.tvnavig.server.user.UserServiceException' due to existing same package import 'com.tvnavig.server.user.*' | 435 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'com.tvnavig.server.user.UserServiceException' due to existing same package import 'com.tvnavig.server.user.*' | 436 |
| Rule | Violation | Line |
|---|---|---|
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.String' due to existing implicit import 'java.lang.*' | 46 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.String' due to existing implicit import 'java.lang.*' | 50 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.String' due to existing implicit import 'java.lang.*' | 57 |
| UseVarargs | Consider using varargs for methods or constructors which take an array the last parameter. | 59 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.String' due to existing implicit import 'java.lang.*' | 59 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.String' due to existing implicit import 'java.lang.*' | 92 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.String' due to existing implicit import 'java.lang.*' | 102 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.String' due to existing implicit import 'java.lang.*' | 132 |
| UseVarargs | Consider using varargs for methods or constructors which take an array the last parameter. | 142 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.String' due to existing implicit import 'java.lang.*' | 142 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.Object' due to existing implicit import 'java.lang.*' | 146 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.Object' due to existing implicit import 'java.lang.*' | 147 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.reflect.Array.get' due to existing implicit import 'java.lang.*' | 186 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.Object' due to existing implicit import 'java.lang.*' | 186 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.String' due to existing implicit import 'java.lang.*' | 240 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.Class' due to existing implicit import 'java.lang.*' | 241 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.String' due to existing implicit import 'java.lang.*' | 252 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.Class' due to existing implicit import 'java.lang.*' | 253 |
| Rule | Violation | Line |
|---|---|---|
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.Integer' due to existing implicit import 'java.lang.*' | 44 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.String' due to existing implicit import 'java.lang.*' | 46 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.Integer' due to existing implicit import 'java.lang.*' | 52 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.String' due to existing implicit import 'java.lang.*' | 53 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.Integer' due to existing implicit import 'java.lang.*' | 64 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.Integer' due to existing implicit import 'java.lang.*' | 74 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.String' due to existing implicit import 'java.lang.*' | 84 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.String' due to existing implicit import 'java.lang.*' | 94 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.Object' due to existing implicit import 'java.lang.*' | 98 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.Object' due to existing implicit import 'java.lang.*' | 99 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.String' due to existing implicit import 'java.lang.*' | 168 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.Class' due to existing implicit import 'java.lang.*' | 169 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.String' due to existing implicit import 'java.lang.*' | 180 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.Class' due to existing implicit import 'java.lang.*' | 181 |
| Rule | Violation | Line |
|---|---|---|
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.String' due to existing implicit import 'java.lang.*' | 44 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.Integer' due to existing implicit import 'java.lang.*' | 46 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.String' due to existing implicit import 'java.lang.*' | 48 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.String' due to existing implicit import 'java.lang.*' | 54 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.Integer' due to existing implicit import 'java.lang.*' | 55 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.String' due to existing implicit import 'java.lang.*' | 56 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.String' due to existing implicit import 'java.lang.*' | 68 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.String' due to existing implicit import 'java.lang.*' | 78 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.Integer' due to existing implicit import 'java.lang.*' | 88 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.Integer' due to existing implicit import 'java.lang.*' | 98 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.String' due to existing implicit import 'java.lang.*' | 108 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.String' due to existing implicit import 'java.lang.*' | 118 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.Object' due to existing implicit import 'java.lang.*' | 122 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.Object' due to existing implicit import 'java.lang.*' | 123 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.String' due to existing implicit import 'java.lang.*' | 204 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.Class' due to existing implicit import 'java.lang.*' | 205 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.String' due to existing implicit import 'java.lang.*' | 216 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.Class' due to existing implicit import 'java.lang.*' | 217 |
| Rule | Violation | Line |
|---|---|---|
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'com.tvnavig.server.user.service.common.MetaDataResult' due to existing same package import 'com.tvnavig.server.user.service.common.*' | 44 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'com.tvnavig.server.user.service.common.UserGroupResult' due to existing same package import 'com.tvnavig.server.user.service.common.*' | 46 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'com.tvnavig.server.user.service.common.MetaDataResult' due to existing same package import 'com.tvnavig.server.user.service.common.*' | 52 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'com.tvnavig.server.user.service.common.UserGroupResult' due to existing same package import 'com.tvnavig.server.user.service.common.*' | 53 |
| UseVarargs | Consider using varargs for methods or constructors which take an array the last parameter. | 53 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'com.tvnavig.server.user.service.common.MetaDataResult' due to existing same package import 'com.tvnavig.server.user.service.common.*' | 64 |
| UseVarargs | Consider using varargs for methods or constructors which take an array the last parameter. | 74 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'com.tvnavig.server.user.service.common.MetaDataResult' due to existing same package import 'com.tvnavig.server.user.service.common.*' | 74 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'com.tvnavig.server.user.service.common.UserGroupResult' due to existing same package import 'com.tvnavig.server.user.service.common.*' | 84 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'com.tvnavig.server.user.service.common.UserGroupResult' due to existing same package import 'com.tvnavig.server.user.service.common.*' | 94 |
| UseVarargs | Consider using varargs for methods or constructors which take an array the last parameter. | 94 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.Object' due to existing implicit import 'java.lang.*' | 98 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.Object' due to existing implicit import 'java.lang.*' | 99 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.Object' due to existing implicit import 'java.lang.*' | 131 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.reflect.Array.get' due to existing implicit import 'java.lang.*' | 131 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.reflect.Array.get' due to existing implicit import 'java.lang.*' | 142 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.Object' due to existing implicit import 'java.lang.*' | 142 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.String' due to existing implicit import 'java.lang.*' | 184 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.Class' due to existing implicit import 'java.lang.*' | 185 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.String' due to existing implicit import 'java.lang.*' | 196 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.Class' due to existing implicit import 'java.lang.*' | 197 |
| Rule | Violation | Line |
|---|---|---|
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.String' due to existing implicit import 'java.lang.*' | 44 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.String' due to existing implicit import 'java.lang.*' | 46 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.String' due to existing implicit import 'java.lang.*' | 48 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.String' due to existing implicit import 'java.lang.*' | 50 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.Integer' due to existing implicit import 'java.lang.*' | 52 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.String' due to existing implicit import 'java.lang.*' | 58 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.String' due to existing implicit import 'java.lang.*' | 59 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.String' due to existing implicit import 'java.lang.*' | 60 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.String' due to existing implicit import 'java.lang.*' | 61 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.Integer' due to existing implicit import 'java.lang.*' | 62 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.String' due to existing implicit import 'java.lang.*' | 76 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.String' due to existing implicit import 'java.lang.*' | 86 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.String' due to existing implicit import 'java.lang.*' | 96 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.String' due to existing implicit import 'java.lang.*' | 106 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.String' due to existing implicit import 'java.lang.*' | 116 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.String' due to existing implicit import 'java.lang.*' | 126 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.String' due to existing implicit import 'java.lang.*' | 136 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.String' due to existing implicit import 'java.lang.*' | 146 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.Integer' due to existing implicit import 'java.lang.*' | 156 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.Integer' due to existing implicit import 'java.lang.*' | 166 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.Object' due to existing implicit import 'java.lang.*' | 170 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.Object' due to existing implicit import 'java.lang.*' | 171 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.String' due to existing implicit import 'java.lang.*' | 276 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.Class' due to existing implicit import 'java.lang.*' | 277 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.String' due to existing implicit import 'java.lang.*' | 288 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.Class' due to existing implicit import 'java.lang.*' | 289 |
| Rule | Violation | Line |
|---|---|---|
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.String' due to existing implicit import 'java.lang.*' | 44 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.String' due to existing implicit import 'java.lang.*' | 50 |
| UseVarargs | Consider using varargs for methods or constructors which take an array the last parameter. | 50 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.String' due to existing implicit import 'java.lang.*' | 60 |
| UseVarargs | Consider using varargs for methods or constructors which take an array the last parameter. | 70 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.String' due to existing implicit import 'java.lang.*' | 70 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.Object' due to existing implicit import 'java.lang.*' | 74 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.Object' due to existing implicit import 'java.lang.*' | 75 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.Object' due to existing implicit import 'java.lang.*' | 104 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.reflect.Array.get' due to existing implicit import 'java.lang.*' | 104 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.String' due to existing implicit import 'java.lang.*' | 140 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.Class' due to existing implicit import 'java.lang.*' | 141 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.String' due to existing implicit import 'java.lang.*' | 152 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.Class' due to existing implicit import 'java.lang.*' | 153 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| MissingSerialVersionUID | Classes implementing Serializable should set a serialVersionUID | 3 | 43–224 |
| FieldNamingConventions | The field name 'id' doesn't match '_[a-z][a-zA-Z0-9]*' | 1 | 44 |
| ShortVariable | Avoid variables with short names like id | 3 | 44 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.Integer' due to existing implicit import 'java.lang.*' | 4 | 44 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.Integer' due to existing implicit import 'java.lang.*' | 4 | 46 |
| FieldNamingConventions | The field name 'metaDataId' doesn't match '_[a-z][a-zA-Z0-9]*' | 1 | 46 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.String' due to existing implicit import 'java.lang.*' | 4 | 48 |
| FieldNamingConventions | The field name 'value' doesn't match '_[a-z][a-zA-Z0-9]*' | 1 | 48 |
| CommentRequired | Public method and constructor comments are required | 3 | 50–51 |
| UncommentedEmptyConstructor | Document empty constructor | 3 | 50–51 |
| CommentRequired | Public method and constructor comments are required | 3 | 53–60 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.Integer' due to existing implicit import 'java.lang.*' | 4 | 54 |
| ShortVariable | Avoid variables with short names like id | 3 | 54 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.Integer' due to existing implicit import 'java.lang.*' | 4 | 55 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.String' due to existing implicit import 'java.lang.*' | 4 | 56 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.Integer' due to existing implicit import 'java.lang.*' | 4 | 68 |
| ShortVariable | Avoid variables with short names like id | 3 | 78 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.Integer' due to existing implicit import 'java.lang.*' | 4 | 78 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.Integer' due to existing implicit import 'java.lang.*' | 4 | 88 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.Integer' due to existing implicit import 'java.lang.*' | 4 | 98 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.String' due to existing implicit import 'java.lang.*' | 4 | 108 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.String' due to existing implicit import 'java.lang.*' | 4 | 118 |
| RedundantFieldInitializer | Avoid using redundant field initializer for '__equalsCalc' | 3 | 122 |
| FieldDeclarationsShouldBeAtStartOfClass | Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. | 3 | 122 |
| FieldNamingConventions | The field name '__equalsCalc' doesn't match '_[a-z][a-zA-Z0-9]*' | 1 | 122 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.Object' due to existing implicit import 'java.lang.*' | 4 | 122 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.Object' due to existing implicit import 'java.lang.*' | 4 | 123 |
| CommentRequired | Public method and constructor comments are required | 3 | 123–145 |
| AvoidSynchronizedAtMethodLevel | Use block level rather than method level synchronization | 3 | 123–145 |
| MissingOverride | The method 'equals(java.lang.Object)' is missing an @Override annotation. | 3 | 123–145 |
| ControlStatementBraces | This statement should have braces | 3 | 124 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 3 | 125 |
| ControlStatementBraces | This statement should have braces | 3 | 126 |
| ControlStatementBraces | This statement should have braces | 3 | 127 |
| CompareObjectsWithEquals | Use equals() to compare object references. | 3 | 129 |
| LocalVariableNamingConventions | The local variable name '_equals' doesn't match '[a-z][a-zA-Z0-9]*' | 1 | 132 |
| NullAssignment | Assigning an Object to null is a code smell. Consider refactoring. | 3 | 143 |
| FieldDeclarationsShouldBeAtStartOfClass | Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. | 3 | 147 |
| FieldNamingConventions | The field name '__hashCodeCalc' doesn't match '_[a-z][a-zA-Z0-9]*' | 1 | 147 |
| RedundantFieldInitializer | Avoid using redundant field initializer for '__hashCodeCalc' | 3 | 147 |
| CommentRequired | Public method and constructor comments are required | 3 | 148–165 |
| AvoidSynchronizedAtMethodLevel | Use block level rather than method level synchronization | 3 | 148–165 |
| MissingOverride | The method 'hashCode()' is missing an @Override annotation. | 3 | 148–165 |
| LocalVariableNamingConventions | The local variable name '_hashCode' doesn't match '[a-z][a-zA-Z0-9]*' | 1 | 153 |
| FieldDeclarationsShouldBeAtStartOfClass | Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. | 3 | 168–169 |
| FieldNamingConventions | The static field name 'typeDesc' doesn't match '_[a-z][a-zA-Z0-9]*' | 1 | 168 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.String' due to existing implicit import 'java.lang.*' | 4 | 204 |
| FormalParameterNamingConventions | The method parameter name '_javaType' doesn't match '[a-z][a-zA-Z0-9]*' | 1 | 205 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.Class' due to existing implicit import 'java.lang.*' | 4 | 205 |
| FormalParameterNamingConventions | The method parameter name '_xmlType' doesn't match '[a-z][a-zA-Z0-9]*' | 1 | 206 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.String' due to existing implicit import 'java.lang.*' | 4 | 216 |
| FormalParameterNamingConventions | The method parameter name '_javaType' doesn't match '[a-z][a-zA-Z0-9]*' | 1 | 217 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.Class' due to existing implicit import 'java.lang.*' | 4 | 217 |
| FormalParameterNamingConventions | The method parameter name '_xmlType' doesn't match '[a-z][a-zA-Z0-9]*' | 1 | 218 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| UnnecessaryModifier | Unnecessary modifier 'public' on method 'getAsset': the method is declared in an interface type | 3 | 44 |
| CommentRequired | Public method and constructor comments are required | 3 | 44 |
| ShortVariable | Avoid variables with short names like id | 3 | 44 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'com.tvnavig.server.searchengine.SearchAssetServiceException' due to existing same package import 'com.tvnavig.server.searchengine.*' | 4 | 44 |
| UnnecessaryModifier | Unnecessary modifier 'public' on method 'getAssetsList': the method is declared in an interface type | 3 | 45 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.String' due to existing implicit import 'java.lang.*' | 4 | 45 |
| CommentRequired | Public method and constructor comments are required | 3 | 45 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.String' due to existing implicit import 'java.lang.*' | 4 | 45 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'com.tvnavig.server.searchengine.SearchAssetServiceException' due to existing same package import 'com.tvnavig.server.searchengine.*' | 4 | 45 |
| UnnecessaryModifier | Unnecessary modifier 'public' on method 'getAssetPlayer': the method is declared in an interface type | 3 | 46 |
| ShortVariable | Avoid variables with short names like id | 3 | 46 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'com.tvnavig.server.searchengine.SearchAssetServiceException' due to existing same package import 'com.tvnavig.server.searchengine.*' | 4 | 46 |
| CommentRequired | Public method and constructor comments are required | 3 | 46 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| MissingSerialVersionUID | Classes implementing Serializable should set a serialVersionUID | 3 | 43–126 |
| CallSuperInConstructor | It is a good practice to call super() in a constructor | 3 | 44–45 |
| CommentRequired | Public method and constructor comments are required | 3 | 44–45 |
| UncommentedEmptyConstructor | Document empty constructor | 3 | 44–45 |
| CommentRequired | Public method and constructor comments are required | 3 | 47–51 |
| UseVarargs | Consider using varargs for methods or constructors which take an array the last parameter. | 4 | 48 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.String' due to existing implicit import 'java.lang.*' | 4 | 48 |
| RedundantFieldInitializer | Avoid using redundant field initializer for '__equalsCalc' | 3 | 53 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.Object' due to existing implicit import 'java.lang.*' | 4 | 53 |
| FieldDeclarationsShouldBeAtStartOfClass | Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. | 3 | 53 |
| FieldNamingConventions | The field name '__equalsCalc' doesn't match '_[a-z][a-zA-Z0-9]*' | 1 | 53 |
| CommentRequired | Public method and constructor comments are required | 3 | 54–67 |
| MissingOverride | The method 'equals(java.lang.Object)' is missing an @Override annotation. | 3 | 54–67 |
| AvoidSynchronizedAtMethodLevel | Use block level rather than method level synchronization | 3 | 54–67 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.Object' due to existing implicit import 'java.lang.*' | 4 | 54 |
| ControlStatementBraces | This statement should have braces | 3 | 55 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 3 | 56 |
| UnusedLocalVariable | Avoid unused local variables such as 'other'. | 3 | 56 |
| ControlStatementBraces | This statement should have braces | 3 | 57 |
| ControlStatementBraces | This statement should have braces | 3 | 58 |
| CompareObjectsWithEquals | Use equals() to compare object references. | 3 | 60 |
| LocalVariableNamingConventions | The local variable name '_equals' doesn't match '[a-z][a-zA-Z0-9]*' | 1 | 63 |
| NullAssignment | Assigning an Object to null is a code smell. Consider refactoring. | 3 | 65 |
| FieldDeclarationsShouldBeAtStartOfClass | Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. | 3 | 69 |
| FieldNamingConventions | The field name '__hashCodeCalc' doesn't match '_[a-z][a-zA-Z0-9]*' | 1 | 69 |
| RedundantFieldInitializer | Avoid using redundant field initializer for '__hashCodeCalc' | 3 | 69 |
| AvoidSynchronizedAtMethodLevel | Use block level rather than method level synchronization | 3 | 70–78 |
| MissingOverride | The method 'hashCode()' is missing an @Override annotation. | 3 | 70–78 |
| CommentRequired | Public method and constructor comments are required | 3 | 70–78 |
| LocalVariableNamingConventions | The local variable name '_hashCode' doesn't match '[a-z][a-zA-Z0-9]*' | 1 | 75 |
| FieldDeclarationsShouldBeAtStartOfClass | Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. | 3 | 81–82 |
| FieldNamingConventions | The static field name 'typeDesc' doesn't match '_[a-z][a-zA-Z0-9]*' | 1 | 81 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.String' due to existing implicit import 'java.lang.*' | 4 | 99 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.Class' due to existing implicit import 'java.lang.*' | 4 | 100 |
| FormalParameterNamingConventions | The method parameter name '_javaType' doesn't match '[a-z][a-zA-Z0-9]*' | 1 | 100 |
| FormalParameterNamingConventions | The method parameter name '_xmlType' doesn't match '[a-z][a-zA-Z0-9]*' | 1 | 101 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.String' due to existing implicit import 'java.lang.*' | 4 | 111 |
| FormalParameterNamingConventions | The method parameter name '_javaType' doesn't match '[a-z][a-zA-Z0-9]*' | 1 | 112 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.Class' due to existing implicit import 'java.lang.*' | 4 | 112 |
| FormalParameterNamingConventions | The method parameter name '_xmlType' doesn't match '[a-z][a-zA-Z0-9]*' | 1 | 113 |
| MissingOverride | The method 'writeDetails(javax.xml.namespace.QName, org.apache.axis.encoding.SerializationContext)' is missing an @Override annotation. | 3 | 123–125 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| UnnecessaryModifier | Unnecessary modifier 'public' on method 'getSearchAssetServiceAddress': the method is declared in an interface type | 3 | 44 |
| CommentRequired | Public method and constructor comments are required | 3 | 44 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.String' due to existing implicit import 'java.lang.*' | 4 | 44 |
| CommentRequired | Public method and constructor comments are required | 3 | 46 |
| UnnecessaryModifier | Unnecessary modifier 'public' on method 'getSearchAssetService': the method is declared in an interface type | 3 | 46 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'com.tvnavig.server.searchengine.SearchAssetService' due to existing same package import 'com.tvnavig.server.searchengine.*' | 4 | 46 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'com.tvnavig.server.searchengine.SearchAssetService' due to existing same package import 'com.tvnavig.server.searchengine.*' | 4 | 48 |
| UnnecessaryModifier | Unnecessary modifier 'public' on method 'getSearchAssetService': the method is declared in an interface type | 3 | 48 |
| CommentRequired | Public method and constructor comments are required | 3 | 48 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'com.tvnavig.server.searchengine.SearchAssetServiceService' due to existing same package import 'com.tvnavig.server.searchengine.*' | 4 | 43 |
| MissingSerialVersionUID | Classes implementing Serializable should set a serialVersionUID | 3 | 43–175 |
| CommentRequired | Public method and constructor comments are required | 3 | 45–46 |
| UncommentedEmptyConstructor | Document empty constructor | 3 | 45–46 |
| CallSuperInConstructor | It is a good practice to call super() in a constructor | 3 | 45–46 |
| CommentRequired | Public method and constructor comments are required | 3 | 49–51 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.String' due to existing implicit import 'java.lang.*' | 4 | 53 |
| CommentRequired | Public method and constructor comments are required | 3 | 53–55 |
| FieldNamingConventions | The field name 'SearchAssetService_address' doesn't match '_[a-z][a-zA-Z0-9]*' | 1 | 58 |
| FieldDeclarationsShouldBeAtStartOfClass | Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. | 3 | 58 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.String' due to existing implicit import 'java.lang.*' | 4 | 58 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.String' due to existing implicit import 'java.lang.*' | 4 | 60 |
| MissingOverride | The method 'getSearchAssetServiceAddress()' is missing an @Override annotation. | 3 | 60–62 |
| CommentRequired | Public method and constructor comments are required | 3 | 60–62 |
| FieldDeclarationsShouldBeAtStartOfClass | Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. | 3 | 65 |
| AvoidDuplicateLiterals | The String literal "SearchAssetService" appears 4 times in this file; the first occurrence is on line 65 | 3 | 65 |
| FieldNamingConventions | The field name 'SearchAssetServiceWSDDServiceName' doesn't match '_[a-z][a-zA-Z0-9]*' | 1 | 65 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.String' due to existing implicit import 'java.lang.*' | 4 | 65 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.String' due to existing implicit import 'java.lang.*' | 4 | 67 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.String' due to existing implicit import 'java.lang.*' | 4 | 71 |
| CommentRequired | Public method and constructor comments are required | 3 | 75–84 |
| MissingOverride | The method 'getSearchAssetService()' is missing an @Override annotation. | 3 | 75–84 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'com.tvnavig.server.searchengine.SearchAssetService' due to existing same package import 'com.tvnavig.server.searchengine.*' | 4 | 75 |
| CommentRequired | Public method and constructor comments are required | 3 | 86–95 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'com.tvnavig.server.searchengine.SearchAssetService' due to existing same package import 'com.tvnavig.server.searchengine.*' | 4 | 86 |
| MissingOverride | The method 'getSearchAssetService(java.net.URL)' is missing an @Override annotation. | 3 | 86–95 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'com.tvnavig.server.searchengine.SearchAssetServiceSoapBindingStub' due to existing same package import 'com.tvnavig.server.searchengine.*' | 4 | 88 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'com.tvnavig.server.searchengine.SearchAssetServiceSoapBindingStub' due to existing same package import 'com.tvnavig.server.searchengine.*' | 4 | 88 |
| LocalVariableNamingConventions | The local variable name '_stub' doesn't match '[a-z][a-zA-Z0-9]*' | 1 | 88 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.String' due to existing implicit import 'java.lang.*' | 4 | 97 |
| CommentRequired | Public method and constructor comments are required | 3 | 97–99 |
| MissingOverride | The method 'getPort(Class)' is missing an @Override annotation. | 3 | 106–118 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'com.tvnavig.server.searchengine.SearchAssetService' due to existing same package import 'com.tvnavig.server.searchengine.*' | 4 | 108 |
| LocalVariableNamingConventions | The local variable name '_stub' doesn't match '[a-z][a-zA-Z0-9]*' | 1 | 109 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'com.tvnavig.server.searchengine.SearchAssetServiceSoapBindingStub' due to existing same package import 'com.tvnavig.server.searchengine.*' | 4 | 109 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'com.tvnavig.server.searchengine.SearchAssetServiceSoapBindingStub' due to existing same package import 'com.tvnavig.server.searchengine.*' | 4 | 109 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.Throwable' due to existing implicit import 'java.lang.*' | 4 | 114 |
| MissingOverride | The method 'getPort(javax.xml.namespace.QName, Class)' is missing an @Override annotation. | 3 | 125–138 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.String' due to existing implicit import 'java.lang.*' | 4 | 129 |
| LocalVariableNamingConventions | The local variable name '_stub' doesn't match '[a-z][a-zA-Z0-9]*' | 1 | 134 |
| CommentRequired | Public method and constructor comments are required | 3 | 140–142 |
| MissingOverride | The method 'getServiceName()' is missing an @Override annotation. | 3 | 140–142 |
| RedundantFieldInitializer | Avoid using redundant field initializer for 'ports' | 3 | 144 |
| LooseCoupling | Avoid using implementation types like 'java.util.HashSet'; use the interface instead | 3 | 144 |
| FieldNamingConventions | The field name 'ports' doesn't match '_[a-z][a-zA-Z0-9]*' | 1 | 144 |
| FieldDeclarationsShouldBeAtStartOfClass | Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. | 3 | 144 |
| MissingOverride | The method 'getPorts()' is missing an @Override annotation. | 3 | 146–152 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.String' due to existing implicit import 'java.lang.*' | 4 | 157 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.String' due to existing implicit import 'java.lang.*' | 4 | 157 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.String' due to existing implicit import 'java.lang.*' | 4 | 171 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'com.tvnavig.server.searchengine.SearchAssetService' due to existing same package import 'com.tvnavig.server.searchengine.*' | 4 | 43 |
| ImmutableField | Private field 'cachedSerClasses' could be made final; it is only initialized in the declaration or constructor. | 3 | 44 |
| LooseCoupling | Avoid using implementation types like 'java.util.Vector'; use the interface instead | 3 | 44 |
| UseArrayListInsteadOfVector | Use ArrayList instead of Vector | 3 | 44 |
| FieldNamingConventions | The field name 'cachedSerClasses' doesn't match '_[a-z][a-zA-Z0-9]*' | 1 | 44 |
| FieldNamingConventions | The field name 'cachedSerQNames' doesn't match '_[a-z][a-zA-Z0-9]*' | 1 | 45 |
| LooseCoupling | Avoid using implementation types like 'java.util.Vector'; use the interface instead | 3 | 45 |
| UseArrayListInsteadOfVector | Use ArrayList instead of Vector | 3 | 45 |
| ImmutableField | Private field 'cachedSerQNames' could be made final; it is only initialized in the declaration or constructor. | 3 | 45 |
| ImmutableField | Private field 'cachedSerFactories' could be made final; it is only initialized in the declaration or constructor. | 3 | 46 |
| FieldNamingConventions | The field name 'cachedSerFactories' doesn't match '_[a-z][a-zA-Z0-9]*' | 1 | 46 |
| UseArrayListInsteadOfVector | Use ArrayList instead of Vector | 3 | 46 |
| LooseCoupling | Avoid using implementation types like 'java.util.Vector'; use the interface instead | 3 | 46 |
| ImmutableField | Private field 'cachedDeserFactories' could be made final; it is only initialized in the declaration or constructor. | 3 | 47 |
| LooseCoupling | Avoid using implementation types like 'java.util.Vector'; use the interface instead | 3 | 47 |
| FieldNamingConventions | The field name 'cachedDeserFactories' doesn't match '_[a-z][a-zA-Z0-9]*' | 1 | 47 |
| UseArrayListInsteadOfVector | Use ArrayList instead of Vector | 3 | 47 |
| DefaultPackage | Use explicit scoping instead of the default package private level | 3 | 49 |
| CommentDefaultAccessModifier | To avoid mistakes add a comment at the beginning of the _operations field if you want a default access modifier | 3 | 49 |
| MethodNamingConventions | The static method name '_initOperationDesc1' doesn't match '[a-z][a-zA-Z0-9]*' | 1 | 56–122 |
| AvoidDuplicateLiterals | The String literal "int" appears 4 times in this file; the first occurrence is on line 61 | 3 | 61 |
| AvoidDuplicateLiterals | The String literal "http://www.w3.org/2001/XMLSchema" appears 9 times in this file; the first occurrence is on line 61 | 3 | 61 |
| AvoidDuplicateLiterals | The String literal "http://common.service.searchengine.server.tvnavig.com" appears 5 times in this file; the first occurrence is on line 65 | 3 | 65 |
| AvoidDuplicateLiterals | The String literal "http://searchengine.server.tvnavig.com" appears 15 times in this file; the first occurrence is on line 71 | 3 | 71 |
| AvoidDuplicateLiterals | The String literal "SearchAssetServiceException" appears 4 times in this file; the first occurrence is on line 73 | 3 | 73 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.String' due to existing implicit import 'java.lang.*' | 4 | 82 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.String' due to existing implicit import 'java.lang.*' | 4 | 90 |
| CommentRequired | Public method and constructor comments are required | 3 | 124–126 |
| CommentRequired | Public method and constructor comments are required | 3 | 128–131 |
| NcssCount | The constructor 'SearchAssetServiceSoapBindingStub(javax.xml.rpc.Service)' has a NCSS line count of 73. | 3 | 133–215 |
| CallSuperInConstructor | It is a good practice to call super() in a constructor | 3 | 133–215 |
| CommentRequired | Public method and constructor comments are required | 3 | 133–215 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.Class' due to existing implicit import 'java.lang.*' | 4 | 140 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.Class' due to existing implicit import 'java.lang.*' | 4 | 143 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.Class' due to existing implicit import 'java.lang.*' | 4 | 144 |
| UnusedLocalVariable | Avoid unused local variables such as 'enumsf'. | 3 | 145 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.Class' due to existing implicit import 'java.lang.*' | 4 | 145 |
| UnusedLocalVariable | Avoid unused local variables such as 'enumdf'. | 3 | 146 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.Class' due to existing implicit import 'java.lang.*' | 4 | 146 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.Class' due to existing implicit import 'java.lang.*' | 4 | 147 |
| UnusedLocalVariable | Avoid unused local variables such as 'arraysf'. | 3 | 147 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.Class' due to existing implicit import 'java.lang.*' | 4 | 148 |
| UnusedLocalVariable | Avoid unused local variables such as 'arraydf'. | 3 | 148 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.Class' due to existing implicit import 'java.lang.*' | 4 | 149 |
| UnusedLocalVariable | Avoid unused local variables such as 'simplesf'. | 3 | 149 |
| UnusedLocalVariable | Avoid unused local variables such as 'simpledf'. | 3 | 150 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.Class' due to existing implicit import 'java.lang.*' | 4 | 150 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.Class' due to existing implicit import 'java.lang.*' | 4 | 151 |
| UnusedLocalVariable | Avoid unused local variables such as 'simplelistsf'. | 3 | 151 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.Class' due to existing implicit import 'java.lang.*' | 4 | 152 |
| UnusedLocalVariable | Avoid unused local variables such as 'simplelistdf'. | 3 | 152 |
| NullAssignment | Assigning an Object to null is a code smell. Consider refactoring. | 3 | 179 |
| NullAssignment | Assigning an Object to null is a code smell. Consider refactoring. | 3 | 188 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.String' due to existing implicit import 'java.lang.*' | 4 | 194 |
| NullAssignment | Assigning an Object to null is a code smell. Consider refactoring. | 3 | 197 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'com.tvnavig.server.searchengine.SearchAssetServiceException' due to existing same package import 'com.tvnavig.server.searchengine.*' | 4 | 203 |
| CyclomaticComplexity | The method 'createCall()' has a cyclomatic complexity of 14. | 3 | 217–280 |
| CommentRequired | Protected method constructor comments are required | 3 | 217–280 |
| NPathComplexity | The method 'createCall()' has an NPath complexity of 641 | 3 | 217–280 |
| LocalVariableNamingConventions | The local variable name '_call' doesn't match '[a-z][a-zA-Z0-9]*' | 1 | 219 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.String' due to existing implicit import 'java.lang.*' | 4 | 240 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.String' due to existing implicit import 'java.lang.*' | 4 | 240 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.Class' due to existing implicit import 'java.lang.*' | 4 | 254 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.Class' due to existing implicit import 'java.lang.*' | 4 | 254 |
| ShortVariable | Avoid variables with short names like x | 3 | 257 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.Object' due to existing implicit import 'java.lang.*' | 4 | 257 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.Class' due to existing implicit import 'java.lang.*' | 4 | 259 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.Class' due to existing implicit import 'java.lang.*' | 4 | 259 |
| ShortVariable | Avoid variables with short names like sf | 3 | 259 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.Class' due to existing implicit import 'java.lang.*' | 4 | 261 |
| ShortVariable | Avoid variables with short names like df | 3 | 261 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.Class' due to existing implicit import 'java.lang.*' | 4 | 261 |
| ShortVariable | Avoid variables with short names like sf | 3 | 266 |
| ShortVariable | Avoid variables with short names like df | 3 | 268 |
| LocalVariableNamingConventions | The exception block parameter name '_t' doesn't match '[a-z][a-zA-Z0-9]*' | 1 | 277 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.Throwable' due to existing implicit import 'java.lang.*' | 4 | 277 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'com.tvnavig.server.searchengine.SearchAssetServiceException' due to existing same package import 'com.tvnavig.server.searchengine.*' | 4 | 282 |
| CommentRequired | Public method and constructor comments are required | 3 | 282–319 |
| ShortVariable | Avoid variables with short names like id | 3 | 282 |
| CyclomaticComplexity | The method 'getAsset(int, int)' has a cyclomatic complexity of 13. | 3 | 282–319 |
| MissingOverride | The method 'getAsset(int, int)' is missing an @Override annotation. | 3 | 282–319 |
| LocalVariableNamingConventions | The local variable name '_call' doesn't match '[a-z][a-zA-Z0-9]*' | 1 | 286 |
| LocalVariableNamingConventions | The local variable name '_resp' doesn't match '[a-z][a-zA-Z0-9]*' | 1 | 295 |
| IntegerInstantiation | Avoid instantiating Integer objects. Call Integer.valueOf() instead. | 2 | 295 |
| IntegerInstantiation | Avoid instantiating Integer objects. Call Integer.valueOf() instead. | 2 | 295 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.Integer' due to existing implicit import 'java.lang.*' | 4 | 295 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.Object' due to existing implicit import 'java.lang.*' | 4 | 295 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.Object' due to existing implicit import 'java.lang.*' | 4 | 295 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.Integer' due to existing implicit import 'java.lang.*' | 4 | 295 |
| LocalVariableNamingConventions | The exception block parameter name '_exception' doesn't match '[a-z][a-zA-Z0-9]*' | 1 | 304 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.Exception' due to existing implicit import 'java.lang.*' | 4 | 304 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'com.tvnavig.server.searchengine.SearchAssetServiceException' due to existing same package import 'com.tvnavig.server.searchengine.*' | 4 | 313 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'com.tvnavig.server.searchengine.SearchAssetServiceException' due to existing same package import 'com.tvnavig.server.searchengine.*' | 4 | 314 |
| MissingOverride | The method 'getAssetsList(int, java.lang.String, com.tvnavig.server.asset.service.common.MetaValueResult, java.util.Calendar, java.util.Calendar, java.lang.String)' is missing an @Override annotation. | 3 | 321–358 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.String' due to existing implicit import 'java.lang.*' | 4 | 321 |
| CommentRequired | Public method and constructor comments are required | 3 | 321–358 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'com.tvnavig.server.searchengine.SearchAssetServiceException' due to existing same package import 'com.tvnavig.server.searchengine.*' | 4 | 321 |
| CyclomaticComplexity | The method 'getAssetsList(int, java.lang.String, com.tvnavig.server.asset.service.common.MetaValueResult, java.util.Calendar, java.util.Calendar, java.lang.String)' has a cyclomatic complexity of 13. | 3 | 321–358 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.String' due to existing implicit import 'java.lang.*' | 4 | 321 |
| LocalVariableNamingConventions | The local variable name '_call' doesn't match '[a-z][a-zA-Z0-9]*' | 1 | 325 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.Object' due to existing implicit import 'java.lang.*' | 4 | 334 |
| IntegerInstantiation | Avoid instantiating Integer objects. Call Integer.valueOf() instead. | 2 | 334 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.Object' due to existing implicit import 'java.lang.*' | 4 | 334 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.Integer' due to existing implicit import 'java.lang.*' | 4 | 334 |
| LocalVariableNamingConventions | The local variable name '_resp' doesn't match '[a-z][a-zA-Z0-9]*' | 1 | 334 |
| LocalVariableNamingConventions | The exception block parameter name '_exception' doesn't match '[a-z][a-zA-Z0-9]*' | 1 | 343 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.Exception' due to existing implicit import 'java.lang.*' | 4 | 343 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'com.tvnavig.server.searchengine.SearchAssetServiceException' due to existing same package import 'com.tvnavig.server.searchengine.*' | 4 | 352 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'com.tvnavig.server.searchengine.SearchAssetServiceException' due to existing same package import 'com.tvnavig.server.searchengine.*' | 4 | 353 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'com.tvnavig.server.searchengine.SearchAssetServiceException' due to existing same package import 'com.tvnavig.server.searchengine.*' | 4 | 360 |
| MissingOverride | The method 'getAssetPlayer(int)' is missing an @Override annotation. | 3 | 360–397 |
| CommentRequired | Public method and constructor comments are required | 3 | 360–397 |
| CyclomaticComplexity | The method 'getAssetPlayer(int)' has a cyclomatic complexity of 13. | 3 | 360–397 |
| ShortVariable | Avoid variables with short names like id | 3 | 360 |
| LocalVariableNamingConventions | The local variable name '_call' doesn't match '[a-z][a-zA-Z0-9]*' | 1 | 364 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.Object' due to existing implicit import 'java.lang.*' | 4 | 373 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.Integer' due to existing implicit import 'java.lang.*' | 4 | 373 |
| IntegerInstantiation | Avoid instantiating Integer objects. Call Integer.valueOf() instead. | 2 | 373 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.Object' due to existing implicit import 'java.lang.*' | 4 | 373 |
| LocalVariableNamingConventions | The local variable name '_resp' doesn't match '[a-z][a-zA-Z0-9]*' | 1 | 373 |
| LocalVariableNamingConventions | The exception block parameter name '_exception' doesn't match '[a-z][a-zA-Z0-9]*' | 1 | 382 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.Exception' due to existing implicit import 'java.lang.*' | 4 | 382 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'com.tvnavig.server.searchengine.SearchAssetServiceException' due to existing same package import 'com.tvnavig.server.searchengine.*' | 4 | 391 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'com.tvnavig.server.searchengine.SearchAssetServiceException' due to existing same package import 'com.tvnavig.server.searchengine.*' | 4 | 392 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| DataClass | The class 'AssetOGC' is suspected to be a Data Class (WOC=23.077%, NOPA=0, NOAM=18, WMC=45) | 3 | 45–461 |
| FieldNamingConventions | The field name 'assetId' doesn't match '_[a-z][a-zA-Z0-9]*' | 1 | 49 |
| FieldNamingConventions | The field name 'creationDate' doesn't match '_[a-z][a-zA-Z0-9]*' | 1 | 51 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.String' due to existing implicit import 'java.lang.*' | 4 | 53 |
| FieldNamingConventions | The field name 'description' doesn't match '_[a-z][a-zA-Z0-9]*' | 1 | 53 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.String' due to existing implicit import 'java.lang.*' | 4 | 55 |
| FieldNamingConventions | The field name 'formats' doesn't match '_[a-z][a-zA-Z0-9]*' | 1 | 55 |
| FieldNamingConventions | The field name 'metaValues' doesn't match '_[a-z][a-zA-Z0-9]*' | 1 | 57 |
| FieldNamingConventions | The field name 'previewURL' doesn't match '_[a-z][a-zA-Z0-9]*' | 1 | 59 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.String' due to existing implicit import 'java.lang.*' | 4 | 59 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.String' due to existing implicit import 'java.lang.*' | 4 | 61 |
| FieldNamingConventions | The field name 'thumbnail' doesn't match '_[a-z][a-zA-Z0-9]*' | 1 | 61 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.String' due to existing implicit import 'java.lang.*' | 4 | 63 |
| FieldNamingConventions | The field name 'title' doesn't match '_[a-z][a-zA-Z0-9]*' | 1 | 63 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.String' due to existing implicit import 'java.lang.*' | 4 | 65 |
| FieldNamingConventions | The field name 'videoURL' doesn't match '_[a-z][a-zA-Z0-9]*' | 1 | 65 |
| UncommentedEmptyConstructor | Document empty constructor | 3 | 67–68 |
| CommentRequired | Public method and constructor comments are required | 3 | 67–68 |
| CommentRequired | Public method and constructor comments are required | 3 | 70–89 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.String' due to existing implicit import 'java.lang.*' | 4 | 73 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.String' due to existing implicit import 'java.lang.*' | 4 | 74 |
| ArrayIsStoredDirectly | The user-supplied array 'formats' is stored directly. | 3 | 74 |
| ArrayIsStoredDirectly | The user-supplied array 'metaValues' is stored directly. | 3 | 75 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.String' due to existing implicit import 'java.lang.*' | 4 | 76 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.String' due to existing implicit import 'java.lang.*' | 4 | 77 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.String' due to existing implicit import 'java.lang.*' | 4 | 78 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.String' due to existing implicit import 'java.lang.*' | 4 | 79 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.String' due to existing implicit import 'java.lang.*' | 4 | 136 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.String' due to existing implicit import 'java.lang.*' | 4 | 146 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.String' due to existing implicit import 'java.lang.*' | 4 | 156 |
| MethodReturnsInternalArray | Returning 'formats' may expose an internal array. | 3 | 157 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.String' due to existing implicit import 'java.lang.*' | 4 | 166 |
| ArrayIsStoredDirectly | The user-supplied array 'formats' is stored directly. | 3 | 166 |
| UseVarargs | Consider using varargs for methods or constructors which take an array the last parameter. | 4 | 166 |
| MethodReturnsInternalArray | Returning 'metaValues' may expose an internal array. | 3 | 177 |
| UseVarargs | Consider using varargs for methods or constructors which take an array the last parameter. | 4 | 186 |
| ArrayIsStoredDirectly | The user-supplied array 'metaValues' is stored directly. | 3 | 186 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.String' due to existing implicit import 'java.lang.*' | 4 | 196 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.String' due to existing implicit import 'java.lang.*' | 4 | 206 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.String' due to existing implicit import 'java.lang.*' | 4 | 216 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.String' due to existing implicit import 'java.lang.*' | 4 | 226 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.String' due to existing implicit import 'java.lang.*' | 4 | 236 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.String' due to existing implicit import 'java.lang.*' | 4 | 246 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.String' due to existing implicit import 'java.lang.*' | 4 | 256 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.String' due to existing implicit import 'java.lang.*' | 4 | 266 |
| RedundantFieldInitializer | Avoid using redundant field initializer for '__equalsCalc' | 3 | 270 |
| FieldDeclarationsShouldBeAtStartOfClass | Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. | 3 | 270 |
| FieldNamingConventions | The field name '__equalsCalc' doesn't match '_[a-z][a-zA-Z0-9]*' | 1 | 270 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.Object' due to existing implicit import 'java.lang.*' | 4 | 270 |
| MissingOverride | The method 'equals(java.lang.Object)' is missing an @Override annotation. | 3 | 271–309 |
| AvoidSynchronizedAtMethodLevel | Use block level rather than method level synchronization | 3 | 271–309 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.Object' due to existing implicit import 'java.lang.*' | 4 | 271 |
| CommentRequired | Public method and constructor comments are required | 3 | 271–309 |
| ControlStatementBraces | This statement should have braces | 3 | 272 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 3 | 273 |
| ControlStatementBraces | This statement should have braces | 3 | 274 |
| ControlStatementBraces | This statement should have braces | 3 | 275 |
| CompareObjectsWithEquals | Use equals() to compare object references. | 3 | 277 |
| LocalVariableNamingConventions | The local variable name '_equals' doesn't match '[a-z][a-zA-Z0-9]*' | 1 | 280 |
| NullAssignment | Assigning an Object to null is a code smell. Consider refactoring. | 3 | 307 |
| FieldNamingConventions | The field name '__hashCodeCalc' doesn't match '_[a-z][a-zA-Z0-9]*' | 1 | 311 |
| RedundantFieldInitializer | Avoid using redundant field initializer for '__hashCodeCalc' | 3 | 311 |
| FieldDeclarationsShouldBeAtStartOfClass | Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. | 3 | 311 |
| AvoidSynchronizedAtMethodLevel | Use block level rather than method level synchronization | 3 | 312–361 |
| CyclomaticComplexity | The method 'hashCode()' has a cyclomatic complexity of 16. | 3 | 312–361 |
| MissingOverride | The method 'hashCode()' is missing an @Override annotation. | 3 | 312–361 |
| CommentRequired | Public method and constructor comments are required | 3 | 312–361 |
| NPathComplexity | The method 'hashCode()' has an NPath complexity of 3200 | 3 | 312–361 |
| LocalVariableNamingConventions | The local variable name '_hashCode' doesn't match '[a-z][a-zA-Z0-9]*' | 1 | 317 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.Object' due to existing implicit import 'java.lang.*' | 4 | 329 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.reflect.Array.get' due to existing implicit import 'java.lang.*' | 4 | 329 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.reflect.Array.get' due to existing implicit import 'java.lang.*' | 4 | 340 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.Object' due to existing implicit import 'java.lang.*' | 4 | 340 |
| FieldNamingConventions | The static field name 'typeDesc' doesn't match '_[a-z][a-zA-Z0-9]*' | 1 | 364 |
| FieldDeclarationsShouldBeAtStartOfClass | Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. | 3 | 364–365 |
| AvoidDuplicateLiterals | The String literal "http://www.w3.org/2001/XMLSchema" appears 8 times in this file; the first occurrence is on line 372 | 3 | 372 |
| AvoidDuplicateLiterals | The String literal "string" appears 6 times in this file; the first occurrence is on line 384 | 3 | 384 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.String' due to existing implicit import 'java.lang.*' | 4 | 436 |
| FormalParameterNamingConventions | The method parameter name '_javaType' doesn't match '[a-z][a-zA-Z0-9]*' | 1 | 437 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.Class' due to existing implicit import 'java.lang.*' | 4 | 437 |
| FormalParameterNamingConventions | The method parameter name '_xmlType' doesn't match '[a-z][a-zA-Z0-9]*' | 1 | 438 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.String' due to existing implicit import 'java.lang.*' | 4 | 448 |
| FormalParameterNamingConventions | The method parameter name '_javaType' doesn't match '[a-z][a-zA-Z0-9]*' | 1 | 449 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.Class' due to existing implicit import 'java.lang.*' | 4 | 449 |
| FormalParameterNamingConventions | The method parameter name '_xmlType' doesn't match '[a-z][a-zA-Z0-9]*' | 1 | 450 |
| CommentRequired | Public method and constructor comments are required | 3 | 456–459 |
| MissingOverride | The method 'compareTo(AssetOGC)' is missing an @Override annotation. | 3 | 456–459 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| MissingSerialVersionUID | Classes implementing Serializable should set a serialVersionUID | 3 | 43–188 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.String' due to existing implicit import 'java.lang.*' | 4 | 44 |
| FieldNamingConventions | The field name 'htmlEmbed' doesn't match '_[a-z][a-zA-Z0-9]*' | 1 | 44 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.String' due to existing implicit import 'java.lang.*' | 4 | 46 |
| FieldNamingConventions | The field name 'thumbnail' doesn't match '_[a-z][a-zA-Z0-9]*' | 1 | 46 |
| UncommentedEmptyConstructor | Document empty constructor | 3 | 48–49 |
| CommentRequired | Public method and constructor comments are required | 3 | 48–49 |
| CommentRequired | Public method and constructor comments are required | 3 | 51–56 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.String' due to existing implicit import 'java.lang.*' | 4 | 52 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.String' due to existing implicit import 'java.lang.*' | 4 | 53 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.String' due to existing implicit import 'java.lang.*' | 4 | 64 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.String' due to existing implicit import 'java.lang.*' | 4 | 74 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.String' due to existing implicit import 'java.lang.*' | 4 | 84 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.String' due to existing implicit import 'java.lang.*' | 4 | 94 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.Object' due to existing implicit import 'java.lang.*' | 4 | 98 |
| FieldNamingConventions | The field name '__equalsCalc' doesn't match '_[a-z][a-zA-Z0-9]*' | 1 | 98 |
| RedundantFieldInitializer | Avoid using redundant field initializer for '__equalsCalc' | 3 | 98 |
| FieldDeclarationsShouldBeAtStartOfClass | Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. | 3 | 98 |
| CommentRequired | Public method and constructor comments are required | 3 | 99–118 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.Object' due to existing implicit import 'java.lang.*' | 4 | 99 |
| MissingOverride | The method 'equals(java.lang.Object)' is missing an @Override annotation. | 3 | 99–118 |
| AvoidSynchronizedAtMethodLevel | Use block level rather than method level synchronization | 3 | 99–118 |
| ControlStatementBraces | This statement should have braces | 3 | 100 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 3 | 101 |
| ControlStatementBraces | This statement should have braces | 3 | 102 |
| ControlStatementBraces | This statement should have braces | 3 | 103 |
| CompareObjectsWithEquals | Use equals() to compare object references. | 3 | 105 |
| LocalVariableNamingConventions | The local variable name '_equals' doesn't match '[a-z][a-zA-Z0-9]*' | 1 | 108 |
| NullAssignment | Assigning an Object to null is a code smell. Consider refactoring. | 3 | 116 |
| RedundantFieldInitializer | Avoid using redundant field initializer for '__hashCodeCalc' | 3 | 120 |
| FieldDeclarationsShouldBeAtStartOfClass | Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. | 3 | 120 |
| FieldNamingConventions | The field name '__hashCodeCalc' doesn't match '_[a-z][a-zA-Z0-9]*' | 1 | 120 |
| AvoidSynchronizedAtMethodLevel | Use block level rather than method level synchronization | 3 | 121–135 |
| CommentRequired | Public method and constructor comments are required | 3 | 121–135 |
| MissingOverride | The method 'hashCode()' is missing an @Override annotation. | 3 | 121–135 |
| LocalVariableNamingConventions | The local variable name '_hashCode' doesn't match '[a-z][a-zA-Z0-9]*' | 1 | 126 |
| FieldDeclarationsShouldBeAtStartOfClass | Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. | 3 | 138–139 |
| FieldNamingConventions | The static field name 'typeDesc' doesn't match '_[a-z][a-zA-Z0-9]*' | 1 | 138 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.String' due to existing implicit import 'java.lang.*' | 4 | 168 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.Class' due to existing implicit import 'java.lang.*' | 4 | 169 |
| FormalParameterNamingConventions | The method parameter name '_javaType' doesn't match '[a-z][a-zA-Z0-9]*' | 1 | 169 |
| FormalParameterNamingConventions | The method parameter name '_xmlType' doesn't match '[a-z][a-zA-Z0-9]*' | 1 | 170 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.String' due to existing implicit import 'java.lang.*' | 4 | 180 |
| FormalParameterNamingConventions | The method parameter name '_javaType' doesn't match '[a-z][a-zA-Z0-9]*' | 1 | 181 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.Class' due to existing implicit import 'java.lang.*' | 4 | 181 |
| FormalParameterNamingConventions | The method parameter name '_xmlType' doesn't match '[a-z][a-zA-Z0-9]*' | 1 | 182 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| UnnecessaryModifier | Unnecessary modifier 'public' on method 'isAlive': the method is declared in an interface type | 3 | 44 |
| CommentRequired | Public method and constructor comments are required | 3 | 44 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.String' due to existing implicit import 'java.lang.*' | 4 | 45 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.String' due to existing implicit import 'java.lang.*' | 4 | 45 |
| UnnecessaryModifier | Unnecessary modifier 'public' on method 'authenticate': the method is declared in an interface type | 3 | 45 |
| CommentRequired | Public method and constructor comments are required | 3 | 45 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'com.tvnavig.server.user.UserServiceException' due to existing same package import 'com.tvnavig.server.user.*' | 4 | 45 |
| CommentRequired | Public method and constructor comments are required | 3 | 46 |
| UnnecessaryModifier | Unnecessary modifier 'public' on method 'getDomainsFromUser': the method is declared in an interface type | 3 | 46 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'com.tvnavig.server.user.UserServiceException' due to existing same package import 'com.tvnavig.server.user.*' | 4 | 46 |
| CommentRequired | Public method and constructor comments are required | 3 | 47 |
| UnnecessaryModifier | Unnecessary modifier 'public' on method 'getGroupsandMetaDatasFromDomain': the method is declared in an interface type | 3 | 47 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'com.tvnavig.server.user.UserServiceException' due to existing same package import 'com.tvnavig.server.user.*' | 4 | 47 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| MissingSerialVersionUID | Classes implementing Serializable should set a serialVersionUID | 3 | 43–126 |
| CallSuperInConstructor | It is a good practice to call super() in a constructor | 3 | 44–45 |
| CommentRequired | Public method and constructor comments are required | 3 | 44–45 |
| UncommentedEmptyConstructor | Document empty constructor | 3 | 44–45 |
| CommentRequired | Public method and constructor comments are required | 3 | 47–51 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.String' due to existing implicit import 'java.lang.*' | 4 | 48 |
| UseVarargs | Consider using varargs for methods or constructors which take an array the last parameter. | 4 | 48 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.Object' due to existing implicit import 'java.lang.*' | 4 | 53 |
| FieldNamingConventions | The field name '__equalsCalc' doesn't match '_[a-z][a-zA-Z0-9]*' | 1 | 53 |
| FieldDeclarationsShouldBeAtStartOfClass | Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. | 3 | 53 |
| RedundantFieldInitializer | Avoid using redundant field initializer for '__equalsCalc' | 3 | 53 |
| AvoidSynchronizedAtMethodLevel | Use block level rather than method level synchronization | 3 | 54–67 |
| CommentRequired | Public method and constructor comments are required | 3 | 54–67 |
| MissingOverride | The method 'equals(java.lang.Object)' is missing an @Override annotation. | 3 | 54–67 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.Object' due to existing implicit import 'java.lang.*' | 4 | 54 |
| ControlStatementBraces | This statement should have braces | 3 | 55 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 3 | 56 |
| UnusedLocalVariable | Avoid unused local variables such as 'other'. | 3 | 56 |
| ControlStatementBraces | This statement should have braces | 3 | 57 |
| ControlStatementBraces | This statement should have braces | 3 | 58 |
| CompareObjectsWithEquals | Use equals() to compare object references. | 3 | 60 |
| LocalVariableNamingConventions | The local variable name '_equals' doesn't match '[a-z][a-zA-Z0-9]*' | 1 | 63 |
| NullAssignment | Assigning an Object to null is a code smell. Consider refactoring. | 3 | 65 |
| RedundantFieldInitializer | Avoid using redundant field initializer for '__hashCodeCalc' | 3 | 69 |
| FieldDeclarationsShouldBeAtStartOfClass | Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. | 3 | 69 |
| FieldNamingConventions | The field name '__hashCodeCalc' doesn't match '_[a-z][a-zA-Z0-9]*' | 1 | 69 |
| MissingOverride | The method 'hashCode()' is missing an @Override annotation. | 3 | 70–78 |
| CommentRequired | Public method and constructor comments are required | 3 | 70–78 |
| AvoidSynchronizedAtMethodLevel | Use block level rather than method level synchronization | 3 | 70–78 |
| LocalVariableNamingConventions | The local variable name '_hashCode' doesn't match '[a-z][a-zA-Z0-9]*' | 1 | 75 |
| FieldDeclarationsShouldBeAtStartOfClass | Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. | 3 | 81–82 |
| FieldNamingConventions | The static field name 'typeDesc' doesn't match '_[a-z][a-zA-Z0-9]*' | 1 | 81 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.String' due to existing implicit import 'java.lang.*' | 4 | 99 |
| FormalParameterNamingConventions | The method parameter name '_javaType' doesn't match '[a-z][a-zA-Z0-9]*' | 1 | 100 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.Class' due to existing implicit import 'java.lang.*' | 4 | 100 |
| FormalParameterNamingConventions | The method parameter name '_xmlType' doesn't match '[a-z][a-zA-Z0-9]*' | 1 | 101 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.String' due to existing implicit import 'java.lang.*' | 4 | 111 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.Class' due to existing implicit import 'java.lang.*' | 4 | 112 |
| FormalParameterNamingConventions | The method parameter name '_javaType' doesn't match '[a-z][a-zA-Z0-9]*' | 1 | 112 |
| FormalParameterNamingConventions | The method parameter name '_xmlType' doesn't match '[a-z][a-zA-Z0-9]*' | 1 | 113 |
| MissingOverride | The method 'writeDetails(javax.xml.namespace.QName, org.apache.axis.encoding.SerializationContext)' is missing an @Override annotation. | 3 | 123–125 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.String' due to existing implicit import 'java.lang.*' | 4 | 44 |
| CommentRequired | Public method and constructor comments are required | 3 | 44 |
| UnnecessaryModifier | Unnecessary modifier 'public' on method 'getUserServiceAddress': the method is declared in an interface type | 3 | 44 |
| CommentRequired | Public method and constructor comments are required | 3 | 46 |
| UnnecessaryModifier | Unnecessary modifier 'public' on method 'getUserService': the method is declared in an interface type | 3 | 46 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'com.tvnavig.server.user.UserService' due to existing same package import 'com.tvnavig.server.user.*' | 4 | 46 |
| CommentRequired | Public method and constructor comments are required | 3 | 48 |
| UnnecessaryModifier | Unnecessary modifier 'public' on method 'getUserService': the method is declared in an interface type | 3 | 48 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'com.tvnavig.server.user.UserService' due to existing same package import 'com.tvnavig.server.user.*' | 4 | 48 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'com.tvnavig.server.user.UserServiceService' due to existing same package import 'com.tvnavig.server.user.*' | 4 | 43 |
| MissingSerialVersionUID | Classes implementing Serializable should set a serialVersionUID | 3 | 43–175 |
| CommentRequired | Public method and constructor comments are required | 3 | 45–46 |
| UncommentedEmptyConstructor | Document empty constructor | 3 | 45–46 |
| CallSuperInConstructor | It is a good practice to call super() in a constructor | 3 | 45–46 |
| CommentRequired | Public method and constructor comments are required | 3 | 49–51 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.String' due to existing implicit import 'java.lang.*' | 4 | 53 |
| CommentRequired | Public method and constructor comments are required | 3 | 53–55 |
| FieldNamingConventions | The field name 'UserService_address' doesn't match '_[a-z][a-zA-Z0-9]*' | 1 | 58 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.String' due to existing implicit import 'java.lang.*' | 4 | 58 |
| FieldDeclarationsShouldBeAtStartOfClass | Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. | 3 | 58 |
| MissingOverride | The method 'getUserServiceAddress()' is missing an @Override annotation. | 3 | 60–62 |
| CommentRequired | Public method and constructor comments are required | 3 | 60–62 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.String' due to existing implicit import 'java.lang.*' | 4 | 60 |
| FieldDeclarationsShouldBeAtStartOfClass | Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. | 3 | 65 |
| FieldNamingConventions | The field name 'UserServiceWSDDServiceName' doesn't match '_[a-z][a-zA-Z0-9]*' | 1 | 65 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.String' due to existing implicit import 'java.lang.*' | 4 | 65 |
| AvoidDuplicateLiterals | The String literal "UserService" appears 4 times in this file; the first occurrence is on line 65 | 3 | 65 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.String' due to existing implicit import 'java.lang.*' | 4 | 67 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.String' due to existing implicit import 'java.lang.*' | 4 | 71 |
| CommentRequired | Public method and constructor comments are required | 3 | 75–84 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'com.tvnavig.server.user.UserService' due to existing same package import 'com.tvnavig.server.user.*' | 4 | 75 |
| MissingOverride | The method 'getUserService()' is missing an @Override annotation. | 3 | 75–84 |
| CommentRequired | Public method and constructor comments are required | 3 | 86–95 |
| MissingOverride | The method 'getUserService(java.net.URL)' is missing an @Override annotation. | 3 | 86–95 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'com.tvnavig.server.user.UserService' due to existing same package import 'com.tvnavig.server.user.*' | 4 | 86 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'com.tvnavig.server.user.UserServiceSoapBindingStub' due to existing same package import 'com.tvnavig.server.user.*' | 4 | 88 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'com.tvnavig.server.user.UserServiceSoapBindingStub' due to existing same package import 'com.tvnavig.server.user.*' | 4 | 88 |
| LocalVariableNamingConventions | The local variable name '_stub' doesn't match '[a-z][a-zA-Z0-9]*' | 1 | 88 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.String' due to existing implicit import 'java.lang.*' | 4 | 97 |
| CommentRequired | Public method and constructor comments are required | 3 | 97–99 |
| MissingOverride | The method 'getPort(Class)' is missing an @Override annotation. | 3 | 106–118 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'com.tvnavig.server.user.UserService' due to existing same package import 'com.tvnavig.server.user.*' | 4 | 108 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'com.tvnavig.server.user.UserServiceSoapBindingStub' due to existing same package import 'com.tvnavig.server.user.*' | 4 | 109 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'com.tvnavig.server.user.UserServiceSoapBindingStub' due to existing same package import 'com.tvnavig.server.user.*' | 4 | 109 |
| LocalVariableNamingConventions | The local variable name '_stub' doesn't match '[a-z][a-zA-Z0-9]*' | 1 | 109 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.Throwable' due to existing implicit import 'java.lang.*' | 4 | 114 |
| MissingOverride | The method 'getPort(javax.xml.namespace.QName, Class)' is missing an @Override annotation. | 3 | 125–138 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.String' due to existing implicit import 'java.lang.*' | 4 | 129 |
| LocalVariableNamingConventions | The local variable name '_stub' doesn't match '[a-z][a-zA-Z0-9]*' | 1 | 134 |
| CommentRequired | Public method and constructor comments are required | 3 | 140–142 |
| MissingOverride | The method 'getServiceName()' is missing an @Override annotation. | 3 | 140–142 |
| FieldDeclarationsShouldBeAtStartOfClass | Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. | 3 | 144 |
| FieldNamingConventions | The field name 'ports' doesn't match '_[a-z][a-zA-Z0-9]*' | 1 | 144 |
| RedundantFieldInitializer | Avoid using redundant field initializer for 'ports' | 3 | 144 |
| LooseCoupling | Avoid using implementation types like 'java.util.HashSet'; use the interface instead | 3 | 144 |
| MissingOverride | The method 'getPorts()' is missing an @Override annotation. | 3 | 146–152 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.String' due to existing implicit import 'java.lang.*' | 4 | 157 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.String' due to existing implicit import 'java.lang.*' | 4 | 157 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.String' due to existing implicit import 'java.lang.*' | 4 | 171 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'com.tvnavig.server.user.UserService' due to existing same package import 'com.tvnavig.server.user.*' | 4 | 43 |
| UseArrayListInsteadOfVector | Use ArrayList instead of Vector | 3 | 44 |
| FieldNamingConventions | The field name 'cachedSerClasses' doesn't match '_[a-z][a-zA-Z0-9]*' | 1 | 44 |
| ImmutableField | Private field 'cachedSerClasses' could be made final; it is only initialized in the declaration or constructor. | 3 | 44 |
| LooseCoupling | Avoid using implementation types like 'java.util.Vector'; use the interface instead | 3 | 44 |
| UseArrayListInsteadOfVector | Use ArrayList instead of Vector | 3 | 45 |
| ImmutableField | Private field 'cachedSerQNames' could be made final; it is only initialized in the declaration or constructor. | 3 | 45 |
| FieldNamingConventions | The field name 'cachedSerQNames' doesn't match '_[a-z][a-zA-Z0-9]*' | 1 | 45 |
| LooseCoupling | Avoid using implementation types like 'java.util.Vector'; use the interface instead | 3 | 45 |
| UseArrayListInsteadOfVector | Use ArrayList instead of Vector | 3 | 46 |
| FieldNamingConventions | The field name 'cachedSerFactories' doesn't match '_[a-z][a-zA-Z0-9]*' | 1 | 46 |
| ImmutableField | Private field 'cachedSerFactories' could be made final; it is only initialized in the declaration or constructor. | 3 | 46 |
| LooseCoupling | Avoid using implementation types like 'java.util.Vector'; use the interface instead | 3 | 46 |
| ImmutableField | Private field 'cachedDeserFactories' could be made final; it is only initialized in the declaration or constructor. | 3 | 47 |
| UseArrayListInsteadOfVector | Use ArrayList instead of Vector | 3 | 47 |
| LooseCoupling | Avoid using implementation types like 'java.util.Vector'; use the interface instead | 3 | 47 |
| FieldNamingConventions | The field name 'cachedDeserFactories' doesn't match '_[a-z][a-zA-Z0-9]*' | 1 | 47 |
| CommentDefaultAccessModifier | To avoid mistakes add a comment at the beginning of the _operations field if you want a default access modifier | 3 | 49 |
| DefaultPackage | Use explicit scoping instead of the default package private level | 3 | 49 |
| MethodNamingConventions | The static method name '_initOperationDesc1' doesn't match '[a-z][a-zA-Z0-9]*' | 1 | 56–119 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.String' due to existing implicit import 'java.lang.*' | 4 | 68 |
| AvoidDuplicateLiterals | The String literal "http://www.w3.org/2001/XMLSchema" appears 5 times in this file; the first occurrence is on line 68 | 3 | 68 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.String' due to existing implicit import 'java.lang.*' | 4 | 70 |
| AvoidDuplicateLiterals | The String literal "http://common.service.user.server.tvnavig.com" appears 10 times in this file; the first occurrence is on line 72 | 3 | 72 |
| AvoidDuplicateLiterals | The String literal "http://user.server.tvnavig.com" appears 16 times in this file; the first occurrence is on line 78 | 3 | 78 |
| AvoidDuplicateLiterals | The String literal "UserServiceException" appears 4 times in this file; the first occurrence is on line 80 | 3 | 80 |
| CommentRequired | Public method and constructor comments are required | 3 | 121–123 |
| CommentRequired | Public method and constructor comments are required | 3 | 125–128 |
| CallSuperInConstructor | It is a good practice to call super() in a constructor | 3 | 130–235 |
| CommentRequired | Public method and constructor comments are required | 3 | 130–235 |
| ExcessiveMethodLength | Avoid really long methods. | 3 | 130–235 |
| NcssCount | The constructor 'UserServiceSoapBindingStub(javax.xml.rpc.Service)' has a NCSS line count of 93. | 3 | 130–235 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.Class' due to existing implicit import 'java.lang.*' | 4 | 137 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.Class' due to existing implicit import 'java.lang.*' | 4 | 140 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.Class' due to existing implicit import 'java.lang.*' | 4 | 141 |
| UnusedLocalVariable | Avoid unused local variables such as 'enumsf'. | 3 | 142 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.Class' due to existing implicit import 'java.lang.*' | 4 | 142 |
| UnusedLocalVariable | Avoid unused local variables such as 'enumdf'. | 3 | 143 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.Class' due to existing implicit import 'java.lang.*' | 4 | 143 |
| UnusedLocalVariable | Avoid unused local variables such as 'arraysf'. | 3 | 144 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.Class' due to existing implicit import 'java.lang.*' | 4 | 144 |
| UnusedLocalVariable | Avoid unused local variables such as 'arraydf'. | 3 | 145 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.Class' due to existing implicit import 'java.lang.*' | 4 | 145 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.Class' due to existing implicit import 'java.lang.*' | 4 | 146 |
| UnusedLocalVariable | Avoid unused local variables such as 'simplesf'. | 3 | 146 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.Class' due to existing implicit import 'java.lang.*' | 4 | 147 |
| UnusedLocalVariable | Avoid unused local variables such as 'simpledf'. | 3 | 147 |
| UnusedLocalVariable | Avoid unused local variables such as 'simplelistsf'. | 3 | 148 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.Class' due to existing implicit import 'java.lang.*' | 4 | 148 |
| UnusedLocalVariable | Avoid unused local variables such as 'simplelistdf'. | 3 | 149 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.Class' due to existing implicit import 'java.lang.*' | 4 | 149 |
| NullAssignment | Assigning an Object to null is a code smell. Consider refactoring. | 3 | 190 |
| NullAssignment | Assigning an Object to null is a code smell. Consider refactoring. | 3 | 199 |
| NullAssignment | Assigning an Object to null is a code smell. Consider refactoring. | 3 | 208 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.String' due to existing implicit import 'java.lang.*' | 4 | 214 |
| NullAssignment | Assigning an Object to null is a code smell. Consider refactoring. | 3 | 217 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'com.tvnavig.server.user.UserServiceException' due to existing same package import 'com.tvnavig.server.user.*' | 4 | 223 |
| CommentRequired | Protected method constructor comments are required | 3 | 237–300 |
| CyclomaticComplexity | The method 'createCall()' has a cyclomatic complexity of 14. | 3 | 237–300 |
| NPathComplexity | The method 'createCall()' has an NPath complexity of 641 | 3 | 237–300 |
| LocalVariableNamingConventions | The local variable name '_call' doesn't match '[a-z][a-zA-Z0-9]*' | 1 | 239 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.String' due to existing implicit import 'java.lang.*' | 4 | 260 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.String' due to existing implicit import 'java.lang.*' | 4 | 260 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.Class' due to existing implicit import 'java.lang.*' | 4 | 274 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.Class' due to existing implicit import 'java.lang.*' | 4 | 274 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.Object' due to existing implicit import 'java.lang.*' | 4 | 277 |
| ShortVariable | Avoid variables with short names like x | 3 | 277 |
| ShortVariable | Avoid variables with short names like sf | 3 | 279 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.Class' due to existing implicit import 'java.lang.*' | 4 | 279 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.Class' due to existing implicit import 'java.lang.*' | 4 | 279 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.Class' due to existing implicit import 'java.lang.*' | 4 | 281 |
| ShortVariable | Avoid variables with short names like df | 3 | 281 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.Class' due to existing implicit import 'java.lang.*' | 4 | 281 |
| ShortVariable | Avoid variables with short names like sf | 3 | 286 |
| ShortVariable | Avoid variables with short names like df | 3 | 288 |
| LocalVariableNamingConventions | The exception block parameter name '_t' doesn't match '[a-z][a-zA-Z0-9]*' | 1 | 297 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.Throwable' due to existing implicit import 'java.lang.*' | 4 | 297 |
| CommentRequired | Public method and constructor comments are required | 3 | 302–324 |
| MissingOverride | The method 'isAlive()' is missing an @Override annotation. | 3 | 302–324 |
| LocalVariableNamingConventions | The local variable name '_call' doesn't match '[a-z][a-zA-Z0-9]*' | 1 | 306 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.Object' due to existing implicit import 'java.lang.*' | 4 | 315 |
| LocalVariableNamingConventions | The local variable name '_resp' doesn't match '[a-z][a-zA-Z0-9]*' | 1 | 315 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.Object' due to existing implicit import 'java.lang.*' | 4 | 315 |
| AvoidRethrowingException | A catch statement that catches an exception only to rethrow it should be avoided. | 3 | 321–323 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.String' due to existing implicit import 'java.lang.*' | 4 | 326 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.String' due to existing implicit import 'java.lang.*' | 4 | 326 |
| CommentRequired | Public method and constructor comments are required | 3 | 326–363 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'com.tvnavig.server.user.UserServiceException' due to existing same package import 'com.tvnavig.server.user.*' | 4 | 326 |
| CyclomaticComplexity | The method 'authenticate(java.lang.String, java.lang.String)' has a cyclomatic complexity of 13. | 3 | 326–363 |
| MissingOverride | The method 'authenticate(java.lang.String, java.lang.String)' is missing an @Override annotation. | 3 | 326–363 |
| LocalVariableNamingConventions | The local variable name '_call' doesn't match '[a-z][a-zA-Z0-9]*' | 1 | 330 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.Object' due to existing implicit import 'java.lang.*' | 4 | 339 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.Object' due to existing implicit import 'java.lang.*' | 4 | 339 |
| LocalVariableNamingConventions | The local variable name '_resp' doesn't match '[a-z][a-zA-Z0-9]*' | 1 | 339 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.Exception' due to existing implicit import 'java.lang.*' | 4 | 348 |
| LocalVariableNamingConventions | The exception block parameter name '_exception' doesn't match '[a-z][a-zA-Z0-9]*' | 1 | 348 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'com.tvnavig.server.user.UserServiceException' due to existing same package import 'com.tvnavig.server.user.*' | 4 | 357 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'com.tvnavig.server.user.UserServiceException' due to existing same package import 'com.tvnavig.server.user.*' | 4 | 358 |
| CommentRequired | Public method and constructor comments are required | 3 | 365–402 |
| CyclomaticComplexity | The method 'getDomainsFromUser(int)' has a cyclomatic complexity of 13. | 3 | 365–402 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'com.tvnavig.server.user.UserServiceException' due to existing same package import 'com.tvnavig.server.user.*' | 4 | 365 |
| MissingOverride | The method 'getDomainsFromUser(int)' is missing an @Override annotation. | 3 | 365–402 |
| LocalVariableNamingConventions | The local variable name '_call' doesn't match '[a-z][a-zA-Z0-9]*' | 1 | 369 |
| IntegerInstantiation | Avoid instantiating Integer objects. Call Integer.valueOf() instead. | 2 | 378 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.Integer' due to existing implicit import 'java.lang.*' | 4 | 378 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.Object' due to existing implicit import 'java.lang.*' | 4 | 378 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.Object' due to existing implicit import 'java.lang.*' | 4 | 378 |
| LocalVariableNamingConventions | The local variable name '_resp' doesn't match '[a-z][a-zA-Z0-9]*' | 1 | 378 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.Exception' due to existing implicit import 'java.lang.*' | 4 | 387 |
| LocalVariableNamingConventions | The exception block parameter name '_exception' doesn't match '[a-z][a-zA-Z0-9]*' | 1 | 387 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'com.tvnavig.server.user.UserServiceException' due to existing same package import 'com.tvnavig.server.user.*' | 4 | 396 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'com.tvnavig.server.user.UserServiceException' due to existing same package import 'com.tvnavig.server.user.*' | 4 | 397 |
| CyclomaticComplexity | The method 'getGroupsandMetaDatasFromDomain(int)' has a cyclomatic complexity of 13. | 3 | 404–441 |
| CommentRequired | Public method and constructor comments are required | 3 | 404–441 |
| MissingOverride | The method 'getGroupsandMetaDatasFromDomain(int)' is missing an @Override annotation. | 3 | 404–441 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'com.tvnavig.server.user.UserServiceException' due to existing same package import 'com.tvnavig.server.user.*' | 4 | 404 |
| LocalVariableNamingConventions | The local variable name '_call' doesn't match '[a-z][a-zA-Z0-9]*' | 1 | 408 |
| IntegerInstantiation | Avoid instantiating Integer objects. Call Integer.valueOf() instead. | 2 | 417 |
| LocalVariableNamingConventions | The local variable name '_resp' doesn't match '[a-z][a-zA-Z0-9]*' | 1 | 417 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.Object' due to existing implicit import 'java.lang.*' | 4 | 417 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.Object' due to existing implicit import 'java.lang.*' | 4 | 417 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.Integer' due to existing implicit import 'java.lang.*' | 4 | 417 |
| LocalVariableNamingConventions | The exception block parameter name '_exception' doesn't match '[a-z][a-zA-Z0-9]*' | 1 | 426 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.Exception' due to existing implicit import 'java.lang.*' | 4 | 426 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'com.tvnavig.server.user.UserServiceException' due to existing same package import 'com.tvnavig.server.user.*' | 4 | 435 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'com.tvnavig.server.user.UserServiceException' due to existing same package import 'com.tvnavig.server.user.*' | 4 | 436 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| MissingSerialVersionUID | Classes implementing Serializable should set a serialVersionUID | 3 | 43–260 |
| FieldNamingConventions | The field name 'metadataId' doesn't match '_[a-z][a-zA-Z0-9]*' | 1 | 44 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.String' due to existing implicit import 'java.lang.*' | 4 | 46 |
| FieldNamingConventions | The field name 'name' doesn't match '_[a-z][a-zA-Z0-9]*' | 1 | 46 |
| FieldNamingConventions | The field name 'type' doesn't match '_[a-z][a-zA-Z0-9]*' | 1 | 48 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.String' due to existing implicit import 'java.lang.*' | 4 | 50 |
| FieldNamingConventions | The field name 'valueList' doesn't match '_[a-z][a-zA-Z0-9]*' | 1 | 50 |
| CommentRequired | Public method and constructor comments are required | 3 | 52–53 |
| UncommentedEmptyConstructor | Document empty constructor | 3 | 52–53 |
| CommentRequired | Public method and constructor comments are required | 3 | 55–64 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.String' due to existing implicit import 'java.lang.*' | 4 | 57 |
| ArrayIsStoredDirectly | The user-supplied array 'valueList' is stored directly. | 3 | 59 |
| UseVarargs | Consider using varargs for methods or constructors which take an array the last parameter. | 4 | 59 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.String' due to existing implicit import 'java.lang.*' | 4 | 59 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.String' due to existing implicit import 'java.lang.*' | 4 | 92 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.String' due to existing implicit import 'java.lang.*' | 4 | 102 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.String' due to existing implicit import 'java.lang.*' | 4 | 132 |
| MethodReturnsInternalArray | Returning 'valueList' may expose an internal array. | 3 | 133 |
| ArrayIsStoredDirectly | The user-supplied array 'valueList' is stored directly. | 3 | 142 |
| UseVarargs | Consider using varargs for methods or constructors which take an array the last parameter. | 4 | 142 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.String' due to existing implicit import 'java.lang.*' | 4 | 142 |
| FieldNamingConventions | The field name '__equalsCalc' doesn't match '_[a-z][a-zA-Z0-9]*' | 1 | 146 |
| RedundantFieldInitializer | Avoid using redundant field initializer for '__equalsCalc' | 3 | 146 |
| FieldDeclarationsShouldBeAtStartOfClass | Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. | 3 | 146 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.Object' due to existing implicit import 'java.lang.*' | 4 | 146 |
| AvoidSynchronizedAtMethodLevel | Use block level rather than method level synchronization | 3 | 147–168 |
| MissingOverride | The method 'equals(java.lang.Object)' is missing an @Override annotation. | 3 | 147–168 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.Object' due to existing implicit import 'java.lang.*' | 4 | 147 |
| CommentRequired | Public method and constructor comments are required | 3 | 147–168 |
| ControlStatementBraces | This statement should have braces | 3 | 148 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 3 | 149 |
| ControlStatementBraces | This statement should have braces | 3 | 150 |
| ControlStatementBraces | This statement should have braces | 3 | 151 |
| CompareObjectsWithEquals | Use equals() to compare object references. | 3 | 153 |
| LocalVariableNamingConventions | The local variable name '_equals' doesn't match '[a-z][a-zA-Z0-9]*' | 1 | 156 |
| NullAssignment | Assigning an Object to null is a code smell. Consider refactoring. | 3 | 166 |
| FieldNamingConventions | The field name '__hashCodeCalc' doesn't match '_[a-z][a-zA-Z0-9]*' | 1 | 170 |
| RedundantFieldInitializer | Avoid using redundant field initializer for '__hashCodeCalc' | 3 | 170 |
| FieldDeclarationsShouldBeAtStartOfClass | Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. | 3 | 170 |
| MissingOverride | The method 'hashCode()' is missing an @Override annotation. | 3 | 171–195 |
| AvoidSynchronizedAtMethodLevel | Use block level rather than method level synchronization | 3 | 171–195 |
| CommentRequired | Public method and constructor comments are required | 3 | 171–195 |
| LocalVariableNamingConventions | The local variable name '_hashCode' doesn't match '[a-z][a-zA-Z0-9]*' | 1 | 176 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.reflect.Array.get' due to existing implicit import 'java.lang.*' | 4 | 186 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.Object' due to existing implicit import 'java.lang.*' | 4 | 186 |
| FieldDeclarationsShouldBeAtStartOfClass | Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. | 3 | 198–199 |
| FieldNamingConventions | The static field name 'typeDesc' doesn't match '_[a-z][a-zA-Z0-9]*' | 1 | 198 |
| AvoidDuplicateLiterals | The String literal "http://www.w3.org/2001/XMLSchema" appears 4 times in this file; the first occurrence is on line 206 | 3 | 206 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.String' due to existing implicit import 'java.lang.*' | 4 | 240 |
| FormalParameterNamingConventions | The method parameter name '_javaType' doesn't match '[a-z][a-zA-Z0-9]*' | 1 | 241 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.Class' due to existing implicit import 'java.lang.*' | 4 | 241 |
| FormalParameterNamingConventions | The method parameter name '_xmlType' doesn't match '[a-z][a-zA-Z0-9]*' | 1 | 242 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.String' due to existing implicit import 'java.lang.*' | 4 | 252 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.Class' due to existing implicit import 'java.lang.*' | 4 | 253 |
| FormalParameterNamingConventions | The method parameter name '_javaType' doesn't match '[a-z][a-zA-Z0-9]*' | 1 | 253 |
| FormalParameterNamingConventions | The method parameter name '_xmlType' doesn't match '[a-z][a-zA-Z0-9]*' | 1 | 254 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| MissingSerialVersionUID | Classes implementing Serializable should set a serialVersionUID | 3 | 43–188 |
| ShortVariable | Avoid variables with short names like id | 3 | 44 |
| FieldNamingConventions | The field name 'id' doesn't match '_[a-z][a-zA-Z0-9]*' | 1 | 44 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.Integer' due to existing implicit import 'java.lang.*' | 4 | 44 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.String' due to existing implicit import 'java.lang.*' | 4 | 46 |
| FieldNamingConventions | The field name 'name' doesn't match '_[a-z][a-zA-Z0-9]*' | 1 | 46 |
| CommentRequired | Public method and constructor comments are required | 3 | 48–49 |
| UncommentedEmptyConstructor | Document empty constructor | 3 | 48–49 |
| CommentRequired | Public method and constructor comments are required | 3 | 51–56 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.Integer' due to existing implicit import 'java.lang.*' | 4 | 52 |
| ShortVariable | Avoid variables with short names like id | 3 | 52 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.String' due to existing implicit import 'java.lang.*' | 4 | 53 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.Integer' due to existing implicit import 'java.lang.*' | 4 | 64 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.Integer' due to existing implicit import 'java.lang.*' | 4 | 74 |
| ShortVariable | Avoid variables with short names like id | 3 | 74 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.String' due to existing implicit import 'java.lang.*' | 4 | 84 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.String' due to existing implicit import 'java.lang.*' | 4 | 94 |
| RedundantFieldInitializer | Avoid using redundant field initializer for '__equalsCalc' | 3 | 98 |
| FieldDeclarationsShouldBeAtStartOfClass | Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. | 3 | 98 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.Object' due to existing implicit import 'java.lang.*' | 4 | 98 |
| FieldNamingConventions | The field name '__equalsCalc' doesn't match '_[a-z][a-zA-Z0-9]*' | 1 | 98 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.Object' due to existing implicit import 'java.lang.*' | 4 | 99 |
| CommentRequired | Public method and constructor comments are required | 3 | 99–118 |
| MissingOverride | The method 'equals(java.lang.Object)' is missing an @Override annotation. | 3 | 99–118 |
| AvoidSynchronizedAtMethodLevel | Use block level rather than method level synchronization | 3 | 99–118 |
| ControlStatementBraces | This statement should have braces | 3 | 100 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 3 | 101 |
| ControlStatementBraces | This statement should have braces | 3 | 102 |
| ControlStatementBraces | This statement should have braces | 3 | 103 |
| CompareObjectsWithEquals | Use equals() to compare object references. | 3 | 105 |
| LocalVariableNamingConventions | The local variable name '_equals' doesn't match '[a-z][a-zA-Z0-9]*' | 1 | 108 |
| NullAssignment | Assigning an Object to null is a code smell. Consider refactoring. | 3 | 116 |
| RedundantFieldInitializer | Avoid using redundant field initializer for '__hashCodeCalc' | 3 | 120 |
| FieldDeclarationsShouldBeAtStartOfClass | Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. | 3 | 120 |
| FieldNamingConventions | The field name '__hashCodeCalc' doesn't match '_[a-z][a-zA-Z0-9]*' | 1 | 120 |
| AvoidSynchronizedAtMethodLevel | Use block level rather than method level synchronization | 3 | 121–135 |
| MissingOverride | The method 'hashCode()' is missing an @Override annotation. | 3 | 121–135 |
| CommentRequired | Public method and constructor comments are required | 3 | 121–135 |
| LocalVariableNamingConventions | The local variable name '_hashCode' doesn't match '[a-z][a-zA-Z0-9]*' | 1 | 126 |
| FieldDeclarationsShouldBeAtStartOfClass | Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. | 3 | 138–139 |
| FieldNamingConventions | The static field name 'typeDesc' doesn't match '_[a-z][a-zA-Z0-9]*' | 1 | 138 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.String' due to existing implicit import 'java.lang.*' | 4 | 168 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.Class' due to existing implicit import 'java.lang.*' | 4 | 169 |
| FormalParameterNamingConventions | The method parameter name '_javaType' doesn't match '[a-z][a-zA-Z0-9]*' | 1 | 169 |
| FormalParameterNamingConventions | The method parameter name '_xmlType' doesn't match '[a-z][a-zA-Z0-9]*' | 1 | 170 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.String' due to existing implicit import 'java.lang.*' | 4 | 180 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.Class' due to existing implicit import 'java.lang.*' | 4 | 181 |
| FormalParameterNamingConventions | The method parameter name '_javaType' doesn't match '[a-z][a-zA-Z0-9]*' | 1 | 181 |
| FormalParameterNamingConventions | The method parameter name '_xmlType' doesn't match '[a-z][a-zA-Z0-9]*' | 1 | 182 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| MissingSerialVersionUID | Classes implementing Serializable should set a serialVersionUID | 3 | 43–224 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.String' due to existing implicit import 'java.lang.*' | 4 | 44 |
| FieldNamingConventions | The field name 'description' doesn't match '_[a-z][a-zA-Z0-9]*' | 1 | 44 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.Integer' due to existing implicit import 'java.lang.*' | 4 | 46 |
| FieldNamingConventions | The field name 'groupId' doesn't match '_[a-z][a-zA-Z0-9]*' | 1 | 46 |
| FieldNamingConventions | The field name 'name' doesn't match '_[a-z][a-zA-Z0-9]*' | 1 | 48 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.String' due to existing implicit import 'java.lang.*' | 4 | 48 |
| CommentRequired | Public method and constructor comments are required | 3 | 50–51 |
| UncommentedEmptyConstructor | Document empty constructor | 3 | 50–51 |
| CommentRequired | Public method and constructor comments are required | 3 | 53–60 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.String' due to existing implicit import 'java.lang.*' | 4 | 54 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.Integer' due to existing implicit import 'java.lang.*' | 4 | 55 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.String' due to existing implicit import 'java.lang.*' | 4 | 56 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.String' due to existing implicit import 'java.lang.*' | 4 | 68 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.String' due to existing implicit import 'java.lang.*' | 4 | 78 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.Integer' due to existing implicit import 'java.lang.*' | 4 | 88 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.Integer' due to existing implicit import 'java.lang.*' | 4 | 98 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.String' due to existing implicit import 'java.lang.*' | 4 | 108 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.String' due to existing implicit import 'java.lang.*' | 4 | 118 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.Object' due to existing implicit import 'java.lang.*' | 4 | 122 |
| RedundantFieldInitializer | Avoid using redundant field initializer for '__equalsCalc' | 3 | 122 |
| FieldDeclarationsShouldBeAtStartOfClass | Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. | 3 | 122 |
| FieldNamingConventions | The field name '__equalsCalc' doesn't match '_[a-z][a-zA-Z0-9]*' | 1 | 122 |
| CommentRequired | Public method and constructor comments are required | 3 | 123–145 |
| MissingOverride | The method 'equals(java.lang.Object)' is missing an @Override annotation. | 3 | 123–145 |
| AvoidSynchronizedAtMethodLevel | Use block level rather than method level synchronization | 3 | 123–145 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.Object' due to existing implicit import 'java.lang.*' | 4 | 123 |
| ControlStatementBraces | This statement should have braces | 3 | 124 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 3 | 125 |
| ControlStatementBraces | This statement should have braces | 3 | 126 |
| ControlStatementBraces | This statement should have braces | 3 | 127 |
| CompareObjectsWithEquals | Use equals() to compare object references. | 3 | 129 |
| LocalVariableNamingConventions | The local variable name '_equals' doesn't match '[a-z][a-zA-Z0-9]*' | 1 | 132 |
| NullAssignment | Assigning an Object to null is a code smell. Consider refactoring. | 3 | 143 |
| FieldDeclarationsShouldBeAtStartOfClass | Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. | 3 | 147 |
| RedundantFieldInitializer | Avoid using redundant field initializer for '__hashCodeCalc' | 3 | 147 |
| FieldNamingConventions | The field name '__hashCodeCalc' doesn't match '_[a-z][a-zA-Z0-9]*' | 1 | 147 |
| AvoidSynchronizedAtMethodLevel | Use block level rather than method level synchronization | 3 | 148–165 |
| CommentRequired | Public method and constructor comments are required | 3 | 148–165 |
| MissingOverride | The method 'hashCode()' is missing an @Override annotation. | 3 | 148–165 |
| LocalVariableNamingConventions | The local variable name '_hashCode' doesn't match '[a-z][a-zA-Z0-9]*' | 1 | 153 |
| FieldDeclarationsShouldBeAtStartOfClass | Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. | 3 | 168–169 |
| FieldNamingConventions | The static field name 'typeDesc' doesn't match '_[a-z][a-zA-Z0-9]*' | 1 | 168 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.String' due to existing implicit import 'java.lang.*' | 4 | 204 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.Class' due to existing implicit import 'java.lang.*' | 4 | 205 |
| FormalParameterNamingConventions | The method parameter name '_javaType' doesn't match '[a-z][a-zA-Z0-9]*' | 1 | 205 |
| FormalParameterNamingConventions | The method parameter name '_xmlType' doesn't match '[a-z][a-zA-Z0-9]*' | 1 | 206 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.String' due to existing implicit import 'java.lang.*' | 4 | 216 |
| FormalParameterNamingConventions | The method parameter name '_javaType' doesn't match '[a-z][a-zA-Z0-9]*' | 1 | 217 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.Class' due to existing implicit import 'java.lang.*' | 4 | 217 |
| FormalParameterNamingConventions | The method parameter name '_xmlType' doesn't match '[a-z][a-zA-Z0-9]*' | 1 | 218 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| MissingSerialVersionUID | Classes implementing Serializable should set a serialVersionUID | 3 | 43–204 |
| FieldNamingConventions | The field name 'metaDataList' doesn't match '_[a-z][a-zA-Z0-9]*' | 1 | 44 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'com.tvnavig.server.user.service.common.MetaDataResult' due to existing same package import 'com.tvnavig.server.user.service.common.*' | 4 | 44 |
| FieldNamingConventions | The field name 'userGroupList' doesn't match '_[a-z][a-zA-Z0-9]*' | 1 | 46 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'com.tvnavig.server.user.service.common.UserGroupResult' due to existing same package import 'com.tvnavig.server.user.service.common.*' | 4 | 46 |
| UncommentedEmptyConstructor | Document empty constructor | 3 | 48–49 |
| CommentRequired | Public method and constructor comments are required | 3 | 48–49 |
| CommentRequired | Public method and constructor comments are required | 3 | 51–56 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'com.tvnavig.server.user.service.common.MetaDataResult' due to existing same package import 'com.tvnavig.server.user.service.common.*' | 4 | 52 |
| ArrayIsStoredDirectly | The user-supplied array 'metaDataList' is stored directly. | 3 | 52 |
| ArrayIsStoredDirectly | The user-supplied array 'userGroupList' is stored directly. | 3 | 53 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'com.tvnavig.server.user.service.common.UserGroupResult' due to existing same package import 'com.tvnavig.server.user.service.common.*' | 4 | 53 |
| UseVarargs | Consider using varargs for methods or constructors which take an array the last parameter. | 4 | 53 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'com.tvnavig.server.user.service.common.MetaDataResult' due to existing same package import 'com.tvnavig.server.user.service.common.*' | 4 | 64 |
| MethodReturnsInternalArray | Returning 'metaDataList' may expose an internal array. | 3 | 65 |
| UseVarargs | Consider using varargs for methods or constructors which take an array the last parameter. | 4 | 74 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'com.tvnavig.server.user.service.common.MetaDataResult' due to existing same package import 'com.tvnavig.server.user.service.common.*' | 4 | 74 |
| ArrayIsStoredDirectly | The user-supplied array 'metaDataList' is stored directly. | 3 | 74 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'com.tvnavig.server.user.service.common.UserGroupResult' due to existing same package import 'com.tvnavig.server.user.service.common.*' | 4 | 84 |
| MethodReturnsInternalArray | Returning 'userGroupList' may expose an internal array. | 3 | 85 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'com.tvnavig.server.user.service.common.UserGroupResult' due to existing same package import 'com.tvnavig.server.user.service.common.*' | 4 | 94 |
| UseVarargs | Consider using varargs for methods or constructors which take an array the last parameter. | 4 | 94 |
| ArrayIsStoredDirectly | The user-supplied array 'userGroupList' is stored directly. | 3 | 94 |
| RedundantFieldInitializer | Avoid using redundant field initializer for '__equalsCalc' | 3 | 98 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.Object' due to existing implicit import 'java.lang.*' | 4 | 98 |
| FieldDeclarationsShouldBeAtStartOfClass | Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. | 3 | 98 |
| FieldNamingConventions | The field name '__equalsCalc' doesn't match '_[a-z][a-zA-Z0-9]*' | 1 | 98 |
| AvoidSynchronizedAtMethodLevel | Use block level rather than method level synchronization | 3 | 99–118 |
| MissingOverride | The method 'equals(java.lang.Object)' is missing an @Override annotation. | 3 | 99–118 |
| CommentRequired | Public method and constructor comments are required | 3 | 99–118 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.Object' due to existing implicit import 'java.lang.*' | 4 | 99 |
| ControlStatementBraces | This statement should have braces | 3 | 100 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 3 | 101 |
| ControlStatementBraces | This statement should have braces | 3 | 102 |
| ControlStatementBraces | This statement should have braces | 3 | 103 |
| CompareObjectsWithEquals | Use equals() to compare object references. | 3 | 105 |
| LocalVariableNamingConventions | The local variable name '_equals' doesn't match '[a-z][a-zA-Z0-9]*' | 1 | 108 |
| NullAssignment | Assigning an Object to null is a code smell. Consider refactoring. | 3 | 116 |
| RedundantFieldInitializer | Avoid using redundant field initializer for '__hashCodeCalc' | 3 | 120 |
| FieldDeclarationsShouldBeAtStartOfClass | Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. | 3 | 120 |
| FieldNamingConventions | The field name '__hashCodeCalc' doesn't match '_[a-z][a-zA-Z0-9]*' | 1 | 120 |
| CommentRequired | Public method and constructor comments are required | 3 | 121–151 |
| MissingOverride | The method 'hashCode()' is missing an @Override annotation. | 3 | 121–151 |
| AvoidSynchronizedAtMethodLevel | Use block level rather than method level synchronization | 3 | 121–151 |
| CyclomaticComplexity | The method 'hashCode()' has a cyclomatic complexity of 10. | 3 | 121–151 |
| LocalVariableNamingConventions | The local variable name '_hashCode' doesn't match '[a-z][a-zA-Z0-9]*' | 1 | 126 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.Object' due to existing implicit import 'java.lang.*' | 4 | 131 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.reflect.Array.get' due to existing implicit import 'java.lang.*' | 4 | 131 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.reflect.Array.get' due to existing implicit import 'java.lang.*' | 4 | 142 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.Object' due to existing implicit import 'java.lang.*' | 4 | 142 |
| FieldDeclarationsShouldBeAtStartOfClass | Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. | 3 | 154–155 |
| FieldNamingConventions | The static field name 'typeDesc' doesn't match '_[a-z][a-zA-Z0-9]*' | 1 | 154 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.String' due to existing implicit import 'java.lang.*' | 4 | 184 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.Class' due to existing implicit import 'java.lang.*' | 4 | 185 |
| FormalParameterNamingConventions | The method parameter name '_javaType' doesn't match '[a-z][a-zA-Z0-9]*' | 1 | 185 |
| FormalParameterNamingConventions | The method parameter name '_xmlType' doesn't match '[a-z][a-zA-Z0-9]*' | 1 | 186 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.String' due to existing implicit import 'java.lang.*' | 4 | 196 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.Class' due to existing implicit import 'java.lang.*' | 4 | 197 |
| FormalParameterNamingConventions | The method parameter name '_javaType' doesn't match '[a-z][a-zA-Z0-9]*' | 1 | 197 |
| FormalParameterNamingConventions | The method parameter name '_xmlType' doesn't match '[a-z][a-zA-Z0-9]*' | 1 | 198 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| DataClass | The class 'UserServiceResult' is suspected to be a Data Class (WOC=29.412%, NOPA=0, NOAM=10, WMC=27) | 3 | 43–296 |
| MissingSerialVersionUID | Classes implementing Serializable should set a serialVersionUID | 3 | 43–296 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.String' due to existing implicit import 'java.lang.*' | 4 | 44 |
| FieldNamingConventions | The field name 'email' doesn't match '_[a-z][a-zA-Z0-9]*' | 1 | 44 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.String' due to existing implicit import 'java.lang.*' | 4 | 46 |
| FieldNamingConventions | The field name 'firstName' doesn't match '_[a-z][a-zA-Z0-9]*' | 1 | 46 |
| FieldNamingConventions | The field name 'lastName' doesn't match '_[a-z][a-zA-Z0-9]*' | 1 | 48 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.String' due to existing implicit import 'java.lang.*' | 4 | 48 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.String' due to existing implicit import 'java.lang.*' | 4 | 50 |
| FieldNamingConventions | The field name 'login' doesn't match '_[a-z][a-zA-Z0-9]*' | 1 | 50 |
| FieldNamingConventions | The field name 'userId' doesn't match '_[a-z][a-zA-Z0-9]*' | 1 | 52 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.Integer' due to existing implicit import 'java.lang.*' | 4 | 52 |
| CommentRequired | Public method and constructor comments are required | 3 | 54–55 |
| UncommentedEmptyConstructor | Document empty constructor | 3 | 54–55 |
| CommentRequired | Public method and constructor comments are required | 3 | 57–68 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.String' due to existing implicit import 'java.lang.*' | 4 | 58 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.String' due to existing implicit import 'java.lang.*' | 4 | 59 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.String' due to existing implicit import 'java.lang.*' | 4 | 60 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.String' due to existing implicit import 'java.lang.*' | 4 | 61 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.Integer' due to existing implicit import 'java.lang.*' | 4 | 62 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.String' due to existing implicit import 'java.lang.*' | 4 | 76 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.String' due to existing implicit import 'java.lang.*' | 4 | 86 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.String' due to existing implicit import 'java.lang.*' | 4 | 96 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.String' due to existing implicit import 'java.lang.*' | 4 | 106 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.String' due to existing implicit import 'java.lang.*' | 4 | 116 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.String' due to existing implicit import 'java.lang.*' | 4 | 126 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.String' due to existing implicit import 'java.lang.*' | 4 | 136 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.String' due to existing implicit import 'java.lang.*' | 4 | 146 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.Integer' due to existing implicit import 'java.lang.*' | 4 | 156 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.Integer' due to existing implicit import 'java.lang.*' | 4 | 166 |
| FieldDeclarationsShouldBeAtStartOfClass | Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. | 3 | 170 |
| RedundantFieldInitializer | Avoid using redundant field initializer for '__equalsCalc' | 3 | 170 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.Object' due to existing implicit import 'java.lang.*' | 4 | 170 |
| FieldNamingConventions | The field name '__equalsCalc' doesn't match '_[a-z][a-zA-Z0-9]*' | 1 | 170 |
| AvoidSynchronizedAtMethodLevel | Use block level rather than method level synchronization | 3 | 171–199 |
| MissingOverride | The method 'equals(java.lang.Object)' is missing an @Override annotation. | 3 | 171–199 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.Object' due to existing implicit import 'java.lang.*' | 4 | 171 |
| CommentRequired | Public method and constructor comments are required | 3 | 171–199 |
| ControlStatementBraces | This statement should have braces | 3 | 172 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 3 | 173 |
| ControlStatementBraces | This statement should have braces | 3 | 174 |
| ControlStatementBraces | This statement should have braces | 3 | 175 |
| CompareObjectsWithEquals | Use equals() to compare object references. | 3 | 177 |
| LocalVariableNamingConventions | The local variable name '_equals' doesn't match '[a-z][a-zA-Z0-9]*' | 1 | 180 |
| NullAssignment | Assigning an Object to null is a code smell. Consider refactoring. | 3 | 197 |
| RedundantFieldInitializer | Avoid using redundant field initializer for '__hashCodeCalc' | 3 | 201 |
| FieldDeclarationsShouldBeAtStartOfClass | Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. | 3 | 201 |
| FieldNamingConventions | The field name '__hashCodeCalc' doesn't match '_[a-z][a-zA-Z0-9]*' | 1 | 201 |
| MissingOverride | The method 'hashCode()' is missing an @Override annotation. | 3 | 202–225 |
| CommentRequired | Public method and constructor comments are required | 3 | 202–225 |
| AvoidSynchronizedAtMethodLevel | Use block level rather than method level synchronization | 3 | 202–225 |
| LocalVariableNamingConventions | The local variable name '_hashCode' doesn't match '[a-z][a-zA-Z0-9]*' | 1 | 207 |
| FieldDeclarationsShouldBeAtStartOfClass | Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. | 3 | 228–229 |
| FieldNamingConventions | The static field name 'typeDesc' doesn't match '_[a-z][a-zA-Z0-9]*' | 1 | 228 |
| AvoidDuplicateLiterals | The String literal "string" appears 4 times in this file; the first occurrence is on line 236 | 3 | 236 |
| AvoidDuplicateLiterals | The String literal "http://www.w3.org/2001/XMLSchema" appears 5 times in this file; the first occurrence is on line 236 | 3 | 236 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.String' due to existing implicit import 'java.lang.*' | 4 | 276 |
| FormalParameterNamingConventions | The method parameter name '_javaType' doesn't match '[a-z][a-zA-Z0-9]*' | 1 | 277 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.Class' due to existing implicit import 'java.lang.*' | 4 | 277 |
| FormalParameterNamingConventions | The method parameter name '_xmlType' doesn't match '[a-z][a-zA-Z0-9]*' | 1 | 278 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.String' due to existing implicit import 'java.lang.*' | 4 | 288 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.Class' due to existing implicit import 'java.lang.*' | 4 | 289 |
| FormalParameterNamingConventions | The method parameter name '_javaType' doesn't match '[a-z][a-zA-Z0-9]*' | 1 | 289 |
| FormalParameterNamingConventions | The method parameter name '_xmlType' doesn't match '[a-z][a-zA-Z0-9]*' | 1 | 290 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| MissingSerialVersionUID | Classes implementing Serializable should set a serialVersionUID | 3 | 43–167 |
| FieldNamingConventions | The field name 'error' doesn't match '_[a-z][a-zA-Z0-9]*' | 1 | 44 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.String' due to existing implicit import 'java.lang.*' | 4 | 44 |
| UncommentedEmptyConstructor | Document empty constructor | 3 | 46–47 |
| CallSuperInConstructor | It is a good practice to call super() in a constructor | 3 | 46–47 |
| CommentRequired | Public method and constructor comments are required | 3 | 46–47 |
| CommentRequired | Public method and constructor comments are required | 3 | 49–52 |
| CallSuperInConstructor | It is a good practice to call super() in a constructor | 3 | 49–52 |
| ArrayIsStoredDirectly | The user-supplied array 'error' is stored directly. | 3 | 50 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.String' due to existing implicit import 'java.lang.*' | 4 | 50 |
| UseVarargs | Consider using varargs for methods or constructors which take an array the last parameter. | 4 | 50 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.String' due to existing implicit import 'java.lang.*' | 4 | 60 |
| MethodReturnsInternalArray | Returning 'error' may expose an internal array. | 3 | 61 |
| ArrayIsStoredDirectly | The user-supplied array 'error' is stored directly. | 3 | 70 |
| UseVarargs | Consider using varargs for methods or constructors which take an array the last parameter. | 4 | 70 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.String' due to existing implicit import 'java.lang.*' | 4 | 70 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.Object' due to existing implicit import 'java.lang.*' | 4 | 74 |
| FieldDeclarationsShouldBeAtStartOfClass | Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. | 3 | 74 |
| RedundantFieldInitializer | Avoid using redundant field initializer for '__equalsCalc' | 3 | 74 |
| FieldNamingConventions | The field name '__equalsCalc' doesn't match '_[a-z][a-zA-Z0-9]*' | 1 | 74 |
| AvoidSynchronizedAtMethodLevel | Use block level rather than method level synchronization | 3 | 75–91 |
| CommentRequired | Public method and constructor comments are required | 3 | 75–91 |
| MissingOverride | The method 'equals(java.lang.Object)' is missing an @Override annotation. | 3 | 75–91 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.Object' due to existing implicit import 'java.lang.*' | 4 | 75 |
| ControlStatementBraces | This statement should have braces | 3 | 76 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 3 | 77 |
| ControlStatementBraces | This statement should have braces | 3 | 78 |
| ControlStatementBraces | This statement should have braces | 3 | 79 |
| CompareObjectsWithEquals | Use equals() to compare object references. | 3 | 81 |
| LocalVariableNamingConventions | The local variable name '_equals' doesn't match '[a-z][a-zA-Z0-9]*' | 1 | 84 |
| NullAssignment | Assigning an Object to null is a code smell. Consider refactoring. | 3 | 89 |
| FieldNamingConventions | The field name '__hashCodeCalc' doesn't match '_[a-z][a-zA-Z0-9]*' | 1 | 93 |
| RedundantFieldInitializer | Avoid using redundant field initializer for '__hashCodeCalc' | 3 | 93 |
| FieldDeclarationsShouldBeAtStartOfClass | Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. | 3 | 93 |
| MissingOverride | The method 'hashCode()' is missing an @Override annotation. | 3 | 94–113 |
| AvoidSynchronizedAtMethodLevel | Use block level rather than method level synchronization | 3 | 94–113 |
| CommentRequired | Public method and constructor comments are required | 3 | 94–113 |
| LocalVariableNamingConventions | The local variable name '_hashCode' doesn't match '[a-z][a-zA-Z0-9]*' | 1 | 99 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.Object' due to existing implicit import 'java.lang.*' | 4 | 104 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.reflect.Array.get' due to existing implicit import 'java.lang.*' | 4 | 104 |
| FieldDeclarationsShouldBeAtStartOfClass | Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. | 3 | 116–117 |
| FieldNamingConventions | The static field name 'typeDesc' doesn't match '_[a-z][a-zA-Z0-9]*' | 1 | 116 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.String' due to existing implicit import 'java.lang.*' | 4 | 140 |
| FormalParameterNamingConventions | The method parameter name '_javaType' doesn't match '[a-z][a-zA-Z0-9]*' | 1 | 141 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.Class' due to existing implicit import 'java.lang.*' | 4 | 141 |
| FormalParameterNamingConventions | The method parameter name '_xmlType' doesn't match '[a-z][a-zA-Z0-9]*' | 1 | 142 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.String' due to existing implicit import 'java.lang.*' | 4 | 152 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.Class' due to existing implicit import 'java.lang.*' | 4 | 153 |
| FormalParameterNamingConventions | The method parameter name '_javaType' doesn't match '[a-z][a-zA-Z0-9]*' | 1 | 153 |
| FormalParameterNamingConventions | The method parameter name '_xmlType' doesn't match '[a-z][a-zA-Z0-9]*' | 1 | 154 |
| MissingOverride | The method 'writeDetails(javax.xml.namespace.QName, org.apache.axis.encoding.SerializationContext)' is missing an @Override annotation. | 3 | 164–166 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| FormalParameterNamingConventions | The method parameter name 'ParisVideo' doesn't match '[a-z][a-zA-Z0-9]*' | 1 | 51 |
| CommentRequired | Public method and constructor comments are required | 3 | 53 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| CommentRequired | Public method and constructor comments are required | 3 | 49–52 |
| ShortVariable | Avoid variables with short names like id | 3 | 54 |
| CommentRequired | Public method and constructor comments are required | 3 | 54–57 |
| CommentRequired | Public method and constructor comments are required | 3 | 59–65 |
| ControlStatementBraces | This statement should have braces | 3 | 62 |
| ControlStatementBraces | This statement should have braces | 3 | 64 |
| CommentRequired | Public method and constructor comments are required | 3 | 67–70 |
| CommentRequired | Public method and constructor comments are required | 3 | 72–78 |
| ControlStatementBraces | This statement should have braces | 3 | 75 |
| ControlStatementBraces | This statement should have braces | 3 | 77 |
| CommentRequired | Public method and constructor comments are required | 3 | 80–83 |
| CommentRequired | Public method and constructor comments are required | 3 | 85–91 |
| ControlStatementBraces | This statement should have braces | 3 | 88 |
| ControlStatementBraces | This statement should have braces | 3 | 90 |
| CommentRequired | Public method and constructor comments are required | 3 | 93–96 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| MissingOverride | The method 'insert(ParisVideoAccount, Plugin)' is missing an @Override annotation. | 3 | 86–98 |
| ShortVariable | Avoid variables with short names like id | 3 | 89 |
| MissingOverride | The method 'selectAll(Plugin)' is missing an @Override annotation. | 3 | 106–126 |
| UseDiamondOperator | Explicit type arguments can be replaced by Diamond Operator | 3 | 108 |
| MissingOverride | The method 'deleteAll(Plugin)' is missing an @Override annotation. | 3 | 128–133 |
| CommentRequired | Public method and constructor comments are required | 3 | 128–133 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| 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 | 45–65 |
| CommentRequired | Public method and constructor comments are required | 3 | 49–54 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| SignatureDeclareThrowsException | A method/constructor should not explicitly throw java.lang.Exception | 3 | 70 |
| UnnecessaryLocalBeforeReturn | Consider simply returning the value vs storing it in local variable 'udrArray' | 3 | 80 |
| SignatureDeclareThrowsException | A method/constructor should not explicitly throw java.lang.Exception | 3 | 95 |
| ShortVariable | Avoid variables with short names like as | 3 | 97 |
| SignatureDeclareThrowsException | A method/constructor should not explicitly throw java.lang.Exception | 3 | 110 |
| ShortVariable | Avoid variables with short names like as | 3 | 112 |
| ShortVariable | Avoid variables with short names like ap | 3 | 116 |
| UnnecessaryLocalBeforeReturn | Consider simply returning the value vs storing it in local variable 'ap' | 3 | 117 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| CommentRequired | Header comments are required | 3 | 43–67 |
| RedundantFieldInitializer | Avoid using redundant field initializer for 'uniqueCookie1' | 3 | 45 |
| FieldNamingConventions | The static field name 'uniqueCookie1' doesn't match '_[a-z][a-zA-Z0-9]*' | 1 | 45 |
| FieldNamingConventions | The static field name 'uniqueCookie2' doesn't match '_[a-z][a-zA-Z0-9]*' | 1 | 46 |
| RedundantFieldInitializer | Avoid using redundant field initializer for 'uniqueCookie2' | 3 | 46 |
| ShortVariable | Avoid variables with short names like mc | 3 | 49 |
| ControlStatementBraces | This statement should have braces | 3 | 53 |
| ControlStatementBraces | This statement should have braces | 3 | 55 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| UseConcurrentHashMap | If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation | 3 | 87 |
| UseDiamondOperator | Explicit type arguments can be replaced by Diamond Operator | 3 | 87 |
| LocalVariableNamingConventions | The local variable name '_collec' doesn't match '[a-z][a-zA-Z0-9]*' | 1 | 99 |
| UseCollectionIsEmpty | Substitute calls to size() == 0 (or size() != 0, size() > 0, size() < 1) with calls to isEmpty() | 3 | 101 |
| LocalVariableNamingConventions | The local variable name '_strAction' doesn't match '[a-z][a-zA-Z0-9]*' | 1 | 116 |
| LocalVariableNamingConventions | The local variable name '_strName' doesn't match '[a-z][a-zA-Z0-9]*' | 1 | 117 |
| LocalVariableNamingConventions | The local variable name '_strLogin' doesn't match '[a-z][a-zA-Z0-9]*' | 1 | 118 |
| LocalVariableNamingConventions | The local variable name '_strPassword' doesn't match '[a-z][a-zA-Z0-9]*' | 1 | 119 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| UnusedPrivateField | Avoid unused private fields such as 'TEMPLATE_AUT_1'. | 3 | 86 |
| ImmutableField | Private field '_nDefaultItemsPerPage' could be made final; it is only initialized in the declaration or constructor. | 3 | 94 |
| RedundantFieldInitializer | Avoid using redundant field initializer for '_plugin' | 3 | 100 |
| MissingOverride | The method 'getInsertServiceSelectorUI(HttpServletRequest)' is missing an @Override annotation. | 3 | 113–153 |
| UseDiamondOperator | Explicit type arguments can be replaced by Diamond Operator | 3 | 117 |
| UseConcurrentHashMap | If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation | 3 | 117 |
| LocalVariableNamingConventions | The local variable name '_collec' doesn't match '[a-z][a-zA-Z0-9]*' | 1 | 122 |
| UseCollectionIsEmpty | Substitute calls to size() == 0 (or size() != 0, size() > 0, size() < 1) with calls to isEmpty() | 3 | 124 |
| UseDiamondOperator | Explicit type arguments can be replaced by Diamond Operator | 3 | 136 |
| EmptyCatchBlock | Avoid empty catch blocks | 3 | 143–145 |
| AvoidCatchingGenericException | Avoid catching generic exceptions such as NullPointerException, RuntimeException, Exception in try-catch block | 3 | 143 |
| SignatureDeclareThrowsException | A method/constructor should not explicitly throw java.lang.Exception | 3 | 158 |
| UseDiamondOperator | Explicit type arguments can be replaced by Diamond Operator | 3 | 160 |
| UseConcurrentHashMap | If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation | 3 | 160 |
| NullAssignment | Assigning an Object to null is a code smell. Consider refactoring. | 3 | 180 |
| ControlStatementBraces | This statement should have braces | 3 | 180 |
| AddEmptyString | Do not add empty strings | 3 | 191 |
| SignatureDeclareThrowsException | A method/constructor should not explicitly throw java.lang.Exception | 3 | 227 |
| UseConcurrentHashMap | If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation | 3 | 229 |
| UseDiamondOperator | Explicit type arguments can be replaced by Diamond Operator | 3 | 229 |
| ControlStatementBraces | This statement should have braces | 3 | 232 |
| ShortVariable | Avoid variables with short names like ap | 3 | 241 |
| SignatureDeclareThrowsException | A method/constructor should not explicitly throw java.lang.Exception | 3 | 251 |
| ShortVariable | Avoid variables with short names like ap | 3 | 257 |
| UseConcurrentHashMap | If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation | 3 | 259 |
| UseDiamondOperator | Explicit type arguments can be replaced by Diamond Operator | 3 | 259 |
