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

NIHVIVO-2038 Improve performance of creating grouped property list by keeping a list of object property uris to check against, so a property doesn't have to be instantiated from the propertyUri in order to check whether it's on the list.

Browse source
This commit is contained in:
rjy7 2011-02-04 19:17:44 +00:00
parent cbfa225c31
commit b8fec2e96c
1 changed files with 23 additions and 12 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

21
webapp/src/edu/cornell/mannlib/vitro/webapp/web/templatemodels/individual/GroupedPropertyList.java
View file

@ -141,14 +141,23 @@ public class GroupedPropertyList extends BaseTemplateModel {
}
private void mergeAllPossibleObjectProperties(List<ObjectProperty> objectPropertyList, List<Property> propertyList) {
// Performance improvement: keep a list of uris on the propertyList to check against, rather than
// having to instantiate a property from propertyUri to check against the propertyList.
List<String> propertyListUris = new ArrayList<String>(propertyList.size());
for (Property p : propertyList) {
propertyListUris.add(p.getURI());
}
PropertyInstanceDao piDao = wdf.getPropertyInstanceDao();
Collection<PropertyInstance> allPropInstColl = piDao.getAllPossiblePropInstForIndividual(subject.getURI());
if (allPropInstColl != null) {
ObjectPropertyDao opDao = wdf.getObjectPropertyDao();
for (PropertyInstance pi : allPropInstColl) {
if (pi != null) {
if (! alreadyOnObjectPropertyList(objectPropertyList, pi)) {
addIfNotAlreadyOnList(propertyList, pi.getPropertyURI(), opDao);
addIfNotAlreadyOnList(propertyList, propertyListUris, pi.getPropertyURI(), opDao);
}
} else {
log.error("a property instance in the Collection created by PropertyInstanceDao.getAllPossiblePropInstForIndividual() is unexpectedly null");
@ -158,22 +167,25 @@ public class GroupedPropertyList extends BaseTemplateModel {
// In future, vitro ns props will be phased out. Vitro public properties should be changed so they do no
// constitute a special case.
for (String propertyUri : VITRO_PROPS_TO_ADD_TO_LIST) {
addIfNotAlreadyOnList(propertyList, propertyUri, opDao);
addIfNotAlreadyOnList(propertyList, propertyListUris, propertyUri, opDao);
}
} else {
log.error("a null Collection is returned from PropertyInstanceDao.getAllPossiblePropInstForIndividual()");
}
}
private void addIfNotAlreadyOnList(List<Property> propertyList, String propertyUri, ObjectPropertyDao opDao) {
private void addIfNotAlreadyOnList(List<Property> propertyList, List<String> propertyListUris, String propertyUri, ObjectPropertyDao opDao) {
if ( ! propertyListUris.contains(propertyUri)) {
ObjectProperty op = opDao.getObjectPropertyByURI(propertyUri);
if (op == null) {
log.error("ObjectProperty op returned null from opDao.getObjectPropertyByURI()");
} else if (op.getURI() == null) {
log.error("ObjectProperty op returned with null propertyURI from opDao.getObjectPropertyByURI()");
} else if (! alreadyOnPropertyList(propertyList, op)) {
} else {
propertyList.add(op);
propertyListUris.add(propertyUri);
}
}
}
@ -186,7 +198,6 @@ public class GroupedPropertyList extends BaseTemplateModel {
if (dp.getURI() == null) {
log.error("DataProperty dp returned with null propertyURI from dpDao.getAllPossibleDatapropsForIndividual()");
} else if (! alreadyOnPropertyList(propertyList, dp)) {
//dp.setLabel(dp.getPublicName());
propertyList.add(dp);
}
} else {