sharpetronics/QGIS
1
0
Fork 0
mirror of https://github.com/qgis/QGIS.git synced 2025-12-15 00:07:25 -05:00
Code Issues Packages Projects Releases Wiki Activity

Clearer ownership, docs for rule based renderer

Browse Source
This commit is contained in:
Nyall Dawson 2018-03-15 15:42:07 +10:00
parent 685adbf7e1
commit 7c0665a331
4 changed files with 62 additions and 55 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

9
python/core/symbology/qgsrulebasedrenderer.sip.in
View File

@ -39,7 +39,12 @@ the rules and draws features with symbols from rules that match.
struct RenderJob
{
RenderJob( QgsRuleBasedRenderer::FeatureToRender &_ftr, QgsSymbol *_s );
QgsRuleBasedRenderer::FeatureToRender &ftr;
%Docstring
Feature to render
%End
QgsSymbol *symbol;
};
@ -48,6 +53,7 @@ the rules and draws features with symbols from rules that match.
explicit RenderLevel( int z );
~RenderLevel();
int zIndex;
QList<QgsRuleBasedRenderer::RenderJob *> jobs;
@ -415,9 +421,6 @@ Check if this rule is an ELSE rule
protected:
void initFilter();
private:
Rule( const QgsRuleBasedRenderer::Rule &rh );
};

27
src/core/symbology/qgsrulebasedrenderer.cpp
View File

