The following document contains the results of PMD 6.13.0.
| Rule | Violation | Line |
|---|---|---|
| FormalParameterNamingConventions | The method parameter name '_bContentcleaned' doesn't match '[a-z][a-zA-Z0-9]*' | 268 |
| Rule | Violation | Line |
|---|---|---|
| ClassNamingConventions | The utility class name 'NotificationToHistory' doesn't match '[A-Z][a-zA-Z0-9]+(Utils?|Helper|Home|Service)' | 59–239 |
| Rule | Violation | Line |
|---|---|---|
| ClassNamingConventions | The utility class name 'ServiceConfigTaskForm' doesn't match '[A-Z][a-zA-Z0-9]+(Utils?|Helper|Home|Service)' | 59–226 |
| Rule | Violation | Line |
|---|---|---|
| ClassNamingConventions | The utility class name 'Constants' doesn't match '[A-Z][a-zA-Z0-9]+(Utils?|Helper|Home|Service)' | 39–299 |
| Rule | Violation | Line |
|---|---|---|
| FieldNamingConventions | The final field name '_request' doesn't match '[a-z][a-zA-Z0-9]*' | 56 |
| Rule | Violation | Line |
|---|---|---|
| FieldNamingConventions | The constant name '_taskNotifyGruConfigService' doesn't match '[A-Z][A-Z_0-9]*' | 107 |
| FieldNamingConventions | The constant name '_notifyGRUService' doesn't match '[A-Z][A-Z_0-9]*' | 108 |
| FieldNamingConventions | The final field name '_task' doesn't match '[a-z][a-zA-Z0-9]*' | 110 |
| FieldNamingConventions | The final field name '_request' doesn't match '[a-z][a-zA-Z0-9]*' | 262 |
| FieldNamingConventions | The final field name '_listNotificationConfig' doesn't match '[a-z][a-zA-Z0-9]*' | 263 |
| FieldNamingConventions | The final field name '_model' doesn't match '[a-z][a-zA-Z0-9]*' | 264 |
| FieldNamingConventions | The final field name '_request' doesn't match '[a-z][a-zA-Z0-9]*' | 446 |
| FieldNamingConventions | The final field name '_listNotificationConfig' doesn't match '[a-z][a-zA-Z0-9]*' | 447 |
| FieldNamingConventions | The final field name '_providerManager' doesn't match '[a-z][a-zA-Z0-9]*' | 448 |
| FieldNamingConventions | The final field name '_strAction' doesn't match '[a-z][a-zA-Z0-9]*' | 449 |
| Rule | Violation | Line |
|---|---|---|
| FieldNamingConventions | The final field name '_request' doesn't match '[a-z][a-zA-Z0-9]*' | 59 |
| FieldNamingConventions | The final field name '_config' doesn't match '[a-z][a-zA-Z0-9]*' | 60 |
| FieldNamingConventions | The final field name '_strMessage' doesn't match '[a-z][a-zA-Z0-9]*' | 61 |
| FieldNamingConventions | The final field name '_strStatutText' doesn't match '[a-z][a-zA-Z0-9]*' | 62 |
| Rule | Violation | Line |
|---|---|---|
| FieldNamingConventions | The final field name '_request' doesn't match '[a-z][a-zA-Z0-9]*' | 72 |
| FieldNamingConventions | The final field name '_config' doesn't match '[a-z][a-zA-Z0-9]*' | 73 |
| FieldNamingConventions | The final field name '_strMailingType' doesn't match '[a-z][a-zA-Z0-9]*' | 74 |
| FieldNamingConventions | The final field name '_strSenderName' doesn't match '[a-z][a-zA-Z0-9]*' | 77 |
| FieldNamingConventions | The final field name '_strSubject' doesn't match '[a-z][a-zA-Z0-9]*' | 78 |
| FieldNamingConventions | The final field name '_strMessage' doesn't match '[a-z][a-zA-Z0-9]*' | 79 |
| FieldNamingConventions | The final field name '_strRecipientsCc' doesn't match '[a-z][a-zA-Z0-9]*' | 80 |
| FieldNamingConventions | The final field name '_strRecipientsCci' doesn't match '[a-z][a-zA-Z0-9]*' | 81 |
| Rule | Violation | Line |
|---|---|---|
| FieldNamingConventions | The final field name '_request' doesn't match '[a-z][a-zA-Z0-9]*' | 62 |
| FieldNamingConventions | The final field name '_config' doesn't match '[a-z][a-zA-Z0-9]*' | 63 |
| FieldNamingConventions | The final field name '_strSubject' doesn't match '[a-z][a-zA-Z0-9]*' | 64 |
| FieldNamingConventions | The final field name '_strMessage' doesn't match '[a-z][a-zA-Z0-9]*' | 65 |
| FieldNamingConventions | The final field name '_strSenderName' doesn't match '[a-z][a-zA-Z0-9]*' | 66 |
| FieldNamingConventions | The final field name '_strRecipientsCc' doesn't match '[a-z][a-zA-Z0-9]*' | 67 |
| FieldNamingConventions | The final field name '_strRecipientsCci' doesn't match '[a-z][a-zA-Z0-9]*' | 68 |
| Rule | Violation | Line |
|---|---|---|
| FieldNamingConventions | The final field name '_request' doesn't match '[a-z][a-zA-Z0-9]*' | 68 |
| FieldNamingConventions | The final field name '_config' doesn't match '[a-z][a-zA-Z0-9]*' | 69 |
| FieldNamingConventions | The final field name '_strMessage' doesn't match '[a-z][a-zA-Z0-9]*' | 70 |
| FieldNamingConventions | The final field name '_strStatusText' doesn't match '[a-z][a-zA-Z0-9]*' | 71 |
| FieldNamingConventions | The final field name '_strSenderName' doesn't match '[a-z][a-zA-Z0-9]*' | 72 |
| FieldNamingConventions | The final field name '_strSubject' doesn't match '[a-z][a-zA-Z0-9]*' | 73 |
| FieldNamingConventions | The final field name '_nDemandMaxStep' doesn't match '[a-z][a-zA-Z0-9]*' | 74 |
| FieldNamingConventions | The final field name '_nDemandUserCurrentStep' doesn't match '[a-z][a-zA-Z0-9]*' | 75 |
| Rule | Violation | Line |
|---|---|---|
| FieldNamingConventions | The final field name '_request' doesn't match '[a-z][a-zA-Z0-9]*' | 58 |
| FieldNamingConventions | The final field name '_config' doesn't match '[a-z][a-zA-Z0-9]*' | 59 |
| FieldNamingConventions | The final field name '_strMessage' doesn't match '[a-z][a-zA-Z0-9]*' | 60 |
| Rule | Violation | Line |
|---|---|---|
| AvoidSynchronizedAtMethodLevel | Use block level rather than method level synchronization | 80–128 |
| Rule | Violation | Line |
|---|---|---|
| ExcessivePublicCount | This class has a bunch of public methods and attributes | 34–895 |
| TooManyFields | Too many fields | 49–895 |
| CommentRequired | Public method and constructor comments are required | 161–164 |
| CallSuperInConstructor | It is a good practice to call super() in a constructor | 161–164 |
| ConfusingTernary | Avoid if (x != y) ..; else ..; | 530–537 |
| Rule | Violation | Line |
|---|---|---|
| AvoidSynchronizedAtMethodLevel | Use block level rather than method level synchronization | 89–100 |
| Rule | Violation | Line |
|---|---|---|
| ConfusingTernary | Avoid if (x != y) ..; else ..; | 134–141 |
| Rule | Violation | Line |
|---|---|---|
| CollapsibleIfStatements | These nested if statements could be combined | 92–96 |
| AvoidLiteralsInIfCondition | Avoid using Literals in Conditional Statements | 138 |
| AvoidLiteralsInIfCondition | Avoid using Literals in Conditional Statements | 143 |
| MissingBreakInSwitch | A switch statement does not contain a break | 193–222 |
| Rule | Violation | Line |
|---|---|---|
| ExcessiveImports | A high number of imports can indicate a high degree of coupling within an object. | 34–568 |
| TooManyMethods | This class has too many methods, consider refactoring it. | 90–568 |
| CyclomaticComplexity | The method 'processTask(int, HttpServletRequest, Locale)' has a cyclomatic complexity of 19. | 113–255 |
| ExcessiveMethodLength | Avoid really long methods. | 113–255 |
| NPathComplexity | The method 'processTask(int, HttpServletRequest, Locale)' has an NPath complexity of 5184 | 113–255 |
| NcssCount | The method 'processTask(int, HttpServletRequest, Locale)' has a NCSS line count of 79. | 113–255 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 119 |
| ConfusingTernary | Avoid if (x != y) ..; else ..; | 210–236 |
| AppendCharacterWithChar | Avoid appending characters as strings in StringBuffer.append. | 229 |
| AppendCharacterWithChar | Avoid appending characters as strings in StringBuffer.append. | 230 |
| AvoidCatchingGenericException | Avoid catching generic exceptions such as NullPointerException, RuntimeException, Exception in try-catch block | 241 |
| ConfusingTernary | Avoid if (x != y) ..; else ..; | 346–363 |
| UselessOperationOnImmutable | An operation on an Immutable object (String, BigDecimal or BigInteger) wont change the object itself | 430 |
| UseConcurrentHashMap | If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation | 523 |
| Rule | Violation | Line |
|---|---|---|
| UnnecessaryLocalBeforeReturn | Consider simply returning the value vs storing it in local variable 'config' | 100 |
| Rule | Violation | Line |
|---|---|---|
| CallSuperInConstructor | It is a good practice to call super() in a constructor | 58–61 |
| NonThreadSafeSingleton | Singleton is not thread safe | 70–73 |
| InsufficientStringBufferDeclaration | StringBuffer constructor is initialized with size 16, but has at least 26 characters appended. | 138 |
| Rule | Violation | Line |
|---|---|---|
| ImmutableField | Private field '_notifyGruHistoryService' could be made final; it is only initialized in the declaration or constructor. | 60 |
| MissingOverride | The method 'run()' is missing an @Override annotation. | 65–112 |
| InsufficientStringBufferDeclaration | StringBuffer constructor is initialized with size 16, but has at least 39 characters appended. | 70 |
| AppendCharacterWithChar | Avoid appending characters as strings in StringBuffer.append. | 85 |
| ConsecutiveLiteralAppends | StringBuffer (or StringBuilder).append is called 2 consecutive times with literals. Use a single append with a single combined String. | 85 |
| InefficientStringBuffering | Avoid concatenating nonliterals in a StringBuffer/StringBuilder constructor or append(). | 87 |
| AppendCharacterWithChar | Avoid appending characters as strings in StringBuffer.append. | 88 |
| ConsecutiveLiteralAppends | StringBuffer (or StringBuilder).append is called 4 consecutive times with literals. Use a single append with a single combined String. | 91 |
| AppendCharacterWithChar | Avoid appending characters as strings in StringBuffer.append. | 93 |
| AppendCharacterWithChar | Avoid appending characters as strings in StringBuffer.append. | 95 |
| ConsecutiveLiteralAppends | StringBuffer (or StringBuilder).append is called 3 consecutive times with literals. Use a single append with a single combined String. | 101 |
| AppendCharacterWithChar | Avoid appending characters as strings in StringBuffer.append. | 102 |
| Rule | Violation | Line |
|---|---|---|
| ShortVariable | Avoid variables with short names like ID | 63 |
| ConfusingTernary | Avoid if (x != y) ..; else ..; | 124 |
| Rule | Violation | Line |
|---|---|---|
| AvoidDuplicateLiterals | The String literal "message" appears 4 times in this file; the first occurrence is on line 61 | 61 |
| ShortVariable | Avoid variables with short names like ID | 127 |
| Rule | Violation | Line |
|---|---|---|
| AvoidCatchingGenericException | Avoid catching generic exceptions such as NullPointerException, RuntimeException, Exception in try-catch block | 161 |
| Rule | Violation | Line |
|---|---|---|
| UnnecessaryModifier | Unnecessary modifier 'public' on method 'getNotificationType': the method is declared in an interface type | 88 |
| Rule | Violation | Line |
|---|---|---|
| UseConcurrentHashMap | If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation | 123 |
| Rule | Violation | Line |
|---|---|---|
| ExcessiveImports | A high number of imports can indicate a high degree of coupling within an object. | 34–908 |
| TooManyMethods | This class has too many methods, consider refactoring it. | 261–433 |
| ConfusingTernary | Avoid if (x != y) ..; else ..; | 393–402 |
| GodClass | Possible God Class (WMC=51, ATFD=37, TCC=7.672%) | 439–907 |
| TooManyMethods | This class has too many methods, consider refactoring it. | 440–907 |
| UseConcurrentHashMap | If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation | 794 |
| ConfusingTernary | Avoid if (x != y) ..; else ..; | 829–836 |
| Rule | Violation | Line |
|---|---|---|
| CommentDefaultAccessModifier | To avoid mistakes add a comment at the beginning of the _strBillingAccount field if you want a default access modifier | 47 |
| DefaultPackage | Use explicit scoping instead of the default package private level | 47 |
| DefaultPackage | Use explicit scoping instead of the default package private level | 50 |
| CommentDefaultAccessModifier | To avoid mistakes add a comment at the beginning of the _strBillingGroup field if you want a default access modifier | 50 |
| Rule | Violation | Line |
|---|---|---|
| NPathComplexity | The method 'validateFieldsWithoutMarker()' has an NPath complexity of 288 | 207–243 |
| CyclomaticComplexity | The method 'validateFieldsWithoutMarker()' has a cyclomatic complexity of 11. | 207–243 |
| Rule | Violation | Line |
|---|---|---|
| UnusedImports | Avoid unused imports such as 'java.util.Date' | 37 |
| Rule | Violation | Line |
|---|---|---|
| UnusedImports | Avoid unused imports such as 'javax.inject.Inject' | 40 |
| UnusedImports | Avoid unused imports such as 'javax.inject.Named' | 41 |
| Rule | Violation | Line |
|---|---|---|
| ShortClassName | Avoid short class names like View | 260–433 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| UnusedImports | Avoid unused imports such as 'java.util.Date' | 4 | 37 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| FormalParameterNamingConventions | The method parameter name '_bContentcleaned' doesn't match '[a-z][a-zA-Z0-9]*' | 1 | 268 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| AvoidSynchronizedAtMethodLevel | Use block level rather than method level synchronization | 3 | 80–128 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| ExcessivePublicCount | This class has a bunch of public methods and attributes | 3 | 34–895 |
| TooManyFields | Too many fields | 3 | 49–895 |
| CommentRequired | Public method and constructor comments are required | 3 | 161–164 |
| CallSuperInConstructor | It is a good practice to call super() in a constructor | 3 | 161–164 |
| ConfusingTernary | Avoid if (x != y) ..; else ..; | 3 | 530–537 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| AvoidSynchronizedAtMethodLevel | Use block level rather than method level synchronization | 3 | 89–100 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| ClassNamingConventions | The utility class name 'NotificationToHistory' doesn't match '[A-Z][a-zA-Z0-9]+(Utils?|Helper|Home|Service)' | 1 | 59–239 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| ConfusingTernary | Avoid if (x != y) ..; else ..; | 3 | 134–141 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| ClassNamingConventions | The utility class name 'ServiceConfigTaskForm' doesn't match '[A-Z][a-zA-Z0-9]+(Utils?|Helper|Home|Service)' | 1 | 59–226 |
| CollapsibleIfStatements | These nested if statements could be combined | 3 | 92–96 |
| AvoidLiteralsInIfCondition | Avoid using Literals in Conditional Statements | 3 | 138 |
| AvoidLiteralsInIfCondition | Avoid using Literals in Conditional Statements | 3 | 143 |
| MissingBreakInSwitch | A switch statement does not contain a break | 3 | 193–222 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| ExcessiveImports | A high number of imports can indicate a high degree of coupling within an object. | 3 | 34–568 |
| TooManyMethods | This class has too many methods, consider refactoring it. | 3 | 90–568 |
| CyclomaticComplexity | The method 'processTask(int, HttpServletRequest, Locale)' has a cyclomatic complexity of 19. | 3 | 113–255 |
| ExcessiveMethodLength | Avoid really long methods. | 3 | 113–255 |
| NPathComplexity | The method 'processTask(int, HttpServletRequest, Locale)' has an NPath complexity of 5184 | 3 | 113–255 |
| NcssCount | The method 'processTask(int, HttpServletRequest, Locale)' has a NCSS line count of 79. | 3 | 113–255 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 3 | 119 |
| ConfusingTernary | Avoid if (x != y) ..; else ..; | 3 | 210–236 |
| AppendCharacterWithChar | Avoid appending characters as strings in StringBuffer.append. | 3 | 229 |
| AppendCharacterWithChar | Avoid appending characters as strings in StringBuffer.append. | 3 | 230 |
| AvoidCatchingGenericException | Avoid catching generic exceptions such as NullPointerException, RuntimeException, Exception in try-catch block | 3 | 241 |
| ConfusingTernary | Avoid if (x != y) ..; else ..; | 3 | 346–363 |
| UselessOperationOnImmutable | An operation on an Immutable object (String, BigDecimal or BigInteger) wont change the object itself | 3 | 430 |
| UseConcurrentHashMap | If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation | 3 | 523 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| UnnecessaryLocalBeforeReturn | Consider simply returning the value vs storing it in local variable 'config' | 3 | 100 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| CallSuperInConstructor | It is a good practice to call super() in a constructor | 3 | 58–61 |
| NonThreadSafeSingleton | Singleton is not thread safe | 3 | 70–73 |
| InsufficientStringBufferDeclaration | StringBuffer constructor is initialized with size 16, but has at least 26 characters appended. | 3 | 138 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| UnusedImports | Avoid unused imports such as 'javax.inject.Inject' | 4 | 40 |
| UnusedImports | Avoid unused imports such as 'javax.inject.Named' | 4 | 41 |
| ImmutableField | Private field '_notifyGruHistoryService' could be made final; it is only initialized in the declaration or constructor. | 3 | 60 |
| MissingOverride | The method 'run()' is missing an @Override annotation. | 3 | 65–112 |
| InsufficientStringBufferDeclaration | StringBuffer constructor is initialized with size 16, but has at least 39 characters appended. | 3 | 70 |
| AppendCharacterWithChar | Avoid appending characters as strings in StringBuffer.append. | 3 | 85 |
| ConsecutiveLiteralAppends | StringBuffer (or StringBuilder).append is called 2 consecutive times with literals. Use a single append with a single combined String. | 3 | 85 |
| InefficientStringBuffering | Avoid concatenating nonliterals in a StringBuffer/StringBuilder constructor or append(). | 3 | 87 |
| AppendCharacterWithChar | Avoid appending characters as strings in StringBuffer.append. | 3 | 88 |
| ConsecutiveLiteralAppends | StringBuffer (or StringBuilder).append is called 4 consecutive times with literals. Use a single append with a single combined String. | 3 | 91 |
| AppendCharacterWithChar | Avoid appending characters as strings in StringBuffer.append. | 3 | 93 |
| AppendCharacterWithChar | Avoid appending characters as strings in StringBuffer.append. | 3 | 95 |
| ConsecutiveLiteralAppends | StringBuffer (or StringBuilder).append is called 3 consecutive times with literals. Use a single append with a single combined String. | 3 | 101 |
| AppendCharacterWithChar | Avoid appending characters as strings in StringBuffer.append. | 3 | 102 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| ShortVariable | Avoid variables with short names like ID | 3 | 63 |
| ConfusingTernary | Avoid if (x != y) ..; else ..; | 3 | 124 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| ClassNamingConventions | The utility class name 'Constants' doesn't match '[A-Z][a-zA-Z0-9]+(Utils?|Helper|Home|Service)' | 1 | 39–299 |
| AvoidDuplicateLiterals | The String literal "message" appears 4 times in this file; the first occurrence is on line 61 | 3 | 61 |
| ShortVariable | Avoid variables with short names like ID | 3 | 127 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| FieldNamingConventions | The final field name '_request' doesn't match '[a-z][a-zA-Z0-9]*' | 1 | 56 |
| AvoidCatchingGenericException | Avoid catching generic exceptions such as NullPointerException, RuntimeException, Exception in try-catch block | 3 | 161 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| UnnecessaryModifier | Unnecessary modifier 'public' on method 'getNotificationType': the method is declared in an interface type | 3 | 88 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| UseConcurrentHashMap | If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation | 3 | 123 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| ExcessiveImports | A high number of imports can indicate a high degree of coupling within an object. | 3 | 34–908 |
| FieldNamingConventions | The constant name '_taskNotifyGruConfigService' doesn't match '[A-Z][A-Z_0-9]*' | 1 | 107 |
| FieldNamingConventions | The constant name '_notifyGRUService' doesn't match '[A-Z][A-Z_0-9]*' | 1 | 108 |
| FieldNamingConventions | The final field name '_task' doesn't match '[a-z][a-zA-Z0-9]*' | 1 | 110 |
| ShortClassName | Avoid short class names like View | 4 | 260–433 |
| TooManyMethods | This class has too many methods, consider refactoring it. | 3 | 261–433 |
| FieldNamingConventions | The final field name '_request' doesn't match '[a-z][a-zA-Z0-9]*' | 1 | 262 |
| FieldNamingConventions | The final field name '_listNotificationConfig' doesn't match '[a-z][a-zA-Z0-9]*' | 1 | 263 |
| FieldNamingConventions | The final field name '_model' doesn't match '[a-z][a-zA-Z0-9]*' | 1 | 264 |
| ConfusingTernary | Avoid if (x != y) ..; else ..; | 3 | 393–402 |
| GodClass | Possible God Class (WMC=51, ATFD=37, TCC=7.672%) | 3 | 439–907 |
| TooManyMethods | This class has too many methods, consider refactoring it. | 3 | 440–907 |
| FieldNamingConventions | The final field name '_request' doesn't match '[a-z][a-zA-Z0-9]*' | 1 | 446 |
| FieldNamingConventions | The final field name '_listNotificationConfig' doesn't match '[a-z][a-zA-Z0-9]*' | 1 | 447 |
| FieldNamingConventions | The final field name '_providerManager' doesn't match '[a-z][a-zA-Z0-9]*' | 1 | 448 |
| FieldNamingConventions | The final field name '_strAction' doesn't match '[a-z][a-zA-Z0-9]*' | 1 | 449 |
| UseConcurrentHashMap | If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation | 3 | 794 |
| ConfusingTernary | Avoid if (x != y) ..; else ..; | 3 | 829–836 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| FieldNamingConventions | The final field name '_request' doesn't match '[a-z][a-zA-Z0-9]*' | 1 | 59 |
| FieldNamingConventions | The final field name '_config' doesn't match '[a-z][a-zA-Z0-9]*' | 1 | 60 |
| FieldNamingConventions | The final field name '_strMessage' doesn't match '[a-z][a-zA-Z0-9]*' | 1 | 61 |
| FieldNamingConventions | The final field name '_strStatutText' doesn't match '[a-z][a-zA-Z0-9]*' | 1 | 62 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| CommentDefaultAccessModifier | To avoid mistakes add a comment at the beginning of the _strBillingAccount field if you want a default access modifier | 3 | 47 |
| DefaultPackage | Use explicit scoping instead of the default package private level | 3 | 47 |
| DefaultPackage | Use explicit scoping instead of the default package private level | 3 | 50 |
| CommentDefaultAccessModifier | To avoid mistakes add a comment at the beginning of the _strBillingGroup field if you want a default access modifier | 3 | 50 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| FieldNamingConventions | The final field name '_request' doesn't match '[a-z][a-zA-Z0-9]*' | 1 | 72 |
| FieldNamingConventions | The final field name '_config' doesn't match '[a-z][a-zA-Z0-9]*' | 1 | 73 |
| FieldNamingConventions | The final field name '_strMailingType' doesn't match '[a-z][a-zA-Z0-9]*' | 1 | 74 |
| FieldNamingConventions | The final field name '_strSenderName' doesn't match '[a-z][a-zA-Z0-9]*' | 1 | 77 |
| FieldNamingConventions | The final field name '_strSubject' doesn't match '[a-z][a-zA-Z0-9]*' | 1 | 78 |
| FieldNamingConventions | The final field name '_strMessage' doesn't match '[a-z][a-zA-Z0-9]*' | 1 | 79 |
| FieldNamingConventions | The final field name '_strRecipientsCc' doesn't match '[a-z][a-zA-Z0-9]*' | 1 | 80 |
| FieldNamingConventions | The final field name '_strRecipientsCci' doesn't match '[a-z][a-zA-Z0-9]*' | 1 | 81 |
| NPathComplexity | The method 'validateFieldsWithoutMarker()' has an NPath complexity of 288 | 3 | 207–243 |
| CyclomaticComplexity | The method 'validateFieldsWithoutMarker()' has a cyclomatic complexity of 11. | 3 | 207–243 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| FieldNamingConventions | The final field name '_request' doesn't match '[a-z][a-zA-Z0-9]*' | 1 | 62 |
| FieldNamingConventions | The final field name '_config' doesn't match '[a-z][a-zA-Z0-9]*' | 1 | 63 |
| FieldNamingConventions | The final field name '_strSubject' doesn't match '[a-z][a-zA-Z0-9]*' | 1 | 64 |
| FieldNamingConventions | The final field name '_strMessage' doesn't match '[a-z][a-zA-Z0-9]*' | 1 | 65 |
| FieldNamingConventions | The final field name '_strSenderName' doesn't match '[a-z][a-zA-Z0-9]*' | 1 | 66 |
| FieldNamingConventions | The final field name '_strRecipientsCc' doesn't match '[a-z][a-zA-Z0-9]*' | 1 | 67 |
| FieldNamingConventions | The final field name '_strRecipientsCci' doesn't match '[a-z][a-zA-Z0-9]*' | 1 | 68 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| FieldNamingConventions | The final field name '_request' doesn't match '[a-z][a-zA-Z0-9]*' | 1 | 68 |
| FieldNamingConventions | The final field name '_config' doesn't match '[a-z][a-zA-Z0-9]*' | 1 | 69 |
| FieldNamingConventions | The final field name '_strMessage' doesn't match '[a-z][a-zA-Z0-9]*' | 1 | 70 |
| FieldNamingConventions | The final field name '_strStatusText' doesn't match '[a-z][a-zA-Z0-9]*' | 1 | 71 |
| FieldNamingConventions | The final field name '_strSenderName' doesn't match '[a-z][a-zA-Z0-9]*' | 1 | 72 |
| FieldNamingConventions | The final field name '_strSubject' doesn't match '[a-z][a-zA-Z0-9]*' | 1 | 73 |
| FieldNamingConventions | The final field name '_nDemandMaxStep' doesn't match '[a-z][a-zA-Z0-9]*' | 1 | 74 |
| FieldNamingConventions | The final field name '_nDemandUserCurrentStep' doesn't match '[a-z][a-zA-Z0-9]*' | 1 | 75 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| FieldNamingConventions | The final field name '_request' doesn't match '[a-z][a-zA-Z0-9]*' | 1 | 58 |
| FieldNamingConventions | The final field name '_config' doesn't match '[a-z][a-zA-Z0-9]*' | 1 | 59 |
| FieldNamingConventions | The final field name '_strMessage' doesn't match '[a-z][a-zA-Z0-9]*' | 1 | 60 |
