| Violation |
Line |
| Possible God class (WMC=195, ATFD=219, TCC=0.19913419913419914) |
34–1826 |
| A high number of imports can indicate a high degree of coupling within an object. |
34–1826 |
| Avoid unused imports such as 'java.time.LocalDate' |
108 |
| Avoid unused imports such as 'java.time.LocalDate' |
108 |
| Avoid unused imports such as 'org.apache.commons.lang.time.DateFormatUtils' |
113 |
| Avoid unused imports such as 'org.apache.commons.lang.time.DateFormatUtils' |
113 |
| The class 'Files2DocsJspBean' has a Standard Cyclomatic Complexity of 6 (Highest = 24). |
119–1826 |
| Avoid really long classes. |
119–1826 |
| The class 'Files2DocsJspBean' has a Cyclomatic Complexity of 9 (Highest = 34). |
119–1826 |
| The class 'Files2DocsJspBean' has a Modified Cyclomatic Complexity of 6 (Highest = 24). |
119–1826 |
| Perhaps '_nDefaultItemsPerPage' could be replaced by a local variable. |
269 |
| It is a good practice to call super() in a constructor |
279–281 |
| Avoid unnecessary constructors - the compiler will generate these for you |
279–281 |
| Document empty constructor |
279–281 |
| This statement may have some unnecessary parentheses |
297 |
| Avoid unused private methods such as 'deleteUploadedFile(HttpServletRequest,String)'. |
381 |
| This statement may have some unnecessary parentheses |
407 |
| Avoid declaring a variable if it is unreferenced before a possible exit point. |
456 |
| Avoid declaring a variable if it is unreferenced before a possible exit point. |
457 |
| The method getReplacedTags() has an NPath complexity of 243 |
534–573 |
| The method 'getReplacedTags' has a Cyclomatic Complexity of 11. |
534–573 |
| Avoid really long methods. |
582–806 |
| The method 'getSelectFiles' has a Cyclomatic Complexity of 34. |
582–806 |
| The method getSelectFiles() has an NCSS line count of 109 |
582–806 |
| The method getSelectFiles() has an NPath complexity of 525312 |
582–806 |
| The method 'getSelectFiles' has a Modified Cyclomatic Complexity of 23. |
582–806 |
| The method 'getSelectFiles' has a Standard Cyclomatic Complexity of 23. |
582–806 |
| Avoid using Literals in Conditional Statements |
590 |
| Prefer StringBuffer over += for concatenating strings |
625 |
| Avoid using if statements without curly braces |
637 |
| Avoid using if statements without curly braces |
638 |
| If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation |
670 |
| Prefer StringBuffer over += for concatenating strings |
718 |
| Prefer StringBuffer over += for concatenating strings |
722 |
| Prefer StringBuffer over += for concatenating strings |
730 |
| Prefer StringBuffer over += for concatenating strings |
734 |
| If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation |
744 |
| Avoid using if statements without curly braces |
787 |
| Avoid using if statements without curly braces |
853 |
| The method 'doSelectFiles' has a Modified Cyclomatic Complexity of 16. |
909–1053 |
| Avoid really long methods. |
909–1053 |
| The method 'doSelectFiles' has a Standard Cyclomatic Complexity of 16. |
909–1053 |
| The method 'doSelectFiles' has a Cyclomatic Complexity of 29. |
909–1053 |
| The method doSelectFiles() has an NPath complexity of 71280 |
909–1053 |
| Avoid declaring a variable if it is unreferenced before a possible exit point. |
932 |
| Prefer StringBuffer over += for concatenating strings |
948 |
| Avoid using if statements without curly braces |
1050 |
| The method 'upload' has a Standard Cyclomatic Complexity of 12. |
1062–1161 |
| The method upload() has an NPath complexity of 1176 |
1062–1161 |
| The method 'upload' has a Cyclomatic Complexity of 16. |
1062–1161 |
| The method 'upload' has a Modified Cyclomatic Complexity of 12. |
1062–1161 |
| Prefer StringBuffer over += for concatenating strings |
1225 |
| If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation |
1264 |
| The method 'getAttributesForms' has a Cyclomatic Complexity of 10. |
1302–1400 |
| If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation |
1365 |
| If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation |
1382 |
| The method 'doCreateDocuments' has a Standard Cyclomatic Complexity of 24. |
1409–1673 |
| The method 'doCreateDocuments' has a Cyclomatic Complexity of 33. |
1409–1673 |
| Avoid really long methods. |
1409–1673 |
| The method 'doCreateDocuments' has a Modified Cyclomatic Complexity of 24. |
1409–1673 |
| The method doCreateDocuments() has an NCSS line count of 117 |
1409–1673 |
| The method doCreateDocuments() has an NPath complexity of 47619 |
1409–1673 |
| Avoid declaring a variable if it is unreferenced before a possible exit point. |
1416 |
| Avoid variables with short names like in |
1496 |
| Prefer StringBuffer over += for concatenating strings |
1606 |
| Prefer StringBuffer over += for concatenating strings |
1610 |
| Prefer StringBuffer over += for concatenating strings |
1643 |
| Prefer StringBuffer over += for concatenating strings |
1647 |
| The method 'getImportResult' has a Cyclomatic Complexity of 12. |
1682–1789 |
| Avoid really long methods. |
1682–1789 |
| If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation |
1765 |
| Avoid using if statements without curly braces |
1817 |
| Avoid using if statements without curly braces |
1818 |
| Avoid using if statements without curly braces |
1822 |
| Prefer StringBuffer over += for concatenating strings |
1822 |
