PMD Results

Last Published: Fri Mar 10 13:29:50 EST 2006

SQLElements

Project Documentation

Built by Maven

PMD Results

The following document contains the results of PMD

Files

org/israfil/sqlelements/AbstractAliasedSQLElement.java

Violation	Line
Assigning an Object to null is a code smell. Consider refactoring.	60
It is a good practice to call super() in a constructor	52
It is a good practice to call super() in a constructor	55

org/israfil/sqlelements/AbstractJoin.java

Violation	Line
It is a good practice to call super() in a constructor	48

org/israfil/sqlelements/AbstractParameterizedCommand.java

Violation	Line
A method should have only one exit point, and that should be the last statement in the method	77

org/israfil/sqlelements/AbstractStatement.java

Violation	Line
It is a good practice to call super() in a constructor	58
It is a good practice to call super() in a constructor	61

org/israfil/sqlelements/AliasedSQLElement.java

Violation	Line
Each class should declare at least one constructor	38
Avoid modifiers which are implied by the context	39
Avoid modifiers which are implied by the context	41

org/israfil/sqlelements/Column.java

Violation	Line
Each class should declare at least one constructor	39
Avoid modifiers which are implied by the context	41
Avoid modifiers which are implied by the context	43

org/israfil/sqlelements/Command.java

Violation	Line
Each class should declare at least one constructor	38
Avoid modifiers which are implied by the context	40

org/israfil/sqlelements/constraints/ArbitraryStringConstraint.java

Violation	Line
It is a good practice to call super() in a constructor	49

org/israfil/sqlelements/constraints/BetweenRange.java

Violation	Line
It is a good practice to call super() in a constructor	47

org/israfil/sqlelements/constraints/BinaryConstraint.java

Violation	Line
It is a good practice to call super() in a constructor	46

org/israfil/sqlelements/constraints/ColumnConstraint.java

Violation	Line
It is a good practice to call super() in a constructor	52
A method should have only one exit point, and that should be the last statement in the method	61

org/israfil/sqlelements/constraints/ComplexConstraint.java

Violation	Line
Each class should declare at least one constructor	39

org/israfil/sqlelements/constraints/Constraint.java

Violation	Line
Each class should declare at least one constructor	40

org/israfil/sqlelements/constraints/ConstraintUtils.java

Violation	Line
Each class should declare at least one constructor	40
A method should have only one exit point, and that should be the last statement in the method	44

org/israfil/sqlelements/constraints/JoinConstraint.java

Violation	Line
It is a good practice to call super() in a constructor	45

org/israfil/sqlelements/constraints/NaryConstraint.java

Violation	Line
It is a good practice to call super() in a constructor	63

org/israfil/sqlelements/constraints/Not.java

Violation	Line
It is a good practice to call super() in a constructor	45

org/israfil/sqlelements/constraints/SimpleConstraint.java

Violation	Line
Each class should declare at least one constructor	39

org/israfil/sqlelements/Join.java

Violation	Line
Each class should declare at least one constructor	40
Avoid modifiers which are implied by the context	42
Avoid modifiers which are implied by the context	43
Avoid modifiers which are implied by the context	44

org/israfil/sqlelements/LiteralColumn.java

Violation	Line
A method should have only one exit point, and that should be the last statement in the method	66
A method should have only one exit point, and that should be the last statement in the method	67
A method should have only one exit point, and that should be the last statement in the method	68

org/israfil/sqlelements/ParameterizedCommand.java

Violation	Line
Each class should declare at least one constructor	39

org/israfil/sqlelements/Query.java

Violation	Line
Each class should declare at least one constructor	43
Avoid modifiers which are implied by the context	45
Avoid modifiers which are implied by the context	47
Avoid modifiers which are implied by the context	49
Avoid modifiers which are implied by the context	51
Avoid modifiers which are implied by the context	53

org/israfil/sqlelements/render/CommandRenderer.java

Violation	Line
Each class should declare at least one constructor	40
Avoid modifiers which are implied by the context	41

org/israfil/sqlelements/render/DatabaseType.java

Violation	Line
It is a good practice to call super() in a constructor	100

org/israfil/sqlelements/render/DateRenderer.java

Violation	Line
Each class should declare at least one constructor	40
Avoid modifiers which are implied by the context	41

org/israfil/sqlelements/render/DelegatedSQLRenderEngine.java

Violation	Line
Each class should declare at least one constructor	47

org/israfil/sqlelements/render/GenericSQLRenderEngine.java

Violation	Line
Each class should declare at least one constructor	48

org/israfil/sqlelements/render/MockSQLRenderEngine.java

Violation	Line
Each class should declare at least one constructor	43

org/israfil/sqlelements/render/MySqlSQLRenderEngine.java

Violation	Line
Each class should declare at least one constructor	38

org/israfil/sqlelements/render/OracleSQLRenderEngine.java

Violation	Line
Each class should declare at least one constructor	48

org/israfil/sqlelements/render/QueryRenderer.java

Violation	Line
Each class should declare at least one constructor	41
Avoid modifiers which are implied by the context	42

org/israfil/sqlelements/render/SQLRenderContext.java

Violation	Line
Assigning an Object to null is a code smell. Consider refactoring.	63
It is a good practice to call super() in a constructor	58
A method should have only one exit point, and that should be the last statement in the method	68
A method should have only one exit point, and that should be the last statement in the method	76

org/israfil/sqlelements/SelectTable.java

Violation	Line
A method should have only one exit point, and that should be the last statement in the method	78

org/israfil/sqlelements/SimpleTable.java

Violation	Line
A method should have only one exit point, and that should be the last statement in the method	61

org/israfil/sqlelements/SQLElement.java

Violation	Line
Each class should declare at least one constructor	38
Avoid modifiers which are implied by the context	39

org/israfil/sqlelements/SQLParameter.java

Violation	Line
Assigning an Object to null is a code smell. Consider refactoring.	46
It is a good practice to call super() in a constructor	43
It is a good practice to call super() in a constructor	49

org/israfil/sqlelements/Statement.java

Violation	Line
Each class should declare at least one constructor	41
Avoid modifiers which are implied by the context	43
Avoid modifiers which are implied by the context	45
Avoid modifiers which are implied by the context	47
Avoid modifiers which are implied by the context	49

org/israfil/sqlelements/Table.java

Violation	Line
Each class should declare at least one constructor	39
Avoid modifiers which are implied by the context	40