PMD Audit |
|
| Designed for use with PMD 4.2.1 and Ant. |
| Files | Total | Priority 1 | Priority 2 | Priority 3 | Priority 4 | Priority 5 |
|---|---|---|---|---|---|---|
| 56 | 95 | 8 |
1 |
86 |
0 |
0 |

| Priority | Error Description | Line |
|---|---|---|
|
3
|
Possible unsafe assignment to a non-final static field in a constructor. | 42 |
| Priority | Error Description | Line |
|---|---|---|
|
3
|
Too many static imports may lead to messy code | 21 |
| Priority | Error Description | Line |
|---|---|---|
|
3
|
Deeply nested if..then statements are hard to read | 85 |
|
3
|
Assigning an Object to null is a code smell. Consider refactoring. | 91 |
| Priority | Error Description | Line |
|---|---|---|
|
3
|
Avoid unused local variables such as 'buff'. | 41 |
|
2
|
System.out.print is used | 77 |
| Priority | Error Description | Line |
|---|---|---|
|
3
|
These nested if statements could be combined | 63 |
| Priority | Error Description | Line |
|---|---|---|
|
3
|
A high number of imports can indicate a high degree of coupling within an object. | 21 |
| Priority | Error Description | Line |
|---|---|---|
|
3
|
Perhaps 'gnuHeaderFile' could be replaced by a local variable. | 48 |
|
3
|
Perhaps 'javadocProp' could be replaced by a local variable. | 53 |
| Priority | Error Description | Line |
|---|---|---|
|
1
|
An empty method in an abstract class should be abstract instead | 107 |
| Priority | Error Description | Line |
|---|---|---|
|
3
|
A high number of imports can indicate a high degree of coupling within an object. | 21 |
| Priority | Error Description | Line |
|---|---|---|
|
1
|
An empty method in an abstract class should be abstract instead | 81 |
|
1
|
An empty method in an abstract class should be abstract instead | 85 |
| Priority | Error Description | Line |
|---|---|---|
|
3
|
This class has too many methods, consider refactoring it. | 33 |
| Priority | Error Description | Line |
|---|---|---|
|
3
|
Possible unsafe assignment to a non-final static field in a constructor. | 46 |
| Priority | Error Description | Line |
|---|---|---|
|
3
|
Perhaps 'bgColorField' could be replaced by a local variable. | 42 |
| Priority | Error Description | Line |
|---|---|---|
|
3
|
Possible unsafe assignment to a non-final static field in a constructor. | 35 |
| Priority | Error Description | Line |
|---|---|---|
|
3
|
Avoid empty if statements | 123 |
|
3
|
Avoid empty if statements | 153 |
| Priority | Error Description | Line |
|---|---|---|
|
3
|
Possible unsafe assignment to a non-final static field in a constructor. | 40 |
| Priority | Error Description | Line |
|---|---|---|
|
3
|
Perhaps 'dotExeFile' could be replaced by a local variable. | 52 |
| Priority | Error Description | Line |
|---|---|---|
|
3
|
Avoid using final local variables, turn them into fields | 54 |
| Priority | Error Description | Line |
|---|---|---|
|
3
|
Avoid using final local variables, turn them into fields | 60 |
| Priority | Error Description | Line |
|---|---|---|
|
3
|
This class has too many methods, consider refactoring it. | 44 |
| Priority | Error Description | Line |
|---|---|---|
|
3
|
This class has too many methods, consider refactoring it. | 41 |
| Priority | Error Description | Line |
|---|---|---|
|
3
|
This call to String.startsWith can be rewritten using String.charAt(0) | 110 |
| Priority | Error Description | Line |
|---|---|---|
|
3
|
Avoid using final local variables, turn them into fields | 77 |
| Priority | Error Description | Line |
|---|---|---|
|
3
|
Use block level rather than method level synchronization | 48 |
| Priority | Error Description | Line |
|---|---|---|
|
3
|
Assigning an Object to null is a code smell. Consider refactoring. | 106 |
|
3
|
Assigning an Object to null is a code smell. Consider refactoring. | 113 |
| Priority | Error Description | Line |
|---|---|---|
|
3
|
Avoid empty if statements | 303 |
| Priority | Error Description | Line |
|---|---|---|
|
3
|
A high number of imports can indicate a high degree of coupling within an object. | 21 |
| Priority | Error Description | Line |
|---|---|---|
|
3
|
A catch statement should never catch throwable since it includes errors. | 131 |
| Priority | Error Description | Line |
|---|---|---|
|
3
|
A high number of imports can indicate a high degree of coupling within an object. | 21 |
|
3
|
Deeply nested if..then statements are hard to read | 143 |
| Priority | Error Description | Line |
|---|---|---|
|
3
|
Use bitwise inversion to invert boolean values | 74 |
|
3
|
Use bitwise inversion to invert boolean values | 78 |
| Priority | Error Description | Line |
|---|---|---|
|
3
|
Perhaps 'parser' could be replaced by a local variable. | 70 |
| Priority | Error Description | Line |
|---|---|---|
|
3
|
A high number of imports can indicate a high degree of coupling within an object. | 21 |
| Priority | Error Description | Line |
|---|---|---|
|
3
|
A high number of imports can indicate a high degree of coupling within an object. | 21 |
| Priority | Error Description | Line |
|---|---|---|
|
3
|
This class has too many methods, consider refactoring it. | 52 |
| Priority | Error Description | Line |
|---|---|---|
|
3
|
A high number of imports can indicate a high degree of coupling within an object. | 21 |
|
3
|
Perhaps 'table' could be replaced by a local variable. | 84 |
| Priority | Error Description | Line |
|---|---|---|
|
3
|
Possible unsafe assignment to a non-final static field in a constructor. | 37 |
| Priority | Error Description | Line |
|---|---|---|
|
3
|
Avoid empty if statements | 107 |
| Priority | Error Description | Line |
|---|---|---|
|
3
|
Avoid empty if statements | 124 |
| Priority | Error Description | Line |
|---|---|---|
|
3
|
Perhaps 'projectionUpdater' could be replaced by a local variable. | 64 |
| Priority | Error Description | Line |
|---|---|---|
|
3
|
Avoid empty if statements | 106 |
| Priority | Error Description | Line |
|---|---|---|
|
1
|
Return an empty array rather than null. | 61 |
| Priority | Error Description | Line |
|---|---|---|
|
3
|
Assigning an Object to null is a code smell. Consider refactoring. | 60 |
|
3
|
Caught exception is rethrown, original stack trace may be lost | 69 |
|
3
|
Singleton is not thread safe | 83 |
| Priority | Error Description | Line |
|---|---|---|
|
3
|
Caught exception is rethrown, original stack trace may be lost | 93 |
| Priority | Error Description | Line |
|---|---|---|
|
3
|
This class has too many methods, consider refactoring it. | 46 |
| Priority | Error Description | Line |
|---|---|---|
|
3
|
A method/constructor shouldn't explicitly throw java.lang.Exception | 172 |
| Priority | Error Description | Line |
|---|---|---|
|
3
|
Avoid printStackTrace(); use a logger call instead. | 76 |

