added a QgsOgrProvider::createSpatialIndex() method. This method creates a .qix index file for shape files. Note that there is currently no access from the GUI. If you already want to try this feature, just add a call to createSpatialIndex() to the constructor of QgsOgrProvider

mhugent · mhugent · commit f0c743801f61 · 2005-05-06T18:35:17.000Z
git-svn-id: http://svn.osgeo.org/qgis/trunk@3326 c8812cc2-4d05-0410-92ff-de0c093fc19c
diff --git a/providers/ogr/qgsogrprovider.cpp b/providers/ogr/qgsogrprovider.cpp
@@ -31,6 +31,7 @@ email                : sherman at mrcc.com
 #include <cpl_error.h>
 #include "ogr_api.h"//only for a test
 
+#include <qfile.h>
 #include <qmessagebox.h>
 #include <qmap.h>
 
@@ -78,7 +79,9 @@ QgsOgrProvider::QgsOgrProvider(QString uri): QgsVectorDataProvider(), dataSource
     std::cerr << "Data source is valid" << std::endl;
 #endif
     valid = true;
+
     ogrLayer = ogrDataSource->GetLayer(0);
+
     // get the extent_ (envelope) of the layer
 #ifdef QGISDEBUG
     std::cerr << "Starting get extent\n";
@@ -122,6 +125,8 @@ QgsOgrProvider::QgsOgrProvider(QString uri): QgsVectorDataProvider(), dataSource
     valid = false;
   }
 
+  //createSpatialIndex();
+
   //resize the cache matrix
   minmaxcache=new double*[fieldCount()];
   for(int i=0;i<fieldCount();i++)