@ -56,8 +56,6 @@ QgsRuleBasedRenderer::Rule::Rule( QgsSymbol *symbol, int scaleMinDenom, int scal
QgsRuleBasedRenderer::Rule::~Rule()
{
delete mSymbol;
delete mFilter;
qDeleteAll( mChildren );
// do NOT delete parent
}
@ -67,20 +65,17 @@ void QgsRuleBasedRenderer::Rule::initFilter()
if ( mFilterExp.trimmed().compare( QLatin1String( "ELSE" ), Qt::CaseInsensitive ) == 0 )
{
mElseRule = true;
delete mFilter;
mFilter = nullptr;
mFilter.reset();
}
else if ( mFilterExp.trimmed().isEmpty() )
{
mElseRule = false;
delete mFilter;
mFilter = nullptr;
mFilter.reset();
}
else
{
mElseRule = false;
delete mFilter;
mFilter = new QgsExpression( mFilterExp );
mFilter = qgis::make_unique< QgsExpression >( mFilterExp );
}
}
@ -157,8 +152,7 @@ void QgsRuleBasedRenderer::Rule::setIsElse( bool iselse )
{
mFilterExp = QStringLiteral( "ELSE" );
mElseRule = iselse;
delete mFilter;
mFilter = nullptr;
mFilter.reset();
}
@ -215,7 +209,7 @@ QgsSymbolList QgsRuleBasedRenderer::Rule::symbols( const QgsRenderContext &conte
{
QgsSymbolList lst;
if ( mSymbol )
lst.append( mSymbol );
lst.append( mSymbol.get() );
Q_FOREACH ( Rule *rule, mChildren )
{
@ -226,8 +220,7 @@ QgsSymbolList QgsRuleBasedRenderer::Rule::symbols( const QgsRenderContext &conte
void QgsRuleBasedRenderer::Rule::setSymbol( QgsSymbol *sym )
{
delete mSymbol;
mSymbol = sym;
mSymbol.reset( sym );
}
void QgsRuleBasedRenderer::Rule::setFilterExpression( const QString &filterExp )
@ -241,7 +234,7 @@ QgsLegendSymbolList QgsRuleBasedRenderer::Rule::legendSymbolItems( int currentLe
QgsLegendSymbolList lst;
if ( currentLevel != -1 ) // root rule should not be shown
{
lst << QgsLegendSymbolItem( mSymbol, mLabel, mRuleKey, true, mMaximumScale, mMinimumScale, currentLevel, mParent ? mParent->mRuleKey : QString() );
lst << QgsLegendSymbolItem( mSymbol.get(), mLabel, mRuleKey, true, mMaximumScale, mMinimumScale, currentLevel, mParent ? mParent->mRuleKey : QString() );
}
for ( RuleList::const_iterator it = mChildren.constBegin(); it != mChildren.constEnd(); ++it )
@ -294,7 +287,7 @@ QDomElement QgsRuleBasedRenderer::Rule::save( QDomDocument &doc, QgsSymbolMap &s
if ( mSymbol )
{
int symbolIndex = symbolMap.size();
symbolMap[QString::number( symbolIndex )] = mSymbol;
symbolMap[QString::number( symbolIndex )] = mSymbol.get();
ruleElem.setAttribute( QStringLiteral( "symbol" ), symbolIndex );
}
if ( !mFilterExp.isEmpty() )
@ -508,7 +501,7 @@ QgsRuleBasedRenderer::Rule::RenderResult QgsRuleBasedRenderer::Rule::renderFeatu
Q_FOREACH ( int normZLevel, mSymbolNormZLevels )
{
//QgsDebugMsg(QString("add job at level %1").arg(normZLevel));
renderQueue[normZLevel].jobs.append( new RenderJob( featToRender, mSymbol ) );
renderQueue[normZLevel].jobs.append( new RenderJob( featToRender, mSymbol.get() ) );
rendered = true;
}
}
@ -565,7 +558,7 @@ QgsSymbolList QgsRuleBasedRenderer::Rule::symbolsForFeature( QgsFeature &feat, Q
if ( !isFilterOK( feat, context ) )
return lst;
if ( mSymbol )
lst.append( mSymbol );
lst.append( mSymbol.get() );
Q_FOREACH ( Rule *rule, mActiveChildren )
{

65
src/core/symbology/qgsrulebasedrenderer.h
View File

@ -45,7 +45,10 @@ class CORE_EXPORT QgsRuleBasedRenderer : public QgsFeatureRenderer
FeatDrawMarkers = 2
};
// feature for rendering: QgsFeature and some flags
/**
* Feature for rendering by a QgsRuleBasedRenderer. Contains a QgsFeature and some flags.
* \ingroup core
*/
struct FeatureToRender
{
FeatureToRender( QgsFeature &_f, int _flags )
@ -56,25 +59,35 @@ class CORE_EXPORT QgsRuleBasedRenderer : public QgsFeatureRenderer
int flags; // selected and/or draw markers
};
// rendering job: a feature to be rendered with a particular symbol
// (both f, symbol are _not_ owned by this class)
/**
* A QgsRuleBasedRenderer rendering job, consisting of a feature to be rendered with a particular symbol.
* \ingroup core
*/
struct RenderJob
{
RenderJob( QgsRuleBasedRenderer::FeatureToRender &_ftr, QgsSymbol *_s )
: ftr( _ftr )
, symbol( _s )
{}
//! Feature to render
QgsRuleBasedRenderer::FeatureToRender &ftr;
//! Symbol to render feature with (not owned by this object).
QgsSymbol *symbol = nullptr;
};
// render level: a list of jobs to be drawn at particular level
// (jobs are owned by this class)
/**
* Render level: a list of jobs to be drawn at particular level for a QgsRuleBasedRenderer.
* \ingroup core
*/
struct RenderLevel
{
explicit RenderLevel( int z ): zIndex( z ) {}
~RenderLevel() { Q_FOREACH ( RenderJob *j, jobs ) delete j; }
int zIndex;
//! List of jobs to render, owned by this object.
QList<QgsRuleBasedRenderer::RenderJob *> jobs;
QgsRuleBasedRenderer::RenderLevel &operator=( const QgsRuleBasedRenderer::RenderLevel &rh )
@ -100,7 +113,7 @@ class CORE_EXPORT QgsRuleBasedRenderer : public QgsFeatureRenderer
};
// rendering queue: a list of rendering levels
//! Rendering queue: a list of rendering levels
typedef QList<QgsRuleBasedRenderer::RenderLevel> RenderQueue;
class Rule;
@ -178,7 +191,7 @@ class CORE_EXPORT QgsRuleBasedRenderer : public QgsFeatureRenderer
*/
bool isScaleOK( double scale ) const;
QgsSymbol *symbol() { return mSymbol; }
QgsSymbol *symbol() { return mSymbol.get(); }
QString label() const { return mLabel; }
bool dependsOnScale() const { return mMaximumScale != 0 || mMinimumScale != 0; }
@ -206,7 +219,7 @@ class CORE_EXPORT QgsRuleBasedRenderer : public QgsFeatureRenderer
* A filter that will check if this rule applies
* \returns An expression
*/
QgsExpression *filter() const { return mFilter; }
QgsExpression *filter() const { return mFilter.get(); }
/**
* A filter that will check if this rule applies
@ -411,29 +424,29 @@ class CORE_EXPORT QgsRuleBasedRenderer : public QgsFeatureRenderer
protected:
void initFilter();
Rule *mParent; // parent rule (NULL only for root rule)
QgsSymbol *mSymbol = nullptr;
double mMaximumScale = 0;
double mMinimumScale = 0;
QString mFilterExp, mLabel, mDescription;
bool mElseRule;
RuleList mChildren;
RuleList mElseRules;
bool mIsActive; // whether it is enabled or not
QString mRuleKey; // string used for unique identification of rule within renderer
// temporary
QgsExpression *mFilter = nullptr;
// temporary while rendering
QSet<int> mSymbolNormZLevels;
RuleList mActiveChildren;
private:
#ifdef SIP_RUN
Rule( const QgsRuleBasedRenderer::Rule &rh );
#endif
Rule *mParent = nullptr; // parent rule (NULL only for root rule)
std::unique_ptr< QgsSymbol > mSymbol;
double mMaximumScale = 0;
double mMinimumScale = 0;
QString mFilterExp, mLabel, mDescription;
bool mElseRule = false;
RuleList mChildren;
RuleList mElseRules;
bool mIsActive = true; // whether it is enabled or not
QString mRuleKey; // string used for unique identification of rule within renderer
// temporary
std::unique_ptr< QgsExpression > mFilter;
// temporary while rendering
QSet<int> mSymbolNormZLevels;
RuleList mActiveChildren;
/**
* Check which child rules are else rules and update the internal list of else rules
*

16
tests/src/python/test_qgsrulebasedrenderer.py
View File

@ -30,6 +30,7 @@ import os
from qgis.PyQt.QtCore import QSize
from qgis.core import (QgsVectorLayer,
QgsMapSettings,
QgsProject,
QgsRectangle,
QgsMultiRenderChecker,
@ -42,7 +43,6 @@ from qgis.core import (QgsVectorLayer,
QgsRendererRange
)
from qgis.testing import start_app, unittest
from qgis.testing.mocked import get_iface
from utilities import unitTestDataPath
# Convenience instances in case you may need them
@ -54,7 +54,6 @@ TEST_DATA_DIR = unitTestDataPath()
class TestQgsRulebasedRenderer(unittest.TestCase):
def setUp(self):
self.iface = get_iface()
myShpFile = os.path.join(TEST_DATA_DIR, 'rectangles.shp')
layer = QgsVectorLayer(myShpFile, 'Points', 'ogr')
QgsProject.instance().addMapLayer(layer)
@ -68,14 +67,13 @@ class TestQgsRulebasedRenderer(unittest.TestCase):
self.r2 = QgsRuleBasedRenderer.Rule(sym2, 0, 0, '"id" = 2')
self.r3 = QgsRuleBasedRenderer.Rule(sym3, 0, 0, 'ELSE')
self.rootrule = QgsRuleBasedRenderer.Rule(None)
self.rootrule.appendChild(self.r1)
self.rootrule.appendChild(self.r2)
self.rootrule.appendChild(self.r3)
rootrule = QgsRuleBasedRenderer.Rule(None)
rootrule.appendChild(self.r1)
rootrule.appendChild(self.r2)
rootrule.appendChild(self.r3)
self.renderer = QgsRuleBasedRenderer(self.rootrule)
layer.setRenderer(self.renderer)
self.mapsettings = self.iface.mapCanvas().mapSettings()
layer.setRenderer(QgsRuleBasedRenderer(rootrule))
self.mapsettings = QgsMapSettings()
self.mapsettings.setOutputSize(QSize(400, 400))
self.mapsettings.setOutputDpi(96)
self.mapsettings.setExtent(QgsRectangle(-163, 22, -70, 52))