Skip to content

Commit

Permalink
Merge pull request #234 from homann/test_qgsclipper
Browse files Browse the repository at this point in the history
Test qgsclipper, improved testing and added constructor in QgsRectangle.
  • Loading branch information
homann committed Sep 16, 2012
2 parents 8a347eb + 9629501 commit 3fb8093
Show file tree
Hide file tree
Showing 5 changed files with 80 additions and 23 deletions.
10 changes: 8 additions & 2 deletions python/core/qgsrectangle.sip
Original file line number Diff line number Diff line change
Expand Up @@ -10,12 +10,15 @@ class QgsRectangle
%TypeHeaderCode
#include <qgsrectangle.h>
%End

public:
//! Constructor
QgsRectangle(double xmin=0, double ymin=0, double xmax=0, double ymax=0);
//! Construct a rectangle from two points. The rectangle is normalized after construction.
QgsRectangle(const QgsPoint & p1, const QgsPoint & p2);
//! Construct a rectangle from a QRectF. The rectangle is normalized after construction.
//@note added in 2.0
QgsRectangle(const QRectF & qRectF );
//! Copy constructor
QgsRectangle(const QgsRectangle &other);
//! Destructor
Expand Down Expand Up @@ -77,8 +80,11 @@ class QgsRectangle
//! returns string representation in Wkt form
QString asWktCoordinates() const;
//! returns string representation as WKT Polygon
//@note added om 2.0
//@note added in 2.0
QString asWktPolygon() const;
//! returns a QRectF with same coordinates.
//@note added in 2.0
QRectF QgsRectangle::toRectF() const;
//! returns string representation of form xmin,ymin xmax,ymax
QString toString(bool automaticPrecision = false) const;
//! overloaded toString that allows precision of numbers to be set
Expand Down
20 changes: 18 additions & 2 deletions src/core/qgsrectangle.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -18,6 +18,7 @@
#include <algorithm>
#include <cmath>
#include <limits>
#include <QRectF>
#include <QString>
#include <QTextStream>
#include <qnumeric.h>
Expand All @@ -37,6 +38,14 @@ QgsRectangle::QgsRectangle( QgsPoint const & p1, QgsPoint const & p2 )
set( p1, p2 );
}

QgsRectangle::QgsRectangle( QRectF const & qRectF )
{
xmin = qRectF.topLeft().x();
ymin = qRectF.topLeft().y();
xmax = qRectF.bottomRight().x();
ymax = qRectF.bottomRight().y();
}

