PMD Results

The following document contains the results of PMD 7.14.0.

PMD found no problems in your source code.

Suppressed Violations

Filename Rule message Suppression type Reason
com/reallifedeveloper/tools/FullThreadDump.java Thrown exception does not preserve the stack trace of exception 'x' on all code paths @suppresswarnings
com/reallifedeveloper/tools/FullThreadDump.java The initializer for variable 'port' is never used (overwritten on line 151) @suppresswarnings
com/reallifedeveloper/tools/FullThreadDump.java Consider using varargs for methods or constructors which take an array the last parameter. @suppresswarnings
com/reallifedeveloper/tools/FullThreadDump.java Missing commented default access modifier on constructor 'ThreadMonitor(MBeanServerConnection)' @suppresswarnings
com/reallifedeveloper/tools/FullThreadDump.java The method 'main(String[])' has a cyclomatic complexity of 11. @suppresswarnings
com/reallifedeveloper/tools/FullThreadDump.java Field 'tmbean' may be declared final @suppresswarnings
com/reallifedeveloper/tools/FullThreadDump.java Perhaps 'jmxc' could be replaced by a local variable. @suppresswarnings
com/reallifedeveloper/tools/FullThreadDump.java Avoid concatenating nonliterals in a StringBuffer/StringBuilder constructor or append(). @suppresswarnings
com/reallifedeveloper/tools/FullThreadDump.java Avoid concatenating nonliterals in a StringBuffer/StringBuilder constructor or append(). @suppresswarnings
com/reallifedeveloper/tools/gis/GeometryDrawingPanel.java Avoid instantiating new objects inside loops @suppresswarnings
com/reallifedeveloper/tools/rabbitmq/MoveMessages.java Rather than using a lot of String arguments, consider using a container object for those values. @suppresswarnings
com/reallifedeveloper/tools/test/TestUtil.java You should not modify visibility of constructors, methods or fields using setAccessible() @suppresswarnings
com/reallifedeveloper/tools/test/TestUtil.java You should not modify visibility of constructors, methods or fields using setAccessible() @suppresswarnings
com/reallifedeveloper/tools/test/database/dbunit/AbstractDbTest.java An empty method in an abstract class should be abstract instead @suppresswarnings
com/reallifedeveloper/tools/test/database/dbunit/AbstractDbTest.java A method/constructor should not explicitly throw java.lang.Exception @suppresswarnings
com/reallifedeveloper/tools/test/database/dbunit/AbstractDbTest.java A method/constructor should not explicitly throw java.lang.Exception @suppresswarnings
com/reallifedeveloper/tools/test/database/dbunit/AbstractDbTest.java Avoid empty finalize methods @suppresswarnings
com/reallifedeveloper/tools/test/database/dbunit/DbTestHelper.java A method/constructor should not explicitly throw java.lang.Exception @suppresswarnings
com/reallifedeveloper/tools/test/database/dbunit/DbTestHelper.java A method/constructor should not explicitly throw java.lang.Exception @suppresswarnings
com/reallifedeveloper/tools/test/database/dbunit/DbUnitFlatXmlReader.java Unnecessary cast (Node) @suppresswarnings
com/reallifedeveloper/tools/test/database/dbunit/DbUnitFlatXmlReader.java Unnecessary cast (Node) @suppresswarnings
com/reallifedeveloper/tools/test/database/dbunit/DbUnitFlatXmlReader.java Unnecessary cast (NamedNodeMap) @suppresswarnings
com/reallifedeveloper/tools/test/database/dbunit/DbUnitFlatXmlReader.java A value of 25 may denote a high amount of coupling within the class (threshold: 20) @suppresswarnings
com/reallifedeveloper/tools/test/database/dbunit/DbUnitFlatXmlReader.java The method 'createObjectFromString(String, Class<?>)' has a cyclomatic complexity of 13. @suppresswarnings
com/reallifedeveloper/tools/test/database/dbunit/DbUnitFlatXmlReader.java A high number of imports can indicate a high degree of coupling within an object. @suppresswarnings
com/reallifedeveloper/tools/test/database/dbunit/DbUnitFlatXmlReader.java Possible God Class (WMC=75, ATFD=49, TCC=1.905%) @suppresswarnings
com/reallifedeveloper/tools/test/database/dbunit/DbUnitFlatXmlReader.java This class has too many methods, consider refactoring it. @suppresswarnings
com/reallifedeveloper/tools/test/database/dbunit/DbUnitFlatXmlReader.java You should not modify visibility of constructors, methods or fields using setAccessible() @suppresswarnings
com/reallifedeveloper/tools/test/database/dbunit/DbUnitFlatXmlReader.java You should not modify visibility of constructors, methods or fields using setAccessible() @suppresswarnings
com/reallifedeveloper/tools/test/database/dbunit/DbUnitFlatXmlReader.java You should not modify visibility of constructors, methods or fields using setAccessible() @suppresswarnings
com/reallifedeveloper/tools/test/database/dbunit/DbUnitFlatXmlReader.java You should not modify visibility of constructors, methods or fields using setAccessible() @suppresswarnings
com/reallifedeveloper/tools/test/database/inmemory/AbstractInMemoryCrudRepository.java The initializer for variable 'id' is never used (overwritten on lines 312, 314 and 316) @suppresswarnings
com/reallifedeveloper/tools/test/database/inmemory/AbstractInMemoryCrudRepository.java An empty method in an abstract class should be abstract instead @suppresswarnings
com/reallifedeveloper/tools/test/database/inmemory/AbstractInMemoryCrudRepository.java Consider simply returning the value vs storing it in local variable 'page' @suppresswarnings
com/reallifedeveloper/tools/test/database/inmemory/AbstractInMemoryCrudRepository.java Explicit type arguments can be replaced by a diamond: `new ArrayList<>()` @suppresswarnings
com/reallifedeveloper/tools/test/database/inmemory/AbstractInMemoryCrudRepository.java A value of 32 may denote a high amount of coupling within the class (threshold: 20) @suppresswarnings
com/reallifedeveloper/tools/test/database/inmemory/AbstractInMemoryCrudRepository.java The class 'AbstractInMemoryCrudRepository' has a total cyclomatic complexity of 109 (highest 7). @suppresswarnings
com/reallifedeveloper/tools/test/database/inmemory/AbstractInMemoryCrudRepository.java Possible God Class (WMC=109, ATFD=32, TCC=6.757%) @suppresswarnings
com/reallifedeveloper/tools/test/database/inmemory/AbstractInMemoryCrudRepository.java This class has too many methods, consider refactoring it. @suppresswarnings
com/reallifedeveloper/tools/test/database/inmemory/AbstractInMemoryCrudRepository.java You should not modify visibility of constructors, methods or fields using setAccessible() @suppresswarnings
com/reallifedeveloper/tools/test/database/inmemory/AbstractInMemoryCrudRepository.java You should not modify visibility of constructors, methods or fields using setAccessible() @suppresswarnings
com/reallifedeveloper/tools/test/database/inmemory/AbstractInMemoryCrudRepository.java You should not modify visibility of constructors, methods or fields using setAccessible() @suppresswarnings
com/reallifedeveloper/tools/test/database/inmemory/AbstractInMemoryCrudRepository.java You should not modify visibility of constructors, methods or fields using setAccessible() @suppresswarnings
com/reallifedeveloper/tools/test/database/inmemory/AbstractInMemoryCrudRepository.java The String literal "Not yet implemented" appears 6 times in this file; the first occurrence is on line 477 @suppresswarnings
com/reallifedeveloper/tools/test/database/inmemory/AbstractInMemoryCrudRepository.java Avoid empty finalize methods @suppresswarnings
com/reallifedeveloper/tools/test/fitnesse/BaseFitNesseFixture.java Avoid empty finalize methods @suppresswarnings