summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorEike Rathke <erack@redhat.com>2017-05-23 14:50:41 +0200
committerEike Rathke <erack@redhat.com>2017-05-23 14:50:47 +0200
commitd24a9a7513805f1baee3ff7c9b8a25eb467a0f6b (patch)
tree92aaca5cd91ae836f9fbb6a00f54a25aca96993e
parentcd87661b4761ee9f512fc01f95d84f786c97bb66 (diff)
Move pushing the result token into IterateParametersIfs, tdf#58874
Change-Id: I7c44afad1610fd04b0b2acc51c654a583ed991e6
-rw-r--r--sc/source/core/inc/interpre.hxx2
-rw-r--r--sc/source/core/tool/interpr1.cxx111
2 files changed, 70 insertions, 43 deletions
diff --git a/sc/source/core/inc/interpre.hxx b/sc/source/core/inc/interpre.hxx
index 8e6d89206c3a..53190038bd9b 100644
--- a/sc/source/core/inc/interpre.hxx
+++ b/sc/source/core/inc/interpre.hxx
@@ -600,7 +600,7 @@ void IterateParametersIf( ScIterFuncIf );
void ScCountIf();
void ScSumIf();
void ScAverageIf();
-void IterateParametersIfs( sc::ParamIfsResult& rRes );
+void IterateParametersIfs( double(*ResultFunc)( const sc::ParamIfsResult& rRes ) );
void ScSumIfs();
void ScAverageIfs();
void ScCountIfs();
diff --git a/sc/source/core/tool/interpr1.cxx b/sc/source/core/tool/interpr1.cxx
index ba1a5d916d4d..4eb3e3bbe518 100644
--- a/sc/source/core/tool/interpr1.cxx
+++ b/sc/source/core/tool/interpr1.cxx
@@ -5624,11 +5624,12 @@ void ScInterpreter::ScCountIf()
}
}
-void ScInterpreter::IterateParametersIfs( sc::ParamIfsResult& rRes )
+void ScInterpreter::IterateParametersIfs( double(*ResultFunc)( const sc::ParamIfsResult& rRes ) )
{
sal_uInt8 nParamCount = GetByte();
sal_uInt8 nQueryCount = nParamCount / 2;
+ sc::ParamIfsResult aRes;
std::vector<sal_uInt8> aResArray;
size_t nRowSize = 0;
size_t nColSize = 0;
@@ -5651,7 +5652,10 @@ void ScInterpreter::IterateParametersIfs( sc::ParamIfsResult& rRes )
{
ScAddress aAdr;
if ( !PopDoubleRefOrSingleRef( aAdr ) )
+ {
+ PushError( nGlobalError);
return;
+ }
ScRefCellValue aCell(*pDok, aAdr);
switch (aCell.meType)
@@ -5713,7 +5717,10 @@ void ScInterpreter::IterateParametersIfs( sc::ParamIfsResult& rRes )
}
if (nGlobalError != FormulaError::NONE)
+ {
+ PushError( nGlobalError);
return; // and bail out, no need to evaluate other arguments
+ }
// take range
nParam = 1;
@@ -5750,7 +5757,7 @@ void ScInterpreter::IterateParametersIfs( sc::ParamIfsResult& rRes )
pQueryMatrix = PopMatrix();
if (!pQueryMatrix)
{
- SetError( FormulaError::IllegalParameter);
+ PushError( FormulaError::IllegalParameter);
return;
}
nCol1 = 0;
@@ -5764,12 +5771,12 @@ void ScInterpreter::IterateParametersIfs( sc::ParamIfsResult& rRes )
}
break;
default:
- SetError( FormulaError::IllegalParameter);
+ PushError( FormulaError::IllegalParameter);
return;
}
if ( nTab1 != nTab2 )
{
- SetError( FormulaError::IllegalArgument);
+ PushError( FormulaError::IllegalArgument);
return;
}
@@ -5780,13 +5787,16 @@ void ScInterpreter::IterateParametersIfs( sc::ParamIfsResult& rRes )
nDimensionRows = nRow2 - nRow1 + 1;
if ((nDimensionCols != (nCol2 - nCol1 + 1)) || (nDimensionRows != (nRow2 - nRow1 + 1)))
{
- SetError ( FormulaError::IllegalArgument);
+ PushError ( FormulaError::IllegalArgument);
return;
}
// recalculate matrix values
if (nGlobalError != FormulaError::NONE)
+ {
+ PushError( nGlobalError);
return;
+ }
// initialize temporary result matrix
if (aResArray.empty())
@@ -5829,7 +5839,7 @@ void ScInterpreter::IterateParametersIfs( sc::ParamIfsResult& rRes )
ScMatrixRef pResultMatrix = QueryMat( pQueryMatrix, aOptions);
if (nGlobalError != FormulaError::NONE || !pResultMatrix)
{
- SetError( FormulaError::IllegalParameter);
+ PushError( FormulaError::IllegalParameter);
return;
}
@@ -5838,7 +5848,7 @@ void ScInterpreter::IterateParametersIfs( sc::ParamIfsResult& rRes )
pResultMatrix->GetDoubleArray(aResValues);
if (aResArray.size() != aResValues.size())
{
- SetError( FormulaError::IllegalParameter);
+ PushError( FormulaError::IllegalParameter);
return;
}
@@ -5866,7 +5876,10 @@ void ScInterpreter::IterateParametersIfs( sc::ParamIfsResult& rRes )
}
if (nGlobalError != FormulaError::NONE)
+ {
+ PushError( nGlobalError);
return; // bail out
+ }
// main range - only for AVERAGEIFS, SUMIFS, MINIFS and MAXIFS
if (nParamCount == 1)
@@ -5906,7 +5919,7 @@ void ScInterpreter::IterateParametersIfs( sc::ParamIfsResult& rRes )
pMainMatrix = PopMatrix();
if (!pMainMatrix)
{
- SetError( FormulaError::IllegalParameter);
+ PushError( FormulaError::IllegalParameter);
return;
}
nMainCol1 = 0;
@@ -5920,24 +5933,27 @@ void ScInterpreter::IterateParametersIfs( sc::ParamIfsResult& rRes )
}
break;
default:
- SetError( FormulaError::IllegalParameter);
+ PushError( FormulaError::IllegalParameter);
return;
}
if ( nMainTab1 != nMainTab2 )
{
- SetError( FormulaError::IllegalArgument);
+ PushError( FormulaError::IllegalArgument);
return;
}
// All reference ranges must be of same dimension and size.
if ((nDimensionCols != (nMainCol2 - nMainCol1 + 1)) || (nDimensionRows != (nMainRow2 - nMainRow1 + 1)))
{
- SetError ( FormulaError::IllegalArgument);
+ PushError ( FormulaError::IllegalArgument);
return;
}
if (nGlobalError != FormulaError::NONE)
+ {
+ PushError( nGlobalError);
return; // bail out
+ }
// end-result calculation
ScAddress aAdr;
@@ -5948,7 +5964,7 @@ void ScInterpreter::IterateParametersIfs( sc::ParamIfsResult& rRes )
pMainMatrix->GetDoubleArray(aMainValues, false); // Map empty values to NaN's.
if (aResArray.size() != aMainValues.size())
{
- SetError( FormulaError::IllegalArgument);
+ PushError( FormulaError::IllegalArgument);
return;
}
@@ -5963,18 +5979,18 @@ void ScInterpreter::IterateParametersIfs( sc::ParamIfsResult& rRes )
if (GetDoubleErrorValue(fVal) == FormulaError::ElementNaN)
continue;
- ++rRes.mfCount;
+ ++aRes.mfCount;
if (bNull && fVal != 0.0)
{
bNull = false;
- rRes.mfMem = fVal;
+ aRes.mfMem = fVal;
}
else
- rRes.mfSum += fVal;
- if ( rRes.mfMin > fVal )
- rRes.mfMin = fVal;
- if ( rRes.mfMax < fVal )
- rRes.mfMax = fVal;
+ aRes.mfSum += fVal;
+ if ( aRes.mfMin > fVal )
+ aRes.mfMin = fVal;
+ if ( aRes.mfMax < fVal )
+ aRes.mfMax = fVal;
}
}
else
@@ -5992,18 +6008,18 @@ void ScInterpreter::IterateParametersIfs( sc::ParamIfsResult& rRes )
if (aCell.hasNumeric())
{
fVal = GetCellValue(aAdr, aCell);
- ++rRes.mfCount;
+ ++aRes.mfCount;
if ( bNull && fVal != 0.0 )
{
bNull = false;
- rRes.mfMem = fVal;
+ aRes.mfMem = fVal;
}
else
- rRes.mfSum += fVal;
- if ( rRes.mfMin > fVal )
- rRes.mfMin = fVal;
- if ( rRes.mfMax < fVal )
- rRes.mfMax = fVal;
+ aRes.mfSum += fVal;
+ if ( aRes.mfMin > fVal )
+ aRes.mfMin = fVal;
+ if ( aRes.mfMax < fVal )
+ aRes.mfMax = fVal;
}
}
}
@@ -6015,8 +6031,9 @@ void ScInterpreter::IterateParametersIfs( sc::ParamIfsResult& rRes )
std::vector<sal_uInt8>::const_iterator itRes = aResArray.begin(), itResEnd = aResArray.end();
for (; itRes != itResEnd; ++itRes)
if (*itRes == nQueryCount)
- ++rRes.mfCount;
+ ++aRes.mfCount;
}
+ PushDouble( ResultFunc( aRes));
}
void ScInterpreter::ScSumIfs()
@@ -6029,9 +6046,11 @@ void ScInterpreter::ScSumIfs()
return;
}
- sc::ParamIfsResult aRes;
- IterateParametersIfs(aRes);
- PushDouble(rtl::math::approxAdd(aRes.mfSum, aRes.mfMem));
+ auto ResultFunc = []( const sc::ParamIfsResult& rRes )
+ {
+ return rtl::math::approxAdd(rRes.mfSum, rRes.mfMem);
+ };
+ IterateParametersIfs(ResultFunc);
}
void ScInterpreter::ScAverageIfs()
@@ -6044,9 +6063,11 @@ void ScInterpreter::ScAverageIfs()
return;
}
- sc::ParamIfsResult aRes;
- IterateParametersIfs(aRes);
- PushDouble(div(rtl::math::approxAdd(aRes.mfSum, aRes.mfMem), aRes.mfCount));
+ auto ResultFunc = []( const sc::ParamIfsResult& rRes )
+ {
+ return sc::div( rtl::math::approxAdd( rRes.mfSum, rRes.mfMem), rRes.mfCount);
+ };
+ IterateParametersIfs(ResultFunc);
}
void ScInterpreter::ScCountIfs()
@@ -6059,9 +6080,11 @@ void ScInterpreter::ScCountIfs()
return;
}
- sc::ParamIfsResult aRes;
- IterateParametersIfs(aRes);
- PushDouble(aRes.mfCount);
+ auto ResultFunc = []( const sc::ParamIfsResult& rRes )
+ {
+ return rRes.mfCount;
+ };
+ IterateParametersIfs(ResultFunc);
}
void ScInterpreter::ScMinIfs_MS()
@@ -6074,9 +6097,11 @@ void ScInterpreter::ScMinIfs_MS()
return;
}
- sc::ParamIfsResult aRes;
- IterateParametersIfs(aRes);
- PushDouble((aRes.mfMin < std::numeric_limits<double>::max()) ? aRes.mfMin : 0.0);
+ auto ResultFunc = []( const sc::ParamIfsResult& rRes )
+ {
+ return (rRes.mfMin < std::numeric_limits<double>::max()) ? rRes.mfMin : 0.0;
+ };
+ IterateParametersIfs(ResultFunc);
}
@@ -6090,9 +6115,11 @@ void ScInterpreter::ScMaxIfs_MS()
return;
}
- sc::ParamIfsResult aRes;
- IterateParametersIfs(aRes);
- PushDouble((aRes.mfMax > std::numeric_limits<double>::lowest()) ? aRes.mfMax : 0.0);
+ auto ResultFunc = []( const sc::ParamIfsResult& rRes )
+ {
+ return (rRes.mfMax > std::numeric_limits<double>::lowest()) ? rRes.mfMax : 0.0;
+ };
+ IterateParametersIfs(ResultFunc);
}
void ScInterpreter::ScLookup()