flightsimulatorplatfrom/pilotclient
1
0
Fork 0
mirror of https://github.com/swift-project/pilotclient.git synced 2026-04-30 14:15:35 +08:00
Code Issues Packages Projects Releases Wiki Activity

Fixed div. by 0 ("zero") warning

Browse Source
This commit is contained in:
Klaus Basan
2018-05-26 21:09:56 +02:00
parent 8a522b5e6d
commit 65ad35bfab
1 changed files with 8 additions and 6 deletions
Download Patch File Download Diff File Expand all files Collapse all files

14
src/blackmisc/geo/geoobjectlist.cpp
View File

@@ -100,7 +100,8 @@ namespace BlackMisc
typename IGeoObjectList<OBJ, CONTAINER>::MinMaxAverageHeight IGeoObjectList<OBJ, CONTAINER>::findMinMaxAverageHeight() const typename IGeoObjectList<OBJ, CONTAINER>::MinMaxAverageHeight IGeoObjectList<OBJ, CONTAINER>::findMinMaxAverageHeight() const
{ {
MinMaxAverageHeight stats{ CAltitude::null(), CAltitude::null(), CAltitude::null(), 0 }; MinMaxAverageHeight stats{ CAltitude::null(), CAltitude::null(), CAltitude::null(), 0 };
int c = 0; if (this->container().isEmpty()) { return stats; } // avoid div by zero
int count = 0;
double avgFt = 0; double avgFt = 0;
for (const OBJ &obj : this->container()) for (const OBJ &obj : this->container())
{ {
@@ -108,17 +109,18 @@ namespace BlackMisc
const CAltitude alt = obj.geodeticHeight(); const CAltitude alt = obj.geodeticHeight();
if (std::get<0>(stats).isNull() || std::get<0>(stats) > alt) if (std::get<0>(stats).isNull() || std::get<0>(stats) > alt)
{ {
std::get<0>(stats) = alt; std::get<0>(stats) = alt; // min.
} }
if (std::get<1>(stats).isNull() || std::get<1>(stats) < alt) if (std::get<1>(stats).isNull() || std::get<1>(stats) < alt)
{ {
std::get<1>(stats) = alt; std::get<1>(stats) = alt; //max.
} }
avgFt += alt.value(CLengthUnit::ft()); avgFt += alt.value(CLengthUnit::ft()); // add up
count++;
} }
std::get<2>(stats) = CAltitude(avgFt / c, CAltitude::MeanSeaLevel, CLengthUnit::ft()); if (count > 0) { std::get<2>(stats) = CAltitude(avgFt / count, CAltitude::MeanSeaLevel, CLengthUnit::ft()); }
std::get<3>(stats) = c; std::get<3>(stats) = count;
return stats; return stats;
} }