PMD 4.2.5 Report [module - data]
2009-11-18 - 21:33:06

Summary

Files Total Priority 1 Priority 2 Priority 3 Priority 4 Priority 5
23 83
1
4
78
0
0
Prio File Line Description
3
 org.openscience.cdk.Atom 63-416 The class 'Atom' has a Cyclomatic Complexity of 2 (Highest = 12).
3
 org.openscience.cdk.Atom 342-359 The method 'compare' has a Cyclomatic Complexity of 12.
3
 org.openscience.cdk.AtomContainer 24-1734 This class has a bunch of public methods and attributes
3
 org.openscience.cdk.AtomContainer 62-1732 Avoid really long classes.
3
 org.openscience.cdk.AtomContainer 62-1732 The class 'AtomContainer' has a Cyclomatic Complexity of 3 (Highest = 11).
3
 org.openscience.cdk.AtomContainer 63-1732 This class has too many methods, consider refactoring it.
3
 org.openscience.cdk.AtomContainer 104 It is somewhat confusing to have a field name with the same name as a method
3
 org.openscience.cdk.AtomContainer 109 It is somewhat confusing to have a field name with the same name as a method
3
 org.openscience.cdk.AtomContainer 114 It is somewhat confusing to have a field name with the same name as a method
3
 org.openscience.cdk.AtomContainer 119 It is somewhat confusing to have a field name with the same name as a method
3
 org.openscience.cdk.AtomContainer 1221 Avoid variables with short names like lp
3
 org.openscience.cdk.AtomContainer 1251 Avoid variables with short names like se
3
 org.openscience.cdk.AtomContainer 1542-1580 The method 'toString' has a Cyclomatic Complexity of 11.
3
 org.openscience.cdk.AtomContainer 1542-1580 The method toString() has an NPath complexity of 243
3
 org.openscience.cdk.AtomContainer 1611 Avoid variables with short names like lp
3
 org.openscience.cdk.AtomContainer 1619 Avoid variables with short names like se
3
 org.openscience.cdk.AtomContainerSet 38-385 This class has too many methods, consider refactoring it.
3
 org.openscience.cdk.AtomContainerSet 50 It is somewhat confusing to have a field name with the same name as a method
3
 org.openscience.cdk.AtomType 377-407 The method toString() has an NPath complexity of 256
3
 org.openscience.cdk.BioPolymer 267-281 clone() method should be implemented only if implementing Cloneable interface
3
 org.openscience.cdk.Bond 82 It is somewhat confusing to have a field name with the same name as a method
3
 org.openscience.cdk.Bond 484 Avoid concatenating nonliterals in a StringBuffer constructor or append().
3
 org.openscience.cdk.Bond 484 StringBuffer.append is called 2 consecutive times with literal Strings. Use a single append with a single String.
3
 org.openscience.cdk.ChemFile 46-247 This class has too many methods, consider refactoring it.
3
 org.openscience.cdk.ChemFile 61 It is somewhat confusing to have a field name with the same name as a method
3
 org.openscience.cdk.ChemSequence 47-233 This class has too many methods, consider refactoring it.
3
 org.openscience.cdk.ChemSequence 62 It is somewhat confusing to have a field name with the same name as a method
3
 org.openscience.cdk.ConformerContainer 63-476 This class has too many methods, consider refactoring it.
1
 org.openscience.cdk.ConformerContainer 73 Avoid throwing null pointer exceptions.
3
 org.openscience.cdk.ConformerContainer 177 Avoid variables with short names like o
3
 org.openscience.cdk.ConformerContainer 221 Avoid variables with short names like ts
3
 org.openscience.cdk.ConformerContainer 268 Avoid variables with short names like o
3
 org.openscience.cdk.ConformerContainer 293 Avoid variables with short names like i
3
 org.openscience.cdk.ConformerContainer 322 Avoid variables with short names like i
3
 org.openscience.cdk.ConformerContainer 332 Avoid variables with short names like i
3
 org.openscience.cdk.ConformerContainer 342 Avoid variables with short names like i
3
 org.openscience.cdk.ConformerContainer 365 Avoid variables with short names like i
3
 org.openscience.cdk.ConformerContainer 382 Avoid variables with short names like o
3
 org.openscience.cdk.ConformerContainer 393 Avoid variables with short names like p
3
 org.openscience.cdk.ConformerContainer 416 Avoid variables with short names like o
3
 org.openscience.cdk.ConformerContainer 427 Avoid variables with short names like p
3
 org.openscience.cdk.ConformerContainer 444 Avoid variables with short names like i
3
 org.openscience.cdk.ConformerContainer 449 Avoid variables with short names like i
3
 org.openscience.cdk.ConformerContainer 449 Avoid variables with short names like i1
3
 org.openscience.cdk.DefaultChemObjectBuilder 24-367 This class has a bunch of public methods and attributes
3
 org.openscience.cdk.DefaultChemObjectBuilder 45-365 This class has too many methods, consider refactoring it.
