Simplified oxNum, central atom is always a carbon so any of the hetero atoms is okay. Wow forgot about "destroyBondOrder", definitely not clear it simply "unboxed" the enum to a number!

johnmay · johnmay · commit 3e7dbb9b48ce · 2019-01-30T10:34:43.000Z
diff --git a/descriptor/qsarmolecular/src/main/java/org/openscience/cdk/qsar/descriptors/molecular/ALOGPDescriptor.java b/descriptor/qsarmolecular/src/main/java/org/openscience/cdk/qsar/descriptors/molecular/ALOGPDescriptor.java
@@ -1455,16 +1455,8 @@ else if (bj.getOrder() == IBond.Order.TRIPLE) {
 
         for (int j = 0; j <= ca2.size() - 1; j++) {
             IAtom ca2j = (IAtom) ca2.get(j);
-
-            // // F,O,Cl,Br,N
-
-            // if (s.equals("F") || s.equals("O") || s.equals("Cl")
-            // || s.equals("Br") || s.equals("N") || s.equals("S"))
-
-            if (ap.getNormalizedElectronegativity(ca2j.getSymbol()) > 1) {
-                oxNum += BondManipulator.destroyBondOrder(atomContainer.getBond(ca0, ca2j).getOrder());
-            }
-
+            if (isHetero(ca2j))
+                oxNum += atomContainer.getBond(ca0, ca2j).getOrder().numeric();
         }
 
         if (hybrid.equals("sp3") && oxNum == 1) {
