PMD Results
The following document contains the results of PMD 6.29.0.
Violations By Priority
Priority 3
org/apache/fulcrum/quartz/impl/QuartzSchedulerImpl.java
| Rule | Violation | Line |
|---|---|---|
| ExcessiveImports | A high number of imports can indicate a high degree of coupling within an object. | 1–304 |
| AtLeastOneConstructor | Each class should declare at least one constructor | 58–304 |
| LongVariable | Avoid excessively long variable names like CONFIG_CONFIGURATION | 63 |
| LongVariable | Avoid excessively long variable names like CONFIG_PROPERTY_FILE | 66 |
| BeanMembersShouldSerialize | Found non-transient, non-static member. Please mark as transient or provide accessors. | 74 |
| BeanMembersShouldSerialize | Found non-transient, non-static member. Please mark as transient or provide accessors. | 79 |
| BeanMembersShouldSerialize | Found non-transient, non-static member. Please mark as transient or provide accessors. | 84 |
| LongVariable | Avoid excessively long variable names like quartzPropertyFile | 84 |
| BeanMembersShouldSerialize | Found non-transient, non-static member. Please mark as transient or provide accessors. | 89 |
| MethodArgumentCouldBeFinal | Parameter 'conf' is not assigned and could be declared final | 97 |
| LocalVariableCouldBeFinal | Local variable 'quartzConf' could be declared final | 99 |
| LawOfDemeter | Potential violation of Law of Demeter (object not created locally) | 101 |
| ConfusingTernary | Avoid if (x != y) ..; else ..; | 101–108 |
| LawOfDemeter | Potential violation of Law of Demeter (object not created locally) | 105 |
| LawOfDemeter | Potential violation of Law of Demeter (method chain calls) | 107 |
| MethodArgumentCouldBeFinal | Parameter 'manager' is not assigned and could be declared final | 115 |
| LocalVariableCouldBeFinal | Local variable 'schedulerFactory' could be declared final | 127 |
| ConfusingTernary | Avoid if (x != y) ..; else ..; | 128–142 |
| LawOfDemeter | Potential violation of Law of Demeter (method chain calls) | 130 |
| ConfusingTernary | Avoid if (x != y) ..; else ..; | 133–142 |
| LawOfDemeter | Potential violation of Law of Demeter (method chain calls) | 135 |
| LawOfDemeter | Potential violation of Law of Demeter (method chain calls) | 140 |
| LawOfDemeter | Potential violation of Law of Demeter (method chain calls) | 153 |
| LawOfDemeter | Potential violation of Law of Demeter (method chain calls) | 155 |
| LawOfDemeter | Potential violation of Law of Demeter (method chain calls) | 165 |
| NullAssignment | Assigning an Object to null is a code smell. Consider refactoring. | 184 |
| NullAssignment | Assigning an Object to null is a code smell. Consider refactoring. | 185 |
| LawOfDemeter | Potential violation of Law of Demeter (method chain calls) | 207 |
| MethodArgumentCouldBeFinal | Parameter 'context' is not assigned and could be declared final | 217 |
| LocalVariableCouldBeFinal | Local variable 'job' could be declared final | 219 |
| LawOfDemeter | Potential violation of Law of Demeter (method chain calls) | 236 |
| ShortVariable | Avoid variables with short names like ex | 245 |
| MethodArgumentCouldBeFinal | Parameter 'ex' is not assigned and could be declared final | 245 |
| MethodArgumentCouldBeFinal | Parameter 'context' is not assigned and could be declared final | 245 |
| ConfusingTernary | Avoid if (x != y) ..; else ..; | 247–258 |
| LawOfDemeter | Potential violation of Law of Demeter (method chain calls) | 249 |
| LocalVariableCouldBeFinal | Local variable 'msg' could be declared final | 249 |
| LawOfDemeter | Potential violation of Law of Demeter (method chain calls) | 250 |
| LawOfDemeter | Potential violation of Law of Demeter (method chain calls) | 254 |
| LawOfDemeter | Potential violation of Law of Demeter (method chain calls) | 256 |
| LawOfDemeter | Potential violation of Law of Demeter (method chain calls) | 256 |
| MethodArgumentCouldBeFinal | Parameter 'context' is not assigned and could be declared final | 265 |
| LawOfDemeter | Potential violation of Law of Demeter (method chain calls) | 276 |
| LocalVariableCouldBeFinal | Local variable 'jobGroup' could be declared final | 276 |
| LawOfDemeter | Potential violation of Law of Demeter (method chain calls) | 278 |
| LocalVariableCouldBeFinal | Local variable 'jobsInGroup' could be declared final | 278 |
| LawOfDemeter | Potential violation of Law of Demeter (object not created locally) | 279 |
| LawOfDemeter | Potential violation of Law of Demeter (method chain calls) | 279 |
| LocalVariableCouldBeFinal | Local variable 'jobKey' could be declared final | 280 |
| AvoidInstantiatingObjectsInLoops | Avoid instantiating new objects inside loops | 282 |
| LocalVariableCouldBeFinal | Local variable 'buffer' could be declared final | 282 |
| LocalVariableCouldBeFinal | Local variable 'jobDetail' could be declared final | 283 |
| LawOfDemeter | Potential violation of Law of Demeter (method chain calls) | 283 |
| LawOfDemeter | Potential violation of Law of Demeter (method chain calls) | 284 |
| LocalVariableCouldBeFinal | Local variable 'jobTriggers' could be declared final | 284 |
| ConsecutiveAppendsShouldReuse | StringBuffer (or StringBuilder).append is called consecutively without reusing the target variable. | 285 |
| LawOfDemeter | Potential violation of Law of Demeter (object not created locally) | 285 |
| LawOfDemeter | Potential violation of Law of Demeter (object not created locally) | 287 |
| ConfusingTernary | Avoid if (x != y) ..; else ..; | 287–298 |
| ShortVariable | Avoid variables with short names like jt | 289 |
| LocalVariableCouldBeFinal | Local variable 'jt' could be declared final | 289 |
| LawOfDemeter | Potential violation of Law of Demeter (object not created locally) | 289 |
| LawOfDemeter | Potential violation of Law of Demeter (object not created locally) | 290 |
| LawOfDemeter | Potential violation of Law of Demeter (object not created locally) | 292 |
| LawOfDemeter | Potential violation of Law of Demeter (method chain calls) | 300 |
Files
org/apache/fulcrum/quartz/impl/QuartzSchedulerImpl.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| ExcessiveImports | A high number of imports can indicate a high degree of coupling within an object. | 3 | 1–304 |
| AtLeastOneConstructor | Each class should declare at least one constructor | 3 | 58–304 |
| LongVariable | Avoid excessively long variable names like CONFIG_CONFIGURATION | 3 | 63 |
| LongVariable | Avoid excessively long variable names like CONFIG_PROPERTY_FILE | 3 | 66 |
| BeanMembersShouldSerialize | Found non-transient, non-static member. Please mark as transient or provide accessors. | 3 | 74 |
| BeanMembersShouldSerialize | Found non-transient, non-static member. Please mark as transient or provide accessors. | 3 | 79 |
| BeanMembersShouldSerialize | Found non-transient, non-static member. Please mark as transient or provide accessors. | 3 | 84 |
| LongVariable | Avoid excessively long variable names like quartzPropertyFile | 3 | 84 |
| BeanMembersShouldSerialize | Found non-transient, non-static member. Please mark as transient or provide accessors. | 3 | 89 |
| MethodArgumentCouldBeFinal | Parameter 'conf' is not assigned and could be declared final | 3 | 97 |
| LocalVariableCouldBeFinal | Local variable 'quartzConf' could be declared final | 3 | 99 |
| LawOfDemeter | Potential violation of Law of Demeter (object not created locally) | 3 | 101 |
| ConfusingTernary | Avoid if (x != y) ..; else ..; | 3 | 101–108 |
| LawOfDemeter | Potential violation of Law of Demeter (object not created locally) | 3 | 105 |
| LawOfDemeter | Potential violation of Law of Demeter (method chain calls) | 3 | 107 |
| MethodArgumentCouldBeFinal | Parameter 'manager' is not assigned and could be declared final | 3 | 115 |
| LocalVariableCouldBeFinal | Local variable 'schedulerFactory' could be declared final | 3 | 127 |
| ConfusingTernary | Avoid if (x != y) ..; else ..; | 3 | 128–142 |
| LawOfDemeter | Potential violation of Law of Demeter (method chain calls) | 3 | 130 |
| ConfusingTernary | Avoid if (x != y) ..; else ..; | 3 | 133–142 |
| LawOfDemeter | Potential violation of Law of Demeter (method chain calls) | 3 | 135 |
| LawOfDemeter | Potential violation of Law of Demeter (method chain calls) | 3 | 140 |
| LawOfDemeter | Potential violation of Law of Demeter (method chain calls) | 3 | 153 |
| LawOfDemeter | Potential violation of Law of Demeter (method chain calls) | 3 | 155 |
| LawOfDemeter | Potential violation of Law of Demeter (method chain calls) | 3 | 165 |
| NullAssignment | Assigning an Object to null is a code smell. Consider refactoring. | 3 | 184 |
| NullAssignment | Assigning an Object to null is a code smell. Consider refactoring. | 3 | 185 |
| LawOfDemeter | Potential violation of Law of Demeter (method chain calls) | 3 | 207 |
| MethodArgumentCouldBeFinal | Parameter 'context' is not assigned and could be declared final | 3 | 217 |
| LocalVariableCouldBeFinal | Local variable 'job' could be declared final | 3 | 219 |
| LawOfDemeter | Potential violation of Law of Demeter (method chain calls) | 3 | 236 |
| ShortVariable | Avoid variables with short names like ex | 3 | 245 |
| MethodArgumentCouldBeFinal | Parameter 'ex' is not assigned and could be declared final | 3 | 245 |
| MethodArgumentCouldBeFinal | Parameter 'context' is not assigned and could be declared final | 3 | 245 |
| ConfusingTernary | Avoid if (x != y) ..; else ..; | 3 | 247–258 |
| LawOfDemeter | Potential violation of Law of Demeter (method chain calls) | 3 | 249 |
| LocalVariableCouldBeFinal | Local variable 'msg' could be declared final | 3 | 249 |
| LawOfDemeter | Potential violation of Law of Demeter (method chain calls) | 3 | 250 |
| LawOfDemeter | Potential violation of Law of Demeter (method chain calls) | 3 | 254 |
| LawOfDemeter | Potential violation of Law of Demeter (method chain calls) | 3 | 256 |
| LawOfDemeter | Potential violation of Law of Demeter (method chain calls) | 3 | 256 |
| MethodArgumentCouldBeFinal | Parameter 'context' is not assigned and could be declared final | 3 | 265 |
| LawOfDemeter | Potential violation of Law of Demeter (method chain calls) | 3 | 276 |
| LocalVariableCouldBeFinal | Local variable 'jobGroup' could be declared final | 3 | 276 |
| LawOfDemeter | Potential violation of Law of Demeter (method chain calls) | 3 | 278 |
| LocalVariableCouldBeFinal | Local variable 'jobsInGroup' could be declared final | 3 | 278 |
| LawOfDemeter | Potential violation of Law of Demeter (object not created locally) | 3 | 279 |
| LawOfDemeter | Potential violation of Law of Demeter (method chain calls) | 3 | 279 |
| LocalVariableCouldBeFinal | Local variable 'jobKey' could be declared final | 3 | 280 |
| AvoidInstantiatingObjectsInLoops | Avoid instantiating new objects inside loops | 3 | 282 |
| LocalVariableCouldBeFinal | Local variable 'buffer' could be declared final | 3 | 282 |
| LocalVariableCouldBeFinal | Local variable 'jobDetail' could be declared final | 3 | 283 |
| LawOfDemeter | Potential violation of Law of Demeter (method chain calls) | 3 | 283 |
| LawOfDemeter | Potential violation of Law of Demeter (method chain calls) | 3 | 284 |
| LocalVariableCouldBeFinal | Local variable 'jobTriggers' could be declared final | 3 | 284 |
| ConsecutiveAppendsShouldReuse | StringBuffer (or StringBuilder).append is called consecutively without reusing the target variable. | 3 | 285 |
| LawOfDemeter | Potential violation of Law of Demeter (object not created locally) | 3 | 285 |
| LawOfDemeter | Potential violation of Law of Demeter (object not created locally) | 3 | 287 |
| ConfusingTernary | Avoid if (x != y) ..; else ..; | 3 | 287–298 |
| ShortVariable | Avoid variables with short names like jt | 3 | 289 |
| LocalVariableCouldBeFinal | Local variable 'jt' could be declared final | 3 | 289 |
| LawOfDemeter | Potential violation of Law of Demeter (object not created locally) | 3 | 289 |
| LawOfDemeter | Potential violation of Law of Demeter (object not created locally) | 3 | 290 |
| LawOfDemeter | Potential violation of Law of Demeter (object not created locally) | 3 | 292 |
| LawOfDemeter | Potential violation of Law of Demeter (method chain calls) | 3 | 300 |