QgsRectangle::QgsRectangle( const QgsRectangle &r )
{
xmin = r.xMinimum();
Expand Down Expand Up @@ -187,7 +196,7 @@ QString QgsRectangle::asWktCoordinates() const
QString QgsRectangle::asWktPolygon() const
{
QString rep =
QString("POLYGON((") +
QString( "POLYGON((" ) +
QString::number( xmin, 'f', 16 ) + " " +
QString::number( ymin, 'f', 16 ) + ", " +
QString::number( xmax, 'f', 16 ) + " " +
Expand All @@ -198,11 +207,18 @@ QString QgsRectangle::asWktPolygon() const
QString::number( ymax, 'f', 16 ) + ", " +
QString::number( xmin, 'f', 16 ) + " " +
QString::number( ymin, 'f', 16 ) +
QString("))");
QString( "))" );

return rep;
}

//! returns a QRectF with same coordinates.
//@note added in 2.0
QRectF QgsRectangle::toRectF() const
{
return QRectF(( qreal )xmin, ( qreal )ymin, ( qreal )xmax - xmin, ( qreal )ymax - ymin );
}

// Return a string representation of the rectangle with automatic or high precision
QString QgsRectangle::toString( bool automaticPrecision ) const
{
Expand Down
10 changes: 8 additions & 2 deletions src/core/qgsrectangle.h
Original file line number Diff line number Diff line change
Expand Up @@ -21,7 +21,7 @@
#include <iosfwd>

class QString;

class QRectF;
#include "qgspoint.h"


Expand All @@ -38,6 +38,9 @@ class CORE_EXPORT QgsRectangle
QgsRectangle( double xmin = 0, double ymin = 0, double xmax = 0, double ymax = 0 );
//! Construct a rectangle from two points. The rectangle is normalized after construction.
QgsRectangle( QgsPoint const & p1, QgsPoint const & p2 );
//! Construct a rectangle from a QRectF. The rectangle is normalized after construction.
//@note added in 2.0
QgsRectangle( const QRectF & qRectF );
//! Copy constructor
QgsRectangle( const QgsRectangle &other );
//! Destructor
Expand Down Expand Up @@ -98,8 +101,11 @@ class CORE_EXPORT QgsRectangle
//! returns string representation in Wkt form
QString asWktCoordinates() const;
//! returns string representation as WKT Polygon
//@note added om 2.0
//@note added in 2.0
QString asWktPolygon() const;
//! returns a QRectF with same coordinates.
//@note added in 2.0
QRectF toRectF() const;
//! returns string representation of form xmin,ymin xmax,ymax
QString toString( bool automaticPrecision = false ) const;
//! overloaded toString that allows precision of numbers to be set
Expand Down
54 changes: 37 additions & 17 deletions tests/src/core/testqgsclipper.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -34,36 +34,56 @@ class TestQgsClipper: public QObject
void init() {};// will be called before each testfunction is executed.
void cleanup() {};// will be called after every testfunction.
void basic();
private:
bool TestQgsClipper::checkBoundingBox( QPolygonF polygon, QgsRectangle clipRect );
};

void TestQgsClipper::initTestCase()
{
//
// Runs once before any tests are run
//
// init QGIS's paths - true means that all path will be inited from prefix
// QgsApplication::init();
// QgsApplication::initQgis();
// QgsApplication::showSettings();

}

void TestQgsClipper::basic()
{
// CQgsClipper is static only
//QgsClipper snipsnip;
// QgsClipper is static only

QPolygonF polygon;
polygon << QPointF(10.4, 20.5) << QPointF(20.2, 30.2);
QgsRectangle clipRect(10, 10, 25, 30 );
polygon << QPointF( 10.4, 20.5 ) << QPointF( 20.2, 30.2 );

QgsRectangle clipRect( 10, 10, 25, 30 );

QgsClipper::trimPolygon( polygon, clipRect );

QRectF bBox( polygon.boundingRect() );
QgsRectangle boundingRect( bBox.bottomLeft().x(), bBox.bottomLeft().y(), bBox.topRight().x(), bBox.topRight().y() );

QVERIFY( clipRect.contains( boundingRect ) );
// Check nothing sticks out.
QVERIFY( checkBoundingBox( polygon , clipRect ) );
// Check that it didn't clip too much
QgsRectangle clipRectInner( clipRect );
clipRectInner.scale( 0.999 );
QVERIFY( ! checkBoundingBox( polygon , clipRectInner ) );

// A more complex example
polygon.clear();
polygon << QPointF( 1.0, 9.0 ) << QPointF( 11.0, 11.0 ) << QPointF( 9.0, 1.0 );
clipRect.set( 0.0, 0.0, 10.0, 10.0 );

QgsClipper::trimPolygon( polygon, clipRect );

// We should have 5 vertices now?
QCOMPARE( polygon.size(), 5 );
// Check nothing sticks out.
QVERIFY( checkBoundingBox( polygon , clipRect ) );
// Check that it didn't clip too much
clipRectInner = clipRect;
clipRectInner.scale( 0.999 );
QVERIFY( ! checkBoundingBox( polygon , clipRectInner ) );
};

bool TestQgsClipper::checkBoundingBox( QPolygonF polygon, QgsRectangle clipRect )
{
QgsRectangle bBox( polygon.boundingRect() );

return clipRect.contains( bBox );
}

QTEST_MAIN( TestQgsClipper )
#include "moc_testqgsclipper.cxx"
9 changes: 9 additions & 0 deletions tests/src/core/testqgsrectangle.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -59,6 +59,15 @@ void TestQgsRectangle::regression6194()
// 100 wide, 200 high
QgsRectangle rect1 = QgsRectangle( 10.0, 20.0, 110.0, 220.0 );

// Test conversion to QRectF and back
QRectF qRectF = rect1.toRectF();
QCOMPARE( qRectF.width(), 100.0 );
QCOMPARE( qRectF.height(), 200.0 );
QCOMPARE( qRectF.x(), 10.0 );
QCOMPARE( qRectF.y(), 20.0 );
QgsRectangle rect4 = QgsRectangle( qRectF );
QCOMPARE( rect4.toString( 2 ), QString( "10.00,20.00 : 110.00,220.00" ) );

// 250 wide, 500 high
QgsRectangle rect2;
rect2.setXMinimum( 10.0 );
Expand Down

0 comments on commit 3fb8093

Please sign in to comment.