litvinovg/vitro
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

Minor performance improvements

Browse source
This commit is contained in:
Graham Triggs 2017-09-18 10:27:38 +01:00
parent e224736323
commit 60017600a3
9 changed files with 9 additions and 28 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

5
api/src/main/java/edu/cornell/mannlib/vitro/webapp/controller/edit/IndividualTypeRetryController.java
View file

@ -66,10 +66,7 @@ public class IndividualTypeRetryController extends BaseEditController {
continue; continue;
} }
prohibitedURIset.add(vc.getURI()); prohibitedURIset.add(vc.getURI());
for (Iterator<String> djURIIt = vcDao.getDisjointWithClassURIs(vc.getURI()).iterator(); djURIIt.hasNext(); ) { prohibitedURIset.addAll(vcDao.getDisjointWithClassURIs(vc.getURI()));
String djURI = djURIIt.next();
prohibitedURIset.add(djURI);
}
} }
List<VClass> eligibleVClasses = new ArrayList<VClass>(); List<VClass> eligibleVClasses = new ArrayList<VClass>();

4
api/src/main/java/edu/cornell/mannlib/vitro/webapp/controller/jena/JenaAdminActions.java
View file

@ -133,9 +133,7 @@ public class JenaAdminActions extends BaseEditController {
typeSet.add((Resource) stmt.getObject()); typeSet.add((Resource) stmt.getObject());
} }
} }
for (Resource classRes : ontModel.listClasses().toList()) { typeSet.addAll(ontModel.listClasses().toList());
typeSet.add(classRes);
}
for (Resource ontClass : typeSet) { for (Resource ontClass : typeSet) {
if (!ontClass.isAnon()) { // Only query for named classes if (!ontClass.isAnon()) { // Only query for named classes
System.out.println("Describing "+ontClass.getURI()); System.out.println("Describing "+ontClass.getURI());

4
api/src/main/java/edu/cornell/mannlib/vitro/webapp/controller/json/GetAllVClasses.java
View file

@ -37,9 +37,7 @@ public class GetAllVClasses extends JsonObjectProducer {
VClassGroupsForRequest vcgc = VClassGroupCache.getVClassGroups(vreq); VClassGroupsForRequest vcgc = VClassGroupCache.getVClassGroups(vreq);
List<VClassGroup> groups = vcgc.getGroups(); List<VClassGroup> groups = vcgc.getGroups();
for(VClassGroup vcg: groups) { for(VClassGroup vcg: groups) {
for( VClass vc : vcg){ vclasses.addAll(vcg);
vclasses.add(vc);
}
} }

4
api/src/main/java/edu/cornell/mannlib/vitro/webapp/edit/n3editing/VTwo/EditConfigurationUtils.java
View file

@ -296,9 +296,7 @@ public class EditConfigurationUtils {
/** Make a copy of list of Strings. */ /** Make a copy of list of Strings. */
public static List<String> copy(List<String> list) { public static List<String> copy(List<String> list) {
List<String> copyList = new ArrayList<String>(); List<String> copyList = new ArrayList<String>();
for(String l: list) { copyList.addAll(list);
copyList.add(l);
}
return copyList; return copyList;
} }

4
api/src/main/java/edu/cornell/mannlib/vitro/webapp/edit/n3editing/configuration/generators/DefaultObjectPropertyFormGenerator.java
View file

@ -145,9 +145,7 @@ public class DefaultObjectPropertyFormGenerator implements EditConfigurationGene
if (!rangeVClass.isUnion()) { if (!rangeVClass.isUnion()) {
types.add(rangeVClass); types.add(rangeVClass);
} else { } else {
for (VClass unionComponent : rangeVClass.getUnionComponents()) { types.addAll(rangeVClass.getUnionComponents());
types.add(unionComponent);
}
} }
return types; return types;
} else { } else {

4
api/src/main/java/edu/cornell/mannlib/vitro/webapp/edit/n3editing/configuration/generators/ManagePageGenerator.java
View file

@ -679,9 +679,7 @@ private String getExistingIsSelfContainedTemplateQuery() {
VClassGroupsForRequest vcgc = VClassGroupCache.getVClassGroups(vreq); VClassGroupsForRequest vcgc = VClassGroupCache.getVClassGroups(vreq);
List<VClassGroup> groups = vcgc.getGroups(); List<VClassGroup> groups = vcgc.getGroups();
for(VClassGroup vcg: groups) { for(VClassGroup vcg: groups) {
for( VClass vc : vcg){ vclasses.addAll(vcg);
vclasses.add(vc);
}
} }

4
api/src/main/java/edu/cornell/mannlib/vitro/webapp/edit/n3editing/controller/ProcessRdfFormController.java
View file

@ -207,9 +207,7 @@ public class ProcessRdfFormController extends FreemarkerHttpServlet{
public static List<String> makeListCopy(List<String> list) { public static List<String> makeListCopy(List<String> list) {
List<String> copyOfN3 = new ArrayList<String>(); List<String> copyOfN3 = new ArrayList<String>();
for( String str : list){ copyOfN3.addAll(list);
copyOfN3.add(str);
}
return copyOfN3; return copyOfN3;
} }

4
api/src/main/java/edu/cornell/mannlib/vitro/webapp/reasoner/ABoxRecomputer.java
View file

@ -239,9 +239,7 @@ public class ABoxRecomputer {
} }
} }
if(rebuildModel.size() - prevRebuildSize > 0) { if(rebuildModel.size() - prevRebuildSize > 0) {
for (String sameAsInd : sameAsInds) { individualQueue.addAll(sameAsInds);
individualQueue.add(sameAsInd);
}
} }
} }
return additionalInferences; return additionalInferences;

4
api/src/main/java/edu/ucsf/vitro/opensocial/OpenSocialManager.java
View file

@ -240,9 +240,7 @@ public class OpenSocialManager {
} }
} }
} }
for (PreparedGadget gadget : removedGadgets) { gadgets.removeAll(removedGadgets);
gadgets.remove(gadget);
}
} }
public void removeGadget(String name) { public void removeGadget(String name) {