3
 org.openscience.cdk.FragmentAtom 45 It is somewhat confusing to have a field name with the same name as a method
3
 org.openscience.cdk.FragmentAtom 85 StringBuffer constructor is initialized with size 16, but has at least 18 characters appended.
3
 org.openscience.cdk.Monomer 130 Avoid appending characters as strings in StringBuffer.append.
3
 org.openscience.cdk.Polymer 149-189 clone() method should be implemented only if implementing Cloneable interface
3
 org.openscience.cdk.Polymer 179 Avoid variables with short names like lp
3
 org.openscience.cdk.Reaction 50-476 This class has too many methods, consider refactoring it.
3
 org.openscience.cdk.ReactionScheme 47 It is somewhat confusing to have a field name matching the declaring class name
3
 org.openscience.cdk.ReactionScheme 117-130 clone() method should be implemented only if implementing Cloneable interface
2
 org.openscience.cdk.ReactionScheme 118 Object clone() should be implemented with super.clone()
3
 org.openscience.cdk.ReactionSet 60-251 This class has too many methods, consider refactoring it.
3
 org.openscience.cdk.ReactionSet 75 It is somewhat confusing to have a field name with the same name as a method
3
 org.openscience.cdk.Strand 246-257 clone() method should be implemented only if implementing Cloneable interface
3
 org.openscience.cdk.formula.AdductFormula 50-359 This class has too many methods, consider refactoring it.
2
 org.openscience.cdk.formula.AdductFormula 317 Object clone() should be implemented with super.clone()
3
 org.openscience.cdk.formula.MolecularFormula 55-403 This class has too many methods, consider refactoring it.
3
 org.openscience.cdk.formula.MolecularFormula 67 It is somewhat confusing to have a field name with the same name as a method
2
 org.openscience.cdk.formula.MolecularFormula 255 Object clone() should be implemented with super.clone()
2
 org.openscience.cdk.formula.MolecularFormulaSet 195 Object clone() should be implemented with super.clone()
3
 org.openscience.cdk.protein.data.PDBAtom 322 StringBuffer constructor is initialized with size 16, but has at least 109 characters appended.
3
 org.openscience.cdk.protein.data.PDBAtom 324 StringBuffer.append is called 2 consecutive times with literal Strings. Use a single append with a single String.
3
 org.openscience.cdk.protein.data.PDBAtom 325 StringBuffer.append is called 2 consecutive times with literal Strings. Use a single append with a single String.
3
 org.openscience.cdk.protein.data.PDBAtom 326 StringBuffer.append is called 2 consecutive times with literal Strings. Use a single append with a single String.
3
 org.openscience.cdk.protein.data.PDBAtom 327 StringBuffer.append is called 2 consecutive times with literal Strings. Use a single append with a single String.
3
 org.openscience.cdk.protein.data.PDBAtom 328 StringBuffer.append is called 2 consecutive times with literal Strings. Use a single append with a single String.
3
 org.openscience.cdk.protein.data.PDBAtom 329 StringBuffer.append is called 2 consecutive times with literal Strings. Use a single append with a single String.
3
 org.openscience.cdk.protein.data.PDBAtom 330 StringBuffer.append is called 2 consecutive times with literal Strings. Use a single append with a single String.
3
 org.openscience.cdk.protein.data.PDBAtom 331 StringBuffer.append is called 2 consecutive times with literal Strings. Use a single append with a single String.
3
 org.openscience.cdk.protein.data.PDBAtom 332 StringBuffer.append is called 2 consecutive times with literal Strings. Use a single append with a single String.
3
 org.openscience.cdk.protein.data.PDBAtom 333 StringBuffer.append is called 2 consecutive times with literal Strings. Use a single append with a single String.
3
 org.openscience.cdk.protein.data.PDBAtom 334 StringBuffer.append is called 2 consecutive times with literal Strings. Use a single append with a single String.
3
 org.openscience.cdk.protein.data.PDBAtom 337 Avoid appending characters as strings in StringBuffer.append.
3
 org.openscience.cdk.protein.data.PDBMonomer 89 StringBuffer constructor is initialized with size 16, but has at least 32 characters appended.
3
 org.openscience.cdk.protein.data.PDBMonomer 91 StringBuffer.append is called 2 consecutive times with literal Strings. Use a single append with a single String.
3
 org.openscience.cdk.protein.data.PDBMonomer 92 StringBuffer.append is called 2 consecutive times with literal Strings. Use a single append with a single String.
3
 org.openscience.cdk.protein.data.PDBMonomer 95 Avoid appending characters as strings in StringBuffer.append.
3
 org.openscience.cdk.protein.data.PDBPolymer 81-83 Overriding method merely calls super
3
 org.openscience.cdk.protein.data.PDBPolymer 126 Avoid appending characters as strings in StringBuffer.append.

Generated by PMD 4.2.5 on 2009-11-18 - 21:33:06.