OilfieldToolsNet/ResInsight
4
0
Fork 0
mirror of https://github.com/OPM/ResInsight.git synced 2025-01-01 03:37:15 -06:00
Code Issues Packages Projects Releases Wiki Activity

Look Up Time Step Object Exactly Once

Browse Source 
Using .contains() followed by .at() with the same argument will
perform the same map lookup twice.
This commit is contained in:
Bård Skaflestad 2023-01-26 17:13:50 +01:00 committed by Kristian Bendiksen
parent 62ce2b18e0
commit 6eebd01bd4
1 changed files with 4 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

7
ApplicationLibCode/ProjectDataModel/Flow/RimWellAllocationOverTimeCollection.cpp
View File

@ -48,10 +48,11 @@ RimWellAllocationOverTimeCollection::RimWellAllocationOverTimeCollection(
QDateTime prevValidTimeStep;
for ( auto it = m_timeStepDates.rbegin(); it != m_timeStepDates.rend(); ++it )
{
const QDateTime& timeStep = *it;
if ( timeStepAndCalculatorPairs.contains( timeStep ) )
const QDateTime& timeStep = *it;
auto timeStepCalculatorIt = timeStepAndCalculatorPairs.find( timeStep );
if ( timeStepCalculatorIt != timeStepAndCalculatorPairs.end() )
{
m_timeStepAndCalculatorPairs.emplace( timeStep, timeStepAndCalculatorPairs.at( timeStep ) );
m_timeStepAndCalculatorPairs.emplace( timeStep, timeStepCalculatorIt->second );
prevValidTimeStep = timeStep;
}
else if ( prevValidTimeStep.isValid() )