PMD Results

The following document contains the results of PMD

Files

org/israfil/sqlelements/AbstractAliasedSQLElement.java

ViolationLine
Assigning an Object to null is a code smell. Consider refactoring.60
It is a good practice to call super() in a constructor52
It is a good practice to call super() in a constructor55

org/israfil/sqlelements/AbstractJoin.java

ViolationLine
It is a good practice to call super() in a constructor48

org/israfil/sqlelements/AbstractParameterizedCommand.java

ViolationLine
A method should have only one exit point, and that should be the last statement in the method77

org/israfil/sqlelements/AbstractStatement.java

ViolationLine
It is a good practice to call super() in a constructor58
It is a good practice to call super() in a constructor61

org/israfil/sqlelements/AliasedSQLElement.java

ViolationLine
Each class should declare at least one constructor38
Avoid modifiers which are implied by the context39
Avoid modifiers which are implied by the context41

org/israfil/sqlelements/Column.java

ViolationLine
Each class should declare at least one constructor39
Avoid modifiers which are implied by the context41
Avoid modifiers which are implied by the context43

org/israfil/sqlelements/Command.java

ViolationLine
Each class should declare at least one constructor38
Avoid modifiers which are implied by the context40

org/israfil/sqlelements/constraints/ArbitraryStringConstraint.java

ViolationLine
It is a good practice to call super() in a constructor49

org/israfil/sqlelements/constraints/BetweenRange.java

ViolationLine
It is a good practice to call super() in a constructor47

org/israfil/sqlelements/constraints/BinaryConstraint.java

ViolationLine
It is a good practice to call super() in a constructor46

org/israfil/sqlelements/constraints/ColumnConstraint.java

ViolationLine
It is a good practice to call super() in a constructor52
A method should have only one exit point, and that should be the last statement in the method61

org/israfil/sqlelements/constraints/ComplexConstraint.java

ViolationLine
Each class should declare at least one constructor39

org/israfil/sqlelements/constraints/Constraint.java

ViolationLine
Each class should declare at least one constructor40

org/israfil/sqlelements/constraints/ConstraintUtils.java

ViolationLine
Each class should declare at least one constructor40
A method should have only one exit point, and that should be the last statement in the method44

org/israfil/sqlelements/constraints/JoinConstraint.java

ViolationLine
It is a good practice to call super() in a constructor45

org/israfil/sqlelements/constraints/NaryConstraint.java

ViolationLine
It is a good practice to call super() in a constructor63

org/israfil/sqlelements/constraints/Not.java

ViolationLine
It is a good practice to call super() in a constructor45

org/israfil/sqlelements/constraints/SimpleConstraint.java

ViolationLine
Each class should declare at least one constructor39

org/israfil/sqlelements/Join.java

ViolationLine
Each class should declare at least one constructor40
Avoid modifiers which are implied by the context42
Avoid modifiers which are implied by the context43
Avoid modifiers which are implied by the context44

org/israfil/sqlelements/LiteralColumn.java

ViolationLine
A method should have only one exit point, and that should be the last statement in the method66
A method should have only one exit point, and that should be the last statement in the method67
A method should have only one exit point, and that should be the last statement in the method68

org/israfil/sqlelements/ParameterizedCommand.java

ViolationLine
Each class should declare at least one constructor39

org/israfil/sqlelements/Query.java

ViolationLine
Each class should declare at least one constructor43
Avoid modifiers which are implied by the context45
Avoid modifiers which are implied by the context47
Avoid modifiers which are implied by the context49
Avoid modifiers which are implied by the context51
Avoid modifiers which are implied by the context53

org/israfil/sqlelements/render/CommandRenderer.java

ViolationLine
Each class should declare at least one constructor40
Avoid modifiers which are implied by the context41

org/israfil/sqlelements/render/DatabaseType.java

ViolationLine
It is a good practice to call super() in a constructor100

org/israfil/sqlelements/render/DateRenderer.java

ViolationLine
Each class should declare at least one constructor40
Avoid modifiers which are implied by the context41

org/israfil/sqlelements/render/DelegatedSQLRenderEngine.java

ViolationLine
Each class should declare at least one constructor47

org/israfil/sqlelements/render/GenericSQLRenderEngine.java

ViolationLine
Each class should declare at least one constructor48

org/israfil/sqlelements/render/MockSQLRenderEngine.java

ViolationLine
Each class should declare at least one constructor43

org/israfil/sqlelements/render/MySqlSQLRenderEngine.java

ViolationLine
Each class should declare at least one constructor38

org/israfil/sqlelements/render/OracleSQLRenderEngine.java

ViolationLine
Each class should declare at least one constructor48

org/israfil/sqlelements/render/QueryRenderer.java

ViolationLine
Each class should declare at least one constructor41
Avoid modifiers which are implied by the context42

org/israfil/sqlelements/render/SQLRenderContext.java

ViolationLine
Assigning an Object to null is a code smell. Consider refactoring.63
It is a good practice to call super() in a constructor58
A method should have only one exit point, and that should be the last statement in the method68
A method should have only one exit point, and that should be the last statement in the method76

org/israfil/sqlelements/SelectTable.java

ViolationLine
A method should have only one exit point, and that should be the last statement in the method78

org/israfil/sqlelements/SimpleTable.java

ViolationLine
A method should have only one exit point, and that should be the last statement in the method61

org/israfil/sqlelements/SQLElement.java

ViolationLine
Each class should declare at least one constructor38
Avoid modifiers which are implied by the context39

org/israfil/sqlelements/SQLParameter.java

ViolationLine
Assigning an Object to null is a code smell. Consider refactoring.46
It is a good practice to call super() in a constructor43
It is a good practice to call super() in a constructor49

org/israfil/sqlelements/Statement.java

ViolationLine
Each class should declare at least one constructor41
Avoid modifiers which are implied by the context43
Avoid modifiers which are implied by the context45
Avoid modifiers which are implied by the context47
Avoid modifiers which are implied by the context49

org/israfil/sqlelements/Table.java

ViolationLine
Each class should declare at least one constructor39
Avoid modifiers which are implied by the context40