| Files | Total | Priority 1 | Priority 2 | Priority 3 | Priority 4 | Priority 5 |
|---|---|---|---|---|---|---|
| 8 | 100 | 17 |
0 |
82 |
1 |
0 |
| Prio | File | Line | Description |
|---|---|---|---|
1 |
org.openscience.cdk.charges.AtomTypeCharges | 48 | Variables that are not final should not contain underscores (except for underscores in standard prefix/suffix). |
1 |
org.openscience.cdk.charges.AtomTypeCharges | 49 | Variables that are not final should not contain underscores (except for underscores in standard prefix/suffix). |
3 |
org.openscience.cdk.charges.AtomTypeCharges | 73 | A method/constructor shouldn't explicitly throw java.lang.Exception |
3 |
org.openscience.cdk.charges.AtomTypeCharges | 84-85 | Avoid empty if statements |
3 |
org.openscience.cdk.charges.AtomTypeCharges | 99 | Avoid variables with short names like ac |
3 |
org.openscience.cdk.charges.Electronegativity | 79 | Avoid variables with short names like ac |
3 |
org.openscience.cdk.charges.Electronegativity | 95 | Avoid variables with short names like ac |
3 |
org.openscience.cdk.charges.GasteigerMarsiliPartialCharges | 54-365 | The class 'GasteigerMarsiliPartialCharges' has a Cyclomatic Complexity of 5 (Highest = 35). |
3 |
org.openscience.cdk.charges.GasteigerMarsiliPartialCharges | 56 | The field name indicates a constant but its modifiers do not |
1 |
org.openscience.cdk.charges.GasteigerMarsiliPartialCharges | 56 | Variables should start with a lowercase character |
1 |
org.openscience.cdk.charges.GasteigerMarsiliPartialCharges | 56 | Variables that are not final should not contain underscores (except for underscores in standard prefix/suffix). |
3 |
org.openscience.cdk.charges.GasteigerMarsiliPartialCharges | 57 | The field name indicates a constant but its modifiers do not |
1 |
org.openscience.cdk.charges.GasteigerMarsiliPartialCharges | 57 | Variables should start with a lowercase character |
1 |
org.openscience.cdk.charges.GasteigerMarsiliPartialCharges | 57 | Variables that are not final should not contain underscores (except for underscores in standard prefix/suffix). |
3 |
org.openscience.cdk.charges.GasteigerMarsiliPartialCharges | 58 | The field name indicates a constant but its modifiers do not |
1 |
org.openscience.cdk.charges.GasteigerMarsiliPartialCharges | 58 | Variables should start with a lowercase character |
1 |
org.openscience.cdk.charges.GasteigerMarsiliPartialCharges | 58 | Variables that are not final should not contain underscores (except for underscores in standard prefix/suffix). |
3 |
org.openscience.cdk.charges.GasteigerMarsiliPartialCharges | 59 | The field name indicates a constant but its modifiers do not |
1 |
org.openscience.cdk.charges.GasteigerMarsiliPartialCharges | 59 | Variables should start with a lowercase character |
1 |
org.openscience.cdk.charges.GasteigerMarsiliPartialCharges | 59 | Variables that are not final should not contain underscores (except for underscores in standard prefix/suffix). |
3 |
org.openscience.cdk.charges.GasteigerMarsiliPartialCharges | 146 | A method/constructor shouldn't explicitly throw java.lang.Exception |
3 |
org.openscience.cdk.charges.GasteigerMarsiliPartialCharges | 146 | Avoid variables with short names like ac |
3 |
org.openscience.cdk.charges.GasteigerMarsiliPartialCharges | 146-217 | The method 'assignGasteigerMarsiliSigmaPartialCharges' has a Cyclomatic Complexity of 12. |
3 |
org.openscience.cdk.charges.GasteigerMarsiliPartialCharges | 146-217 | The method assignGasteigerMarsiliSigmaPartialCharges() has an NPath complexity of 248 |
3 |
org.openscience.cdk.charges.GasteigerMarsiliPartialCharges | 156 | Avoid variables with short names like q |
3 |
org.openscience.cdk.charges.GasteigerMarsiliPartialCharges | 260-364 | Avoid really long methods. |
3 |
org.openscience.cdk.charges.GasteigerMarsiliPartialCharges | 260 | Avoid variables with short names like ac |
3 |
org.openscience.cdk.charges.GasteigerMarsiliPartialCharges | 260-364 | The method 'assignGasteigerSigmaMarsiliFactors' has a Cyclomatic Complexity of 35. |
3 |
org.openscience.cdk.charges.GasteigerMarsiliPartialCharges | 260-364 | The method assignGasteigerSigmaMarsiliFactors() has an NCSS line count of 100 |
3 |
org.openscience.cdk.charges.GasteigerPEPEPartialCharges | 77-878 | The class 'GasteigerPEPEPartialCharges' has a Cyclomatic Complexity of 8 (Highest = 49). |
3 |
org.openscience.cdk.charges.GasteigerPEPEPartialCharges | 79 | The field name indicates a constant but its modifiers do not |
1 |
org.openscience.cdk.charges.GasteigerPEPEPartialCharges | 79 | Variables should start with a lowercase character |
1 |
org.openscience.cdk.charges.GasteigerPEPEPartialCharges | 79 | Variables that are not final should not contain underscores (except for underscores in standard prefix/suffix). |
3 |
org.openscience.cdk.charges.GasteigerPEPEPartialCharges | 81 | The field name indicates a constant but its modifiers do not |
1 |
org.openscience.cdk.charges.GasteigerPEPEPartialCharges | 81 | Variables should start with a lowercase character |
1 |
org.openscience.cdk.charges.GasteigerPEPEPartialCharges | 81 | Variables that are not final should not contain underscores (except for underscores in standard prefix/suffix). |
3 |
org.openscience.cdk.charges.GasteigerPEPEPartialCharges | 82 | The field name indicates a constant but its modifiers do not |
1 |
org.openscience.cdk.charges.GasteigerPEPEPartialCharges | 82 | Variables should start with a lowercase character |
1 |
org.openscience.cdk.charges.GasteigerPEPEPartialCharges | 82 | Variables that are not final should not contain underscores (except for underscores in standard prefix/suffix). |
3 |
org.openscience.cdk.charges.GasteigerPEPEPartialCharges | 85 | The field name indicates a constant but its modifiers do not |
1 |
org.openscience.cdk.charges.GasteigerPEPEPartialCharges | 85 | Variables should start with a lowercase character |
3 |
org.openscience.cdk.charges.GasteigerPEPEPartialCharges | 89 | Avoid variables with short names like fE |
3 |
org.openscience.cdk.charges.GasteigerPEPEPartialCharges | 91 | Avoid variables with short names like fS |
3 |
org.openscience.cdk.charges.GasteigerPEPEPartialCharges | 149 | A method/constructor shouldn't explicitly throw java.lang.Exception |
3 |
org.openscience.cdk.charges.GasteigerPEPEPartialCharges | 149-407 | Avoid really long methods. |
3 |
org.openscience.cdk.charges.GasteigerPEPEPartialCharges | 149 | Avoid variables with short names like ac |
3 |
org.openscience.cdk.charges.GasteigerPEPEPartialCharges | 149-407 | The method 'assignGasteigerPiPartialCharges' has a Cyclomatic Complexity of 49. |
3 |
org.openscience.cdk.charges.GasteigerPEPEPartialCharges | 149-407 | The method assignGasteigerPiPartialCharges() has an NCSS line count of 179 |
3 |
org.openscience.cdk.charges.GasteigerPEPEPartialCharges | 221 | Avoid variables with short names like a |
3 |
org.openscience.cdk.charges.GasteigerPEPEPartialCharges | 231 | Avoid variables with short names like aa |
3 |
org.openscience.cdk.charges.GasteigerPEPEPartialCharges | 233 | Avoid variables with short names like ab |
3 |
org.openscience.cdk.charges.GasteigerPEPEPartialCharges | 274-278 | These nested if statements could be combined |
3 |
org.openscience.cdk.charges.GasteigerPEPEPartialCharges | 282 | An empty statement (semicolon) not part of a loop |
3 |
org.openscience.cdk.charges.GasteigerPEPEPartialCharges | 290 | Avoid variables with short names like Wt |
3 |
org.openscience.cdk.charges.GasteigerPEPEPartialCharges | 327 | Avoid variables with short names like q1 |
3 |
org.openscience.cdk.charges.GasteigerPEPEPartialCharges | 337 | Avoid variables with short names like DX |
3 |
org.openscience.cdk.charges.GasteigerPEPEPartialCharges | 343 | Avoid variables with short names like Dq |
3 |
org.openscience.cdk.charges.GasteigerPEPEPartialCharges | 351 | Avoid variables with short names like WE |
3 |
org.openscience.cdk.charges.GasteigerPEPEPartialCharges | 356 | Avoid variables with short names like W |
3 |
org.openscience.cdk.charges.GasteigerPEPEPartialCharges | 427 | Avoid variables with short names like ac |
3 |
org.openscience.cdk.charges.GasteigerPEPEPartialCharges | 443 | Avoid variables with short names like ac |
3 |
org.openscience.cdk.charges.GasteigerPEPEPartialCharges | 443 | Avoid variables with short names like b |
3 |
org.openscience.cdk.charges.GasteigerPEPEPartialCharges | 466 | Avoid variables with short names like ac |
3 |
org.openscience.cdk.charges.GasteigerPEPEPartialCharges | 466 | Avoid variables with short names like b |
3 |
org.openscience.cdk.charges.GasteigerPEPEPartialCharges | 487 | Avoid variables with short names like ac |
3 |
org.openscience.cdk.charges.GasteigerPEPEPartialCharges | 487-556 | The method 'getHyperconjugationInteractions' has a Cyclomatic Complexity of 18. |
3 |
org.openscience.cdk.charges.GasteigerPEPEPartialCharges | 501 | Avoid variables with short names like a0 |
3 |
org.openscience.cdk.charges.GasteigerPEPEPartialCharges | 502 | Avoid variables with short names like a1 |
3 |
org.openscience.cdk.charges.GasteigerPEPEPartialCharges | 504-510 | These nested if statements could be combined |
3 |
org.openscience.cdk.charges.GasteigerPEPEPartialCharges | 508-509 | These nested if statements could be combined |
3 |
org.openscience.cdk.charges.GasteigerPEPEPartialCharges | 565 | Avoid variables with short names like ac |
3 |
org.openscience.cdk.charges.GasteigerPEPEPartialCharges | 565 | Avoid variables with short names like ds |
3 |
org.openscience.cdk.charges.GasteigerPEPEPartialCharges | 603 | Avoid variables with short names like ac |
3 |
org.openscience.cdk.charges.GasteigerPEPEPartialCharges | 603-651 | The method 'getTopologicalFactors' has a Cyclomatic Complexity of 14. |
3 |
org.openscience.cdk.charges.GasteigerPEPEPartialCharges | 603-651 | The method getTopologicalFactors() has an NPath complexity of 600 |
3 |
org.openscience.cdk.charges.GasteigerPEPEPartialCharges | 608 | Avoid variables with short names like fQ |
3 |
org.openscience.cdk.charges.GasteigerPEPEPartialCharges | 620 | Avoid variables with short names like fB |
3 |
org.openscience.cdk.charges.GasteigerPEPEPartialCharges | 640 | Avoid variables with short names like fA |
3 |
org.openscience.cdk.charges.GasteigerPEPEPartialCharges | 643-644 | These nested if statements could be combined |
3 |
org.openscience.cdk.charges.GasteigerPEPEPartialCharges | 678-763 | The method 'assignPiFactors' has a Cyclomatic Complexity of 18. |
3 |
org.openscience.cdk.charges.GasteigerPEPEPartialCharges | 684 | Avoid variables with short names like ac |
3 |
org.openscience.cdk.charges.GasteigerPEPEPartialCharges | 770-866 | The method 'assignrPiMarsilliFactors' has a Cyclomatic Complexity of 20. |
3 |
org.openscience.cdk.charges.GasteigerPEPEPartialCharges | 776 | Avoid variables with short names like ac |
3 |
org.openscience.cdk.charges.GasteigerPEPEPartialCharges | 872 | Avoid variables with short names like ac |
3 |
org.openscience.cdk.charges.InductivePartialCharges | 58-409 | The class 'InductivePartialCharges' has a Cyclomatic Complexity of 7 (Highest = 17). |
3 |
org.openscience.cdk.charges.InductivePartialCharges | 94 | A method/constructor shouldn't explicitly throw java.lang.Exception |
3 |
org.openscience.cdk.charges.InductivePartialCharges | 94 | Avoid variables with short names like ac |
3 |
org.openscience.cdk.charges.InductivePartialCharges | 151 | Avoid variables with short names like ac |
3 |
org.openscience.cdk.charges.InductivePartialCharges | 151-206 | The method 'getPaulingElectronegativities' has a Cyclomatic Complexity of 17. |
3 |
org.openscience.cdk.charges.InductivePartialCharges | 222 | Avoid variables with short names like ac |
3 |
org.openscience.cdk.charges.InductivePartialCharges | 246 | The String literal "Problems with AtomTypeFactory due to " appears 4 times in this file; the first occurrence is on line 246 |
3 |
org.openscience.cdk.charges.InductivePartialCharges | 292 | Avoid variables with short names like ac |
3 |
org.openscience.cdk.charges.InductivePartialCharges | 292 | Avoid variables with short names like as |
3 |
org.openscience.cdk.charges.InductivePartialCharges | 353-391 | The method 'getCovalentRadius' has a Cyclomatic Complexity of 15. |
3 |
org.openscience.cdk.charges.PiElectronegativity | 84 | Avoid variables with short names like ac |
3 |
org.openscience.cdk.charges.PiElectronegativity | 100 | Avoid variables with short names like ac |
3 |
org.openscience.cdk.charges.PiElectronegativity | 127 | Avoid variables with short names like q |
3 |
org.openscience.cdk.charges.Polarizability | 57-325 | The class 'Polarizability' has a Cyclomatic Complexity of 6 (Highest = 33). |
3 |
org.openscience.cdk.charges.Polarizability | 233-303 | The method 'getKJPolarizabilityFactor' has a Cyclomatic Complexity of 33. |
4 |
org.openscience.cdk.charges.StabilizationCharges | 27 | Avoid unused imports such as 'org.openscience.cdk.exception.CDKException' |
Generated by PMD 4.2.5 on 2009-11-18 - 21:35:21.