Le document suivant contient les résultats de PMD 6.13.0.
| Rule | Violation | Ligne |
|---|---|---|
| FormalParameterNamingConventions | The method parameter name '_strLibelleResetButton' doesn't match '[a-z][a-zA-Z0-9]*' | 185 |
| Rule | Violation | Ligne |
|---|---|---|
| FormalParameterNamingConventions | The method parameter name '_bUnique' doesn't match '[a-z][a-zA-Z0-9]*' | 529 |
| Rule | Violation | Ligne |
|---|---|---|
| FieldNamingConventions | The final field name '_template_create' doesn't match '[a-z][a-zA-Z0-9]*' | 57 |
| FieldNamingConventions | The final field name '_template_modify' doesn't match '[a-z][a-zA-Z0-9]*' | 58 |
| FieldNamingConventions | The final field name '_template_html_code' doesn't match '[a-z][a-zA-Z0-9]*' | 59 |
| Rule | Violation | Ligne |
|---|---|---|
| FieldNamingConventions | The final field name '_template_create' doesn't match '[a-z][a-zA-Z0-9]*' | 52 |
| FieldNamingConventions | The final field name '_template_modify' doesn't match '[a-z][a-zA-Z0-9]*' | 53 |
| FieldNamingConventions | The final field name '_template_html_code' doesn't match '[a-z][a-zA-Z0-9]*' | 54 |
| Rule | Violation | Ligne |
|---|---|---|
| FieldNamingConventions | The constant name '_message_illogical_date' doesn't match '[A-Z][A-Z_0-9]*' | 60 |
| FieldNamingConventions | The final field name '_template_create' doesn't match '[a-z][a-zA-Z0-9]*' | 61 |
| FieldNamingConventions | The final field name '_template_modify' doesn't match '[a-z][a-zA-Z0-9]*' | 62 |
| FieldNamingConventions | The final field name '_template_html_code' doesn't match '[a-z][a-zA-Z0-9]*' | 63 |
| Rule | Violation | Ligne |
|---|---|---|
| FieldNamingConventions | The final field name '_template_create' doesn't match '[a-z][a-zA-Z0-9]*' | 66 |
| FieldNamingConventions | The final field name '_template_modify' doesn't match '[a-z][a-zA-Z0-9]*' | 67 |
| FieldNamingConventions | The final field name '_template_html_code' doesn't match '[a-z][a-zA-Z0-9]*' | 68 |
| Rule | Violation | Ligne |
|---|---|---|
| FieldNamingConventions | The final field name '_template_modify' doesn't match '[a-z][a-zA-Z0-9]*' | 53 |
| FieldNamingConventions | The final field name '_template_html_code' doesn't match '[a-z][a-zA-Z0-9]*' | 54 |
| Rule | Violation | Ligne |
|---|---|---|
| FieldNamingConventions | The final field name '_template_create' doesn't match '[a-z][a-zA-Z0-9]*' | 56 |
| FieldNamingConventions | The final field name '_template_modify' doesn't match '[a-z][a-zA-Z0-9]*' | 57 |
| FieldNamingConventions | The final field name '_template_html_code' doesn't match '[a-z][a-zA-Z0-9]*' | 58 |
| Rule | Violation | Ligne |
|---|---|---|
| FieldNamingConventions | The final field name '_template_create' doesn't match '[a-z][a-zA-Z0-9]*' | 56 |
| FieldNamingConventions | The final field name '_template_modify' doesn't match '[a-z][a-zA-Z0-9]*' | 57 |
| FieldNamingConventions | The final field name '_template_html_code' doesn't match '[a-z][a-zA-Z0-9]*' | 58 |
| Rule | Violation | Ligne |
|---|---|---|
| FieldNamingConventions | The final field name '_template_create' doesn't match '[a-z][a-zA-Z0-9]*' | 58 |
| FieldNamingConventions | The final field name '_template_modify' doesn't match '[a-z][a-zA-Z0-9]*' | 59 |
| FieldNamingConventions | The final field name '_template_html_code' doesn't match '[a-z][a-zA-Z0-9]*' | 60 |
| Rule | Violation | Ligne |
|---|---|---|
| FieldNamingConventions | The final field name '_template_create' doesn't match '[a-z][a-zA-Z0-9]*' | 64 |
| FieldNamingConventions | The final field name '_template_modify' doesn't match '[a-z][a-zA-Z0-9]*' | 65 |
| FieldNamingConventions | The final field name '_template_html_code' doesn't match '[a-z][a-zA-Z0-9]*' | 66 |
| Rule | Violation | Ligne |
|---|---|---|
| FieldNamingConventions | The final field name '_template_create' doesn't match '[a-z][a-zA-Z0-9]*' | 56 |
| FieldNamingConventions | The final field name '_template_modify' doesn't match '[a-z][a-zA-Z0-9]*' | 57 |
| FieldNamingConventions | The final field name '_template_html_code' doesn't match '[a-z][a-zA-Z0-9]*' | 58 |
| Rule | Violation | Ligne |
|---|---|---|
| FormalParameterNamingConventions | The method parameter name '_tDateBeginDisponibility' doesn't match '[a-z][a-zA-Z0-9]*' | 471 |
| FormalParameterNamingConventions | The method parameter name '_bAutoPublicationActive' doesn't match '[a-z][a-zA-Z0-9]*' | 489 |
| FormalParameterNamingConventions | The method parameter name '_strLibelleResetButton' doesn't match '[a-z][a-zA-Z0-9]*' | 516 |
| Rule | Violation | Ligne |
|---|---|---|
| FormalParameterNamingConventions | The method parameter name '_bDateBeginDisponibilityAfterCurrentDate' doesn't match '[a-z][a-zA-Z0-9]*' | 131 |
| Rule | Violation | Ligne |
|---|---|---|
| ClassWithOnlyPrivateConstructorsShouldBeFinal | A class which only has private constructors should be final | 52–150 |
| FieldNamingConventions | The field name '_OutputProcessorSet' doesn't match '_[a-z][a-zA-Z0-9]*' | 56 |
| Rule | Violation | Ligne |
|---|---|---|
| AvoidReassigningParameters | Avoid reassigning parameters such as 'entryParent' | 1089 |
| Rule | Violation | Ligne |
|---|---|---|
| MissingOverride | The method 'getResourceTypeCode()' is missing an @Override annotation. | 167–170 |
| MissingOverride | The method 'getResourceId()' is missing an @Override annotation. | 176–179 |
| Rule | Violation | Ligne |
|---|---|---|
| MissingOverride | The method 'store(DefaultMessage, Plugin)' is missing an @Override annotation. | 59–73 |
| MissingOverride | The method 'load(Plugin)' is missing an @Override annotation. | 81–103 |
| Rule | Violation | Ligne |
|---|---|---|
| ExcessivePublicCount | This class has a bunch of public methods and attributes | 34–541 |
| TooManyFields | Too many fields | 51–541 |
| MissingOverride | The method 'getChildren()' is missing an @Override annotation. | 119–122 |
| MissingOverride | The method 'getComment()' is missing an @Override annotation. | 127–130 |
| MissingOverride | The method 'getEntryType()' is missing an @Override annotation. | 135–138 |
| MissingOverride | The method 'getFields()' is missing an @Override annotation. | 143–146 |
| MissingOverride | The method 'getHelpMessage()' is missing an @Override annotation. | 151–154 |
| MissingOverride | The method 'getIdEntry()' is missing an @Override annotation. | 159–162 |
| MissingOverride | The method 'getParent()' is missing an @Override annotation. | 167–170 |
| MissingOverride | The method 'getPosition()' is missing an @Override annotation. | 175–178 |
| MissingOverride | The method 'getTitle()' is missing an @Override annotation. | 183–186 |
| MissingOverride | The method 'isFieldInLine()' is missing an @Override annotation. | 191–194 |
| MissingOverride | The method 'isMandatory()' is missing an @Override annotation. | 199–202 |
| MissingOverride | The method 'setChildren(List)' is missing an @Override annotation. | 208–211 |
| MissingOverride | The method 'setComment(String)' is missing an @Override annotation. | 217–220 |
| MissingOverride | The method 'setEntryType(EntryType)' is missing an @Override annotation. | 226–229 |
| MissingOverride | The method 'setFieldInLine(boolean)' is missing an @Override annotation. | 235–238 |
| MissingOverride | The method 'setFields(List)' is missing an @Override annotation. | 244–247 |
| MissingOverride | The method 'setHelpMessage(String)' is missing an @Override annotation. | 253–256 |
| MissingOverride | The method 'setIdEntry(int)' is missing an @Override annotation. | 262–265 |
| MissingOverride | The method 'setMandatory(boolean)' is missing an @Override annotation. | 271–274 |
| MissingOverride | The method 'setParent(IEntry)' is missing an @Override annotation. | 280–283 |
| MissingOverride | The method 'setPosition(int)' is missing an @Override annotation. | 289–292 |
| MissingOverride | The method 'setTitle(String)' is missing an @Override annotation. | 298–301 |
| MissingOverride | The method 'getForm()' is missing an @Override annotation. | 307–310 |
| MissingOverride | The method 'setForm(Form)' is missing an @Override annotation. | 316–319 |
| MissingOverride | The method 'getFieldDepend()' is missing an @Override annotation. | 324–327 |
| MissingOverride | The method 'setFieldDepend(Field)' is missing an @Override annotation. | 333–336 |
| MissingOverride | The method 'getHtmlCode()' is missing an @Override annotation. | 343–346 |
| MissingOverride | The method 'getRequestData(HttpServletRequest, Locale)' is missing an @Override annotation. | 354–357 |
| MissingOverride | The method 'getResponseData(HttpServletRequest, List, Locale)' is missing an @Override annotation. | 366–369 |
| MissingOverride | The method 'getTemplateCreate()' is missing an @Override annotation. | 375–378 |
| MissingOverride | The method 'getTemplateModify()' is missing an @Override annotation. | 384–387 |
| MissingOverride | The method 'getNumberConditionalQuestion()' is missing an @Override annotation. | 392–395 |
| MissingOverride | The method 'getResponseValueForExport(HttpServletRequest, Response, Locale)' is missing an @Override annotation. | 404–407 |
| MissingOverride | The method 'getResponseValueForRecap(HttpServletRequest, Response, Locale)' is missing an @Override annotation. | 416–419 |
| MissingOverride | The method 'setNumberConditionalQuestion(int)' is missing an @Override annotation. | 426–429 |
| MissingOverride | The method 'getPaginator(int, String, String, String)' is missing an @Override annotation. | 439–443 |
| MissingOverride | The method 'getReferenceListRegularExpression(IEntry, Plugin)' is missing an @Override annotation. | 451–454 |
| MissingOverride | The method 'isLastInTheList()' is missing an @Override annotation. | 459–462 |
| MissingOverride | The method 'setLastInTheList(boolean)' is missing an @Override annotation. | 468–471 |
| MissingOverride | The method 'isFirstInTheList()' is missing an @Override annotation. | 476–479 |
| MissingOverride | The method 'setFirstInTheList(boolean)' is missing an @Override annotation. | 485–488 |
| MissingOverride | The method 'setConfirmField(boolean)' is missing an @Override annotation. | 494–497 |
| MissingOverride | The method 'isConfirmField()' is missing an @Override annotation. | 503–506 |
| MissingOverride | The method 'setConfirmFieldTitle(String)' is missing an @Override annotation. | 512–515 |
| MissingOverride | The method 'getConfirmFieldTitle()' is missing an @Override annotation. | 521–524 |
| MissingOverride | The method 'setUnique(boolean)' is missing an @Override annotation. | 529–532 |
| MissingOverride | The method 'isUnique()' is missing an @Override annotation. | 537–540 |
| Rule | Violation | Ligne |
|---|---|---|
| GodClass | Possible God Class (WMC=53, ATFD=104, TCC=2.778%) | 47–583 |
| MissingOverride | The method 'insert(IEntry, Plugin)' is missing an @Override annotation. | 165–207 |
| AvoidSynchronizedAtMethodLevel | Use block level rather than method level synchronization | 165–207 |
| ConfusingTernary | Avoid if (x != y) ..; else ..; | 174–181 |
| ConfusingTernary | Avoid if (x != y) ..; else ..; | 190–197 |
| MissingOverride | The method 'load(int, Plugin)' is missing an @Override annotation. | 216–302 |
| IdenticalCatchBranches | 'catch' branch identical to 'ClassNotFoundException' branch | 248–254 |
| IdenticalCatchBranches | 'catch' branch identical to 'ClassNotFoundException' branch | 255–261 |
| MissingOverride | The method 'delete(int, Plugin)' is missing an @Override annotation. | 310–316 |
| MissingOverride | The method 'store(IEntry, Plugin)' is missing an @Override annotation. | 324–364 |
| ConfusingTernary | Avoid if (x != y) ..; else ..; | 332–339 |
| ConfusingTernary | Avoid if (x != y) ..; else ..; | 348–355 |
| NPathComplexity | The method 'selectEntryListByFilter(EntryFilter, Plugin)' has an NPath complexity of 26112 | 372–502 |
| ExcessiveMethodLength | Avoid really long methods. | 372–502 |
| MissingOverride | The method 'selectEntryListByFilter(EntryFilter, Plugin)' is missing an @Override annotation. | 372–502 |
| CyclomaticComplexity | The method 'selectEntryListByFilter(EntryFilter, Plugin)' has a cyclomatic complexity of 18. | 372–502 |
| NcssCount | The method 'selectEntryListByFilter(EntryFilter, Plugin)' has a NCSS line count of 76. | 372–502 |
| UseDiamondOperator | Explicit type arguments can be replaced by Diamond Operator | 374 |
| UseStringBufferForStringAppends | Prefer StringBuilder (non-synchronized) or StringBuffer (synchronized) over += for concatenating strings | 382 |
| UseStringBufferForStringAppends | Prefer StringBuilder (non-synchronized) or StringBuffer (synchronized) over += for concatenating strings | 383 |
| UseStringBufferForStringAppends | Prefer StringBuilder (non-synchronized) or StringBuffer (synchronized) over += for concatenating strings | 384 |
| UseStringBufferForStringAppends | Prefer StringBuilder (non-synchronized) or StringBuffer (synchronized) over += for concatenating strings | 385 |
| UseStringBufferForStringAppends | Prefer StringBuilder (non-synchronized) or StringBuffer (synchronized) over += for concatenating strings | 386 |
| UseStringBufferForStringAppends | Prefer StringBuilder (non-synchronized) or StringBuffer (synchronized) over += for concatenating strings | 387 |
| UseStringBufferForStringAppends | Prefer StringBuilder (non-synchronized) or StringBuffer (synchronized) over += for concatenating strings | 389 |
| IdenticalCatchBranches | 'catch' branch identical to 'ClassNotFoundException' branch | 448–454 |
| IdenticalCatchBranches | 'catch' branch identical to 'ClassNotFoundException' branch | 455–461 |
| NPathComplexity | The method 'selectNumberEntryByFilter(EntryFilter, Plugin)' has an NPath complexity of 9216 | 510–582 |
| MissingOverride | The method 'selectNumberEntryByFilter(EntryFilter, Plugin)' is missing an @Override annotation. | 510–582 |
| CyclomaticComplexity | The method 'selectNumberEntryByFilter(EntryFilter, Plugin)' has a cyclomatic complexity of 15. | 510–582 |
| UseStringBufferForStringAppends | Prefer StringBuilder (non-synchronized) or StringBuffer (synchronized) over += for concatenating strings | 514 |
| UseStringBufferForStringAppends | Prefer StringBuilder (non-synchronized) or StringBuffer (synchronized) over += for concatenating strings | 515 |
| UseStringBufferForStringAppends | Prefer StringBuilder (non-synchronized) or StringBuffer (synchronized) over += for concatenating strings | 516 |
| UseStringBufferForStringAppends | Prefer StringBuilder (non-synchronized) or StringBuffer (synchronized) over += for concatenating strings | 517 |
| UseStringBufferForStringAppends | Prefer StringBuilder (non-synchronized) or StringBuffer (synchronized) over += for concatenating strings | 518 |
| UseStringBufferForStringAppends | Prefer StringBuilder (non-synchronized) or StringBuffer (synchronized) over += for concatenating strings | 519 |
| UseStringBufferForStringAppends | Prefer StringBuilder (non-synchronized) or StringBuffer (synchronized) over += for concatenating strings | 521 |
| Rule | Violation | Ligne |
|---|---|---|
| AvoidReassigningLoopVariables | Avoid reassigning the loop control variable 'field' | 85 |
| AvoidReassigningLoopVariables | Avoid reassigning the loop control variable 'entryChild' | 100 |
| Rule | Violation | Ligne |
|---|---|---|
| FinalFieldCouldBeStatic | This final field could be made static | 57 |
| FinalFieldCouldBeStatic | This final field could be made static | 58 |
| FinalFieldCouldBeStatic | This final field could be made static | 59 |
| MissingOverride | The method 'getHtmlCode()' is missing an @Override annotation. | 66–69 |
| MissingOverride | The method 'getRequestData(HttpServletRequest, Locale)' is missing an @Override annotation. | 77–137 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 80–81 |
| ConfusingTernary | Avoid if (x != y) ..; else ..; | 80–81 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 82 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 83 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 84 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 85 |
| ConfusingTernary | Avoid if (x != y) ..; else ..; | 109–116 |
| AvoidLiteralsInIfCondition | Avoid using Literals in Conditional Statements | 127 |
| MissingOverride | The method 'getTemplateCreate()' is missing an @Override annotation. | 143–146 |
| MissingOverride | The method 'getTemplateModify()' is missing an @Override annotation. | 152–155 |
| MissingOverride | The method 'getPaginator(int, String, String, String)' is missing an @Override annotation. | 165–169 |
| CyclomaticComplexity | The method 'getResponseData(HttpServletRequest, List, Locale)' has a cyclomatic complexity of 11. | 178–249 |
| MissingOverride | The method 'getResponseData(HttpServletRequest, List, Locale)' is missing an @Override annotation. | 178–249 |
| UseDiamondOperator | Explicit type arguments can be replaced by Diamond Operator | 181 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 184 |
| ForLoopCanBeForeach | This for loop can be replaced by a foreach loop | 188–205 |
| UseCollectionIsEmpty | Substitute calls to size() == 0 (or size() != 0, size() > 0, size() < 1) with calls to isEmpty() | 230 |
| MissingOverride | The method 'getResponseValueForExport(HttpServletRequest, Response, Locale)' is missing an @Override annotation. | 258–261 |
| MissingOverride | The method 'getResponseValueForRecap(HttpServletRequest, Response, Locale)' is missing an @Override annotation. | 270–273 |
| Rule | Violation | Ligne |
|---|---|---|
| FinalFieldCouldBeStatic | This final field could be made static | 52 |
| FinalFieldCouldBeStatic | This final field could be made static | 53 |
| FinalFieldCouldBeStatic | This final field could be made static | 54 |
| MissingOverride | The method 'getHtmlCode()' is missing an @Override annotation. | 61–64 |
| MissingOverride | The method 'getRequestData(HttpServletRequest, Locale)' is missing an @Override annotation. | 72–93 |
| MissingOverride | The method 'getTemplateCreate()' is missing an @Override annotation. | 99–102 |
| MissingOverride | The method 'getTemplateModify()' is missing an @Override annotation. | 108–111 |
| Rule | Violation | Ligne |
|---|---|---|
| MissingOverride | The method 'load(int, Plugin)' is missing an @Override annotation. | 62–83 |
| MissingOverride | The method 'select(Plugin)' is missing an @Override annotation. | 90–112 |
| UseDiamondOperator | Explicit type arguments can be replaced by Diamond Operator | 92 |
| Rule | Violation | Ligne |
|---|---|---|
| FinalFieldCouldBeStatic | This final field could be made static | 61 |
| FinalFieldCouldBeStatic | This final field could be made static | 62 |
| FinalFieldCouldBeStatic | This final field could be made static | 63 |
| MissingOverride | The method 'getHtmlCode()' is missing an @Override annotation. | 70–73 |
| CyclomaticComplexity | The method 'getRequestData(HttpServletRequest, Locale)' has a cyclomatic complexity of 10. | 81–141 |
| MissingOverride | The method 'getRequestData(HttpServletRequest, Locale)' is missing an @Override annotation. | 81–141 |
| ConfusingTernary | Avoid if (x != y) ..; else ..; | 84–85 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 84–85 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 86 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 87 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 88 |
| UseDiamondOperator | Explicit type arguments can be replaced by Diamond Operator | 123 |
| ConfusingTernary | Avoid if (x != y) ..; else ..; | 131–138 |
| MissingOverride | The method 'getTemplateCreate()' is missing an @Override annotation. | 147–150 |
| MissingOverride | The method 'getTemplateModify()' is missing an @Override annotation. | 156–159 |
| MissingOverride | The method 'getResponseData(HttpServletRequest, List, Locale)' is missing an @Override annotation. | 168–223 |
| CollapsibleIfStatements | These nested if statements could be combined | 189–196 |
| MissingOverride | The method 'getResponseValueForExport(HttpServletRequest, Response, Locale)' is missing an @Override annotation. | 232–238 |
| MissingOverride | The method 'getResponseValueForRecap(HttpServletRequest, Response, Locale)' is missing an @Override annotation. | 247–253 |
| Rule | Violation | Ligne |
|---|---|---|
| FinalFieldCouldBeStatic | This final field could be made static | 66 |
| FinalFieldCouldBeStatic | This final field could be made static | 67 |
| FinalFieldCouldBeStatic | This final field could be made static | 68 |
| MissingOverride | The method 'getHtmlCode()' is missing an @Override annotation. | 75–78 |
| MissingOverride | The method 'getRequestData(HttpServletRequest, Locale)' is missing an @Override annotation. | 86–156 |
| CyclomaticComplexity | The method 'getRequestData(HttpServletRequest, Locale)' has a cyclomatic complexity of 11. | 86–156 |
| NPathComplexity | The method 'getRequestData(HttpServletRequest, Locale)' has an NPath complexity of 320 | 86–156 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 89–90 |
| ConfusingTernary | Avoid if (x != y) ..; else ..; | 89–90 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 91 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 92 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 96 |
| UseDiamondOperator | Explicit type arguments can be replaced by Diamond Operator | 138 |
| ConfusingTernary | Avoid if (x != y) ..; else ..; | 146–153 |
| MissingOverride | The method 'getTemplateCreate()' is missing an @Override annotation. | 162–165 |
| MissingOverride | The method 'getTemplateModify()' is missing an @Override annotation. | 171–174 |
| CyclomaticComplexity | The method 'getResponseData(HttpServletRequest, List, Locale)' has a cyclomatic complexity of 11. | 183–232 |
| MissingOverride | The method 'getResponseData(HttpServletRequest, List, Locale)' is missing an @Override annotation. | 183–232 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 188 |
| CollapsibleIfStatements | These nested if statements could be combined | 195–202 |
| UseCollectionIsEmpty | Substitute calls to size() == 0 (or size() != 0, size() > 0, size() < 1) with calls to isEmpty() | 208 |
| MissingOverride | The method 'getPaginator(int, String, String, String)' is missing an @Override annotation. | 242–247 |
| MissingOverride | The method 'getReferenceListRegularExpression(IEntry, Plugin)' is missing an @Override annotation. | 255–277 |
| MissingOverride | The method 'getResponseValueForExport(HttpServletRequest, Response, Locale)' is missing an @Override annotation. | 286–289 |
| MissingOverride | The method 'getResponseValueForRecap(HttpServletRequest, Response, Locale)' is missing an @Override annotation. | 298–308 |
| ConfusingTernary | Avoid if (x != y) ..; else ..; | 300–307 |
| Rule | Violation | Ligne |
|---|---|---|
| FinalFieldCouldBeStatic | This final field could be made static | 53 |
| FinalFieldCouldBeStatic | This final field could be made static | 54 |
| MissingOverride | The method 'getHtmlCode()' is missing an @Override annotation. | 61–64 |
| MissingOverride | The method 'getRequestData(HttpServletRequest, Locale)' is missing an @Override annotation. | 72–93 |
| MissingOverride | The method 'getTemplateModify()' is missing an @Override annotation. | 99–102 |
| Rule | Violation | Ligne |
|---|---|---|
| FinalFieldCouldBeStatic | This final field could be made static | 56 |
| FinalFieldCouldBeStatic | This final field could be made static | 57 |
| FinalFieldCouldBeStatic | This final field could be made static | 58 |
| MissingOverride | The method 'getHtmlCode()' is missing an @Override annotation. | 65–68 |
| MissingOverride | The method 'getRequestData(HttpServletRequest, Locale)' is missing an @Override annotation. | 76–136 |
| ConfusingTernary | Avoid if (x != y) ..; else ..; | 79–80 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 79–80 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 81 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 82 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 83 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 84 |
| ConfusingTernary | Avoid if (x != y) ..; else ..; | 108–115 |
| AvoidLiteralsInIfCondition | Avoid using Literals in Conditional Statements | 126 |
| MissingOverride | The method 'getTemplateCreate()' is missing an @Override annotation. | 142–145 |
| MissingOverride | The method 'getTemplateModify()' is missing an @Override annotation. | 151–154 |
| MissingOverride | The method 'getPaginator(int, String, String, String)' is missing an @Override annotation. | 164–168 |
| MissingOverride | The method 'getResponseData(HttpServletRequest, List, Locale)' is missing an @Override annotation. | 177–223 |
| CollapsibleIfStatements | These nested if statements could be combined | 204–211 |
| MissingOverride | The method 'getResponseValueForExport(HttpServletRequest, Response, Locale)' is missing an @Override annotation. | 232–235 |
| MissingOverride | The method 'getResponseValueForRecap(HttpServletRequest, Response, Locale)' is missing an @Override annotation. | 244–247 |
| Rule | Violation | Ligne |
|---|---|---|
| FinalFieldCouldBeStatic | This final field could be made static | 56 |
| FinalFieldCouldBeStatic | This final field could be made static | 57 |
| FinalFieldCouldBeStatic | This final field could be made static | 58 |
| MissingOverride | The method 'getHtmlCode()' is missing an @Override annotation. | 65–68 |
| MissingOverride | The method 'getRequestData(HttpServletRequest, Locale)' is missing an @Override annotation. | 76–116 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 79–80 |
| ConfusingTernary | Avoid if (x != y) ..; else ..; | 79–80 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 81 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 82 |
| ConfusingTernary | Avoid if (x != y) ..; else ..; | 106–113 |
| MissingOverride | The method 'getTemplateCreate()' is missing an @Override annotation. | 122–125 |
| MissingOverride | The method 'getTemplateModify()' is missing an @Override annotation. | 131–134 |
| MissingOverride | The method 'getPaginator(int, String, String, String)' is missing an @Override annotation. | 144–148 |
| MissingOverride | The method 'getResponseData(HttpServletRequest, List, Locale)' is missing an @Override annotation. | 157–203 |
| CollapsibleIfStatements | These nested if statements could be combined | 184–191 |
| MissingOverride | The method 'getResponseValueForExport(HttpServletRequest, Response, Locale)' is missing an @Override annotation. | 212–215 |
| MissingOverride | The method 'getResponseValueForRecap(HttpServletRequest, Response, Locale)' is missing an @Override annotation. | 224–227 |
| Rule | Violation | Ligne |
|---|---|---|
| FinalFieldCouldBeStatic | This final field could be made static | 58 |
| FinalFieldCouldBeStatic | This final field could be made static | 59 |
| FinalFieldCouldBeStatic | This final field could be made static | 60 |
| ConfusingTernary | Avoid if (x != y) ..; else ..; | 85–86 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 85–86 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 87 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 88 |
| ConfusingTernary | Avoid if (x != y) ..; else ..; | 111–118 |
| CollapsibleIfStatements | These nested if statements could be combined | 193–200 |
| UseDiamondOperator | Explicit type arguments can be replaced by Diamond Operator | 258 |
| Rule | Violation | Ligne |
|---|---|---|
| GodClass | Possible God Class (WMC=50, ATFD=40, TCC=0.000%) | 62–406 |
| FinalFieldCouldBeStatic | This final field could be made static | 64 |
| FinalFieldCouldBeStatic | This final field could be made static | 65 |
| FinalFieldCouldBeStatic | This final field could be made static | 66 |
| MissingOverride | The method 'getHtmlCode()' is missing an @Override annotation. | 73–76 |
| NcssCount | The method 'getRequestData(HttpServletRequest, Locale)' has a NCSS line count of 60. | 84–202 |
| NPathComplexity | The method 'getRequestData(HttpServletRequest, Locale)' has an NPath complexity of 20480 | 84–202 |
| ExcessiveMethodLength | Avoid really long methods. | 84–202 |
| MissingOverride | The method 'getRequestData(HttpServletRequest, Locale)' is missing an @Override annotation. | 84–202 |
| CyclomaticComplexity | The method 'getRequestData(HttpServletRequest, Locale)' has a cyclomatic complexity of 19. | 84–202 |
| ConfusingTernary | Avoid if (x != y) ..; else ..; | 87–88 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 87–88 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 89 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 90 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 91 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 93 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 96 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 98 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 99 |
| UseDiamondOperator | Explicit type arguments can be replaced by Diamond Operator | 162 |
| ConfusingTernary | Avoid if (x != y) ..; else ..; | 172–179 |
| ConfusingTernary | Avoid if (x != y) ..; else ..; | 181–190 |
| ConfusingTernary | Avoid if (x != y) ..; else ..; | 192–199 |
| MissingOverride | The method 'getTemplateCreate()' is missing an @Override annotation. | 208–211 |
| MissingOverride | The method 'getTemplateModify()' is missing an @Override annotation. | 217–220 |
| MissingOverride | The method 'getPaginator(int, String, String, String)' is missing an @Override annotation. | 230–235 |
| MissingOverride | The method 'getReferenceListRegularExpression(IEntry, Plugin)' is missing an @Override annotation. | 243–265 |
| ExcessiveMethodLength | Avoid really long methods. | 274–381 |
| NPathComplexity | The method 'getResponseData(HttpServletRequest, List, Locale)' has an NPath complexity of 2354 | 274–381 |
| MissingOverride | The method 'getResponseData(HttpServletRequest, List, Locale)' is missing an @Override annotation. | 274–381 |
| CyclomaticComplexity | The method 'getResponseData(HttpServletRequest, List, Locale)' has a cyclomatic complexity of 21. | 274–381 |
| CollapsibleIfStatements | These nested if statements could be combined | 306–313 |
| UseCollectionIsEmpty | Substitute calls to size() == 0 (or size() != 0, size() > 0, size() < 1) with calls to isEmpty() | 317 |
| MissingOverride | The method 'getResponseValueForExport(HttpServletRequest, Response, Locale)' is missing an @Override annotation. | 390–393 |
| MissingOverride | The method 'getResponseValueForRecap(HttpServletRequest, Response, Locale)' is missing an @Override annotation. | 402–405 |
| Rule | Violation | Ligne |
|---|---|---|
| FinalFieldCouldBeStatic | This final field could be made static | 56 |
| FinalFieldCouldBeStatic | This final field could be made static | 57 |
| FinalFieldCouldBeStatic | This final field could be made static | 58 |
| MissingOverride | The method 'getHtmlCode()' is missing an @Override annotation. | 65–68 |
| CyclomaticComplexity | The method 'getRequestData(HttpServletRequest, Locale)' has a cyclomatic complexity of 14. | 76–166 |
| MissingOverride | The method 'getRequestData(HttpServletRequest, Locale)' is missing an @Override annotation. | 76–166 |
| NPathComplexity | The method 'getRequestData(HttpServletRequest, Locale)' has an NPath complexity of 896 | 76–166 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 79–80 |
| ConfusingTernary | Avoid if (x != y) ..; else ..; | 79–80 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 81 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 82 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 83 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 87 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 88 |
| UseDiamondOperator | Explicit type arguments can be replaced by Diamond Operator | 146 |
| ConfusingTernary | Avoid if (x != y) ..; else ..; | 156–163 |
| MissingOverride | The method 'getTemplateCreate()' is missing an @Override annotation. | 172–175 |
| MissingOverride | The method 'getTemplateModify()' is missing an @Override annotation. | 181–184 |
| MissingOverride | The method 'getResponseData(HttpServletRequest, List, Locale)' is missing an @Override annotation. | 193–230 |
| CollapsibleIfStatements | These nested if statements could be combined | 214–221 |
| MissingOverride | The method 'getResponseValueForExport(HttpServletRequest, Response, Locale)' is missing an @Override annotation. | 239–242 |
| MissingOverride | The method 'getResponseValueForRecap(HttpServletRequest, Response, Locale)' is missing an @Override annotation. | 251–254 |
| Rule | Violation | Ligne |
|---|---|---|
| MethodReturnsInternalArray | Returning '_xsl' may expose an internal array. | 113 |
| ArrayIsStoredDirectly | The user-supplied array 'donnees' is stored directly. | 120 |
| MissingOverride | The method 'getResourceTypeCode()' is missing an @Override annotation. | 129–132 |
| MissingOverride | The method 'getResourceId()' is missing an @Override annotation. | 138–141 |
| AddEmptyString | Do not add empty strings | 140 |
| Rule | Violation | Ligne |
|---|---|---|
| AvoidSynchronizedAtMethodLevel | Use block level rather than method level synchronization | 91–102 |
| MissingOverride | The method 'insert(ExportFormat, Plugin)' is missing an @Override annotation. | 91–102 |
| MissingOverride | The method 'load(int, Plugin)' is missing an @Override annotation. | 111–132 |
| MissingOverride | The method 'delete(int, Plugin)' is missing an @Override annotation. | 140–146 |
| MissingOverride | The method 'store(ExportFormat, Plugin)' is missing an @Override annotation. | 154–165 |
| MissingOverride | The method 'selectListExport(Plugin)' is missing an @Override annotation. | 173–193 |
| UseDiamondOperator | Explicit type arguments can be replaced by Diamond Operator | 175 |
| MissingOverride | The method 'getListExport(Plugin)' is missing an @Override annotation. | 200–218 |
| Rule | Violation | Ligne |
|---|---|---|
| MissingOverride | The method 'insert(Field, Plugin)' is missing an @Override annotation. | 125–145 |
| AvoidSynchronizedAtMethodLevel | Use block level rather than method level synchronization | 125–145 |
| NullAssignment | Assigning an Object to null is a code smell. Consider refactoring. | 139 |
| MissingOverride | The method 'load(int, Plugin)' is missing an @Override annotation. | 154–185 |
| MissingOverride | The method 'delete(int, Plugin)' is missing an @Override annotation. | 193–199 |
| MissingOverride | The method 'store(Field, Plugin)' is missing an @Override annotation. | 207–226 |
| NullAssignment | Assigning an Object to null is a code smell. Consider refactoring. | 220 |
| MissingOverride | The method 'selectFieldListByIdEntry(int, Plugin)' is missing an @Override annotation. | 234–267 |
| UseDiamondOperator | Explicit type arguments can be replaced by Diamond Operator | 236 |
| MissingOverride | The method 'deleteVerifyBy(int, int, Plugin)' is missing an @Override annotation. | 276–283 |
| MissingOverride | The method 'insertVerifyBy(int, int, Plugin)' is missing an @Override annotation. | 292–299 |
| MissingOverride | The method 'selectListRegularExpressionKeyByIdField(int, Plugin)' is missing an @Override annotation. | 307–322 |
| UseDiamondOperator | Explicit type arguments can be replaced by Diamond Operator | 309 |
| MissingOverride | The method 'isRegularExpressionIsUse(int, Plugin)' is missing an @Override annotation. | 331–347 |
| Rule | Violation | Ligne |
|---|---|---|
| UseDiamondOperator | Explicit type arguments can be replaced by Diamond Operator | 153 |
| AvoidDeeplyNestedIfStmts | Deeply nested if..then statements are hard to read | 159–173 |
| UseCollectionIsEmpty | Substitute calls to size() == 0 (or size() != 0, size() > 0, size() < 1) with calls to isEmpty() | 159 |
| Rule | Violation | Ligne |
|---|---|---|
| ExcessivePublicCount | This class has a bunch of public methods and attributes | 34–637 |
| TooManyFields | Too many fields | 52–637 |
| NonThreadSafeSingleton | Singleton is not thread safe | 94–98 |
| NonThreadSafeSingleton | Singleton is not thread safe | 100–104 |
| MissingOverride | The method 'getWorkgroup()' is missing an @Override annotation. | 309–312 |
| MissingOverride | The method 'getResourceTypeCode()' is missing an @Override annotation. | 417–420 |
| MissingOverride | The method 'getResourceId()' is missing an @Override annotation. | 426–429 |
| AddEmptyString | Do not add empty strings | 428 |
| Rule | Violation | Ligne |
|---|---|---|
| MissingOverride | The method 'getPermission()' is missing an @Override annotation. | 184–187 |
| MissingOverride | The method 'setLocale(Locale)' is missing an @Override annotation. | 203–206 |
| Rule | Violation | Ligne |
|---|---|---|
| MissingOverride | The method 'selectActionsByFormState(int, Plugin)' is missing an @Override annotation. | 57–79 |
| UseDiamondOperator | Explicit type arguments can be replaced by Diamond Operator | 59 |
| Rule | Violation | Ligne |
|---|---|---|
| AvoidSynchronizedAtMethodLevel | Use block level rather than method level synchronization | 116–156 |
| MissingOverride | The method 'insert(Form, Plugin)' is missing an @Override annotation. | 116–156 |
| ConfusingTernary | Avoid if (x != y) ..; else ..; | 135–136 |
| NullAssignment | Assigning an Object to null is a code smell. Consider refactoring. | 136 |
| NullAssignment | Assigning an Object to null is a code smell. Consider refactoring. | 138 |
| ConfusingTernary | Avoid if (x != y) ..; else ..; | 138 |
| MissingOverride | The method 'load(int, Plugin)' is missing an @Override annotation. | 165–211 |
| MissingOverride | The method 'delete(int, Plugin)' is missing an @Override annotation. | 219–225 |
| MissingOverride | The method 'store(Form, Plugin)' is missing an @Override annotation. | 233–269 |
| ConfusingTernary | Avoid if (x != y) ..; else ..; | 250–251 |
| NullAssignment | Assigning an Object to null is a code smell. Consider refactoring. | 251 |
| NullAssignment | Assigning an Object to null is a code smell. Consider refactoring. | 253 |
| ConfusingTernary | Avoid if (x != y) ..; else ..; | 253 |
| ExcessiveMethodLength | Avoid really long methods. | 277–394 |
| NPathComplexity | The method 'selectFormList(FormFilter, Plugin)' has an NPath complexity of 3072 | 277–394 |
| CyclomaticComplexity | The method 'selectFormList(FormFilter, Plugin)' has a cyclomatic complexity of 14. | 277–394 |
| NcssCount | The method 'selectFormList(FormFilter, Plugin)' has a NCSS line count of 71. | 277–394 |
| MissingOverride | The method 'selectFormList(FormFilter, Plugin)' is missing an @Override annotation. | 277–394 |
| UseDiamondOperator | Explicit type arguments can be replaced by Diamond Operator | 279 |
| UseDiamondOperator | Explicit type arguments can be replaced by Diamond Operator | 282 |
| MissingOverride | The method 'getEnableFormList(Plugin)' is missing an @Override annotation. | 401–423 |
| Rule | Violation | Ligne |
|---|---|---|
| AvoidReassigningLoopVariables | Avoid reassigning the loop control variable 'entry' | 96 |
| Rule | Violation | Ligne |
|---|---|---|
| MissingOverride | The method 'canBeRemoved(String)' is missing an @Override annotation. | 57–77 |
| MissingOverride | The method 'getRemovalRefusedMessage(String, Locale)' is missing an @Override annotation. | 85–89 |
| Rule | Violation | Ligne |
|---|---|---|
| ShortVariable | Avoid variables with short names like ip | 104 |
| Rule | Violation | Ligne |
|---|---|---|
| AvoidSynchronizedAtMethodLevel | Use block level rather than method level synchronization | 104–120 |
| MissingOverride | The method 'insert(FormSubmit, Plugin)' is missing an @Override annotation. | 104–120 |
| MissingOverride | The method 'load(int, Plugin)' is missing an @Override annotation. | 129–151 |
| MissingOverride | The method 'delete(int, Plugin)' is missing an @Override annotation. | 159–165 |
| MissingOverride | The method 'store(FormSubmit, Plugin)' is missing an @Override annotation. | 173–183 |
| MissingOverride | The method 'selectListByFilter(ResponseFilter, Plugin)' is missing an @Override annotation. | 191–253 |
| UseDiamondOperator | Explicit type arguments can be replaced by Diamond Operator | 193 |
| UseDiamondOperator | Explicit type arguments can be replaced by Diamond Operator | 196 |
| MissingOverride | The method 'selectCountByFilter(ResponseFilter, Plugin)' is missing an @Override annotation. | 261–313 |
| UseDiamondOperator | Explicit type arguments can be replaced by Diamond Operator | 264 |
| MissingOverride | The method 'selectStatisticFormSubmit(ResponseFilter, Plugin)' is missing an @Override annotation. | 321–394 |
| CyclomaticComplexity | The method 'selectStatisticFormSubmit(ResponseFilter, Plugin)' has a cyclomatic complexity of 11. | 321–394 |
| NPathComplexity | The method 'selectStatisticFormSubmit(ResponseFilter, Plugin)' has an NPath complexity of 1024 | 321–394 |
| UseDiamondOperator | Explicit type arguments can be replaced by Diamond Operator | 323 |
| UseDiamondOperator | Explicit type arguments can be replaced by Diamond Operator | 325 |
| UseDiamondOperator | Explicit type arguments can be replaced by Diamond Operator | 326 |
| Rule | Violation | Ligne |
|---|---|---|
| MissingOverride | The method 'canBeRemoved(String)' is missing an @Override annotation. | 57–76 |
| MissingOverride | The method 'getRemovalRefusedMessage(String, Locale)' is missing an @Override annotation. | 84–88 |
| Rule | Violation | Ligne |
|---|---|---|
| MissingOverride | The method 'createChart(List, String, boolean, boolean)' is missing an @Override annotation. | 72–76 |
| Rule | Violation | Ligne |
|---|---|---|
| MissingOverride | The method 'load(int, Plugin)' is missing an @Override annotation. | 61–104 |
| IdenticalCatchBranches | 'catch' branch identical to 'ClassNotFoundException' branch | 85–91 |
| IdenticalCatchBranches | 'catch' branch identical to 'ClassNotFoundException' branch | 92–98 |
| MissingOverride | The method 'select(Plugin)' is missing an @Override annotation. | 111–130 |
| UseDiamondOperator | Explicit type arguments can be replaced by Diamond Operator | 116 |
| Rule | Violation | Ligne |
|---|---|---|
| MissingOverride | The method 'createChart(List, String, boolean, boolean)' is missing an @Override annotation. | 62–66 |
| Rule | Violation | Ligne |
|---|---|---|
| ExcessivePublicCount | This class has a bunch of public methods and attributes | 34–335 |
| Rule | Violation | Ligne |
|---|---|---|
| MissingOverride | The method 'insert(Recap, Plugin)' is missing an @Override annotation. | 89–116 |
| AvoidSynchronizedAtMethodLevel | Use block level rather than method level synchronization | 89–116 |
| MissingOverride | The method 'load(int, Plugin)' is missing an @Override annotation. | 125–160 |
| MissingOverride | The method 'delete(int, Plugin)' is missing an @Override annotation. | 168–174 |
| MissingOverride | The method 'store(Recap, Plugin)' is missing an @Override annotation. | 182–207 |
| Rule | Violation | Ligne |
|---|---|---|
| MethodReturnsInternalArray | Returning '_byValueResponse' may expose an internal array. | 113 |
| ArrayIsStoredDirectly | The user-supplied array 'valueResponse' is stored directly. | 120 |
| Rule | Violation | Ligne |
|---|---|---|
| MissingOverride | The method 'insert(Response, Plugin)' is missing an @Override annotation. | 104–128 |
| AvoidSynchronizedAtMethodLevel | Use block level rather than method level synchronization | 104–128 |
| ConfusingTernary | Avoid if (x != y) ..; else ..; | 113–120 |
| MissingOverride | The method 'load(int, Plugin)' is missing an @Override annotation. | 137–207 |
| IdenticalCatchBranches | 'catch' branch identical to 'ClassNotFoundException' branch | 170–175 |
| IdenticalCatchBranches | 'catch' branch identical to 'ClassNotFoundException' branch | 176–181 |
| MissingOverride | The method 'delete(int, Plugin)' is missing an @Override annotation. | 215–221 |
| MissingOverride | The method 'store(Response, Plugin)' is missing an @Override annotation. | 229–252 |
| ConfusingTernary | Avoid if (x != y) ..; else ..; | 238–245 |
| MissingOverride | The method 'selectListByFilter(ResponseFilter, Plugin)' is missing an @Override annotation. | 260–358 |
| CyclomaticComplexity | The method 'selectListByFilter(ResponseFilter, Plugin)' has a cyclomatic complexity of 13. | 260–358 |
| NPathComplexity | The method 'selectListByFilter(ResponseFilter, Plugin)' has an NPath complexity of 1088 | 260–358 |
| UseDiamondOperator | Explicit type arguments can be replaced by Diamond Operator | 263 |
| UseStringBufferForStringAppends | Prefer StringBuilder (non-synchronized) or StringBuffer (synchronized) over += for concatenating strings | 271 |
| UseStringBufferForStringAppends | Prefer StringBuilder (non-synchronized) or StringBuffer (synchronized) over += for concatenating strings | 272 |
| UseStringBufferForStringAppends | Prefer StringBuilder (non-synchronized) or StringBuffer (synchronized) over += for concatenating strings | 273 |
| UseStringBufferForStringAppends | Prefer StringBuilder (non-synchronized) or StringBuffer (synchronized) over += for concatenating strings | 274 |
| IdenticalCatchBranches | 'catch' branch identical to 'ClassNotFoundException' branch | 320–325 |
| IdenticalCatchBranches | 'catch' branch identical to 'ClassNotFoundException' branch | 326–331 |
| MissingOverride | The method 'getStatisticByIdEntry(int, Plugin)' is missing an @Override annotation. | 366–385 |
| UseDiamondOperator | Explicit type arguments can be replaced by Diamond Operator | 368 |
| Rule | Violation | Ligne |
|---|---|---|
| CommentRequired | Public method and constructor comments are required | 60–80 |
| MissingOverride | The method 'selectByIdForm(int, Plugin)' is missing an @Override annotation. | 60–80 |
| UseDiamondOperator | Explicit type arguments can be replaced by Diamond Operator | 62 |
| CommentRequired | Public method and constructor comments are required | 85–105 |
| MissingOverride | The method 'selectAll(Plugin)' is missing an @Override annotation. | 85–105 |
| UseDiamondOperator | Explicit type arguments can be replaced by Diamond Operator | 87 |
| CommentRequired | Public method and constructor comments are required | 110–119 |
| MissingOverride | The method 'insert(FormProcessor, Plugin)' is missing an @Override annotation. | 110–119 |
| CommentRequired | Public method and constructor comments are required | 124–132 |
| MissingOverride | The method 'delete(FormProcessor, Plugin)' is missing an @Override annotation. | 124–132 |
| Rule | Violation | Ligne |
|---|---|---|
| CommentRequired | Header comments are required | 41–71 |
| Rule | Violation | Ligne |
|---|---|---|
| CommentRequired | Header comments are required | 40–60 |
| Rule | Violation | Ligne |
|---|---|---|
| CommentRequired | Public method and constructor comments are required | 50–53 |
| MissingOverride | The method 'getKey()' is missing an @Override annotation. | 50–53 |
| CommentRequired | Public method and constructor comments are required | 58–61 |
| MissingOverride | The method 'setKey(String)' is missing an @Override annotation. | 58–61 |
| Rule | Violation | Ligne |
|---|---|---|
| CommentRequired | Public method and constructor comments are required | 51–60 |
| MissingOverride | The method 'setOutputProcessorMap(Map)' is missing an @Override annotation. | 51–60 |
| MissingOverride | The method 'getOutputProcessor(String)' is missing an @Override annotation. | 65–70 |
| CommentRequired | Public method and constructor comments are required | 65–70 |
| UnnecessaryLocalBeforeReturn | Consider simply returning the value vs storing it in local variable 'outputProcessor' | 69 |
| MissingOverride | The method 'getAllOutputProcessor()' is missing an @Override annotation. | 76–82 |
| CommentRequired | Public method and constructor comments are required | 76–82 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 78 |
| UnusedLocalVariable | Avoid unused local variables such as 'test'. | 78 |
| UnnecessaryLocalBeforeReturn | Consider simply returning the value vs storing it in local variable 'list' | 81 |
| Rule | Violation | Ligne |
|---|---|---|
| UncommentedEmptyConstructor | Document empty constructor | 72–74 |
| UnnecessaryConstructor | Avoid unnecessary constructors - the compiler will generate these for you | 72–74 |
| CallSuperInConstructor | It is a good practice to call super() in a constructor | 72–74 |
| MissingOverride | The method 'getXml(HttpServletRequest)' is missing an @Override annotation. | 82–127 |
| ConfusingTernary | Avoid if (x != y) ..; else ..; | 87–94 |
| UseStringBufferForStringAppends | Prefer StringBuilder (non-synchronized) or StringBuffer (synchronized) over += for concatenating strings | 112 |
| UnnecessaryLocalBeforeReturn | Consider simply returning the value vs storing it in local variable 'str' | 126 |
| MissingOverride | The method 'getXmlDocument(HttpServletRequest)' is missing an @Override annotation. | 135–138 |
| MissingOverride | The method 'remove()' is missing an @Override annotation. | 151–154 |
| MissingOverride | The method 'getStatus()' is missing an @Override annotation. | 201–204 |
| MissingOverride | The method 'setStatus(int)' is missing an @Override annotation. | 211–214 |
| Rule | Violation | Ligne |
|---|---|---|
| MissingOverride | The method 'insert(Portlet)' is missing an @Override annotation. | 60–70 |
| ShortVariable | Avoid variables with short names like p | 62 |
| MissingOverride | The method 'delete(int)' is missing an @Override annotation. | 78–84 |
| MissingOverride | The method 'load(int)' is missing an @Override annotation. | 93–110 |
| MissingOverride | The method 'selectCountPortletByIdForm(int)' is missing an @Override annotation. | 117–132 |
| MissingOverride | The method 'store(Portlet)' is missing an @Override annotation. | 140–149 |
| ShortVariable | Avoid variables with short names like p | 142 |
| Rule | Violation | Ligne |
|---|---|---|
| AssignmentToNonFinalStatic | Possible unsafe assignment to a non-final static field in a constructor. | 58 |
| CallSuperInConstructor | It is a good practice to call super() in a constructor | 63–69 |
| NonThreadSafeSingleton | Singleton is not thread safe | 78–81 |
| MissingOverride | The method 'getPortletTypeId()' is missing an @Override annotation. | 91–97 |
| UnnecessaryLocalBeforeReturn | Consider simply returning the value vs storing it in local variable 'strPortletTypeId' | 96 |
| MissingOverride | The method 'getDAO()' is missing an @Override annotation. | 104–107 |
| Rule | Violation | Ligne |
|---|---|---|
| MissingOverride | The method 'delete(int)' is missing an @Override annotation. | 51 |
| MissingOverride | The method 'insert(Portlet)' is missing an @Override annotation. | 59 |
| MissingOverride | The method 'load(int)' is missing an @Override annotation. | 68 |
| MissingOverride | The method 'store(Portlet)' is missing an @Override annotation. | 83 |
| Rule | Violation | Ligne |
|---|---|---|
| MissingOverride | The method 'insert(NotifySenderConfiguration, Plugin)' is missing an @Override annotation. | 58–68 |
| CommentRequired | Public method and constructor comments are required | 58–68 |
| MissingOverride | The method 'load(int, Plugin)' is missing an @Override annotation. | 73–93 |
| CommentRequired | Public method and constructor comments are required | 73–93 |
| CommentRequired | Public method and constructor comments are required | 98–104 |
| MissingOverride | The method 'delete(int, Plugin)' is missing an @Override annotation. | 98–104 |
| MissingOverride | The method 'store(NotifySenderConfiguration, Plugin)' is missing an @Override annotation. | 109–120 |
| CommentRequired | Public method and constructor comments are required | 109–120 |
| MissingOverride | The method 'selectNotifySenderConfigurationsList(Plugin)' is missing an @Override annotation. | 125–146 |
| CommentRequired | Public method and constructor comments are required | 125–146 |
| UseDiamondOperator | Explicit type arguments can be replaced by Diamond Operator | 127 |
| Rule | Violation | Ligne |
|---|---|---|
| MissingOverride | The method 'getOutputConfigForm(HttpServletRequest, Form, Locale, Plugin)' is missing an @Override annotation. | 93–111 |
| CommentRequired | Public method and constructor comments are required | 93–111 |
| UseDiamondOperator | Explicit type arguments can be replaced by Diamond Operator | 100 |
| UseConcurrentHashMap | If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation | 100 |
| MissingOverride | The method 'doOutputConfigForm(HttpServletRequest, Locale, Plugin)' is missing an @Override annotation. | 117–152 |
| CommentRequired | Public method and constructor comments are required | 117–152 |
| ConfusingTernary | Avoid if (x != y) ..; else ..; | 142–149 |
| MissingOverride | The method 'process(FormSubmit, HttpServletRequest, Plugin)' is missing an @Override annotation. | 158–237 |
| CommentRequired | Public method and constructor comments are required | 158–237 |
| UseDiamondOperator | Explicit type arguments can be replaced by Diamond Operator | 187 |
| ConfusingTernary | Avoid if (x != y) ..; else ..; | 197–206 |
| ShortVariable | Avoid variables with short names like t | 223 |
| AvoidCatchingGenericException | Avoid catching generic exceptions such as NullPointerException, RuntimeException, Exception in try-catch block | 231 |
| UnusedFormalParameter | Avoid unused method parameters such as 'locale'. | 247 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 252 |
| UnusedLocalVariable | Avoid unused local variables such as 'strSendRecap'. | 253 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 253 |
| Rule | Violation | Ligne |
|---|---|---|
| CallSuperInConstructor | It is a good practice to call super() in a constructor | 61–64 |
| MissingOverride | The method 'register()' is missing an @Override annotation. | 69–82 |
| ShortVariable | Avoid variables with short names like rt | 71 |
| ShortVariable | Avoid variables with short names like p | 77 |
| MissingOverride | The method 'getResourceIdList(Locale)' is missing an @Override annotation. | 89–92 |
| MissingOverride | The method 'getTitle(String, Locale)' is missing an @Override annotation. | 100–103 |
| Rule | Violation | Ligne |
|---|---|---|
| CallSuperInConstructor | It is a good practice to call super() in a constructor | 64–67 |
| MissingOverride | The method 'register()' is missing an @Override annotation. | 72–85 |
| ShortVariable | Avoid variables with short names like rt | 74 |
| ShortVariable | Avoid variables with short names like p | 80 |
| MissingOverride | The method 'getResourceIdList(Locale)' is missing an @Override annotation. | 92–95 |
| MissingOverride | The method 'getTitle(String, Locale)' is missing an @Override annotation. | 103–119 |
| Rule | Violation | Ligne |
|---|---|---|
| MissingOverride | The method 'init()' is missing an @Override annotation. | 52–56 |
| Rule | Violation | Ligne |
|---|---|---|
| CallSuperInConstructor | It is a good practice to call super() in a constructor | 96–99 |
| MissingOverride | The method 'register()' is missing an @Override annotation. | 104–158 |
| ShortVariable | Avoid variables with short names like rt | 106 |
| ShortVariable | Avoid variables with short names like p | 112 |
| MissingOverride | The method 'getResourceIdList(Locale)' is missing an @Override annotation. | 165–168 |
| MissingOverride | The method 'getTitle(String, Locale)' is missing an @Override annotation. | 176–192 |
| Rule | Violation | Ligne |
|---|---|---|
| ImmutableField | Private field '_OutputProcessorSet' could be made final; it is only initialized in the declaration or constructor. | 56 |
| CommentRequired | Public method and constructor comments are required | 65–73 |
| NonThreadSafeSingleton | Singleton is not thread safe | 67–70 |
| CommentRequired | Public method and constructor comments are required | 75–78 |
| CommentRequired | Public method and constructor comments are required | 80–99 |
| UseDiamondOperator | Explicit type arguments can be replaced by Diamond Operator | 82 |
| CommentRequired | Public method and constructor comments are required | 101–110 |
| CommentRequired | Public method and constructor comments are required | 112–119 |
| CommentRequired | Public method and constructor comments are required | 121–128 |
| CommentRequired | Public method and constructor comments are required | 130–144 |
| CommentRequired | Public method and constructor comments are required | 146–149 |
| Rule | Violation | Ligne |
|---|---|---|
| MissingOverride | The method 'run()' is missing an @Override annotation. | 50–53 |
| Rule | Violation | Ligne |
|---|---|---|
| UnusedLocalVariable | Avoid unused local variables such as 'autoPublicationActive'. | 83 |
| Rule | Violation | Ligne |
|---|---|---|
| ExcessiveImports | A high number of imports can indicate a high degree of coupling within an object. | 34–1197 |
| ExcessiveClassLength | Avoid really long classes. | 101–1197 |
| CyclomaticComplexity | The class 'FormUtils' has a total cyclomatic complexity of 126 (highest 14). | 101–1197 |
| GodClass | Possible God Class (WMC=126, ATFD=237, TCC=2.151%) | 101–1197 |
| TooManyMethods | This class has too many methods, consider refactoring it. | 102–1197 |
| ShortVariable | Avoid variables with short names like t | 174 |
| AvoidCatchingGenericException | Avoid catching generic exceptions such as NullPointerException, RuntimeException, Exception in try-catch block | 184 |
| ShortVariable | Avoid variables with short names like t | 208 |
| AvoidCatchingGenericException | Avoid catching generic exceptions such as NullPointerException, RuntimeException, Exception in try-catch block | 218 |
| UnnecessaryLocalBeforeReturn | Consider simply returning the value vs storing it in local variable 'timeStamp' | 246 |
| UnnecessaryLocalBeforeReturn | Consider simply returning the value vs storing it in local variable 'timeStamp' | 271 |
| UnnecessaryLocalBeforeReturn | Consider simply returning the value vs storing it in local variable 'timeStamp1' | 362 |
| CyclomaticComplexity | The method 'sameDate(Timestamp, Timestamp, String)' has a cyclomatic complexity of 10. | 372–400 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 452 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 453 |
| IdenticalCatchBranches | 'catch' branch identical to 'ClassNotFoundException' branch | 486–490 |
| IdenticalCatchBranches | 'catch' branch identical to 'ClassNotFoundException' branch | 491–495 |
| LinguisticNaming | Linguistics Antipattern - The getter 'getHtmlEntry' should not return void linguistically | 595–653 |
| CyclomaticComplexity | The method 'getResponseEntry(HttpServletRequest, int, Plugin, FormSubmit, boolean, Locale)' has a cyclomatic complexity of 14. | 666–746 |
| UseDiamondOperator | Explicit type arguments can be replaced by Diamond Operator | 672 |
| UseDiamondOperator | Explicit type arguments can be replaced by Diamond Operator | 675 |
| AvoidReassigningLoopVariables | Avoid reassigning the loop control variable 'field' | 679 |
| ConfusingTernary | Avoid if (x != y) ..; else ..; | 699–708 |
| CyclomaticComplexity | The method 'getXmlResponses(HttpServletRequest, Form, List, Locale, Plugin)' has a cyclomatic complexity of 12. | 795–871 |
| ConfusingTernary | Avoid if (x != y) ..; else ..; | 812–819 |
| ConfusingTernary | Avoid if (x != y) ..; else ..; | 846–854 |
| PositionLiteralsFirstInComparisons | Position literals first in String comparisons | 885 |
| ConfusingTernary | Avoid if (x != y) ..; else ..; | 893–900 |
| CommentRequired | Public method and constructor comments are required | 1025–1042 |
| ConfusingTernary | Avoid if (x != y) ..; else ..; | 1031–1038 |
| UseDiamondOperator | Explicit type arguments can be replaced by Diamond Operator | 1052 |
| AvoidReassigningLoopVariables | Avoid reassigning the loop control variable 'field' | 1095 |
| UseDiamondOperator | Explicit type arguments can be replaced by Diamond Operator | 1116 |
| UseDiamondOperator | Explicit type arguments can be replaced by Diamond Operator | 1130 |
| AvoidLiteralsInIfCondition | Avoid using Literals in Conditional Statements | 1169 |
| AssignmentInOperand | Avoid assignments in operands | 1169 |
| Rule | Violation | Ligne |
|---|---|---|
| MissingSerialVersionUID | Classes implementing Serializable should set a serialVersionUID | 61–204 |
| UnusedPrivateField | Avoid unused private fields such as 'FIELD_LIBELLE_RESET_BUTTON'. | 92 |
| CyclomaticComplexity | The method 'getDefaultMessageData(HttpServletRequest, DefaultMessage)' has a cyclomatic complexity of 14. | 150–203 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 157 |
| Rule | Violation | Ligne |
|---|---|---|
| CyclomaticComplexity | The method 'doGenerateGraph(HttpServletRequest, HttpServletResponse)' has a cyclomatic complexity of 14. | 80–149 |
| AvoidCatchingGenericException | Avoid catching generic exceptions such as NullPointerException, RuntimeException, Exception in try-catch block | 144 |
| Rule | Violation | Ligne |
|---|---|---|
| MissingSerialVersionUID | Classes implementing Serializable should set a serialVersionUID | 78–456 |
| ImmutableField | Private field '_nDefaultItemsPerPage' could be made final; it is only initialized in the declaration or constructor. | 123 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 216 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 217 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 218 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 220 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 260 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 261 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 327 |
| CyclomaticComplexity | The method 'getExportFormatData(MultipartHttpServletRequest, ExportFormat)' has a cyclomatic complexity of 16. | 362–421 |
| NPathComplexity | The method 'getExportFormatData(MultipartHttpServletRequest, ExportFormat)' has an NPath complexity of 240 | 362–421 |
| LinguisticNaming | Linguistics Antipattern - The method 'isValid' indicates linguistically it returns a boolean, but it returns 'String' | 428–445 |
| ShortVariable | Avoid variables with short names like is | 436 |
| AvoidCatchingGenericException | Avoid catching generic exceptions such as NullPointerException, RuntimeException, Exception in try-catch block | 439 |
| Rule | Violation | Ligne |
|---|---|---|
| ExcessiveImports | A high number of imports can indicate a high degree of coupling within an object. | 34–646 |
| MissingSerialVersionUID | Classes implementing Serializable should set a serialVersionUID | 84–646 |
| UnusedPrivateField | Avoid unused private fields such as 'PARAMETER_PAGE_ID'. | 115 |
| MissingOverride | The method 'getPage(HttpServletRequest, int, Plugin)' is missing an @Override annotation. | 146–225 |
| CyclomaticComplexity | The method 'getPage(HttpServletRequest, int, Plugin)' has a cyclomatic complexity of 16. | 146–225 |
| NPathComplexity | The method 'getPage(HttpServletRequest, int, Plugin)' has an NPath complexity of 480 | 146–225 |
| ConfusingTernary | Avoid if (x != y) ..; else ..; | 187–222 |
| ConfusingTernary | Avoid if (x != y) ..; else ..; | 195–222 |
| ConfusingTernary | Avoid if (x != y) ..; else ..; | 202–222 |
| ConfusingTernary | Avoid if (x != y) ..; else ..; | 210–222 |
| UnusedFormalParameter | Avoid unused method parameters such as 'nMode'. | 235 |
| CyclomaticComplexity | The method 'getResult(HttpServletRequest, HttpSession, int, Plugin)' has a cyclomatic complexity of 16. | 235–296 |
| UseDiamondOperator | Explicit type arguments can be replaced by Diamond Operator | 287 |
| UseConcurrentHashMap | If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation | 287 |
| UnusedFormalParameter | Avoid unused method parameters such as 'nMode'. | 307 |
| UnusedFormalParameter | Avoid unused method parameters such as 'session'. | 307 |
| UseDiamondOperator | Explicit type arguments can be replaced by Diamond Operator | 313 |
| UnusedFormalParameter | Avoid unused method parameters such as 'nMode'. | 332 |
| UnusedFormalParameter | Avoid unused method parameters such as 'session'. | 332 |
| UseDiamondOperator | Explicit type arguments can be replaced by Diamond Operator | 336 |
| ConfusingTernary | Avoid if (x != y) ..; else ..; | 354–370 |
| UseStringBufferForStringAppends | Prefer StringBuilder (non-synchronized) or StringBuffer (synchronized) over += for concatenating strings | 364 |
| UnusedFormalParameter | Avoid unused method parameters such as 'nMode'. | 388 |
| ExcessiveMethodLength | Avoid really long methods. | 388–503 |
| CyclomaticComplexity | The method 'getRecap(HttpServletRequest, HttpSession, int, Plugin)' has a cyclomatic complexity of 20. | 388–503 |
| NPathComplexity | The method 'getRecap(HttpServletRequest, HttpSession, int, Plugin)' has an NPath complexity of 11520 | 388–503 |
| UseConcurrentHashMap | If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation | 392 |
| UseDiamondOperator | Explicit type arguments can be replaced by Diamond Operator | 392 |
| ConfusingTernary | Avoid if (x != y) ..; else ..; | 468–476 |
| UnusedFormalParameter | Avoid unused method parameters such as 'nMode'. | 513 |
| UseDiamondOperator | Explicit type arguments can be replaced by Diamond Operator | 517 |
| UseConcurrentHashMap | If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation | 517 |
| UseDiamondOperator | Explicit type arguments can be replaced by Diamond Operator | 571 |
| ConfusingTernary | Avoid if (x != y) ..; else ..; | 617–624 |
| Rule | Violation | Ligne |
|---|---|---|
| ExcessiveImports | A high number of imports can indicate a high degree of coupling within an object. | 34–3884 |
| ExcessivePublicCount | This class has a bunch of public methods and attributes | 34–3884 |
| CyclomaticComplexity | The class 'FormJspBean' has a total cyclomatic complexity of 456 (highest 29). | 127–3884 |
| ExcessiveClassLength | Avoid really long classes. | 127–3884 |
| MissingSerialVersionUID | Classes implementing Serializable should set a serialVersionUID | 127–3884 |
| NcssCount | The class 'FormJspBean' has a NCSS line count of 1753 (Highest = 90). | 127–3884 |
| TooManyMethods | This class has too many methods, consider refactoring it. | 128–3884 |
| ImmutableField | Private field '_nDefaultItemsPerPage' could be made final; it is only initialized in the declaration or constructor. | 320 |
| UseConcurrentHashMap | If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation | 388 |
| UseDiamondOperator | Explicit type arguments can be replaced by Diamond Operator | 388 |
| ExcessiveMethodLength | Avoid really long methods. | 445–639 |
| CyclomaticComplexity | The method 'getFormData(HttpServletRequest, Form)' has a cyclomatic complexity of 29. | 445–639 |
| NPathComplexity | The method 'getFormData(HttpServletRequest, Form)' has an NPath complexity of 1679360 | 445–639 |
| NcssCount | The method 'getFormData(HttpServletRequest, Form)' has a NCSS line count of 90. | 445–639 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 449 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 450 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 451 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 452 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 453 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 455 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 456 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 457 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 458 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 460 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 461 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 462 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 463 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 464 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 465 |
| ConfusingTernary | Avoid if (x != y) ..; else ..; | 516–523 |
| ConfusingTernary | Avoid if (x != y) ..; else ..; | 525–532 |
| ConfusingTernary | Avoid if (x != y) ..; else ..; | 534–541 |
| ConfusingTernary | Avoid if (x != y) ..; else ..; | 543–550 |
| ConfusingTernary | Avoid if (x != y) ..; else ..; | 552–559 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 659 |
| UseDiamondOperator | Explicit type arguments can be replaced by Diamond Operator | 667 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 692 |
| CyclomaticComplexity | The method 'getModifyForm(HttpServletRequest)' has a cyclomatic complexity of 11. | 725–845 |
| NcssCount | The method 'getModifyForm(HttpServletRequest)' has a NCSS line count of 74. | 725–845 |
| ExcessiveMethodLength | Avoid really long methods. | 725–845 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 727 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 728 |
| UseDiamondOperator | Explicit type arguments can be replaced by Diamond Operator | 728 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 729 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 730 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 731 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 734 |
| UseCollectionIsEmpty | Substitute calls to size() == 0 (or size() != 0, size() > 0, size() < 1) with calls to isEmpty() | 771 |
| AvoidDuplicateLiterals | The String literal "?id_form=" appears 4 times in this file; the first occurrence is on line 806 | 806 |
| UseDiamondOperator | Explicit type arguments can be replaced by Diamond Operator | 827 |
| UseConcurrentHashMap | If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation | 827 |
| CyclomaticComplexity | The method 'doModifyForm(HttpServletRequest)' has a cyclomatic complexity of 10. | 852–902 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 911 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 913 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 914 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 961 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 962 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 963 |
| UseDiamondOperator | Explicit type arguments can be replaced by Diamond Operator | 989 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 1014 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 1015 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 1016 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 1017 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 1061 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 1064 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 1065 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 1066 |
| UseConcurrentHashMap | If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation | 1095 |
| UseDiamondOperator | Explicit type arguments can be replaced by Diamond Operator | 1095 |
| CyclomaticComplexity | The method 'getRecapData(HttpServletRequest, Recap)' has a cyclomatic complexity of 15. | 1115–1233 |
| NPathComplexity | The method 'getRecapData(HttpServletRequest, Recap)' has an NPath complexity of 980 | 1115–1233 |
| ExcessiveMethodLength | Avoid really long methods. | 1115–1233 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 1117 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 1118 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 1121 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 1122 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 1123 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 1124 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 1128 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 1129 |
| ConfusingTernary | Avoid if (x != y) ..; else ..; | 1153–1160 |
| ConfusingTernary | Avoid if (x != y) ..; else ..; | 1162–1230 |
| ConfusingTernary | Avoid if (x != y) ..; else ..; | 1213–1220 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 1286 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 1289 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 1290 |
| UseConcurrentHashMap | If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation | 1321 |
| UseDiamondOperator | Explicit type arguments can be replaced by Diamond Operator | 1321 |
| CyclomaticComplexity | The method 'doCreateEntry(HttpServletRequest)' has a cyclomatic complexity of 12. | 1345–1415 |
| NPathComplexity | The method 'doCreateEntry(HttpServletRequest)' has an NPath complexity of 208 | 1345–1415 |
| ConfusingTernary | Avoid if (x != y) ..; else ..; | 1407–1414 |
| NPathComplexity | The method 'getModifyEntry(HttpServletRequest)' has an NPath complexity of 288 | 1422–1508 |
| CyclomaticComplexity | The method 'getModifyEntry(HttpServletRequest)' has a cyclomatic complexity of 11. | 1422–1508 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 1424 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 1425 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 1426 |
| UseDiamondOperator | Explicit type arguments can be replaced by Diamond Operator | 1454 |
| AvoidReassigningLoopVariables | Avoid reassigning the loop control variable 'field' | 1458 |
| UseConcurrentHashMap | If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation | 1464 |
| UseDiamondOperator | Explicit type arguments can be replaced by Diamond Operator | 1464 |
| NPathComplexity | The method 'doModifyEntry(HttpServletRequest)' has an NPath complexity of 252 | 1515–1578 |
| CyclomaticComplexity | The method 'doModifyEntry(HttpServletRequest)' has a cyclomatic complexity of 12. | 1515–1578 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 1517 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 1518 |
| ConfusingTernary | Avoid if (x != y) ..; else ..; | 1565–1572 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 1587 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 1588 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 1590 |
| ConfusingTernary | Avoid if (x != y) ..; else ..; | 1618–1625 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 1645 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 1647 |
| UseDiamondOperator | Explicit type arguments can be replaced by Diamond Operator | 1673 |
| ConfusingTernary | Avoid if (x != y) ..; else ..; | 1685–1692 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 1702 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 1704 |
| ConfusingTernary | Avoid if (x != y) ..; else ..; | 1741–1748 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 1758 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 1759 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 1760 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 1761 |
| UseDiamondOperator | Explicit type arguments can be replaced by Diamond Operator | 1794 |
| UseConcurrentHashMap | If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation | 1794 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 1811 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 1812 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 1813 |
| CyclomaticComplexity | The method 'doMoveUpEntry(HttpServletRequest)' has a cyclomatic complexity of 10. | 1863–1942 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 1865 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 1866 |
| ConfusingTernary | Avoid if (x != y) ..; else ..; | 1898–1905 |
| ConfusingTernary | Avoid if (x != y) ..; else ..; | 1907–1914 |
| ConfusingTernary | Avoid if (x != y) ..; else ..; | 1934–1941 |
| CyclomaticComplexity | The method 'doMoveDownEntry(HttpServletRequest)' has a cyclomatic complexity of 10. | 1949–2028 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 1951 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 1952 |
| ConfusingTernary | Avoid if (x != y) ..; else ..; | 1984–1991 |
| ConfusingTernary | Avoid if (x != y) ..; else ..; | 1993–2000 |
| ConfusingTernary | Avoid if (x != y) ..; else ..; | 2020–2027 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 2037 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 2038 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 2106 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 2107 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 2108 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 2180 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 2181 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 2183 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 2247 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 2248 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 2250 |
| UnusedFormalParameter | Avoid unused method parameters such as 'plugin'. | 2367 |
| UnusedFormalParameter | Avoid unused method parameters such as 'locale'. | 2388 |
| ConfusingTernary | Avoid if (x != y) ..; else ..; | 2395–2402 |
| UnusedFormalParameter | Avoid unused method parameters such as 'locale'. | 2414 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 2438 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 2439 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 2495 |
| UseDiamondOperator | Explicit type arguments can be replaced by Diamond Operator | 2507 |
| UseConcurrentHashMap | If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation | 2507 |
| ConfusingTernary | Avoid if (x != y) ..; else ..; | 2510–2517 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 2535 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 2536 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 2537 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 2538 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 2539 |
| UseDiamondOperator | Explicit type arguments can be replaced by Diamond Operator | 2573 |
| ConfusingTernary | Avoid if (x != y) ..; else ..; | 2659–2680 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 2722 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 2759 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 2760 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 2761 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 2762 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 2763 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 2811 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 2812 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 2813 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 2814 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 2815 |
| UseDiamondOperator | Explicit type arguments can be replaced by Diamond Operator | 2935 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 2938 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 2939 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 2943 |
| UseDiamondOperator | Explicit type arguments can be replaced by Diamond Operator | 2969 |
| UseConcurrentHashMap | If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation | 2969 |
| UseStringBufferForStringAppends | Prefer StringBuilder (non-synchronized) or StringBuffer (synchronized) over += for concatenating strings | 2974 |
| NPathComplexity | The method 'doTestForm(HttpServletRequest)' has an NPath complexity of 1152 | 2990–3088 |
| CyclomaticComplexity | The method 'doTestForm(HttpServletRequest)' has a cyclomatic complexity of 15. | 2990–3088 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 2992 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 2993 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 2994 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 2996 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 2997 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 2998 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 3000 |
| UseDiamondOperator | Explicit type arguments can be replaced by Diamond Operator | 3057 |
| CyclomaticComplexity | The method 'doExportResponseTestForm(HttpServletRequest, HttpServletResponse)' has a cyclomatic complexity of 13. | 3098–3172 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 3102 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 3103 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 3109 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 3110 |
| ConfusingTernary | Avoid if (x != y) ..; else ..; | 3138–3168 |
| UseCollectionIsEmpty | Substitute calls to size() == 0 (or size() != 0, size() > 0, size() < 1) with calls to isEmpty() | 3138 |
| ShortVariable | Avoid variables with short names like os | 3156 |
| NPathComplexity | The method 'getResult(HttpServletRequest)' has an NPath complexity of 1920 | 3179–3284 |
| NcssCount | The method 'getResult(HttpServletRequest)' has a NCSS line count of 61. | 3179–3284 |
| ExcessiveMethodLength | Avoid really long methods. | 3179–3284 |
| CyclomaticComplexity | The method 'getResult(HttpServletRequest)' has a cyclomatic complexity of 15. | 3179–3284 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 3181 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 3183 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 3184 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 3185 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 3186 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 3187 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 3189 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 3194 |
| ConfusingTernary | Avoid if (x != y) ..; else ..; | 3247–3266 |
| UseDiamondOperator | Explicit type arguments can be replaced by Diamond Operator | 3268 |
| UseConcurrentHashMap | If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation | 3268 |
| NullAssignment | Assigning an Object to null is a code smell. Consider refactoring. | 3273 |
| NullAssignment | Assigning an Object to null is a code smell. Consider refactoring. | 3276 |
| NPathComplexity | The method 'doExportResult(HttpServletRequest, HttpServletResponse)' has an NPath complexity of 769 | 3293–3396 |
| ExcessiveMethodLength | Avoid really long methods. | 3293–3396 |
| CyclomaticComplexity | The method 'doExportResult(HttpServletRequest, HttpServletResponse)' has a cyclomatic complexity of 16. | 3293–3396 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 3297 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 3320 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 3321 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 3322 |
| ConfusingTernary | Avoid if (x != y) ..; else ..; | 3362–3392 |
| UseCollectionIsEmpty | Substitute calls to size() == 0 (or size() != 0, size() > 0, size() < 1) with calls to isEmpty() | 3362 |
| ShortVariable | Avoid variables with short names like os | 3380 |
| ExcessiveMethodLength | Avoid really long methods. | 3404–3526 |
| CyclomaticComplexity | The method 'doGenerateGraph(HttpServletRequest, HttpServletResponse)' has a cyclomatic complexity of 17. | 3404–3526 |
| NPathComplexity | The method 'doGenerateGraph(HttpServletRequest, HttpServletResponse)' has an NPath complexity of 3840 | 3404–3526 |
| NcssCount | The method 'doGenerateGraph(HttpServletRequest, HttpServletResponse)' has a NCSS line count of 65. | 3404–3526 |
| ConfusingTernary | Avoid if (x != y) ..; else ..; | 3445–3464 |
| UseDiamondOperator | Explicit type arguments can be replaced by Diamond Operator | 3480 |
| UseCollectionIsEmpty | Substitute calls to size() == 0 (or size() != 0, size() > 0, size() < 1) with calls to isEmpty() | 3483 |
| AvoidCatchingGenericException | Avoid catching generic exceptions such as NullPointerException, RuntimeException, Exception in try-catch block | 3522 |
| CyclomaticComplexity | The method 'doDownloadFile(HttpServletRequest, HttpServletResponse)' has a cyclomatic complexity of 10. | 3535–3610 |
| UseDiamondOperator | Explicit type arguments can be replaced by Diamond Operator | 3564 |
| UseCollectionIsEmpty | Substitute calls to size() == 0 (or size() != 0, size() > 0, size() < 1) with calls to isEmpty() | 3574 |
| UseCollectionIsEmpty | Substitute calls to size() == 0 (or size() != 0, size() > 0, size() < 1) with calls to isEmpty() | 3575 |
| ConfusingTernary | Avoid if (x != y) ..; else ..; | 3584–3606 |
| ShortVariable | Avoid variables with short names like os | 3593 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 3621 |
| UseDiamondOperator | Explicit type arguments can be replaced by Diamond Operator | 3646 |
| UseDiamondOperator | Explicit type arguments can be replaced by Diamond Operator | 3648 |
| LinguisticNaming | Linguistics Antipattern - The variable 'hasMaProcess' indicates linguistically it is a boolean, but it is 'HashMap' | 3650 |
| UseDiamondOperator | Explicit type arguments can be replaced by Diamond Operator | 3655 |
| CyclomaticComplexity | The method 'doManageOutputProcessor(HttpServletRequest)' has a cyclomatic complexity of 16. | 3678–3750 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 3684 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 3685 |
| ConfusingTernary | Avoid if (x != y) ..; else ..; | 3717–3744 |
| ConfusingTernary | Avoid if (x != y) ..; else ..; | 3721–3728 |
| UseConcurrentHashMap | If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation | 3794 |
| UseDiamondOperator | Explicit type arguments can be replaced by Diamond Operator | 3794 |
| CyclomaticComplexity | The method 'doModifyMessage(HttpServletRequest)' has a cyclomatic complexity of 15. | 3811–3883 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 3842 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 3846 |
| Rule | Violation | Ligne |
|---|---|---|
| MissingSerialVersionUID | Classes implementing Serializable should set a serialVersionUID | 60–133 |
| UseDiamondOperator | Explicit type arguments can be replaced by Diamond Operator | 92 |
| UseConcurrentHashMap | If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation | 92 |
| UseCollectionIsEmpty | Substitute calls to size() == 0 (or size() != 0, size() > 0, size() < 1) with calls to isEmpty() | 98 |
| Rule | Violation | Ligne |
|---|---|---|
| MissingSerialVersionUID | Classes implementing Serializable should set a serialVersionUID | 60–225 |
| MissingOverride | The method 'getCreate(HttpServletRequest)' is missing an @Override annotation. | 79–93 |
| MissingOverride | The method 'getModify(HttpServletRequest)' is missing an @Override annotation. | 101–128 |
| MissingOverride | The method 'doCreate(HttpServletRequest)' is missing an @Override annotation. | 136–176 |
| MissingOverride | The method 'doModify(HttpServletRequest)' is missing an @Override annotation. | 184–224 |
| Rule | Violation | Ligne |
|---|---|---|
| ShortClassName | Avoid short class names like Form | 52–637 |
| Rule | Violation | Ligne |
|---|---|---|
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.util.Date' due to existing import 'java.util.Date' | 581 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.util.Date' due to existing import 'java.util.Date' | 599 |
| Rule | Violation | Priority | Ligne |
|---|---|---|---|
| MissingOverride | The method 'getResourceTypeCode()' is missing an @Override annotation. | 3 | 167–170 |
| MissingOverride | The method 'getResourceId()' is missing an @Override annotation. | 3 | 176–179 |
| FormalParameterNamingConventions | The method parameter name '_strLibelleResetButton' doesn't match '[a-z][a-zA-Z0-9]*' | 1 | 185 |
| Rule | Violation | Priority | Ligne |
|---|---|---|---|
| MissingOverride | The method 'store(DefaultMessage, Plugin)' is missing an @Override annotation. | 3 | 59–73 |
| MissingOverride | The method 'load(Plugin)' is missing an @Override annotation. | 3 | 81–103 |
| Rule | Violation | Priority | Ligne |
|---|---|---|---|
| ExcessivePublicCount | This class has a bunch of public methods and attributes | 3 | 34–541 |
| TooManyFields | Too many fields | 3 | 51–541 |
| MissingOverride | The method 'getChildren()' is missing an @Override annotation. | 3 | 119–122 |
| MissingOverride | The method 'getComment()' is missing an @Override annotation. | 3 | 127–130 |
| MissingOverride | The method 'getEntryType()' is missing an @Override annotation. | 3 | 135–138 |
| MissingOverride | The method 'getFields()' is missing an @Override annotation. | 3 | 143–146 |
| MissingOverride | The method 'getHelpMessage()' is missing an @Override annotation. | 3 | 151–154 |
| MissingOverride | The method 'getIdEntry()' is missing an @Override annotation. | 3 | 159–162 |
| MissingOverride | The method 'getParent()' is missing an @Override annotation. | 3 | 167–170 |
| MissingOverride | The method 'getPosition()' is missing an @Override annotation. | 3 | 175–178 |
| MissingOverride | The method 'getTitle()' is missing an @Override annotation. | 3 | 183–186 |
| MissingOverride | The method 'isFieldInLine()' is missing an @Override annotation. | 3 | 191–194 |
| MissingOverride | The method 'isMandatory()' is missing an @Override annotation. | 3 | 199–202 |
| MissingOverride | The method 'setChildren(List)' is missing an @Override annotation. | 3 | 208–211 |
| MissingOverride | The method 'setComment(String)' is missing an @Override annotation. | 3 | 217–220 |
| MissingOverride | The method 'setEntryType(EntryType)' is missing an @Override annotation. | 3 | 226–229 |
| MissingOverride | The method 'setFieldInLine(boolean)' is missing an @Override annotation. | 3 | 235–238 |
| MissingOverride | The method 'setFields(List)' is missing an @Override annotation. | 3 | 244–247 |
| MissingOverride | The method 'setHelpMessage(String)' is missing an @Override annotation. | 3 | 253–256 |
| MissingOverride | The method 'setIdEntry(int)' is missing an @Override annotation. | 3 | 262–265 |
| MissingOverride | The method 'setMandatory(boolean)' is missing an @Override annotation. | 3 | 271–274 |
| MissingOverride | The method 'setParent(IEntry)' is missing an @Override annotation. | 3 | 280–283 |
| MissingOverride | The method 'setPosition(int)' is missing an @Override annotation. | 3 | 289–292 |
| MissingOverride | The method 'setTitle(String)' is missing an @Override annotation. | 3 | 298–301 |
| MissingOverride | The method 'getForm()' is missing an @Override annotation. | 3 | 307–310 |
| MissingOverride | The method 'setForm(Form)' is missing an @Override annotation. | 3 | 316–319 |
| MissingOverride | The method 'getFieldDepend()' is missing an @Override annotation. | 3 | 324–327 |
| MissingOverride | The method 'setFieldDepend(Field)' is missing an @Override annotation. | 3 | 333–336 |
| MissingOverride | The method 'getHtmlCode()' is missing an @Override annotation. | 3 | 343–346 |
| MissingOverride | The method 'getRequestData(HttpServletRequest, Locale)' is missing an @Override annotation. | 3 | 354–357 |
| MissingOverride | The method 'getResponseData(HttpServletRequest, List, Locale)' is missing an @Override annotation. | 3 | 366–369 |
| MissingOverride | The method 'getTemplateCreate()' is missing an @Override annotation. | 3 | 375–378 |
| MissingOverride | The method 'getTemplateModify()' is missing an @Override annotation. | 3 | 384–387 |
| MissingOverride | The method 'getNumberConditionalQuestion()' is missing an @Override annotation. | 3 | 392–395 |
| MissingOverride | The method 'getResponseValueForExport(HttpServletRequest, Response, Locale)' is missing an @Override annotation. | 3 | 404–407 |
| MissingOverride | The method 'getResponseValueForRecap(HttpServletRequest, Response, Locale)' is missing an @Override annotation. | 3 | 416–419 |
| MissingOverride | The method 'setNumberConditionalQuestion(int)' is missing an @Override annotation. | 3 | 426–429 |
| MissingOverride | The method 'getPaginator(int, String, String, String)' is missing an @Override annotation. | 3 | 439–443 |
| MissingOverride | The method 'getReferenceListRegularExpression(IEntry, Plugin)' is missing an @Override annotation. | 3 | 451–454 |
| MissingOverride | The method 'isLastInTheList()' is missing an @Override annotation. | 3 | 459–462 |
| MissingOverride | The method 'setLastInTheList(boolean)' is missing an @Override annotation. | 3 | 468–471 |
| MissingOverride | The method 'isFirstInTheList()' is missing an @Override annotation. | 3 | 476–479 |
| MissingOverride | The method 'setFirstInTheList(boolean)' is missing an @Override annotation. | 3 | 485–488 |
| MissingOverride | The method 'setConfirmField(boolean)' is missing an @Override annotation. | 3 | 494–497 |
| MissingOverride | The method 'isConfirmField()' is missing an @Override annotation. | 3 | 503–506 |
| MissingOverride | The method 'setConfirmFieldTitle(String)' is missing an @Override annotation. | 3 | 512–515 |
| MissingOverride | The method 'getConfirmFieldTitle()' is missing an @Override annotation. | 3 | 521–524 |
| MissingOverride | The method 'setUnique(boolean)' is missing an @Override annotation. | 3 | 529–532 |
| FormalParameterNamingConventions | The method parameter name '_bUnique' doesn't match '[a-z][a-zA-Z0-9]*' | 1 | 529 |
| MissingOverride | The method 'isUnique()' is missing an @Override annotation. | 3 | 537–540 |
| Rule | Violation | Priority | Ligne |
|---|---|---|---|
| GodClass | Possible God Class (WMC=53, ATFD=104, TCC=2.778%) | 3 | 47–583 |
| MissingOverride | The method 'insert(IEntry, Plugin)' is missing an @Override annotation. | 3 | 165–207 |
| AvoidSynchronizedAtMethodLevel | Use block level rather than method level synchronization | 3 | 165–207 |
| ConfusingTernary | Avoid if (x != y) ..; else ..; | 3 | 174–181 |
| ConfusingTernary | Avoid if (x != y) ..; else ..; | 3 | 190–197 |
| MissingOverride | The method 'load(int, Plugin)' is missing an @Override annotation. | 3 | 216–302 |
| IdenticalCatchBranches | 'catch' branch identical to 'ClassNotFoundException' branch | 3 | 248–254 |
| IdenticalCatchBranches | 'catch' branch identical to 'ClassNotFoundException' branch | 3 | 255–261 |
| MissingOverride | The method 'delete(int, Plugin)' is missing an @Override annotation. | 3 | 310–316 |
| MissingOverride | The method 'store(IEntry, Plugin)' is missing an @Override annotation. | 3 | 324–364 |
| ConfusingTernary | Avoid if (x != y) ..; else ..; | 3 | 332–339 |
| ConfusingTernary | Avoid if (x != y) ..; else ..; | 3 | 348–355 |
| NPathComplexity | The method 'selectEntryListByFilter(EntryFilter, Plugin)' has an NPath complexity of 26112 | 3 | 372–502 |
| ExcessiveMethodLength | Avoid really long methods. | 3 | 372–502 |
| MissingOverride | The method 'selectEntryListByFilter(EntryFilter, Plugin)' is missing an @Override annotation. | 3 | 372–502 |
| CyclomaticComplexity | The method 'selectEntryListByFilter(EntryFilter, Plugin)' has a cyclomatic complexity of 18. | 3 | 372–502 |
| NcssCount | The method 'selectEntryListByFilter(EntryFilter, Plugin)' has a NCSS line count of 76. | 3 | 372–502 |
| UseDiamondOperator | Explicit type arguments can be replaced by Diamond Operator | 3 | 374 |
| UseStringBufferForStringAppends | Prefer StringBuilder (non-synchronized) or StringBuffer (synchronized) over += for concatenating strings | 3 | 382 |
| UseStringBufferForStringAppends | Prefer StringBuilder (non-synchronized) or StringBuffer (synchronized) over += for concatenating strings | 3 | 383 |
| UseStringBufferForStringAppends | Prefer StringBuilder (non-synchronized) or StringBuffer (synchronized) over += for concatenating strings | 3 | 384 |
| UseStringBufferForStringAppends | Prefer StringBuilder (non-synchronized) or StringBuffer (synchronized) over += for concatenating strings | 3 | 385 |
| UseStringBufferForStringAppends | Prefer StringBuilder (non-synchronized) or StringBuffer (synchronized) over += for concatenating strings | 3 | 386 |
| UseStringBufferForStringAppends | Prefer StringBuilder (non-synchronized) or StringBuffer (synchronized) over += for concatenating strings | 3 | 387 |
| UseStringBufferForStringAppends | Prefer StringBuilder (non-synchronized) or StringBuffer (synchronized) over += for concatenating strings | 3 | 389 |
| IdenticalCatchBranches | 'catch' branch identical to 'ClassNotFoundException' branch | 3 | 448–454 |
| IdenticalCatchBranches | 'catch' branch identical to 'ClassNotFoundException' branch | 3 | 455–461 |
| NPathComplexity | The method 'selectNumberEntryByFilter(EntryFilter, Plugin)' has an NPath complexity of 9216 | 3 | 510–582 |
| MissingOverride | The method 'selectNumberEntryByFilter(EntryFilter, Plugin)' is missing an @Override annotation. | 3 | 510–582 |
| CyclomaticComplexity | The method 'selectNumberEntryByFilter(EntryFilter, Plugin)' has a cyclomatic complexity of 15. | 3 | 510–582 |
| UseStringBufferForStringAppends | Prefer StringBuilder (non-synchronized) or StringBuffer (synchronized) over += for concatenating strings | 3 | 514 |
| UseStringBufferForStringAppends | Prefer StringBuilder (non-synchronized) or StringBuffer (synchronized) over += for concatenating strings | 3 | 515 |
| UseStringBufferForStringAppends | Prefer StringBuilder (non-synchronized) or StringBuffer (synchronized) over += for concatenating strings | 3 | 516 |
| UseStringBufferForStringAppends | Prefer StringBuilder (non-synchronized) or StringBuffer (synchronized) over += for concatenating strings | 3 | 517 |
| UseStringBufferForStringAppends | Prefer StringBuilder (non-synchronized) or StringBuffer (synchronized) over += for concatenating strings | 3 | 518 |
| UseStringBufferForStringAppends | Prefer StringBuilder (non-synchronized) or StringBuffer (synchronized) over += for concatenating strings | 3 | 519 |
| UseStringBufferForStringAppends | Prefer StringBuilder (non-synchronized) or StringBuffer (synchronized) over += for concatenating strings | 3 | 521 |
| Rule | Violation | Priority | Ligne |
|---|---|---|---|
| AvoidReassigningLoopVariables | Avoid reassigning the loop control variable 'field' | 3 | 85 |
| AvoidReassigningLoopVariables | Avoid reassigning the loop control variable 'entryChild' | 3 | 100 |
| Rule | Violation | Priority | Ligne |
|---|---|---|---|
| FieldNamingConventions | The final field name '_template_create' doesn't match '[a-z][a-zA-Z0-9]*' | 1 | 57 |
| FinalFieldCouldBeStatic | This final field could be made static | 3 | 57 |
| FinalFieldCouldBeStatic | This final field could be made static | 3 | 58 |
| FieldNamingConventions | The final field name '_template_modify' doesn't match '[a-z][a-zA-Z0-9]*' | 1 | 58 |
| FieldNamingConventions | The final field name '_template_html_code' doesn't match '[a-z][a-zA-Z0-9]*' | 1 | 59 |
| FinalFieldCouldBeStatic | This final field could be made static | 3 | 59 |
| MissingOverride | The method 'getHtmlCode()' is missing an @Override annotation. | 3 | 66–69 |
| MissingOverride | The method 'getRequestData(HttpServletRequest, Locale)' is missing an @Override annotation. | 3 | 77–137 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 3 | 80–81 |
| ConfusingTernary | Avoid if (x != y) ..; else ..; | 3 | 80–81 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 3 | 82 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 3 | 83 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 3 | 84 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 3 | 85 |
| ConfusingTernary | Avoid if (x != y) ..; else ..; | 3 | 109–116 |
| AvoidLiteralsInIfCondition | Avoid using Literals in Conditional Statements | 3 | 127 |
| MissingOverride | The method 'getTemplateCreate()' is missing an @Override annotation. | 3 | 143–146 |
| MissingOverride | The method 'getTemplateModify()' is missing an @Override annotation. | 3 | 152–155 |
| MissingOverride | The method 'getPaginator(int, String, String, String)' is missing an @Override annotation. | 3 | 165–169 |
| CyclomaticComplexity | The method 'getResponseData(HttpServletRequest, List, Locale)' has a cyclomatic complexity of 11. | 3 | 178–249 |
| MissingOverride | The method 'getResponseData(HttpServletRequest, List, Locale)' is missing an @Override annotation. | 3 | 178–249 |
| UseDiamondOperator | Explicit type arguments can be replaced by Diamond Operator | 3 | 181 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 3 | 184 |
| ForLoopCanBeForeach | This for loop can be replaced by a foreach loop | 3 | 188–205 |
| UseCollectionIsEmpty | Substitute calls to size() == 0 (or size() != 0, size() > 0, size() < 1) with calls to isEmpty() | 3 | 230 |
| MissingOverride | The method 'getResponseValueForExport(HttpServletRequest, Response, Locale)' is missing an @Override annotation. | 3 | 258–261 |
| MissingOverride | The method 'getResponseValueForRecap(HttpServletRequest, Response, Locale)' is missing an @Override annotation. | 3 | 270–273 |
| Rule | Violation | Priority | Ligne |
|---|---|---|---|
| FieldNamingConventions | The final field name '_template_create' doesn't match '[a-z][a-zA-Z0-9]*' | 1 | 52 |
| FinalFieldCouldBeStatic | This final field could be made static | 3 | 52 |
| FinalFieldCouldBeStatic | This final field could be made static | 3 | 53 |
| FieldNamingConventions | The final field name '_template_modify' doesn't match '[a-z][a-zA-Z0-9]*' | 1 | 53 |
| FinalFieldCouldBeStatic | This final field could be made static | 3 | 54 |
| FieldNamingConventions | The final field name '_template_html_code' doesn't match '[a-z][a-zA-Z0-9]*' | 1 | 54 |
| MissingOverride | The method 'getHtmlCode()' is missing an @Override annotation. | 3 | 61–64 |
| MissingOverride | The method 'getRequestData(HttpServletRequest, Locale)' is missing an @Override annotation. | 3 | 72–93 |
| MissingOverride | The method 'getTemplateCreate()' is missing an @Override annotation. | 3 | 99–102 |
| MissingOverride | The method 'getTemplateModify()' is missing an @Override annotation. | 3 | 108–111 |
| Rule | Violation | Priority | Ligne |
|---|---|---|---|
| MissingOverride | The method 'load(int, Plugin)' is missing an @Override annotation. | 3 | 62–83 |
| MissingOverride | The method 'select(Plugin)' is missing an @Override annotation. | 3 | 90–112 |
| UseDiamondOperator | Explicit type arguments can be replaced by Diamond Operator | 3 | 92 |
| Rule | Violation | Priority | Ligne |
|---|---|---|---|
| FieldNamingConventions | The constant name '_message_illogical_date' doesn't match '[A-Z][A-Z_0-9]*' | 1 | 60 |
| FinalFieldCouldBeStatic | This final field could be made static | 3 | 61 |
| FieldNamingConventions | The final field name '_template_create' doesn't match '[a-z][a-zA-Z0-9]*' | 1 | 61 |
| FieldNamingConventions | The final field name '_template_modify' doesn't match '[a-z][a-zA-Z0-9]*' | 1 | 62 |
| FinalFieldCouldBeStatic | This final field could be made static | 3 | 62 |
| FieldNamingConventions | The final field name '_template_html_code' doesn't match '[a-z][a-zA-Z0-9]*' | 1 | 63 |
| FinalFieldCouldBeStatic | This final field could be made static | 3 | 63 |
| MissingOverride | The method 'getHtmlCode()' is missing an @Override annotation. | 3 | 70–73 |
| CyclomaticComplexity | The method 'getRequestData(HttpServletRequest, Locale)' has a cyclomatic complexity of 10. | 3 | 81–141 |
| MissingOverride | The method 'getRequestData(HttpServletRequest, Locale)' is missing an @Override annotation. | 3 | 81–141 |
| ConfusingTernary | Avoid if (x != y) ..; else ..; | 3 | 84–85 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 3 | 84–85 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 3 | 86 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 3 | 87 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 3 | 88 |
| UseDiamondOperator | Explicit type arguments can be replaced by Diamond Operator | 3 | 123 |
| ConfusingTernary | Avoid if (x != y) ..; else ..; | 3 | 131–138 |
| MissingOverride | The method 'getTemplateCreate()' is missing an @Override annotation. | 3 | 147–150 |
| MissingOverride | The method 'getTemplateModify()' is missing an @Override annotation. | 3 | 156–159 |
| MissingOverride | The method 'getResponseData(HttpServletRequest, List, Locale)' is missing an @Override annotation. | 3 | 168–223 |
| CollapsibleIfStatements | These nested if statements could be combined | 3 | 189–196 |
| MissingOverride | The method 'getResponseValueForExport(HttpServletRequest, Response, Locale)' is missing an @Override annotation. | 3 | 232–238 |
| MissingOverride | The method 'getResponseValueForRecap(HttpServletRequest, Response, Locale)' is missing an @Override annotation. | 3 | 247–253 |
| Rule | Violation | Priority | Ligne |
|---|---|---|---|
| FieldNamingConventions | The final field name '_template_create' doesn't match '[a-z][a-zA-Z0-9]*' | 1 | 66 |
| FinalFieldCouldBeStatic | This final field could be made static | 3 | 66 |
| FinalFieldCouldBeStatic | This final field could be made static | 3 | 67 |
| FieldNamingConventions | The final field name '_template_modify' doesn't match '[a-z][a-zA-Z0-9]*' | 1 | 67 |
| FieldNamingConventions | The final field name '_template_html_code' doesn't match '[a-z][a-zA-Z0-9]*' | 1 | 68 |
| FinalFieldCouldBeStatic | This final field could be made static | 3 | 68 |
| MissingOverride | The method 'getHtmlCode()' is missing an @Override annotation. | 3 | 75–78 |
| MissingOverride | The method 'getRequestData(HttpServletRequest, Locale)' is missing an @Override annotation. | 3 | 86–156 |
| CyclomaticComplexity | The method 'getRequestData(HttpServletRequest, Locale)' has a cyclomatic complexity of 11. | 3 | 86–156 |
| NPathComplexity | The method 'getRequestData(HttpServletRequest, Locale)' has an NPath complexity of 320 | 3 | 86–156 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 3 | 89–90 |
| ConfusingTernary | Avoid if (x != y) ..; else ..; | 3 | 89–90 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 3 | 91 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 3 | 92 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 3 | 96 |
| UseDiamondOperator | Explicit type arguments can be replaced by Diamond Operator | 3 | 138 |
| ConfusingTernary | Avoid if (x != y) ..; else ..; | 3 | 146–153 |
| MissingOverride | The method 'getTemplateCreate()' is missing an @Override annotation. | 3 | 162–165 |
| MissingOverride | The method 'getTemplateModify()' is missing an @Override annotation. | 3 | 171–174 |
| CyclomaticComplexity | The method 'getResponseData(HttpServletRequest, List, Locale)' has a cyclomatic complexity of 11. | 3 | 183–232 |
| MissingOverride | The method 'getResponseData(HttpServletRequest, List, Locale)' is missing an @Override annotation. | 3 | 183–232 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 3 | 188 |
| CollapsibleIfStatements | These nested if statements could be combined | 3 | 195–202 |
| UseCollectionIsEmpty | Substitute calls to size() == 0 (or size() != 0, size() > 0, size() < 1) with calls to isEmpty() | 3 | 208 |
| MissingOverride | The method 'getPaginator(int, String, String, String)' is missing an @Override annotation. | 3 | 242–247 |
| MissingOverride | The method 'getReferenceListRegularExpression(IEntry, Plugin)' is missing an @Override annotation. | 3 | 255–277 |
| MissingOverride | The method 'getResponseValueForExport(HttpServletRequest, Response, Locale)' is missing an @Override annotation. | 3 | 286–289 |
| MissingOverride | The method 'getResponseValueForRecap(HttpServletRequest, Response, Locale)' is missing an @Override annotation. | 3 | 298–308 |
| ConfusingTernary | Avoid if (x != y) ..; else ..; | 3 | 300–307 |
| Rule | Violation | Priority | Ligne |
|---|---|---|---|
| FieldNamingConventions | The final field name '_template_modify' doesn't match '[a-z][a-zA-Z0-9]*' | 1 | 53 |
| FinalFieldCouldBeStatic | This final field could be made static | 3 | 53 |
| FieldNamingConventions | The final field name '_template_html_code' doesn't match '[a-z][a-zA-Z0-9]*' | 1 | 54 |
| FinalFieldCouldBeStatic | This final field could be made static | 3 | 54 |
| MissingOverride | The method 'getHtmlCode()' is missing an @Override annotation. | 3 | 61–64 |
| MissingOverride | The method 'getRequestData(HttpServletRequest, Locale)' is missing an @Override annotation. | 3 | 72–93 |
| MissingOverride | The method 'getTemplateModify()' is missing an @Override annotation. | 3 | 99–102 |
| Rule | Violation | Priority | Ligne |
|---|---|---|---|
| FinalFieldCouldBeStatic | This final field could be made static | 3 | 56 |
| FieldNamingConventions | The final field name '_template_create' doesn't match '[a-z][a-zA-Z0-9]*' | 1 | 56 |
| FinalFieldCouldBeStatic | This final field could be made static | 3 | 57 |
| FieldNamingConventions | The final field name '_template_modify' doesn't match '[a-z][a-zA-Z0-9]*' | 1 | 57 |
| FieldNamingConventions | The final field name '_template_html_code' doesn't match '[a-z][a-zA-Z0-9]*' | 1 | 58 |
| FinalFieldCouldBeStatic | This final field could be made static | 3 | 58 |
| MissingOverride | The method 'getHtmlCode()' is missing an @Override annotation. | 3 | 65–68 |
| MissingOverride | The method 'getRequestData(HttpServletRequest, Locale)' is missing an @Override annotation. | 3 | 76–136 |
| ConfusingTernary | Avoid if (x != y) ..; else ..; | 3 | 79–80 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 3 | 79–80 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 3 | 81 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 3 | 82 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 3 | 83 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 3 | 84 |
| ConfusingTernary | Avoid if (x != y) ..; else ..; | 3 | 108–115 |
| AvoidLiteralsInIfCondition | Avoid using Literals in Conditional Statements | 3 | 126 |
| MissingOverride | The method 'getTemplateCreate()' is missing an @Override annotation. | 3 | 142–145 |
| MissingOverride | The method 'getTemplateModify()' is missing an @Override annotation. | 3 | 151–154 |
| MissingOverride | The method 'getPaginator(int, String, String, String)' is missing an @Override annotation. | 3 | 164–168 |
| MissingOverride | The method 'getResponseData(HttpServletRequest, List, Locale)' is missing an @Override annotation. | 3 | 177–223 |
| CollapsibleIfStatements | These nested if statements could be combined | 3 | 204–211 |
| MissingOverride | The method 'getResponseValueForExport(HttpServletRequest, Response, Locale)' is missing an @Override annotation. | 3 | 232–235 |
| MissingOverride | The method 'getResponseValueForRecap(HttpServletRequest, Response, Locale)' is missing an @Override annotation. | 3 | 244–247 |
| Rule | Violation | Priority | Ligne |
|---|---|---|---|
| FinalFieldCouldBeStatic | This final field could be made static | 3 | 56 |
| FieldNamingConventions | The final field name '_template_create' doesn't match '[a-z][a-zA-Z0-9]*' | 1 | 56 |
| FinalFieldCouldBeStatic | This final field could be made static | 3 | 57 |
| FieldNamingConventions | The final field name '_template_modify' doesn't match '[a-z][a-zA-Z0-9]*' | 1 | 57 |
| FinalFieldCouldBeStatic | This final field could be made static | 3 | 58 |
| FieldNamingConventions | The final field name '_template_html_code' doesn't match '[a-z][a-zA-Z0-9]*' | 1 | 58 |
| MissingOverride | The method 'getHtmlCode()' is missing an @Override annotation. | 3 | 65–68 |
| MissingOverride | The method 'getRequestData(HttpServletRequest, Locale)' is missing an @Override annotation. | 3 | 76–116 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 3 | 79–80 |
| ConfusingTernary | Avoid if (x != y) ..; else ..; | 3 | 79–80 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 3 | 81 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 3 | 82 |
| ConfusingTernary | Avoid if (x != y) ..; else ..; | 3 | 106–113 |
| MissingOverride | The method 'getTemplateCreate()' is missing an @Override annotation. | 3 | 122–125 |
| MissingOverride | The method 'getTemplateModify()' is missing an @Override annotation. | 3 | 131–134 |
| MissingOverride | The method 'getPaginator(int, String, String, String)' is missing an @Override annotation. | 3 | 144–148 |
| MissingOverride | The method 'getResponseData(HttpServletRequest, List, Locale)' is missing an @Override annotation. | 3 | 157–203 |
| CollapsibleIfStatements | These nested if statements could be combined | 3 | 184–191 |
| MissingOverride | The method 'getResponseValueForExport(HttpServletRequest, Response, Locale)' is missing an @Override annotation. | 3 | 212–215 |
| MissingOverride | The method 'getResponseValueForRecap(HttpServletRequest, Response, Locale)' is missing an @Override annotation. | 3 | 224–227 |
| Rule | Violation | Priority | Ligne |
|---|---|---|---|
| FinalFieldCouldBeStatic | This final field could be made static | 3 | 58 |
| FieldNamingConventions | The final field name '_template_create' doesn't match '[a-z][a-zA-Z0-9]*' | 1 | 58 |
| FieldNamingConventions | The final field name '_template_modify' doesn't match '[a-z][a-zA-Z0-9]*' | 1 | 59 |
| FinalFieldCouldBeStatic | This final field could be made static | 3 | 59 |
| FieldNamingConventions | The final field name '_template_html_code' doesn't match '[a-z][a-zA-Z0-9]*' | 1 | 60 |
| FinalFieldCouldBeStatic | This final field could be made static | 3 | 60 |
| ConfusingTernary | Avoid if (x != y) ..; else ..; | 3 | 85–86 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 3 | 85–86 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 3 | 87 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 3 | 88 |
| ConfusingTernary | Avoid if (x != y) ..; else ..; | 3 | 111–118 |
| CollapsibleIfStatements | These nested if statements could be combined | 3 | 193–200 |
| UseDiamondOperator | Explicit type arguments can be replaced by Diamond Operator | 3 | 258 |
| Rule | Violation | Priority | Ligne |
|---|---|---|---|
| GodClass | Possible God Class (WMC=50, ATFD=40, TCC=0.000%) | 3 | 62–406 |
| FieldNamingConventions | The final field name '_template_create' doesn't match '[a-z][a-zA-Z0-9]*' | 1 | 64 |
| FinalFieldCouldBeStatic | This final field could be made static | 3 | 64 |
| FieldNamingConventions | The final field name '_template_modify' doesn't match '[a-z][a-zA-Z0-9]*' | 1 | 65 |
| FinalFieldCouldBeStatic | This final field could be made static | 3 | 65 |
| FieldNamingConventions | The final field name '_template_html_code' doesn't match '[a-z][a-zA-Z0-9]*' | 1 | 66 |
| FinalFieldCouldBeStatic | This final field could be made static | 3 | 66 |
| MissingOverride | The method 'getHtmlCode()' is missing an @Override annotation. | 3 | 73–76 |
| NcssCount | The method 'getRequestData(HttpServletRequest, Locale)' has a NCSS line count of 60. | 3 | 84–202 |
| NPathComplexity | The method 'getRequestData(HttpServletRequest, Locale)' has an NPath complexity of 20480 | 3 | 84–202 |
| ExcessiveMethodLength | Avoid really long methods. | 3 | 84–202 |
| MissingOverride | The method 'getRequestData(HttpServletRequest, Locale)' is missing an @Override annotation. | 3 | 84–202 |
| CyclomaticComplexity | The method 'getRequestData(HttpServletRequest, Locale)' has a cyclomatic complexity of 19. | 3 | 84–202 |
| ConfusingTernary | Avoid if (x != y) ..; else ..; | 3 | 87–88 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 3 | 87–88 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 3 | 89 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 3 | 90 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 3 | 91 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 3 | 93 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 3 | 96 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 3 | 98 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 3 | 99 |
| UseDiamondOperator | Explicit type arguments can be replaced by Diamond Operator | 3 | 162 |
| ConfusingTernary | Avoid if (x != y) ..; else ..; | 3 | 172–179 |
| ConfusingTernary | Avoid if (x != y) ..; else ..; | 3 | 181–190 |
| ConfusingTernary | Avoid if (x != y) ..; else ..; | 3 | 192–199 |
| MissingOverride | The method 'getTemplateCreate()' is missing an @Override annotation. | 3 | 208–211 |
| MissingOverride | The method 'getTemplateModify()' is missing an @Override annotation. | 3 | 217–220 |
| MissingOverride | The method 'getPaginator(int, String, String, String)' is missing an @Override annotation. | 3 | 230–235 |
| MissingOverride | The method 'getReferenceListRegularExpression(IEntry, Plugin)' is missing an @Override annotation. | 3 | 243–265 |
| ExcessiveMethodLength | Avoid really long methods. | 3 | 274–381 |
| NPathComplexity | The method 'getResponseData(HttpServletRequest, List, Locale)' has an NPath complexity of 2354 | 3 | 274–381 |
| MissingOverride | The method 'getResponseData(HttpServletRequest, List, Locale)' is missing an @Override annotation. | 3 | 274–381 |
| CyclomaticComplexity | The method 'getResponseData(HttpServletRequest, List, Locale)' has a cyclomatic complexity of 21. | 3 | 274–381 |
| CollapsibleIfStatements | These nested if statements could be combined | 3 | 306–313 |
| UseCollectionIsEmpty | Substitute calls to size() == 0 (or size() != 0, size() > 0, size() < 1) with calls to isEmpty() | 3 | 317 |
| MissingOverride | The method 'getResponseValueForExport(HttpServletRequest, Response, Locale)' is missing an @Override annotation. | 3 | 390–393 |
| MissingOverride | The method 'getResponseValueForRecap(HttpServletRequest, Response, Locale)' is missing an @Override annotation. | 3 | 402–405 |
| Rule | Violation | Priority | Ligne |
|---|---|---|---|
| FinalFieldCouldBeStatic | This final field could be made static | 3 | 56 |
| FieldNamingConventions | The final field name '_template_create' doesn't match '[a-z][a-zA-Z0-9]*' | 1 | 56 |
| FinalFieldCouldBeStatic | This final field could be made static | 3 | 57 |
| FieldNamingConventions | The final field name '_template_modify' doesn't match '[a-z][a-zA-Z0-9]*' | 1 | 57 |
| FinalFieldCouldBeStatic | This final field could be made static | 3 | 58 |
| FieldNamingConventions | The final field name '_template_html_code' doesn't match '[a-z][a-zA-Z0-9]*' | 1 | 58 |
| MissingOverride | The method 'getHtmlCode()' is missing an @Override annotation. | 3 | 65–68 |
| CyclomaticComplexity | The method 'getRequestData(HttpServletRequest, Locale)' has a cyclomatic complexity of 14. | 3 | 76–166 |
| MissingOverride | The method 'getRequestData(HttpServletRequest, Locale)' is missing an @Override annotation. | 3 | 76–166 |
| NPathComplexity | The method 'getRequestData(HttpServletRequest, Locale)' has an NPath complexity of 896 | 3 | 76–166 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 3 | 79–80 |
| ConfusingTernary | Avoid if (x != y) ..; else ..; | 3 | 79–80 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 3 | 81 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 3 | 82 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 3 | 83 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 3 | 87 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 3 | 88 |
| UseDiamondOperator | Explicit type arguments can be replaced by Diamond Operator | 3 | 146 |
| ConfusingTernary | Avoid if (x != y) ..; else ..; | 3 | 156–163 |
| MissingOverride | The method 'getTemplateCreate()' is missing an @Override annotation. | 3 | 172–175 |
| MissingOverride | The method 'getTemplateModify()' is missing an @Override annotation. | 3 | 181–184 |
| MissingOverride | The method 'getResponseData(HttpServletRequest, List, Locale)' is missing an @Override annotation. | 3 | 193–230 |
| CollapsibleIfStatements | These nested if statements could be combined | 3 | 214–221 |
| MissingOverride | The method 'getResponseValueForExport(HttpServletRequest, Response, Locale)' is missing an @Override annotation. | 3 | 239–242 |
| MissingOverride | The method 'getResponseValueForRecap(HttpServletRequest, Response, Locale)' is missing an @Override annotation. | 3 | 251–254 |
| Rule | Violation | Priority | Ligne |
|---|---|---|---|
| MethodReturnsInternalArray | Returning '_xsl' may expose an internal array. | 3 | 113 |
| ArrayIsStoredDirectly | The user-supplied array 'donnees' is stored directly. | 3 | 120 |
| MissingOverride | The method 'getResourceTypeCode()' is missing an @Override annotation. | 3 | 129–132 |
| MissingOverride | The method 'getResourceId()' is missing an @Override annotation. | 3 | 138–141 |
| AddEmptyString | Do not add empty strings | 3 | 140 |
| Rule | Violation | Priority | Ligne |
|---|---|---|---|
| AvoidSynchronizedAtMethodLevel | Use block level rather than method level synchronization | 3 | 91–102 |
| MissingOverride | The method 'insert(ExportFormat, Plugin)' is missing an @Override annotation. | 3 | 91–102 |
| MissingOverride | The method 'load(int, Plugin)' is missing an @Override annotation. | 3 | 111–132 |
| MissingOverride | The method 'delete(int, Plugin)' is missing an @Override annotation. | 3 | 140–146 |
| MissingOverride | The method 'store(ExportFormat, Plugin)' is missing an @Override annotation. | 3 | 154–165 |
| MissingOverride | The method 'selectListExport(Plugin)' is missing an @Override annotation. | 3 | 173–193 |
| UseDiamondOperator | Explicit type arguments can be replaced by Diamond Operator | 3 | 175 |
| MissingOverride | The method 'getListExport(Plugin)' is missing an @Override annotation. | 3 | 200–218 |
| Rule | Violation | Priority | Ligne |
|---|---|---|---|
| MissingOverride | The method 'insert(Field, Plugin)' is missing an @Override annotation. | 3 | 125–145 |
| AvoidSynchronizedAtMethodLevel | Use block level rather than method level synchronization | 3 | 125–145 |
| NullAssignment | Assigning an Object to null is a code smell. Consider refactoring. | 3 | 139 |
| MissingOverride | The method 'load(int, Plugin)' is missing an @Override annotation. | 3 | 154–185 |
| MissingOverride | The method 'delete(int, Plugin)' is missing an @Override annotation. | 3 | 193–199 |
| MissingOverride | The method 'store(Field, Plugin)' is missing an @Override annotation. | 3 | 207–226 |
| NullAssignment | Assigning an Object to null is a code smell. Consider refactoring. | 3 | 220 |
| MissingOverride | The method 'selectFieldListByIdEntry(int, Plugin)' is missing an @Override annotation. | 3 | 234–267 |
| UseDiamondOperator | Explicit type arguments can be replaced by Diamond Operator | 3 | 236 |
| MissingOverride | The method 'deleteVerifyBy(int, int, Plugin)' is missing an @Override annotation. | 3 | 276–283 |
| MissingOverride | The method 'insertVerifyBy(int, int, Plugin)' is missing an @Override annotation. | 3 | 292–299 |
| MissingOverride | The method 'selectListRegularExpressionKeyByIdField(int, Plugin)' is missing an @Override annotation. | 3 | 307–322 |
| UseDiamondOperator | Explicit type arguments can be replaced by Diamond Operator | 3 | 309 |
| MissingOverride | The method 'isRegularExpressionIsUse(int, Plugin)' is missing an @Override annotation. | 3 | 331–347 |
| Rule | Violation | Priority | Ligne |
|---|---|---|---|
| UseDiamondOperator | Explicit type arguments can be replaced by Diamond Operator | 3 | 153 |
| AvoidDeeplyNestedIfStmts | Deeply nested if..then statements are hard to read | 3 | 159–173 |
| UseCollectionIsEmpty | Substitute calls to size() == 0 (or size() != 0, size() > 0, size() < 1) with calls to isEmpty() | 3 | 159 |
| Rule | Violation | Priority | Ligne |
|---|---|---|---|
| ExcessivePublicCount | This class has a bunch of public methods and attributes | 3 | 34–637 |
| ShortClassName | Avoid short class names like Form | 4 | 52–637 |
| TooManyFields | Too many fields | 3 | 52–637 |
| NonThreadSafeSingleton | Singleton is not thread safe | 3 | 94–98 |
| NonThreadSafeSingleton | Singleton is not thread safe | 3 | 100–104 |
| MissingOverride | The method 'getWorkgroup()' is missing an @Override annotation. | 3 | 309–312 |
| MissingOverride | The method 'getResourceTypeCode()' is missing an @Override annotation. | 3 | 417–420 |
| MissingOverride | The method 'getResourceId()' is missing an @Override annotation. | 3 | 426–429 |
| AddEmptyString | Do not add empty strings | 3 | 428 |
| FormalParameterNamingConventions | The method parameter name '_tDateBeginDisponibility' doesn't match '[a-z][a-zA-Z0-9]*' | 1 | 471 |
| FormalParameterNamingConventions | The method parameter name '_bAutoPublicationActive' doesn't match '[a-z][a-zA-Z0-9]*' | 1 | 489 |
| FormalParameterNamingConventions | The method parameter name '_strLibelleResetButton' doesn't match '[a-z][a-zA-Z0-9]*' | 1 | 516 |
| Rule | Violation | Priority | Ligne |
|---|---|---|---|
| MissingOverride | The method 'getPermission()' is missing an @Override annotation. | 3 | 184–187 |
| MissingOverride | The method 'setLocale(Locale)' is missing an @Override annotation. | 3 | 203–206 |
| Rule | Violation | Priority | Ligne |
|---|---|---|---|
| MissingOverride | The method 'selectActionsByFormState(int, Plugin)' is missing an @Override annotation. | 3 | 57–79 |
| UseDiamondOperator | Explicit type arguments can be replaced by Diamond Operator | 3 | 59 |
| Rule | Violation | Priority | Ligne |
|---|---|---|---|
| AvoidSynchronizedAtMethodLevel | Use block level rather than method level synchronization | 3 | 116–156 |
| MissingOverride | The method 'insert(Form, Plugin)' is missing an @Override annotation. | 3 | 116–156 |
| ConfusingTernary | Avoid if (x != y) ..; else ..; | 3 | 135–136 |
| NullAssignment | Assigning an Object to null is a code smell. Consider refactoring. | 3 | 136 |
| NullAssignment | Assigning an Object to null is a code smell. Consider refactoring. | 3 | 138 |
| ConfusingTernary | Avoid if (x != y) ..; else ..; | 3 | 138 |
| MissingOverride | The method 'load(int, Plugin)' is missing an @Override annotation. | 3 | 165–211 |
| MissingOverride | The method 'delete(int, Plugin)' is missing an @Override annotation. | 3 | 219–225 |
| MissingOverride | The method 'store(Form, Plugin)' is missing an @Override annotation. | 3 | 233–269 |
| ConfusingTernary | Avoid if (x != y) ..; else ..; | 3 | 250–251 |
| NullAssignment | Assigning an Object to null is a code smell. Consider refactoring. | 3 | 251 |
| NullAssignment | Assigning an Object to null is a code smell. Consider refactoring. | 3 | 253 |
| ConfusingTernary | Avoid if (x != y) ..; else ..; | 3 | 253 |
| ExcessiveMethodLength | Avoid really long methods. | 3 | 277–394 |
| NPathComplexity | The method 'selectFormList(FormFilter, Plugin)' has an NPath complexity of 3072 | 3 | 277–394 |
| CyclomaticComplexity | The method 'selectFormList(FormFilter, Plugin)' has a cyclomatic complexity of 14. | 3 | 277–394 |
| NcssCount | The method 'selectFormList(FormFilter, Plugin)' has a NCSS line count of 71. | 3 | 277–394 |
| MissingOverride | The method 'selectFormList(FormFilter, Plugin)' is missing an @Override annotation. | 3 | 277–394 |
| UseDiamondOperator | Explicit type arguments can be replaced by Diamond Operator | 3 | 279 |
| UseDiamondOperator | Explicit type arguments can be replaced by Diamond Operator | 3 | 282 |
| MissingOverride | The method 'getEnableFormList(Plugin)' is missing an @Override annotation. | 3 | 401–423 |
| Rule | Violation | Priority | Ligne |
|---|---|---|---|
| FormalParameterNamingConventions | The method parameter name '_bDateBeginDisponibilityAfterCurrentDate' doesn't match '[a-z][a-zA-Z0-9]*' | 1 | 131 |
| Rule | Violation | Priority | Ligne |
|---|---|---|---|
| AvoidReassigningLoopVariables | Avoid reassigning the loop control variable 'entry' | 3 | 96 |
| Rule | Violation | Priority | Ligne |
|---|---|---|---|
| MissingOverride | The method 'canBeRemoved(String)' is missing an @Override annotation. | 3 | 57–77 |
| MissingOverride | The method 'getRemovalRefusedMessage(String, Locale)' is missing an @Override annotation. | 3 | 85–89 |
| Rule | Violation | Priority | Ligne |
|---|---|---|---|
| ShortVariable | Avoid variables with short names like ip | 3 | 104 |
| Rule | Violation | Priority | Ligne |
|---|---|---|---|
| AvoidSynchronizedAtMethodLevel | Use block level rather than method level synchronization | 3 | 104–120 |
| MissingOverride | The method 'insert(FormSubmit, Plugin)' is missing an @Override annotation. | 3 | 104–120 |
| MissingOverride | The method 'load(int, Plugin)' is missing an @Override annotation. | 3 | 129–151 |
| MissingOverride | The method 'delete(int, Plugin)' is missing an @Override annotation. | 3 | 159–165 |
| MissingOverride | The method 'store(FormSubmit, Plugin)' is missing an @Override annotation. | 3 | 173–183 |
| MissingOverride | The method 'selectListByFilter(ResponseFilter, Plugin)' is missing an @Override annotation. | 3 | 191–253 |
| UseDiamondOperator | Explicit type arguments can be replaced by Diamond Operator | 3 | 193 |
| UseDiamondOperator | Explicit type arguments can be replaced by Diamond Operator | 3 | 196 |
| MissingOverride | The method 'selectCountByFilter(ResponseFilter, Plugin)' is missing an @Override annotation. | 3 | 261–313 |
| UseDiamondOperator | Explicit type arguments can be replaced by Diamond Operator | 3 | 264 |
| MissingOverride | The method 'selectStatisticFormSubmit(ResponseFilter, Plugin)' is missing an @Override annotation. | 3 | 321–394 |
| CyclomaticComplexity | The method 'selectStatisticFormSubmit(ResponseFilter, Plugin)' has a cyclomatic complexity of 11. | 3 | 321–394 |
| NPathComplexity | The method 'selectStatisticFormSubmit(ResponseFilter, Plugin)' has an NPath complexity of 1024 | 3 | 321–394 |
| UseDiamondOperator | Explicit type arguments can be replaced by Diamond Operator | 3 | 323 |
| UseDiamondOperator | Explicit type arguments can be replaced by Diamond Operator | 3 | 325 |
| UseDiamondOperator | Explicit type arguments can be replaced by Diamond Operator | 3 | 326 |
| Rule | Violation | Priority | Ligne |
|---|---|---|---|
| MissingOverride | The method 'canBeRemoved(String)' is missing an @Override annotation. | 3 | 57–76 |
| MissingOverride | The method 'getRemovalRefusedMessage(String, Locale)' is missing an @Override annotation. | 3 | 84–88 |
| Rule | Violation | Priority | Ligne |
|---|---|---|---|
| MissingOverride | The method 'createChart(List, String, boolean, boolean)' is missing an @Override annotation. | 3 | 72–76 |
| Rule | Violation | Priority | Ligne |
|---|---|---|---|
| MissingOverride | The method 'load(int, Plugin)' is missing an @Override annotation. | 3 | 61–104 |
| IdenticalCatchBranches | 'catch' branch identical to 'ClassNotFoundException' branch | 3 | 85–91 |
| IdenticalCatchBranches | 'catch' branch identical to 'ClassNotFoundException' branch | 3 | 92–98 |
| MissingOverride | The method 'select(Plugin)' is missing an @Override annotation. | 3 | 111–130 |
| UseDiamondOperator | Explicit type arguments can be replaced by Diamond Operator | 3 | 116 |
| Rule | Violation | Priority | Ligne |
|---|---|---|---|
| MissingOverride | The method 'createChart(List, String, boolean, boolean)' is missing an @Override annotation. | 3 | 62–66 |
| Rule | Violation | Priority | Ligne |
|---|---|---|---|
| ExcessivePublicCount | This class has a bunch of public methods and attributes | 3 | 34–335 |
| Rule | Violation | Priority | Ligne |
|---|---|---|---|
| MissingOverride | The method 'insert(Recap, Plugin)' is missing an @Override annotation. | 3 | 89–116 |
| AvoidSynchronizedAtMethodLevel | Use block level rather than method level synchronization | 3 | 89–116 |
| MissingOverride | The method 'load(int, Plugin)' is missing an @Override annotation. | 3 | 125–160 |
| MissingOverride | The method 'delete(int, Plugin)' is missing an @Override annotation. | 3 | 168–174 |
| MissingOverride | The method 'store(Recap, Plugin)' is missing an @Override annotation. | 3 | 182–207 |
| Rule | Violation | Priority | Ligne |
|---|---|---|---|
| MethodReturnsInternalArray | Returning '_byValueResponse' may expose an internal array. | 3 | 113 |
| ArrayIsStoredDirectly | The user-supplied array 'valueResponse' is stored directly. | 3 | 120 |
| Rule | Violation | Priority | Ligne |
|---|---|---|---|
| MissingOverride | The method 'insert(Response, Plugin)' is missing an @Override annotation. | 3 | 104–128 |
| AvoidSynchronizedAtMethodLevel | Use block level rather than method level synchronization | 3 | 104–128 |
| ConfusingTernary | Avoid if (x != y) ..; else ..; | 3 | 113–120 |
| MissingOverride | The method 'load(int, Plugin)' is missing an @Override annotation. | 3 | 137–207 |
| IdenticalCatchBranches | 'catch' branch identical to 'ClassNotFoundException' branch | 3 | 170–175 |
| IdenticalCatchBranches | 'catch' branch identical to 'ClassNotFoundException' branch | 3 | 176–181 |
| MissingOverride | The method 'delete(int, Plugin)' is missing an @Override annotation. | 3 | 215–221 |
| MissingOverride | The method 'store(Response, Plugin)' is missing an @Override annotation. | 3 | 229–252 |
| ConfusingTernary | Avoid if (x != y) ..; else ..; | 3 | 238–245 |
| MissingOverride | The method 'selectListByFilter(ResponseFilter, Plugin)' is missing an @Override annotation. | 3 | 260–358 |
| CyclomaticComplexity | The method 'selectListByFilter(ResponseFilter, Plugin)' has a cyclomatic complexity of 13. | 3 | 260–358 |
| NPathComplexity | The method 'selectListByFilter(ResponseFilter, Plugin)' has an NPath complexity of 1088 | 3 | 260–358 |
| UseDiamondOperator | Explicit type arguments can be replaced by Diamond Operator | 3 | 263 |
| UseStringBufferForStringAppends | Prefer StringBuilder (non-synchronized) or StringBuffer (synchronized) over += for concatenating strings | 3 | 271 |
| UseStringBufferForStringAppends | Prefer StringBuilder (non-synchronized) or StringBuffer (synchronized) over += for concatenating strings | 3 | 272 |
| UseStringBufferForStringAppends | Prefer StringBuilder (non-synchronized) or StringBuffer (synchronized) over += for concatenating strings | 3 | 273 |
| UseStringBufferForStringAppends | Prefer StringBuilder (non-synchronized) or StringBuffer (synchronized) over += for concatenating strings | 3 | 274 |
| IdenticalCatchBranches | 'catch' branch identical to 'ClassNotFoundException' branch | 3 | 320–325 |
| IdenticalCatchBranches | 'catch' branch identical to 'ClassNotFoundException' branch | 3 | 326–331 |
| MissingOverride | The method 'getStatisticByIdEntry(int, Plugin)' is missing an @Override annotation. | 3 | 366–385 |
| UseDiamondOperator | Explicit type arguments can be replaced by Diamond Operator | 3 | 368 |
| Rule | Violation | Priority | Ligne |
|---|---|---|---|
| CommentRequired | Public method and constructor comments are required | 3 | 60–80 |
| MissingOverride | The method 'selectByIdForm(int, Plugin)' is missing an @Override annotation. | 3 | 60–80 |
| UseDiamondOperator | Explicit type arguments can be replaced by Diamond Operator | 3 | 62 |
| CommentRequired | Public method and constructor comments are required | 3 | 85–105 |
| MissingOverride | The method 'selectAll(Plugin)' is missing an @Override annotation. | 3 | 85–105 |
| UseDiamondOperator | Explicit type arguments can be replaced by Diamond Operator | 3 | 87 |
| CommentRequired | Public method and constructor comments are required | 3 | 110–119 |
| MissingOverride | The method 'insert(FormProcessor, Plugin)' is missing an @Override annotation. | 3 | 110–119 |
| CommentRequired | Public method and constructor comments are required | 3 | 124–132 |
| MissingOverride | The method 'delete(FormProcessor, Plugin)' is missing an @Override annotation. | 3 | 124–132 |
| Rule | Violation | Priority | Ligne |
|---|---|---|---|
| CommentRequired | Header comments are required | 3 | 41–71 |
| Rule | Violation | Priority | Ligne |
|---|---|---|---|
| CommentRequired | Header comments are required | 3 | 40–60 |
| Rule | Violation | Priority | Ligne |
|---|---|---|---|
| CommentRequired | Public method and constructor comments are required | 3 | 50–53 |
| MissingOverride | The method 'getKey()' is missing an @Override annotation. | 3 | 50–53 |
| CommentRequired | Public method and constructor comments are required | 3 | 58–61 |
| MissingOverride | The method 'setKey(String)' is missing an @Override annotation. | 3 | 58–61 |
| Rule | Violation | Priority | Ligne |
|---|---|---|---|
| CommentRequired | Public method and constructor comments are required | 3 | 51–60 |
| MissingOverride | The method 'setOutputProcessorMap(Map)' is missing an @Override annotation. | 3 | 51–60 |
| MissingOverride | The method 'getOutputProcessor(String)' is missing an @Override annotation. | 3 | 65–70 |
| CommentRequired | Public method and constructor comments are required | 3 | 65–70 |
| UnnecessaryLocalBeforeReturn | Consider simply returning the value vs storing it in local variable 'outputProcessor' | 3 | 69 |
| MissingOverride | The method 'getAllOutputProcessor()' is missing an @Override annotation. | 3 | 76–82 |
| CommentRequired | Public method and constructor comments are required | 3 | 76–82 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 3 | 78 |
| UnusedLocalVariable | Avoid unused local variables such as 'test'. | 3 | 78 |
| UnnecessaryLocalBeforeReturn | Consider simply returning the value vs storing it in local variable 'list' | 3 | 81 |
| Rule | Violation | Priority | Ligne |
|---|---|---|---|
| UncommentedEmptyConstructor | Document empty constructor | 3 | 72–74 |
| UnnecessaryConstructor | Avoid unnecessary constructors - the compiler will generate these for you | 3 | 72–74 |
| CallSuperInConstructor | It is a good practice to call super() in a constructor | 3 | 72–74 |
| MissingOverride | The method 'getXml(HttpServletRequest)' is missing an @Override annotation. | 3 | 82–127 |
| ConfusingTernary | Avoid if (x != y) ..; else ..; | 3 | 87–94 |
| UseStringBufferForStringAppends | Prefer StringBuilder (non-synchronized) or StringBuffer (synchronized) over += for concatenating strings | 3 | 112 |
| UnnecessaryLocalBeforeReturn | Consider simply returning the value vs storing it in local variable 'str' | 3 | 126 |
| MissingOverride | The method 'getXmlDocument(HttpServletRequest)' is missing an @Override annotation. | 3 | 135–138 |
| MissingOverride | The method 'remove()' is missing an @Override annotation. | 3 | 151–154 |
| MissingOverride | The method 'getStatus()' is missing an @Override annotation. | 3 | 201–204 |
| MissingOverride | The method 'setStatus(int)' is missing an @Override annotation. | 3 | 211–214 |
| Rule | Violation | Priority | Ligne |
|---|---|---|---|
| MissingOverride | The method 'insert(Portlet)' is missing an @Override annotation. | 3 | 60–70 |
| ShortVariable | Avoid variables with short names like p | 3 | 62 |
| MissingOverride | The method 'delete(int)' is missing an @Override annotation. | 3 | 78–84 |
| MissingOverride | The method 'load(int)' is missing an @Override annotation. | 3 | 93–110 |
| MissingOverride | The method 'selectCountPortletByIdForm(int)' is missing an @Override annotation. | 3 | 117–132 |
| MissingOverride | The method 'store(Portlet)' is missing an @Override annotation. | 3 | 140–149 |
| ShortVariable | Avoid variables with short names like p | 3 | 142 |
| Rule | Violation | Priority | Ligne |
|---|---|---|---|
| AssignmentToNonFinalStatic | Possible unsafe assignment to a non-final static field in a constructor. | 3 | 58 |
| CallSuperInConstructor | It is a good practice to call super() in a constructor | 3 | 63–69 |
| NonThreadSafeSingleton | Singleton is not thread safe | 3 | 78–81 |
| MissingOverride | The method 'getPortletTypeId()' is missing an @Override annotation. | 3 | 91–97 |
| UnnecessaryLocalBeforeReturn | Consider simply returning the value vs storing it in local variable 'strPortletTypeId' | 3 | 96 |
| MissingOverride | The method 'getDAO()' is missing an @Override annotation. | 3 | 104–107 |
| Rule | Violation | Priority | Ligne |
|---|---|---|---|
| MissingOverride | The method 'delete(int)' is missing an @Override annotation. | 3 | 51 |
| MissingOverride | The method 'insert(Portlet)' is missing an @Override annotation. | 3 | 59 |
| MissingOverride | The method 'load(int)' is missing an @Override annotation. | 3 | 68 |
| MissingOverride | The method 'store(Portlet)' is missing an @Override annotation. | 3 | 83 |
| Rule | Violation | Priority | Ligne |
|---|---|---|---|
| MissingOverride | The method 'insert(NotifySenderConfiguration, Plugin)' is missing an @Override annotation. | 3 | 58–68 |
| CommentRequired | Public method and constructor comments are required | 3 | 58–68 |
| MissingOverride | The method 'load(int, Plugin)' is missing an @Override annotation. | 3 | 73–93 |
| CommentRequired | Public method and constructor comments are required | 3 | 73–93 |
| CommentRequired | Public method and constructor comments are required | 3 | 98–104 |
| MissingOverride | The method 'delete(int, Plugin)' is missing an @Override annotation. | 3 | 98–104 |
| MissingOverride | The method 'store(NotifySenderConfiguration, Plugin)' is missing an @Override annotation. | 3 | 109–120 |
| CommentRequired | Public method and constructor comments are required | 3 | 109–120 |
| MissingOverride | The method 'selectNotifySenderConfigurationsList(Plugin)' is missing an @Override annotation. | 3 | 125–146 |
| CommentRequired | Public method and constructor comments are required | 3 | 125–146 |
| UseDiamondOperator | Explicit type arguments can be replaced by Diamond Operator | 3 | 127 |
| Rule | Violation | Priority | Ligne |
|---|---|---|---|
| MissingOverride | The method 'getOutputConfigForm(HttpServletRequest, Form, Locale, Plugin)' is missing an @Override annotation. | 3 | 93–111 |
| CommentRequired | Public method and constructor comments are required | 3 | 93–111 |
| UseDiamondOperator | Explicit type arguments can be replaced by Diamond Operator | 3 | 100 |
| UseConcurrentHashMap | If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation | 3 | 100 |
| MissingOverride | The method 'doOutputConfigForm(HttpServletRequest, Locale, Plugin)' is missing an @Override annotation. | 3 | 117–152 |
| CommentRequired | Public method and constructor comments are required | 3 | 117–152 |
| ConfusingTernary | Avoid if (x != y) ..; else ..; | 3 | 142–149 |
| MissingOverride | The method 'process(FormSubmit, HttpServletRequest, Plugin)' is missing an @Override annotation. | 3 | 158–237 |
| CommentRequired | Public method and constructor comments are required | 3 | 158–237 |
| UseDiamondOperator | Explicit type arguments can be replaced by Diamond Operator | 3 | 187 |
| ConfusingTernary | Avoid if (x != y) ..; else ..; | 3 | 197–206 |
| ShortVariable | Avoid variables with short names like t | 3 | 223 |
| AvoidCatchingGenericException | Avoid catching generic exceptions such as NullPointerException, RuntimeException, Exception in try-catch block | 3 | 231 |
| UnusedFormalParameter | Avoid unused method parameters such as 'locale'. | 3 | 247 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 3 | 252 |
| UnusedLocalVariable | Avoid unused local variables such as 'strSendRecap'. | 3 | 253 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 3 | 253 |
| Rule | Violation | Priority | Ligne |
|---|---|---|---|
| CallSuperInConstructor | It is a good practice to call super() in a constructor | 3 | 61–64 |
| MissingOverride | The method 'register()' is missing an @Override annotation. | 3 | 69–82 |
| ShortVariable | Avoid variables with short names like rt | 3 | 71 |
| ShortVariable | Avoid variables with short names like p | 3 | 77 |
| MissingOverride | The method 'getResourceIdList(Locale)' is missing an @Override annotation. | 3 | 89–92 |
| MissingOverride | The method 'getTitle(String, Locale)' is missing an @Override annotation. | 3 | 100–103 |
| Rule | Violation | Priority | Ligne |
|---|---|---|---|
| CallSuperInConstructor | It is a good practice to call super() in a constructor | 3 | 64–67 |
| MissingOverride | The method 'register()' is missing an @Override annotation. | 3 | 72–85 |
| ShortVariable | Avoid variables with short names like rt | 3 | 74 |
| ShortVariable | Avoid variables with short names like p | 3 | 80 |
| MissingOverride | The method 'getResourceIdList(Locale)' is missing an @Override annotation. | 3 | 92–95 |
| MissingOverride | The method 'getTitle(String, Locale)' is missing an @Override annotation. | 3 | 103–119 |
| Rule | Violation | Priority | Ligne |
|---|---|---|---|
| MissingOverride | The method 'init()' is missing an @Override annotation. | 3 | 52–56 |
| Rule | Violation | Priority | Ligne |
|---|---|---|---|
| CallSuperInConstructor | It is a good practice to call super() in a constructor | 3 | 96–99 |
| MissingOverride | The method 'register()' is missing an @Override annotation. | 3 | 104–158 |
| ShortVariable | Avoid variables with short names like rt | 3 | 106 |
| ShortVariable | Avoid variables with short names like p | 3 | 112 |
| MissingOverride | The method 'getResourceIdList(Locale)' is missing an @Override annotation. | 3 | 165–168 |
| MissingOverride | The method 'getTitle(String, Locale)' is missing an @Override annotation. | 3 | 176–192 |
| Rule | Violation | Priority | Ligne |
|---|---|---|---|
| ClassWithOnlyPrivateConstructorsShouldBeFinal | A class which only has private constructors should be final | 1 | 52–150 |
| FieldNamingConventions | The field name '_OutputProcessorSet' doesn't match '_[a-z][a-zA-Z0-9]*' | 1 | 56 |
| ImmutableField | Private field '_OutputProcessorSet' could be made final; it is only initialized in the declaration or constructor. | 3 | 56 |
| CommentRequired | Public method and constructor comments are required | 3 | 65–73 |
| NonThreadSafeSingleton | Singleton is not thread safe | 3 | 67–70 |
| CommentRequired | Public method and constructor comments are required | 3 | 75–78 |
| CommentRequired | Public method and constructor comments are required | 3 | 80–99 |
| UseDiamondOperator | Explicit type arguments can be replaced by Diamond Operator | 3 | 82 |
| CommentRequired | Public method and constructor comments are required | 3 | 101–110 |
| CommentRequired | Public method and constructor comments are required | 3 | 112–119 |
| CommentRequired | Public method and constructor comments are required | 3 | 121–128 |
| CommentRequired | Public method and constructor comments are required | 3 | 130–144 |
| CommentRequired | Public method and constructor comments are required | 3 | 146–149 |
| Rule | Violation | Priority | Ligne |
|---|---|---|---|
| MissingOverride | The method 'run()' is missing an @Override annotation. | 3 | 50–53 |
| Rule | Violation | Priority | Ligne |
|---|---|---|---|
| UnusedLocalVariable | Avoid unused local variables such as 'autoPublicationActive'. | 3 | 83 |
| Rule | Violation | Priority | Ligne |
|---|---|---|---|
| ExcessiveImports | A high number of imports can indicate a high degree of coupling within an object. | 3 | 34–1197 |
| ExcessiveClassLength | Avoid really long classes. | 3 | 101–1197 |
| CyclomaticComplexity | The class 'FormUtils' has a total cyclomatic complexity of 126 (highest 14). | 3 | 101–1197 |
| GodClass | Possible God Class (WMC=126, ATFD=237, TCC=2.151%) | 3 | 101–1197 |
| TooManyMethods | This class has too many methods, consider refactoring it. | 3 | 102–1197 |
| ShortVariable | Avoid variables with short names like t | 3 | 174 |
| AvoidCatchingGenericException | Avoid catching generic exceptions such as NullPointerException, RuntimeException, Exception in try-catch block | 3 | 184 |
| ShortVariable | Avoid variables with short names like t | 3 | 208 |
| AvoidCatchingGenericException | Avoid catching generic exceptions such as NullPointerException, RuntimeException, Exception in try-catch block | 3 | 218 |
| UnnecessaryLocalBeforeReturn | Consider simply returning the value vs storing it in local variable 'timeStamp' | 3 | 246 |
| UnnecessaryLocalBeforeReturn | Consider simply returning the value vs storing it in local variable 'timeStamp' | 3 | 271 |
| UnnecessaryLocalBeforeReturn | Consider simply returning the value vs storing it in local variable 'timeStamp1' | 3 | 362 |
| CyclomaticComplexity | The method 'sameDate(Timestamp, Timestamp, String)' has a cyclomatic complexity of 10. | 3 | 372–400 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 3 | 452 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 3 | 453 |
| IdenticalCatchBranches | 'catch' branch identical to 'ClassNotFoundException' branch | 3 | 486–490 |
| IdenticalCatchBranches | 'catch' branch identical to 'ClassNotFoundException' branch | 3 | 491–495 |
| LinguisticNaming | Linguistics Antipattern - The getter 'getHtmlEntry' should not return void linguistically | 3 | 595–653 |
| CyclomaticComplexity | The method 'getResponseEntry(HttpServletRequest, int, Plugin, FormSubmit, boolean, Locale)' has a cyclomatic complexity of 14. | 3 | 666–746 |
| UseDiamondOperator | Explicit type arguments can be replaced by Diamond Operator | 3 | 672 |
| UseDiamondOperator | Explicit type arguments can be replaced by Diamond Operator | 3 | 675 |
| AvoidReassigningLoopVariables | Avoid reassigning the loop control variable 'field' | 3 | 679 |
| ConfusingTernary | Avoid if (x != y) ..; else ..; | 3 | 699–708 |
| CyclomaticComplexity | The method 'getXmlResponses(HttpServletRequest, Form, List, Locale, Plugin)' has a cyclomatic complexity of 12. | 3 | 795–871 |
| ConfusingTernary | Avoid if (x != y) ..; else ..; | 3 | 812–819 |
| ConfusingTernary | Avoid if (x != y) ..; else ..; | 3 | 846–854 |
| PositionLiteralsFirstInComparisons | Position literals first in String comparisons | 3 | 885 |
| ConfusingTernary | Avoid if (x != y) ..; else ..; | 3 | 893–900 |
| CommentRequired | Public method and constructor comments are required | 3 | 1025–1042 |
| ConfusingTernary | Avoid if (x != y) ..; else ..; | 3 | 1031–1038 |
| UseDiamondOperator | Explicit type arguments can be replaced by Diamond Operator | 3 | 1052 |
| AvoidReassigningParameters | Avoid reassigning parameters such as 'entryParent' | 2 | 1089 |
| AvoidReassigningLoopVariables | Avoid reassigning the loop control variable 'field' | 3 | 1095 |
| UseDiamondOperator | Explicit type arguments can be replaced by Diamond Operator | 3 | 1116 |
| UseDiamondOperator | Explicit type arguments can be replaced by Diamond Operator | 3 | 1130 |
| AvoidLiteralsInIfCondition | Avoid using Literals in Conditional Statements | 3 | 1169 |
| AssignmentInOperand | Avoid assignments in operands | 3 | 1169 |
| Rule | Violation | Priority | Ligne |
|---|---|---|---|
| MissingSerialVersionUID | Classes implementing Serializable should set a serialVersionUID | 3 | 61–204 |
| UnusedPrivateField | Avoid unused private fields such as 'FIELD_LIBELLE_RESET_BUTTON'. | 3 | 92 |
| CyclomaticComplexity | The method 'getDefaultMessageData(HttpServletRequest, DefaultMessage)' has a cyclomatic complexity of 14. | 3 | 150–203 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 3 | 157 |
| Rule | Violation | Priority | Ligne |
|---|---|---|---|
| CyclomaticComplexity | The method 'doGenerateGraph(HttpServletRequest, HttpServletResponse)' has a cyclomatic complexity of 14. | 3 | 80–149 |
| AvoidCatchingGenericException | Avoid catching generic exceptions such as NullPointerException, RuntimeException, Exception in try-catch block | 3 | 144 |
| Rule | Violation | Priority | Ligne |
|---|---|---|---|
| MissingSerialVersionUID | Classes implementing Serializable should set a serialVersionUID | 3 | 78–456 |
| ImmutableField | Private field '_nDefaultItemsPerPage' could be made final; it is only initialized in the declaration or constructor. | 3 | 123 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 3 | 216 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 3 | 217 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 3 | 218 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 3 | 220 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 3 | 260 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 3 | 261 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 3 | 327 |
| CyclomaticComplexity | The method 'getExportFormatData(MultipartHttpServletRequest, ExportFormat)' has a cyclomatic complexity of 16. | 3 | 362–421 |
| NPathComplexity | The method 'getExportFormatData(MultipartHttpServletRequest, ExportFormat)' has an NPath complexity of 240 | 3 | 362–421 |
| LinguisticNaming | Linguistics Antipattern - The method 'isValid' indicates linguistically it returns a boolean, but it returns 'String' | 3 | 428–445 |
| ShortVariable | Avoid variables with short names like is | 3 | 436 |
| AvoidCatchingGenericException | Avoid catching generic exceptions such as NullPointerException, RuntimeException, Exception in try-catch block | 3 | 439 |
| Rule | Violation | Priority | Ligne |
|---|---|---|---|
| ExcessiveImports | A high number of imports can indicate a high degree of coupling within an object. | 3 | 34–646 |
| MissingSerialVersionUID | Classes implementing Serializable should set a serialVersionUID | 3 | 84–646 |
| UnusedPrivateField | Avoid unused private fields such as 'PARAMETER_PAGE_ID'. | 3 | 115 |
| MissingOverride | The method 'getPage(HttpServletRequest, int, Plugin)' is missing an @Override annotation. | 3 | 146–225 |
| CyclomaticComplexity | The method 'getPage(HttpServletRequest, int, Plugin)' has a cyclomatic complexity of 16. | 3 | 146–225 |
| NPathComplexity | The method 'getPage(HttpServletRequest, int, Plugin)' has an NPath complexity of 480 | 3 | 146–225 |
| ConfusingTernary | Avoid if (x != y) ..; else ..; | 3 | 187–222 |
| ConfusingTernary | Avoid if (x != y) ..; else ..; | 3 | 195–222 |
| ConfusingTernary | Avoid if (x != y) ..; else ..; | 3 | 202–222 |
| ConfusingTernary | Avoid if (x != y) ..; else ..; | 3 | 210–222 |
| UnusedFormalParameter | Avoid unused method parameters such as 'nMode'. | 3 | 235 |
| CyclomaticComplexity | The method 'getResult(HttpServletRequest, HttpSession, int, Plugin)' has a cyclomatic complexity of 16. | 3 | 235–296 |
| UseDiamondOperator | Explicit type arguments can be replaced by Diamond Operator | 3 | 287 |
| UseConcurrentHashMap | If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation | 3 | 287 |
| UnusedFormalParameter | Avoid unused method parameters such as 'nMode'. | 3 | 307 |
| UnusedFormalParameter | Avoid unused method parameters such as 'session'. | 3 | 307 |
| UseDiamondOperator | Explicit type arguments can be replaced by Diamond Operator | 3 | 313 |
| UnusedFormalParameter | Avoid unused method parameters such as 'nMode'. | 3 | 332 |
| UnusedFormalParameter | Avoid unused method parameters such as 'session'. | 3 | 332 |
| UseDiamondOperator | Explicit type arguments can be replaced by Diamond Operator | 3 | 336 |
| ConfusingTernary | Avoid if (x != y) ..; else ..; | 3 | 354–370 |
| UseStringBufferForStringAppends | Prefer StringBuilder (non-synchronized) or StringBuffer (synchronized) over += for concatenating strings | 3 | 364 |
| UnusedFormalParameter | Avoid unused method parameters such as 'nMode'. | 3 | 388 |
| ExcessiveMethodLength | Avoid really long methods. | 3 | 388–503 |
| CyclomaticComplexity | The method 'getRecap(HttpServletRequest, HttpSession, int, Plugin)' has a cyclomatic complexity of 20. | 3 | 388–503 |
| NPathComplexity | The method 'getRecap(HttpServletRequest, HttpSession, int, Plugin)' has an NPath complexity of 11520 | 3 | 388–503 |
| UseConcurrentHashMap | If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation | 3 | 392 |
| UseDiamondOperator | Explicit type arguments can be replaced by Diamond Operator | 3 | 392 |
| ConfusingTernary | Avoid if (x != y) ..; else ..; | 3 | 468–476 |
| UnusedFormalParameter | Avoid unused method parameters such as 'nMode'. | 3 | 513 |
| UseDiamondOperator | Explicit type arguments can be replaced by Diamond Operator | 3 | 517 |
| UseConcurrentHashMap | If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation | 3 | 517 |
| UseDiamondOperator | Explicit type arguments can be replaced by Diamond Operator | 3 | 571 |
| ConfusingTernary | Avoid if (x != y) ..; else ..; | 3 | 617–624 |
| Rule | Violation | Priority | Ligne |
|---|---|---|---|
| ExcessiveImports | A high number of imports can indicate a high degree of coupling within an object. | 3 | 34–3884 |
| ExcessivePublicCount | This class has a bunch of public methods and attributes | 3 | 34–3884 |
| CyclomaticComplexity | The class 'FormJspBean' has a total cyclomatic complexity of 456 (highest 29). | 3 | 127–3884 |
| ExcessiveClassLength | Avoid really long classes. | 3 | 127–3884 |
| MissingSerialVersionUID | Classes implementing Serializable should set a serialVersionUID | 3 | 127–3884 |
| NcssCount | The class 'FormJspBean' has a NCSS line count of 1753 (Highest = 90). | 3 | 127–3884 |
| TooManyMethods | This class has too many methods, consider refactoring it. | 3 | 128–3884 |
| ImmutableField | Private field '_nDefaultItemsPerPage' could be made final; it is only initialized in the declaration or constructor. | 3 | 320 |
| UseConcurrentHashMap | If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation | 3 | 388 |
| UseDiamondOperator | Explicit type arguments can be replaced by Diamond Operator | 3 | 388 |
| ExcessiveMethodLength | Avoid really long methods. | 3 | 445–639 |
| CyclomaticComplexity | The method 'getFormData(HttpServletRequest, Form)' has a cyclomatic complexity of 29. | 3 | 445–639 |
| NPathComplexity | The method 'getFormData(HttpServletRequest, Form)' has an NPath complexity of 1679360 | 3 | 445–639 |
| NcssCount | The method 'getFormData(HttpServletRequest, Form)' has a NCSS line count of 90. | 3 | 445–639 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 3 | 449 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 3 | 450 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 3 | 451 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 3 | 452 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 3 | 453 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 3 | 455 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 3 | 456 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 3 | 457 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 3 | 458 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 3 | 460 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 3 | 461 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 3 | 462 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 3 | 463 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 3 | 464 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 3 | 465 |
| ConfusingTernary | Avoid if (x != y) ..; else ..; | 3 | 516–523 |
| ConfusingTernary | Avoid if (x != y) ..; else ..; | 3 | 525–532 |
| ConfusingTernary | Avoid if (x != y) ..; else ..; | 3 | 534–541 |
| ConfusingTernary | Avoid if (x != y) ..; else ..; | 3 | 543–550 |
| ConfusingTernary | Avoid if (x != y) ..; else ..; | 3 | 552–559 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.util.Date' due to existing import 'java.util.Date' | 4 | 581 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.util.Date' due to existing import 'java.util.Date' | 4 | 599 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 3 | 659 |
| UseDiamondOperator | Explicit type arguments can be replaced by Diamond Operator | 3 | 667 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 3 | 692 |
| CyclomaticComplexity | The method 'getModifyForm(HttpServletRequest)' has a cyclomatic complexity of 11. | 3 | 725–845 |
| NcssCount | The method 'getModifyForm(HttpServletRequest)' has a NCSS line count of 74. | 3 | 725–845 |
| ExcessiveMethodLength | Avoid really long methods. | 3 | 725–845 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 3 | 727 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 3 | 728 |
| UseDiamondOperator | Explicit type arguments can be replaced by Diamond Operator | 3 | 728 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 3 | 729 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 3 | 730 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 3 | 731 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 3 | 734 |
| UseCollectionIsEmpty | Substitute calls to size() == 0 (or size() != 0, size() > 0, size() < 1) with calls to isEmpty() | 3 | 771 |
| AvoidDuplicateLiterals | The String literal "?id_form=" appears 4 times in this file; the first occurrence is on line 806 | 3 | 806 |
| UseDiamondOperator | Explicit type arguments can be replaced by Diamond Operator | 3 | 827 |
| UseConcurrentHashMap | If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation | 3 | 827 |
| CyclomaticComplexity | The method 'doModifyForm(HttpServletRequest)' has a cyclomatic complexity of 10. | 3 | 852–902 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 3 | 911 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 3 | 913 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 3 | 914 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 3 | 961 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 3 | 962 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 3 | 963 |
| UseDiamondOperator | Explicit type arguments can be replaced by Diamond Operator | 3 | 989 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 3 | 1014 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 3 | 1015 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 3 | 1016 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 3 | 1017 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 3 | 1061 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 3 | 1064 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 3 | 1065 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 3 | 1066 |
| UseConcurrentHashMap | If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation | 3 | 1095 |
| UseDiamondOperator | Explicit type arguments can be replaced by Diamond Operator | 3 | 1095 |
| CyclomaticComplexity | The method 'getRecapData(HttpServletRequest, Recap)' has a cyclomatic complexity of 15. | 3 | 1115–1233 |
| NPathComplexity | The method 'getRecapData(HttpServletRequest, Recap)' has an NPath complexity of 980 | 3 | 1115–1233 |
| ExcessiveMethodLength | Avoid really long methods. | 3 | 1115–1233 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 3 | 1117 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 3 | 1118 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 3 | 1121 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 3 | 1122 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 3 | 1123 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 3 | 1124 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 3 | 1128 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 3 | 1129 |
| ConfusingTernary | Avoid if (x != y) ..; else ..; | 3 | 1153–1160 |
| ConfusingTernary | Avoid if (x != y) ..; else ..; | 3 | 1162–1230 |
| ConfusingTernary | Avoid if (x != y) ..; else ..; | 3 | 1213–1220 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 3 | 1286 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 3 | 1289 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 3 | 1290 |
| UseConcurrentHashMap | If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation | 3 | 1321 |
| UseDiamondOperator | Explicit type arguments can be replaced by Diamond Operator | 3 | 1321 |
| CyclomaticComplexity | The method 'doCreateEntry(HttpServletRequest)' has a cyclomatic complexity of 12. | 3 | 1345–1415 |
| NPathComplexity | The method 'doCreateEntry(HttpServletRequest)' has an NPath complexity of 208 | 3 | 1345–1415 |
| ConfusingTernary | Avoid if (x != y) ..; else ..; | 3 | 1407–1414 |
| NPathComplexity | The method 'getModifyEntry(HttpServletRequest)' has an NPath complexity of 288 | 3 | 1422–1508 |
| CyclomaticComplexity | The method 'getModifyEntry(HttpServletRequest)' has a cyclomatic complexity of 11. | 3 | 1422–1508 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 3 | 1424 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 3 | 1425 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 3 | 1426 |
| UseDiamondOperator | Explicit type arguments can be replaced by Diamond Operator | 3 | 1454 |
| AvoidReassigningLoopVariables | Avoid reassigning the loop control variable 'field' | 3 | 1458 |
| UseConcurrentHashMap | If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation | 3 | 1464 |
| UseDiamondOperator | Explicit type arguments can be replaced by Diamond Operator | 3 | 1464 |
| NPathComplexity | The method 'doModifyEntry(HttpServletRequest)' has an NPath complexity of 252 | 3 | 1515–1578 |
| CyclomaticComplexity | The method 'doModifyEntry(HttpServletRequest)' has a cyclomatic complexity of 12. | 3 | 1515–1578 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 3 | 1517 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 3 | 1518 |
| ConfusingTernary | Avoid if (x != y) ..; else ..; | 3 | 1565–1572 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 3 | 1587 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 3 | 1588 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 3 | 1590 |
| ConfusingTernary | Avoid if (x != y) ..; else ..; | 3 | 1618–1625 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 3 | 1645 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 3 | 1647 |
| UseDiamondOperator | Explicit type arguments can be replaced by Diamond Operator | 3 | 1673 |
| ConfusingTernary | Avoid if (x != y) ..; else ..; | 3 | 1685–1692 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 3 | 1702 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 3 | 1704 |
| ConfusingTernary | Avoid if (x != y) ..; else ..; | 3 | 1741–1748 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 3 | 1758 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 3 | 1759 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 3 | 1760 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 3 | 1761 |
| UseDiamondOperator | Explicit type arguments can be replaced by Diamond Operator | 3 | 1794 |
| UseConcurrentHashMap | If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation | 3 | 1794 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 3 | 1811 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 3 | 1812 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 3 | 1813 |
| CyclomaticComplexity | The method 'doMoveUpEntry(HttpServletRequest)' has a cyclomatic complexity of 10. | 3 | 1863–1942 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 3 | 1865 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 3 | 1866 |
| ConfusingTernary | Avoid if (x != y) ..; else ..; | 3 | 1898–1905 |
| ConfusingTernary | Avoid if (x != y) ..; else ..; | 3 | 1907–1914 |
| ConfusingTernary | Avoid if (x != y) ..; else ..; | 3 | 1934–1941 |
| CyclomaticComplexity | The method 'doMoveDownEntry(HttpServletRequest)' has a cyclomatic complexity of 10. | 3 | 1949–2028 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 3 | 1951 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 3 | 1952 |
| ConfusingTernary | Avoid if (x != y) ..; else ..; | 3 | 1984–1991 |
| ConfusingTernary | Avoid if (x != y) ..; else ..; | 3 | 1993–2000 |
| ConfusingTernary | Avoid if (x != y) ..; else ..; | 3 | 2020–2027 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 3 | 2037 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 3 | 2038 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 3 | 2106 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 3 | 2107 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 3 | 2108 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 3 | 2180 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 3 | 2181 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 3 | 2183 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 3 | 2247 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 3 | 2248 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 3 | 2250 |
| UnusedFormalParameter | Avoid unused method parameters such as 'plugin'. | 3 | 2367 |
| UnusedFormalParameter | Avoid unused method parameters such as 'locale'. | 3 | 2388 |
| ConfusingTernary | Avoid if (x != y) ..; else ..; | 3 | 2395–2402 |
| UnusedFormalParameter | Avoid unused method parameters such as 'locale'. | 3 | 2414 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 3 | 2438 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 3 | 2439 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 3 | 2495 |
| UseDiamondOperator | Explicit type arguments can be replaced by Diamond Operator | 3 | 2507 |
| UseConcurrentHashMap | If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation | 3 | 2507 |
| ConfusingTernary | Avoid if (x != y) ..; else ..; | 3 | 2510–2517 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 3 | 2535 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 3 | 2536 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 3 | 2537 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 3 | 2538 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 3 | 2539 |
| UseDiamondOperator | Explicit type arguments can be replaced by Diamond Operator | 3 | 2573 |
| ConfusingTernary | Avoid if (x != y) ..; else ..; | 3 | 2659–2680 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 3 | 2722 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 3 | 2759 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 3 | 2760 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 3 | 2761 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 3 | 2762 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 3 | 2763 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 3 | 2811 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 3 | 2812 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 3 | 2813 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 3 | 2814 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 3 | 2815 |
| UseDiamondOperator | Explicit type arguments can be replaced by Diamond Operator | 3 | 2935 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 3 | 2938 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 3 | 2939 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 3 | 2943 |
| UseDiamondOperator | Explicit type arguments can be replaced by Diamond Operator | 3 | 2969 |
| UseConcurrentHashMap | If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation | 3 | 2969 |
| UseStringBufferForStringAppends | Prefer StringBuilder (non-synchronized) or StringBuffer (synchronized) over += for concatenating strings | 3 | 2974 |
| NPathComplexity | The method 'doTestForm(HttpServletRequest)' has an NPath complexity of 1152 | 3 | 2990–3088 |
| CyclomaticComplexity | The method 'doTestForm(HttpServletRequest)' has a cyclomatic complexity of 15. | 3 | 2990–3088 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 3 | 2992 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 3 | 2993 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 3 | 2994 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 3 | 2996 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 3 | 2997 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 3 | 2998 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 3 | 3000 |
| UseDiamondOperator | Explicit type arguments can be replaced by Diamond Operator | 3 | 3057 |
| CyclomaticComplexity | The method 'doExportResponseTestForm(HttpServletRequest, HttpServletResponse)' has a cyclomatic complexity of 13. | 3 | 3098–3172 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 3 | 3102 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 3 | 3103 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 3 | 3109 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 3 | 3110 |
| ConfusingTernary | Avoid if (x != y) ..; else ..; | 3 | 3138–3168 |
| UseCollectionIsEmpty | Substitute calls to size() == 0 (or size() != 0, size() > 0, size() < 1) with calls to isEmpty() | 3 | 3138 |
| ShortVariable | Avoid variables with short names like os | 3 | 3156 |
| NPathComplexity | The method 'getResult(HttpServletRequest)' has an NPath complexity of 1920 | 3 | 3179–3284 |
| NcssCount | The method 'getResult(HttpServletRequest)' has a NCSS line count of 61. | 3 | 3179–3284 |
| ExcessiveMethodLength | Avoid really long methods. | 3 | 3179–3284 |
| CyclomaticComplexity | The method 'getResult(HttpServletRequest)' has a cyclomatic complexity of 15. | 3 | 3179–3284 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 3 | 3181 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 3 | 3183 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 3 | 3184 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 3 | 3185 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 3 | 3186 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 3 | 3187 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 3 | 3189 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 3 | 3194 |
| ConfusingTernary | Avoid if (x != y) ..; else ..; | 3 | 3247–3266 |
| UseDiamondOperator | Explicit type arguments can be replaced by Diamond Operator | 3 | 3268 |
| UseConcurrentHashMap | If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation | 3 | 3268 |
| NullAssignment | Assigning an Object to null is a code smell. Consider refactoring. | 3 | 3273 |
| NullAssignment | Assigning an Object to null is a code smell. Consider refactoring. | 3 | 3276 |
| NPathComplexity | The method 'doExportResult(HttpServletRequest, HttpServletResponse)' has an NPath complexity of 769 | 3 | 3293–3396 |
| ExcessiveMethodLength | Avoid really long methods. | 3 | 3293–3396 |
| CyclomaticComplexity | The method 'doExportResult(HttpServletRequest, HttpServletResponse)' has a cyclomatic complexity of 16. | 3 | 3293–3396 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 3 | 3297 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 3 | 3320 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 3 | 3321 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 3 | 3322 |
| ConfusingTernary | Avoid if (x != y) ..; else ..; | 3 | 3362–3392 |
| UseCollectionIsEmpty | Substitute calls to size() == 0 (or size() != 0, size() > 0, size() < 1) with calls to isEmpty() | 3 | 3362 |
| ShortVariable | Avoid variables with short names like os | 3 | 3380 |
| ExcessiveMethodLength | Avoid really long methods. | 3 | 3404–3526 |
| CyclomaticComplexity | The method 'doGenerateGraph(HttpServletRequest, HttpServletResponse)' has a cyclomatic complexity of 17. | 3 | 3404–3526 |
| NPathComplexity | The method 'doGenerateGraph(HttpServletRequest, HttpServletResponse)' has an NPath complexity of 3840 | 3 | 3404–3526 |
| NcssCount | The method 'doGenerateGraph(HttpServletRequest, HttpServletResponse)' has a NCSS line count of 65. | 3 | 3404–3526 |
| ConfusingTernary | Avoid if (x != y) ..; else ..; | 3 | 3445–3464 |
| UseDiamondOperator | Explicit type arguments can be replaced by Diamond Operator | 3 | 3480 |
| UseCollectionIsEmpty | Substitute calls to size() == 0 (or size() != 0, size() > 0, size() < 1) with calls to isEmpty() | 3 | 3483 |
| AvoidCatchingGenericException | Avoid catching generic exceptions such as NullPointerException, RuntimeException, Exception in try-catch block | 3 | 3522 |
| CyclomaticComplexity | The method 'doDownloadFile(HttpServletRequest, HttpServletResponse)' has a cyclomatic complexity of 10. | 3 | 3535–3610 |
| UseDiamondOperator | Explicit type arguments can be replaced by Diamond Operator | 3 | 3564 |
| UseCollectionIsEmpty | Substitute calls to size() == 0 (or size() != 0, size() > 0, size() < 1) with calls to isEmpty() | 3 | 3574 |
| UseCollectionIsEmpty | Substitute calls to size() == 0 (or size() != 0, size() > 0, size() < 1) with calls to isEmpty() | 3 | 3575 |
| ConfusingTernary | Avoid if (x != y) ..; else ..; | 3 | 3584–3606 |
| ShortVariable | Avoid variables with short names like os | 3 | 3593 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 3 | 3621 |
| UseDiamondOperator | Explicit type arguments can be replaced by Diamond Operator | 3 | 3646 |
| UseDiamondOperator | Explicit type arguments can be replaced by Diamond Operator | 3 | 3648 |
| LinguisticNaming | Linguistics Antipattern - The variable 'hasMaProcess' indicates linguistically it is a boolean, but it is 'HashMap' | 3 | 3650 |
| UseDiamondOperator | Explicit type arguments can be replaced by Diamond Operator | 3 | 3655 |
| CyclomaticComplexity | The method 'doManageOutputProcessor(HttpServletRequest)' has a cyclomatic complexity of 16. | 3 | 3678–3750 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 3 | 3684 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 3 | 3685 |
| ConfusingTernary | Avoid if (x != y) ..; else ..; | 3 | 3717–3744 |
| ConfusingTernary | Avoid if (x != y) ..; else ..; | 3 | 3721–3728 |
| UseConcurrentHashMap | If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation | 3 | 3794 |
| UseDiamondOperator | Explicit type arguments can be replaced by Diamond Operator | 3 | 3794 |
| CyclomaticComplexity | The method 'doModifyMessage(HttpServletRequest)' has a cyclomatic complexity of 15. | 3 | 3811–3883 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 3 | 3842 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 3 | 3846 |
| Rule | Violation | Priority | Ligne |
|---|---|---|---|
| MissingSerialVersionUID | Classes implementing Serializable should set a serialVersionUID | 3 | 60–133 |
| UseDiamondOperator | Explicit type arguments can be replaced by Diamond Operator | 3 | 92 |
| UseConcurrentHashMap | If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation | 3 | 92 |
| UseCollectionIsEmpty | Substitute calls to size() == 0 (or size() != 0, size() > 0, size() < 1) with calls to isEmpty() | 3 | 98 |
| Rule | Violation | Priority | Ligne |
|---|---|---|---|
| MissingSerialVersionUID | Classes implementing Serializable should set a serialVersionUID | 3 | 60–225 |
| MissingOverride | The method 'getCreate(HttpServletRequest)' is missing an @Override annotation. | 3 | 79–93 |
| MissingOverride | The method 'getModify(HttpServletRequest)' is missing an @Override annotation. | 3 | 101–128 |
| MissingOverride | The method 'doCreate(HttpServletRequest)' is missing an @Override annotation. | 3 | 136–176 |
| MissingOverride | The method 'doModify(HttpServletRequest)' is missing an @Override annotation. | 3 | 184–224 |