| Priority | File | Error Description | Line |
|---|---|---|---|
|
3
|
org/limy/eclipse/code/LimyCodePlugin.java | Possible unsafe assignment to a non-final static field in a constructor. | ${subEntry.line} |
|
3
|
org/limy/eclipse/core/LimyEclipsePlugin.java | Possible unsafe assignment to a non-final static field in a constructor. | ${subEntry.line} |
|
3
|
org/limy/eclipse/prop/LimyPropPlugin.java | Possible unsafe assignment to a non-final static field in a constructor. | ${subEntry.line} |
|
3
|
org/limy/eclipse/qalab/LimyQalabPlugin.java | Possible unsafe assignment to a non-final static field in a constructor. | ${subEntry.line} |
|
3
|
org/limy/eclipse/web/LimyWebPlugin.java | Possible unsafe assignment to a non-final static field in a constructor. | ${subEntry.line} |
| Priority | File | Error Description | Line |
|---|---|---|---|
|
3
|
org/limy/eclipse/code/accessor/AccessorCreater.java | Too many static imports may lead to messy code | ${subEntry.line} |
| Priority | File | Error Description | Line |
|---|---|---|---|
|
3
|
org/limy/eclipse/code/accessor/AccessorUtils.java | Deeply nested if..then statements are hard to read | ${subEntry.line} |
|
3
|
org/limy/eclipse/code/di/LimyDIUtils.java | Deeply nested if..then statements are hard to read | ${subEntry.line} |
|
3
|
org/limy/eclipse/qalab/mark/cobertura/CoberturaMarkAppender.java | Deeply nested if..then statements are hard to read | ${subEntry.line} |
| Priority | File | Error Description | Line |
|---|---|---|---|
|
3
|
org/limy/eclipse/code/etc/CreateIbatisXmlAction.java | Avoid unused local variables such as 'buff'. | ${subEntry.line} |
|
3
|
org/limy/eclipse/qalab/outline/CcParser.java | Avoid unused local variables such as 'lineNo'. | ${subEntry.line} |
| Priority | File | Error Description | Line |
|---|---|---|---|
|
2
|
org/limy/eclipse/code/etc/CreateIbatisXmlAction.java | System.out.print is used | ${subEntry.line} |
| Priority | File | Error Description | Line |
|---|---|---|---|
|
3
|
org/limy/eclipse/code/header/AddJavaHeaderAction.java | These nested if statements could be combined | ${subEntry.line} |
|
3
|
org/limy/eclipse/qalab/action/CreateBuildXmlAction.java | These nested if statements could be combined | ${subEntry.line} |
|
3
|
org/limy/eclipse/qalab/action/CreateBuildXmlAction.java | These nested if statements could be combined | ${subEntry.line} |
|
3
|
org/limy/eclipse/qalab/builder/LimyQalabBuilder.java | These nested if statements could be combined | ${subEntry.line} |
| Priority | File | Error Description | Line |
|---|---|---|---|
|
3
|
org/limy/eclipse/code/javadoc/LimyStubUtils.java | Caught exception is rethrown, original stack trace may be lost | ${subEntry.line} |
|
3
|
org/limy/eclipse/code/javadoc/LimyStubUtils.java | Caught exception is rethrown, original stack trace may be lost | ${subEntry.line} |
|
3
|
org/limy/velocity/VelocitySupport.java | Caught exception is rethrown, original stack trace may be lost | ${subEntry.line} |
|
3
|
org/limy/velocity/XmlToHtmlImpl.java | Caught exception is rethrown, original stack trace may be lost | ${subEntry.line} |
| Priority | File | Error Description | Line |
|---|---|---|---|
|
3
|
org/limy/eclipse/prop/editor/PropertyRule.java | Avoid empty if statements | ${subEntry.line} |
|
3
|
org/limy/eclipse/prop/editor/PropertyRule.java | Avoid empty if statements | ${subEntry.line} |
|
3
|
org/limy/eclipse/qalab/builder/LimyQalabBuilder.java | Avoid empty if statements | ${subEntry.line} |
|
3
|
org/limy/eclipse/qalab/mark/CoberturaMarkCreator.java | Avoid empty if statements | ${subEntry.line} |
|
3
|
org/limy/eclipse/web/velocityeditor/HtmlTagRule.java | Avoid empty if statements | ${subEntry.line} |
|
3
|
org/limy/eclipse/web/velocityeditor/VelocityDollarRule.java | Avoid empty if statements | ${subEntry.line} |
|
3
|
org/limy/eclipse/web/velocityeditor/VelocityFoldingParser.java | Avoid empty if statements | ${subEntry.line} |
| Priority | File | Error Description | Line |
|---|---|---|---|
|
3
|
org/limy/eclipse/qalab/action/toolbar/AbstractPartAction.java | Avoid returning from a finally block | ${subEntry.line} |
| Priority | File | Error Description | Line |
|---|---|---|---|
|
3
|
org/limy/eclipse/qalab/ant/CreateBuildProperties.java | This call to String.startsWith can be rewritten using String.charAt(0) | ${subEntry.line} |
| Priority | File | Error Description | Line |
|---|---|---|---|
|
3
|
org/limy/eclipse/qalab/builder/LimyQalabBuilder.java | Avoid unused private methods such as 'taskAll(LimyQalabEnvironment,IProject,IProgressMonitor)'. | ${subEntry.line} |
|
3
|
org/limy/eclipse/qalab/outline/CcParser.java | Avoid unused private methods such as 'searchMulti(AST,int...)'. | ${subEntry.line} |
| Priority | File | Error Description | Line |
|---|---|---|---|
|
3
|
org/limy/eclipse/qalab/editor/EditPositionHistoryManager.java | Use block level rather than method level synchronization | ${subEntry.line} |
| Priority | File | Error Description | Line |
|---|---|---|---|
|
3
|
org/limy/eclipse/qalab/mark/cobertura/CoberturaExecuter.java | A catch statement should never catch throwable since it includes errors. | ${subEntry.line} |
| Priority | File | Error Description | Line |
|---|---|---|---|
|
3
|
org/limy/eclipse/qalab/outline/CcImageCreator.java | Use bitwise inversion to invert boolean values | ${subEntry.line} |
|
3
|
org/limy/eclipse/qalab/outline/CcImageCreator.java | Use bitwise inversion to invert boolean values | ${subEntry.line} |
| Priority | File | Error Description | Line |
|---|---|---|---|
|
1
|
org/limy/eclipse/web/velocityeditor/VelocityHyperlinkDetector.java | Return an empty array rather than null. | ${subEntry.line} |
| Priority | File | Error Description | Line |
|---|---|---|---|
|
3
|
org/limy/velocity/VelocitySupport.java | Singleton is not thread safe | ${subEntry.line} |
| Priority | File | Error Description | Line |
|---|---|---|---|
|
3
|
org/limy/velocity/task/VmStyleTask.java | A method/constructor shouldn't explicitly throw java.lang.Exception | ${subEntry.line} |
| Priority | File | Error Description | Line |
|---|---|---|---|
|
3
|
org/limy/xml/XmlUtils.java | Avoid printStackTrace(); use a logger call instead. | ${subEntry.line} |