@@ -571,8 +576,9 @@ void QgsOgrProvider::select(QgsRect *rect, bool useIntersect)
   //TODO   about it. If setting the filter fails, all records will be returned
   if (result == OGRERR_NONE) 
   {
-    std::cerr << "Setting spatial filter using " << wktExtent    << std::endl;
-    ogrLayer->SetSpatialFilter(filter);
+    std::cerr << "Setting spatial filter using " << wktExtent   << std::endl;
+    //ogrLayer->SetSpatialFilter(filter);
+    ogrLayer->SetSpatialFilterRect(rect->xMin(), rect->yMin(), rect->xMax(), rect->yMax());
     std::cerr << "Feature count: " << ogrLayer->GetFeatureCount() << std::endl;
   }else{
 #ifdef QGISDEBUG    
@@ -1070,6 +1076,28 @@ bool QgsOgrProvider::changeAttributeValues(std::map<int,std::map<QString,QString
   return true;
 }
 
+bool QgsOgrProvider::createSpatialIndex()
+{
+    //experimental, try to create a spatial index
+    QString filename=dataSourceUri.section('/',-1,-1);//todo: find out the filename from the uri
+    QString layername=filename.section('.',0,0);
+    QString sql="CREATE SPATIAL INDEX ON "+layername;
+    ogrDataSource->ExecuteSQL (sql.ascii(), ogrLayer->GetSpatialFilter(),"");
+    //todo: find out, if the .qix file is there
+    QString indexname=dataSourceUri;
+    indexname.truncate(dataSourceUri.length()-filename.length());
+    indexname=indexname+layername+".qix";
+    QFile indexfile(indexname);
+    if(indexfile.exists())
+    {
+	return true;
+    }
+    else
+    {
+	return false;
+    }
+}
+
 int QgsOgrProvider::capabilities() const
 {
     return (QgsVectorDataProvider::AddFeatures
diff --git a/providers/ogr/qgsogrprovider.h b/providers/ogr/qgsogrprovider.h
@@ -154,6 +154,10 @@ class QgsOgrProvider:public QgsVectorDataProvider
         return 0;
     }
 
+    /**Tries to create a .qix index file for faster access if only a subset of the features is required
+     @return true in case of success*/
+    bool createSpatialIndex();
+
     /**Returns a bitmask containing the supported capabilities*/
     int capabilities() const;
 
diff --git a/providers/postgres/qgspostgresprovider.cpp b/providers/postgres/qgspostgresprovider.cpp
@@ -322,6 +322,27 @@ QgsPostgresProvider::QgsPostgresProvider(QString uri):dataSourceUri(uri)
       } */
   //  tabledisplay=0;
 
+  //set client encoding to unicode because QString uses UTF-8 anyway
+#ifdef QGISDEBUG
+  qWarning("setting client encoding to UNICODE");
+#endif
+  int errcode=PQsetClientEncoding(connection, "UNICODE");
+ 
+#ifdef QGISDEBUG
+  if(errcode==0)
+  {
+      qWarning("encoding successfully set");
+  }
+  else if(errcode==-1)
+  {
+      qWarning("error in setting encoding");
+  }
+  else
+  {
+      qWarning("undefined return value from encoding setting");
+  }
+#endif
+
   //fill type names into lists
   mNumericalTypes.push_back("double precision");
   mNumericalTypes.push_back("int4");
@@ -727,7 +748,8 @@ void QgsPostgresProvider::getFeatureAttributes(int key, QgsFeature *f){
     // results
     if(fld != geometryColumn){
       // Add the attribute to the feature
-      QString val = mEncoding->toUnicode(PQgetvalue(attr,0, i));
+      //QString val = mEncoding->toUnicode(PQgetvalue(attr,0, i));
+	QString val = QString::fromUtf8 (PQgetvalue(attr,0, i));
       f->addAttribute(fld, val);
     }
   }
@@ -755,7 +777,8 @@ void QgsPostgresProvider::getFeatureAttributes(int key,
     if(fld != geometryColumn)
     {
       // Add the attribute to the feature
-      QString val = mEncoding->toUnicode(PQgetvalue(attr,0, i));
+      //QString val = mEncoding->toUnicode(PQgetvalue(attr,0, i));
+	QString val = QString::fromUtf8(PQgetvalue(attr,0, i));
       //qWarning(val);
       f->addAttribute(fld, val);
     }
diff --git a/src/qgsfeature.cpp b/src/qgsfeature.cpp
@@ -905,7 +905,7 @@ geos::Geometry* QgsFeature::geosGeometry() const
     }
 
 #ifdef QGISDEBUG
-    qWarning("In QgsFeature::geosGeometry()");
+    //qWarning("In QgsFeature::geosGeometry()");
 #endif
 	
     double *x;
@@ -951,7 +951,7 @@ geos::Geometry* QgsFeature::geosGeometry() const
 	case QGis::WKBLineString:
 	{
 #ifdef QGISDEBUG
-    qWarning("Linestring found");
+	    //qWarning("Linestring found");
 #endif
 	    geos::DefaultCoordinateSequence* sequence=new geos::DefaultCoordinateSequence();
 	    ptr = geometry + 5;
@@ -964,7 +964,7 @@ geos::Geometry* QgsFeature::geosGeometry() const
 		y = (double *) ptr;
 		ptr += sizeof(double);
 #ifdef QGISDEBUG
-			qWarning("adding coordinate pair "+QString::number(*x)+"//"+QString::number(*y));
+		//qWarning("adding coordinate pair "+QString::number(*x)+"//"+QString::number(*y));
 #endif
 		sequence->add(geos::Coordinate(*x,*y));
 	    }
@@ -998,7 +998,7 @@ geos::Geometry* QgsFeature::geosGeometry() const
 	case QGis::WKBPolygon: 
 	{
 #ifdef QGISDEBUG
-    qWarning("Polygon found");
+	    //qWarning("Polygon found");
 #endif
 	    // get number of rings in the polygon
 	    numRings = (int *) (geometry + 1 + sizeof(int));
@@ -1010,7 +1010,7 @@ geos::Geometry* QgsFeature::geosGeometry() const
 	    for (idx = 0; idx < *numRings; idx++)
 	    {
 #ifdef QGISDEBUG
-    qWarning("Ring nr: "+QString::number(idx));
+		//qWarning("Ring nr: "+QString::number(idx));
 #endif		
 		geos::DefaultCoordinateSequence* sequence=new geos::DefaultCoordinateSequence();
 		// get number of points in the ring
@@ -1041,7 +1041,7 @@ geos::Geometry* QgsFeature::geosGeometry() const
 	case QGis::WKBMultiPolygon:
 	{
 #ifdef QGISDEBUG
-	    qWarning("Multipolygon found");
+	    //qWarning("Multipolygon found");
 #endif
 	    std::vector<geos::Geometry *> *polygons=new std::vector<geos::Geometry *>;
 	    // get the number of polygons
diff --git a/src/qgsvectorlayer.cpp b/src/qgsvectorlayer.cpp
@@ -789,6 +789,9 @@ void QgsVectorLayer::draw(QPainter * p, QgsRect * viewExtent, QgsMapToPixel * th
       double scale = markerScaleFactor * symbolScale;
       drawFeature(p,fet,theMapToPixelTransform,&marker, scale, 
       projectionsEnabledFlag);
+      //test for geos performance
+      //geos::Geometry* g=fet->geosGeometry();
+      //delete g;
       ++featureCount;
       delete fet;
     }

Original file line number	Diff line number	Diff line change
`@@ -154,6 +154,10 @@ class QgsOgrProvider:public QgsVectorDataProvider`
`154`	`154`	`return 0;`
`155`	`155`	`}`
`156`	`156`
	`157`	`+ /**Tries to create a .qix index file for faster access if only a subset of the features is required`
	`158`	`+ @return true in case of success*/`
	`159`	`+ bool createSpatialIndex();`
	`160`	`+`
`157`	`161`	`/*Returns a bitmask containing the supported capabilities/`
`158`	`162`	`int capabilities() const;`
`159`	`163`