Improve logging when parsing VATSIM METARs

refs #689
This commit is contained in:
Roland Winklmeier
2016-06-29 16:51:33 +02:00
parent 28ec2be1ca
commit a3fbbd7a67

View File

@@ -119,39 +119,25 @@ namespace BlackCore
return;
}
QString invalidMetars;
int invalidLineCount = 0;
CMetarList metars;
int invalidLines = 0;
QTextStream lineReader(&metarData);
while (!lineReader.atEnd())
{
if (this->isAbandoned()) { return; }
QString line = lineReader.readLine();
CMetar metar = m_metarDecoder.decode(line);
if (metar != CMetar())
{
metars.push_back(metar);
}
else
{
invalidMetars += line;
invalidMetars += "\n";
invalidLineCount++;
}
if (metar != CMetar()) { metars.push_back(metar); }
else { invalidLines++; }
}
CLogMessage(this).debug() << "METAR statistic: " << metars.size() << "Metars ( invalid:" << invalidLines << ")";
{
QWriteLocker l(&m_lock);
m_metars = metars;
}
// I could use those for logging, etc.
Q_UNUSED(invalidMetars);
if (invalidLineCount > 0)
{
// Regular issue, log it, but do not show to user
CLogMessage(this).debug() << "Reading METARs failed for entries" << invalidLineCount;
}
emit metarsRead(metars);
emit dataRead(CEntityFlags::MetarEntity, CEntityFlags::ReadFinished, metars.size());
}