ont-api: add OWLOntology#disjointClassesAxioms(OWLClass) graph-optimization (issue #12)

Author: sszuev

src/main/java/com/github/owlcs/ontapi/internal/InternalModel.java

@@ -162,6 +162,7 @@ public class InternalModel extends OntGraphModelImpl
     protected final ByObjectSearcher<OWLSubClassOfAxiom, OWLClass> subClassOfBySubject = new SubClassOfBySubject();
     protected final ByObjectSearcher<OWLSubClassOfAxiom, OWLClass> subClassOfByObject = new SubClassOfByObject();
     protected final ByObjectSearcher<OWLEquivalentClassesAxiom, OWLClass> equivalentClassesByClass = new EquivalentClassesByClass();
+    protected final ByObjectSearcher<OWLDisjointClassesAxiom, OWLClass> disjointClassesByClass = new DisjointClassesByClass();
 
     // To search OWLObjects
     protected final ObjectsSearcher<OWLClass> classSearcher = new ClassSearcher();
@@ -709,7 +710,15 @@ public Stream<OWLEquivalentClassesAxiom> listOWLEquivalentClassesAxioms(OWLClass
      * @return {@code Stream} of {@link OWLDisjointClassesAxiom}s
      */
     public Stream<OWLDisjointClassesAxiom> listOWLDisjointClassesAxioms(OWLClass clazz) {
-        return listOWLAxioms(OWLDisjointClassesAxiom.class).filter(x -> x.operands().anyMatch(clazz::equals));
+        InternalConfig config = getConfig();
+        // bad performance of the graph-reading way if
+        // there is a []-list with many disjoint classes (i.e. owl:AllDisjointClasses) =>
+        // it is faster to use parsing of cached axioms (i.e. a classic way), if the cache is present
+        if (!config.useContentCache() ||
+                (!hasManuallyAddedAxioms() && !containsLocal(null, RDF.type, OWL.AllDisjointClasses))) {
+            return listOWLAxioms(disjointClassesByClass, OWLDisjointClassesAxiom.class, clazz, config);
+        }
+        return listOWLNaryAxiomAxiomsByOperand(clazz, OWLDisjointClassesAxiom.class);
     }
 
     /**
@@ -720,20 +729,26 @@ public Stream<OWLDisjointClassesAxiom> listOWLDisjointClassesAxioms(OWLClass cla
      * @param searcher - {@link ByObjectSearcher} for {@link A} and {@link K}
      * @param <A>      - subtype of {@link OWLNaryAxiom}
      * @param <K>      - subtype of {@link OWLObject}
-     * @return {@code Stream} of {@link A}s
+     * @return a {@code Stream} of {@link A}s
      * @see AbstractNaryTranslator#axioms(OntModel)
      */
-    public <A extends OWLNaryAxiom<? super K>,
+    @SuppressWarnings("SameParameterValue")
+    protected <A extends OWLNaryAxiom<? super K>,
             K extends OWLObject> Stream<A> listOWLNaryAxiomAxiomsByOperand(K operand,
                                                                            Class<A> type,
                                                                            ByObjectSearcher<A, K> searcher) {
         InternalConfig config = getConfig();
         if (!useAxiomsSearchOptimization(config)) {
-            return listOWLAxioms(type).filter(a -> a.operands().anyMatch(operand::equals));
+            return listOWLNaryAxiomAxiomsByOperand(operand, type);
         }
         return listOWLAxioms(searcher, type, operand, config);
     }
 
+    protected <A extends OWLNaryAxiom<? super K>,
+            K extends OWLObject> Stream<A> listOWLNaryAxiomAxiomsByOperand(K operand, Class<A> type) {
+        return listOWLAxioms(type).filter(a -> a.operands().anyMatch(operand::equals));
+    }
+
     /**
      * Auxiliary method to extract axioms from the graph.
      * Note: the method returns non-cached axioms.
@@ -869,7 +884,7 @@ protected boolean useReferencingAxiomsSearchOptimization(OWLComponentType type,
     }
 
     /**
-     * Answers {@code true} if need to use {@link ByObjectSearcher}-search optimization.
+     * Answers {@code true} if need to use {@link ByObjectSearcher}-search optimization instead of parsing cache.
      *
      * @param config {@link InternalConfig}
      * @return boolean

src/main/java/com/github/owlcs/ontapi/internal/searchers/axioms/DisjointClassesByClass.java

@@ -0,0 +1,54 @@
+/*
+ * This file is part of the ONT API.
+ * The contents of this file are subject to the LGPL License, Version 3.0.
+ * Copyright (c) 2020, The University of Manchester, owl.cs group.
+ *
+ * This program is free software: you can redistribute it and/or modify it under the terms of the GNU General Public License as published by the Free Software Foundation, either version 3 of the License, or (at your option) any later version.
+ * This program is distributed in the hope that it will be useful, but WITHOUT ANY WARRANTY; without even the implied warranty of MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the GNU General Public License for more details.
+ * You should have received a copy of the GNU General Public License along with this program.  If not, see http://www.gnu.org/licenses/.
+ *
+ * Alternatively, the contents of this file may be used under the terms of the Apache License, Version 2.0 in which case, the provisions of the Apache License Version 2.0 are applicable instead of those above.
+ * Licensed under the Apache License, Version 2.0 (the "License"); you may not use this file except in compliance with the License. You may obtain a copy of the License at http://www.apache.org/licenses/LICENSE-2.0
+ * Unless required by applicable law or agreed to in writing, software distributed under the License is distributed on an "AS IS" BASIS, WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied. See the License for the specific language governing permissions and limitations under the License.
+ */
+
+package com.github.owlcs.ontapi.internal.searchers.axioms;
+
+import com.github.owlcs.ontapi.config.AxiomsSettings;
+import com.github.owlcs.ontapi.internal.ONTObject;
+import com.github.owlcs.ontapi.internal.ONTObjectFactory;
+import com.github.owlcs.ontapi.internal.OWLTopObjectType;
+import com.github.owlcs.ontapi.internal.axioms.DisjointClassesTranslator;
+import com.github.owlcs.ontapi.jena.model.*;
+import com.github.owlcs.ontapi.jena.utils.Iter;
+import com.github.owlcs.ontapi.jena.utils.OntModels;
+import com.github.owlcs.ontapi.jena.vocabulary.OWL;
+import org.apache.jena.util.iterator.ExtendedIterator;
+import org.semanticweb.owlapi.model.OWLClass;
+import org.semanticweb.owlapi.model.OWLDisjointClassesAxiom;
+
+/**
+ * Created by @ssz on 09.05.2020.
+ */
+public class DisjointClassesByClass extends BaseByObject<OWLDisjointClassesAxiom, OWLClass> {
+    public static final DisjointClassesTranslator TRANSLATOR = toTranslator(OWLTopObjectType.DISJOINT_CLASSES);
+
+    protected ExtendedIterator<OntStatement> listStatements(OntModel model, OntClass clazz) {
+        ExtendedIterator<OntStatement> simple = listBySubjectAndPredicate(model, clazz, OWL.disjointWith)
+                .andThen(listByPredicateAndObject(model, OWL.disjointWith, clazz));
+        ExtendedIterator<OntStatement> bulk = OntModels.listLocalObjects(model, OntDisjoint.Classes.class)
+                .filterKeep(x -> x.getList().contains(clazz))
+                .mapWith(OntObject::getMainStatement);
+        return Iter.concat(simple, bulk);
+    }
+
+    @Override
+    public ExtendedIterator<ONTObject<OWLDisjointClassesAxiom>> listONTAxioms(OWLClass clazz,
+                                                                              OntModel model,
+                                                                              ONTObjectFactory factory,
+                                                                              AxiomsSettings config) {
+        ExtendedIterator<OntStatement> res = listStatements(model, model.getOntClass(clazz.getIRI().getIRIString()))
+                .filterKeep(s -> TRANSLATOR.testStatement(s, config));
+        return translate(TRANSLATOR, res, factory, config);
+    }
+}

src/main/java/com/github/owlcs/ontapi/jena/impl/OntGraphModelImpl.java

@@ -641,6 +641,20 @@ public OntStatementImpl asStatement(Triple triple) {
         return OntStatementImpl.createOntStatementImpl(triple, this);
     }
 
+    /**
+     * Determines if the given {@code (s, p, o)} pattern is present in the base graph,
+     * with {@code null} allowed to represent a wildcard match.
+     *
+     * @param s - {@link Resource} - the subject of the statement tested ({@code null} as wildcard)
+     * @param p - {@link Property} - the predicate of the statement tested ({@code null} as wildcard)
+     * @param o - {@link RDFNode} - the object of the statement tested ({@code null} as wildcard)
+     * @return boolean
+     * @see Model#contains(Resource, Property, RDFNode)
+     */
+    public boolean containsLocal(Resource s, Property p, RDFNode o) {
+        return getBaseGraph().contains(asNode(s), asNode(p), asNode(o));
+    }
+
     /**
      * Wraps the existing given {@link RDFList []-list} as {@link OntList ONT-list}.
      *