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 | This class has too many methods, consider refactoring it. | @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 | Missing commented default access modifier on method 'fromEntity(Object)' | @suppresswarnings | |
| com/reallifedeveloper/tools/test/database/CrudRepositoryWriter.java | Missing commented default access modifier on method 'joinTableField(String)' | @suppresswarnings | |
| com/reallifedeveloper/tools/test/database/CrudRepositoryWriter.java | Missing commented default access modifier on method 'createObjectFromString(String, Class<?>)' | @suppresswarnings | |
| com/reallifedeveloper/tools/test/database/CrudRepositoryWriter.java | Missing commented default access modifier on method 'entities()' | @suppresswarnings | |
| com/reallifedeveloper/tools/test/database/CrudRepositoryWriter.java | Missing commented default access modifier on method 'findEntity(String, Class<?>)' | @suppresswarnings | |
| com/reallifedeveloper/tools/test/database/CrudRepositoryWriter.java | Missing commented default access modifier on method 'addEntity(Object)' | @suppresswarnings | |
| com/reallifedeveloper/tools/test/database/CrudRepositoryWriter.java | This class has only private constructors and may be final | @suppresswarnings | |
| com/reallifedeveloper/tools/test/database/CrudRepositoryWriter.java | The method 'createObjectFromString(String, Class<?>)' has a cognitive complexity of 21, current threshold is 15 | @suppresswarnings | |
| com/reallifedeveloper/tools/test/database/CrudRepositoryWriter.java | A value of 30 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 21. | @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 | A high number of imports (36) can indicate a high degree of coupling within an object; current threshold is 30. | @suppresswarnings | |
| com/reallifedeveloper/tools/test/database/CrudRepositoryWriter.java | Field 'entities' may be declared final | @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/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': 'getField' is already in scope | @suppresswarnings | |
| com/reallifedeveloper/tools/test/database/JpaUtil.java | Unnecessary qualifier 'JpaUtil': 'getIdField' 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 | |
| com/reallifedeveloper/tools/test/database/inmemory/InMemoryJpaRepository.java | A value of 26 may denote a high amount of coupling within the class (threshold: 20) | @suppresswarnings | |
| com/reallifedeveloper/tools/test/database/inmemory/InMemoryJpaRepository.java | This class has too many methods, consider refactoring it. | @suppresswarnings | |
| com/reallifedeveloper/tools/test/database/inmemory/PredicateSpecificationEvaluator.java | Annotate this interface with @FunctionalInterface or with @SuppressWarnings("PMD.ImplicitFunctionalInterface") to clarify your intent. | @suppresswarnings | |
| com/reallifedeveloper/tools/test/database/inmemory/PredicateSpecificationEvaluator.java | Annotate this interface with @FunctionalInterface or with @SuppressWarnings("PMD.ImplicitFunctionalInterface") to clarify your intent. | @suppresswarnings | |
| com/reallifedeveloper/tools/test/database/inmemory/PredicateSpecificationEvaluator.java | Position literals first in String comparisons | @suppresswarnings | |
| com/reallifedeveloper/tools/test/database/inmemory/PredicateSpecificationEvaluator.java | Position literals first in String comparisons | @suppresswarnings | |
| com/reallifedeveloper/tools/test/database/inmemory/PredicateSpecificationEvaluator.java | Position literals first in String comparisons | @suppresswarnings | |
| com/reallifedeveloper/tools/test/database/inmemory/PredicateSpecificationEvaluator.java | Position literals first in String comparisons | @suppresswarnings | |
| com/reallifedeveloper/tools/test/database/inmemory/PredicateSpecificationEvaluator.java | Position literals first in String comparisons | @suppresswarnings | |
| com/reallifedeveloper/tools/test/database/inmemory/PredicateSpecificationEvaluator.java | Position literals first in String comparisons | @suppresswarnings | |
| com/reallifedeveloper/tools/test/database/inmemory/PredicateSpecificationEvaluator.java | Position literals first in String comparisons | @suppresswarnings | |
| com/reallifedeveloper/tools/test/database/inmemory/PredicateSpecificationEvaluator.java | Position literals first in String comparisons | @suppresswarnings | |
| com/reallifedeveloper/tools/test/database/inmemory/PredicateSpecificationEvaluator.java | Position literals first in String comparisons | @suppresswarnings | |
| com/reallifedeveloper/tools/test/database/inmemory/PredicateSpecificationEvaluator.java | Position literals first in String comparisons | @suppresswarnings | |
| com/reallifedeveloper/tools/test/database/inmemory/PredicateSpecificationEvaluator.java | Position literals first in String comparisons | @suppresswarnings | |
| com/reallifedeveloper/tools/test/database/inmemory/PredicateSpecificationEvaluator.java | Position literals first in String comparisons | @suppresswarnings | |
| com/reallifedeveloper/tools/test/database/inmemory/PredicateSpecificationEvaluator.java | Position literals first in String comparisons | @suppresswarnings | |
| com/reallifedeveloper/tools/test/database/inmemory/PredicateSpecificationEvaluator.java | Position literals first in String comparisons | @suppresswarnings | |
| com/reallifedeveloper/tools/test/database/inmemory/PredicateSpecificationEvaluator.java | Position literals first in String comparisons | @suppresswarnings | |
| com/reallifedeveloper/tools/test/database/inmemory/PredicateSpecificationEvaluator.java | Consider using varargs for methods or constructors which take an array the last parameter. | @suppresswarnings | |
| com/reallifedeveloper/tools/test/database/inmemory/PredicateSpecificationEvaluator.java | Consider using varargs for methods or constructors which take an array the last parameter. | @suppresswarnings | |
| com/reallifedeveloper/tools/test/database/inmemory/PredicateSpecificationEvaluator.java | Consider using varargs for methods or constructors which take an array the last parameter. | @suppresswarnings | |
| com/reallifedeveloper/tools/test/database/inmemory/PredicateSpecificationEvaluator.java | Consider using varargs for methods or constructors which take an array the last parameter. | @suppresswarnings | |
| com/reallifedeveloper/tools/test/database/inmemory/PredicateSpecificationEvaluator.java | Avoid unnecessary constructors - the compiler will generate these for you | @suppresswarnings | |
| com/reallifedeveloper/tools/test/database/inmemory/PredicateSpecificationEvaluator.java | Missing commented default access modifier on method 'path(PathExpression)' | @suppresswarnings | |
| com/reallifedeveloper/tools/test/database/inmemory/PredicateSpecificationEvaluator.java | Missing commented default access modifier on method 'root(RecordingState)' | @suppresswarnings | |
| com/reallifedeveloper/tools/test/database/inmemory/PredicateSpecificationEvaluator.java | Missing commented default access modifier on method 'mapJoin(RecordingState, JoinSource)' | @suppresswarnings | |
| com/reallifedeveloper/tools/test/database/inmemory/PredicateSpecificationEvaluator.java | Missing commented default access modifier on method 'in(Expr)' | @suppresswarnings | |
| com/reallifedeveloper/tools/test/database/inmemory/PredicateSpecificationEvaluator.java | Missing commented default access modifier on method 'addMapJoin(Source, String, JoinType)' | @suppresswarnings | |
| com/reallifedeveloper/tools/test/database/inmemory/PredicateSpecificationEvaluator.java | Missing commented default access modifier on method 'addJoin(Source, String, JoinType)' | @suppresswarnings | |
| com/reallifedeveloper/tools/test/database/inmemory/PredicateSpecificationEvaluator.java | Missing commented default access modifier on method 'join(RecordingState, JoinSource)' | @suppresswarnings | |
| com/reallifedeveloper/tools/test/database/inmemory/PredicateSpecificationEvaluator.java | Missing commented default access modifier on method 'criteriaBuilder()' | @suppresswarnings | |
| com/reallifedeveloper/tools/test/database/inmemory/PredicateSpecificationEvaluator.java | Missing commented default access modifier on method 'source(Source)' | @suppresswarnings | |
| com/reallifedeveloper/tools/test/database/inmemory/PredicateSpecificationEvaluator.java | Missing commented default access modifier on method 'expression(Expr)' | @suppresswarnings | |
| com/reallifedeveloper/tools/test/database/inmemory/PredicateSpecificationEvaluator.java | Missing commented default access modifier on method 'read(Object, String)' | @suppresswarnings | |
| com/reallifedeveloper/tools/test/database/inmemory/PredicateSpecificationEvaluator.java | Missing commented default access modifier on method 'expressionOf(Object)' | @suppresswarnings | |
| com/reallifedeveloper/tools/test/database/inmemory/PredicateSpecificationEvaluator.java | Missing commented default access modifier on method 'predicate(Expr)' | @suppresswarnings | |
| com/reallifedeveloper/tools/test/database/inmemory/PredicateSpecificationEvaluator.java | Missing commented default access modifier on method 'bind(int, Object)' | @suppresswarnings | |
| com/reallifedeveloper/tools/test/database/inmemory/PredicateSpecificationEvaluator.java | The method 'invoke(Object, Method, Object[])' has a cognitive complexity of 17, current threshold is 15 | @suppresswarnings | |
| com/reallifedeveloper/tools/test/database/inmemory/PredicateSpecificationEvaluator.java | The method 'expandJoins(RecordingState, Object)' has a cognitive complexity of 16, current threshold is 15 | @suppresswarnings | |
| com/reallifedeveloper/tools/test/database/inmemory/PredicateSpecificationEvaluator.java | A value of 49 may denote a high amount of coupling within the class (threshold: 20) | @suppresswarnings | |
| com/reallifedeveloper/tools/test/database/inmemory/PredicateSpecificationEvaluator.java | The method 'invoke(Object, Method, Object[])' has a cyclomatic complexity of 14. | @suppresswarnings | |
| com/reallifedeveloper/tools/test/database/inmemory/PredicateSpecificationEvaluator.java | The method 'invoke(Object, Method, Object[])' has a cyclomatic complexity of 20. | @suppresswarnings | |
| com/reallifedeveloper/tools/test/database/inmemory/PredicateSpecificationEvaluator.java | The method 'wrap(Class<?>)' has a cyclomatic complexity of 12. | @suppresswarnings | |
| com/reallifedeveloper/tools/test/database/inmemory/PredicateSpecificationEvaluator.java | The method 'invoke(Object, Method, Object[])' has a cyclomatic complexity of 20. | @suppresswarnings | |
| com/reallifedeveloper/tools/test/database/inmemory/PredicateSpecificationEvaluator.java | The class 'PredicateSpecificationEvaluator' has a total cyclomatic complexity of 135 (highest 12). | @suppresswarnings | |
| com/reallifedeveloper/tools/test/database/inmemory/PredicateSpecificationEvaluator.java | A high number of imports (32) can indicate a high degree of coupling within an object; current threshold is 30. | @suppresswarnings | |
| com/reallifedeveloper/tools/test/database/inmemory/PredicateSpecificationEvaluator.java | Possible God Class (WMC=135, ATFD=73, TCC=5.492%) | @suppresswarnings | |
| com/reallifedeveloper/tools/test/database/inmemory/PredicateSpecificationEvaluator.java | The method 'invoke(Object, Method, Object[])' has an NPath complexity of 256, current threshold is 200 | @suppresswarnings | |
| com/reallifedeveloper/tools/test/database/inmemory/PredicateSpecificationEvaluator.java | This class has too many methods, consider refactoring it. | @suppresswarnings | |
| com/reallifedeveloper/tools/test/database/inmemory/PredicateSpecificationEvaluator.java | The String literal "unchecked" appears 7 times in this file; the first occurrence is on line 650 | @suppresswarnings | |
| com/reallifedeveloper/tools/test/database/inmemory/PredicateSpecificationEvaluator.java | In J2EE, getClassLoader() might not work as expected. Use Thread.currentThread().getContextClassLoader() instead. | @suppresswarnings | |
| com/reallifedeveloper/tools/test/database/inmemory/PredicateSpecificationEvaluator.java | In J2EE, getClassLoader() might not work as expected. Use Thread.currentThread().getContextClassLoader() instead. | @suppresswarnings | |
| com/reallifedeveloper/tools/test/database/inmemory/PredicateSpecificationEvaluator.java | In J2EE, getClassLoader() might not work as expected. Use Thread.currentThread().getContextClassLoader() instead. | @suppresswarnings | |
| com/reallifedeveloper/tools/test/database/inmemory/PredicateSpecificationEvaluator.java | In J2EE, getClassLoader() might not work as expected. Use Thread.currentThread().getContextClassLoader() instead. | @suppresswarnings | |
| com/reallifedeveloper/tools/test/database/inmemory/PredicateSpecificationEvaluator.java | In J2EE, getClassLoader() might not work as expected. Use Thread.currentThread().getContextClassLoader() instead. | @suppresswarnings | |
| com/reallifedeveloper/tools/test/database/inmemory/PredicateSpecificationEvaluator.java | In J2EE, getClassLoader() might not work as expected. Use Thread.currentThread().getContextClassLoader() instead. | @suppresswarnings | |
| com/reallifedeveloper/tools/test/database/inmemory/PredicateSpecificationEvaluator.java | In J2EE, getClassLoader() might not work as expected. Use Thread.currentThread().getContextClassLoader() instead. | @suppresswarnings | |
| com/reallifedeveloper/tools/test/database/inmemory/PredicateSpecificationEvaluator.java | In J2EE, getClassLoader() might not work as expected. Use Thread.currentThread().getContextClassLoader() instead. | @suppresswarnings | |
| com/reallifedeveloper/tools/test/database/inmemory/PredicateSpecificationEvaluator.java | Avoid assignment to nextJoinId in operand | @suppresswarnings | |
| com/reallifedeveloper/tools/test/database/inmemory/PredicateSpecificationEvaluator.java | Avoid instantiating new objects inside loops | @suppresswarnings |