5

I want to find the bounding box of several features selected by their attributes. Here's what I've been trying:

tNumList = [8,9,10,11]
selectGeom = QgsGeometry()
for tNum in tNumList:
    request = QgsFeatureRequest().setFilterExpression( u'"Territory" = {}'.format(tNum))
    iter = territoryBorders.getFeatures(request)
    feat = QgsFeature()
    iter.nextFeature(feat)
    selectGeom.combine(feat.geometry())
    print selectGeom.boundingBox().width()

Is there a better way to get a single feature from a layer then using the getFeatures(request)?

This just gives me empty rectangles. What am I doing wrong?

underdark
  • 84,148
  • 21
  • 231
  • 413
J M
  • 1,563
  • 10
  • 27

1 Answers1

16

I've optimized and corrected your code.

  1. All features are selected in one request with SQL in
  2. First feature is taken to be combined with something (this was the mistake in your code)
  3. Only the bounding boxes are combined to get result faster

Optimized code:

request = QgsFeatureRequest().setFilterExpression( u'"Territory" in (8, 9, 10, 11)')
iter = territoryBorders.getFeatures(request)
feat = QgsFeature()
iter.nextFeature(feat)
box = feat.geometry().boundingBox()
while iter.nextFeature(feat):
    box.combineExtentWith(feat.geometry().boundingBox())
box.width()
xunilk
  • 29,891
  • 4
  • 41
  • 80
Zoltan
  • 7,325
  • 17
  • 27