PMD Results
The following document contains the results of PMD 7.23.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 142) | @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 method 'threadDump()' | @suppresswarnings | |
| com/reallifedeveloper/tools/FullThreadDump.java | Missing commented default access modifier on method 'findDeadlock()' | @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 | Consider using varargs for methods or constructors which take an array the last parameter. | @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/TestUtil.java | Return an empty collection rather than null. | @suppresswarnings | |
| com/reallifedeveloper/tools/test/database/CrudRepositoryWriter.java | Annotate this interface with @FunctionalInterface or with @SuppressWarnings("PMD.ImplicitFunctionalInterface") to clarify your intent. | @suppresswarnings | |
| com/reallifedeveloper/tools/test/database/CrudRepositoryWriter.java | Avoid unused method parameters such as 'repositoryEntityType'. | @suppresswarnings | |
| com/reallifedeveloper/tools/test/database/CrudRepositoryWriter.java | Avoid unused method parameters such as 'entityType'. | @suppresswarnings | |
| com/reallifedeveloper/tools/test/database/CrudRepositoryWriter.java | Avoid unused method parameters such as 'repository'. | @suppresswarnings | |
| com/reallifedeveloper/tools/test/database/CrudRepositoryWriter.java | Avoid unused method parameters such as 'tableName'. | @suppresswarnings | |
| com/reallifedeveloper/tools/test/database/CrudRepositoryWriter.java | Avoid the use of value in annotations when its the only element | @suppresswarnings | |
| com/reallifedeveloper/tools/test/database/CrudRepositoryWriter.java | The method 'createObjectFromString(String, Class<?>)' has a cognitive complexity of 20, current threshold is 15 | @suppresswarnings | |
| com/reallifedeveloper/tools/test/database/CrudRepositoryWriter.java | A value of 25 may denote a high amount of coupling within the class (threshold: 20) | @suppresswarnings | |
| com/reallifedeveloper/tools/test/database/CrudRepositoryWriter.java | The method 'createObjectFromString(String, Class<?>)' has a cyclomatic complexity of 20. | @suppresswarnings | |
| com/reallifedeveloper/tools/test/database/CrudRepositoryWriter.java | The method 'handleOneToOne(Object, Field, OneToOne)' has a cyclomatic complexity of 10. | @suppresswarnings | |
| com/reallifedeveloper/tools/test/database/CrudRepositoryWriter.java | The class 'CrudRepositoryWriter' has a total cyclomatic complexity of 97 (highest 20). | @suppresswarnings | |
| com/reallifedeveloper/tools/test/database/CrudRepositoryWriter.java | A high number of imports (34) can indicate a high degree of coupling within an object; current threshold is 30. | @suppresswarnings | |
| com/reallifedeveloper/tools/test/database/CrudRepositoryWriter.java | Possible God Class (WMC=97, ATFD=47, TCC=16.374%) | @suppresswarnings | |
| com/reallifedeveloper/tools/test/database/CrudRepositoryWriter.java | This class has too many methods, consider refactoring it. | @suppresswarnings | |
| com/reallifedeveloper/tools/test/database/CrudRepositoryWriter.java | You should not modify visibility of constructors, methods or fields using setAccessible() | @suppresswarnings | |
| com/reallifedeveloper/tools/test/database/CrudRepositoryWriter.java | You should not modify visibility of constructors, methods or fields using setAccessible() | @suppresswarnings | |
| com/reallifedeveloper/tools/test/database/CrudRepositoryWriter.java | You should not modify visibility of constructors, methods or fields using setAccessible() | @suppresswarnings | |
| com/reallifedeveloper/tools/test/database/CrudRepositoryWriter.java | You should not modify visibility of constructors, methods or fields using setAccessible() | @suppresswarnings | |
| com/reallifedeveloper/tools/test/database/CrudRepositoryWriter.java | You should not modify visibility of constructors, methods or fields using setAccessible() | @suppresswarnings | |
| com/reallifedeveloper/tools/test/database/CrudRepositoryWriter.java | You should not modify visibility of constructors, methods or fields using setAccessible() | @suppresswarnings | |
| com/reallifedeveloper/tools/test/database/CrudRepositoryWriter.java | You should not modify visibility of constructors, methods or fields using setAccessible() | @suppresswarnings | |
| com/reallifedeveloper/tools/test/database/JpaUtil.java | Avoid the use of value in annotations when its the only element | @suppresswarnings | |
| com/reallifedeveloper/tools/test/database/JpaUtil.java | Unnecessary qualifier 'JpaUtil': 'getIdField' is already in scope | @suppresswarnings | |
| com/reallifedeveloper/tools/test/database/JpaUtil.java | Unnecessary qualifier 'JpaUtil': 'getField' is already in scope | @suppresswarnings | |
| com/reallifedeveloper/tools/test/database/JpaUtil.java | Consider simply using the value vs. storing it in local variable 'id'. | @suppresswarnings | |
| com/reallifedeveloper/tools/test/database/JpaUtil.java | The method 'addEntitiesToMapField(Field, Object, List<?>)' has a cyclomatic complexity of 10. | @suppresswarnings | |
| com/reallifedeveloper/tools/test/database/JpaUtil.java | Possible God Class (WMC=55, ATFD=43, TCC=1.515%) | @suppresswarnings | |
| com/reallifedeveloper/tools/test/database/JpaUtil.java | This class has too many methods, consider refactoring it. | @suppresswarnings | |
| com/reallifedeveloper/tools/test/database/JpaUtil.java | You should not modify visibility of constructors, methods or fields using setAccessible() | @suppresswarnings | |
| com/reallifedeveloper/tools/test/database/JpaUtil.java | You should not modify visibility of constructors, methods or fields using setAccessible() | @suppresswarnings | |
| com/reallifedeveloper/tools/test/database/csv/CsvDatabaseReader.java | Consider using varargs for methods or constructors which take an array the last parameter. | @suppresswarnings | |
| com/reallifedeveloper/tools/test/database/csv/CsvDatabaseReader.java | Avoid instantiating new objects inside loops | @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/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 | Avoid instantiating new objects inside loops | @suppresswarnings | |
| com/reallifedeveloper/tools/test/database/inmemory/AbstractInMemoryCrudRepository.java | The initializer for variable 'id' is never used (overwritten on lines 319, 321 and 323) | @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 | Consider simply using the value vs. storing it in local variable 'page'. | @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 112 (highest 8). | @suppresswarnings | |
| com/reallifedeveloper/tools/test/database/inmemory/AbstractInMemoryCrudRepository.java | Possible God Class (WMC=112, 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 493 | @suppresswarnings |