for Qstrings and QByteArrays prefer isEmpty() over size() and length() (#1409)

* use QString::isEmpty()

* use QByteArray::isEmpty()

Author: tsteven4

arcdist.cc

@@ -132,7 +132,7 @@ void ArcDistanceFilter::process()
       arcpt2->latitude = arcpt2->longitude = BADVAL;
       int argsfound = sscanf(CSTR(line), "%lf %lf", &arcpt2->latitude, &arcpt2->longitude);
 
-      if ((argsfound != 2) && (line.trimmed().size() > 0)) {
+      if ((argsfound != 2) && (!line.trimmed().isEmpty())) {
         gbWarning("Warning: Arc file contains unusable vertex on line %d.\n", fileline);
       } else {
         Waypoint* arcpttmp = arcpt1;

csv_util.cc

@@ -102,7 +102,7 @@ QString
 csv_enquote(const QString& str, const QString& enclosure)
 {
   QString retval = str;
-  if (enclosure.size() > 0) {
+  if (!enclosure.isEmpty()) {
     retval = enclosure + retval.replace(enclosure, enclosure + enclosure) + enclosure;
   }
   return retval;

exif.cc

@@ -215,7 +215,7 @@ ExifFormat::exif_read_str(ExifTag* tag)
   if (auto idx = buf.indexOf('\0'); idx >= 0) {
     buf = buf.left(idx);
   }
-  while ((buf.size() > 0) && isspace(buf.back())) {
+  while ((!buf.isEmpty()) && isspace(buf.back())) {
     buf.chop(1);
   }
   return buf;

googletakeout.cc

@@ -59,13 +59,13 @@ Waypoint* GoogleTakeoutFormat::takeout_waypoint(
   auto* waypoint = new Waypoint();
   waypoint->latitude = lat_e7 / 1e7;
   waypoint->longitude = lon_e7 / 1e7;
-  if (shortname && shortname->length() > 0) {
+  if (shortname && !shortname->isEmpty()) {
     waypoint->shortname = *shortname;
   }
-  if (description && description->length() > 0) {
+  if (description && !description->isEmpty()) {
     waypoint->description = *description;
   }
-  if (start_str && start_str->length() > 0) {
+  if (start_str && !start_str->isEmpty()) {
     gpsbabel::DateTime start = QDateTime::fromString(*start_str, Qt::ISODate);
     waypoint->SetCreationTime(start);
   }
@@ -286,12 +286,12 @@ GoogleTakeoutFormat::add_place_visit(const QJsonObject& placeVisit)
   const QString timestamp = placeVisit[DURATION][START_TIMESTAMP].toString();
   Waypoint* waypoint;
 
-  if (loc.contains(NAME) && loc[NAME].toString().length() > 0) {
+  if (loc.contains(NAME) && !loc[NAME].toString().isEmpty()) {
     QString name = loc[NAME].toString();
     waypoint = takeout_waypoint(
       loc[LOCATION_LATE7].toInt(),
       loc[LOCATION_LONE7].toInt(),
-      name.length() > 0 ? &name : nullptr,
+      !name.isEmpty() ? &name : nullptr,
       &address,
       &timestamp
     );

gui/filterdata.cc

@@ -143,7 +143,7 @@ QStringList TrackFilterData::makeOptionString() const
     s += QString(",title=%1").arg(titleString);
   }
 
-  if (s.length() != 0) {
+  if (!s.isEmpty()) {
     args << "-x" << "track" + s;
   }
 

gui/formatload.cc

@@ -87,19 +87,19 @@ bool FormatLoad::processFormat(Format& format)
     } else if (optionType == "string") {
       type = FormatOption::OPTstring;
     } else if (optionType == "integer") {
-      type = (optionMax != "" && optionMin != "") ? FormatOption::OPTboundedInt : FormatOption::OPTint;
-      if (optionMax == "") {
+      type = (!optionMax.isEmpty() && !optionMin.isEmpty()) ? FormatOption::OPTboundedInt : FormatOption::OPTint;
+      if (optionMax.isEmpty()) {
         optionMax = "2147483647";
       }
-      if (optionMin == "") {
+      if (optionMin.isEmpty()) {
         optionMin = "-2147483647";
       }
     } else if (optionType == "float") {
       type = FormatOption::OPTfloat;
-      if (optionMax == "") {
+      if (optionMax.isEmpty()) {
         optionMax = "1.0E308";
       }
-      if (optionMin == "") {
+      if (optionMin.isEmpty()) {
         optionMin = "-1.0E308";
       }
     } else if (optionType == "file") {
@@ -125,7 +125,7 @@ bool FormatLoad::processFormat(Format& format)
                   optionList,
                   optionList2, htmlPage);
 #ifndef GENERATE_CORE_STRINGS
-  if (htmlPage.length() > 0 && Format::getHtmlBase().length() == 0) {
+  if (!htmlPage.isEmpty() && Format::getHtmlBase().isEmpty()) {
     QString base = htmlPage;
     static const QRegularExpression re("/[^/]+$");
     base.replace(re, "/");

gui/gmapdlg.cc

@@ -82,10 +82,10 @@ void GMapDialog::appendWaypointInfo(QStandardItem* it, const GpxWaypoint& wpt)
 {
   it->appendRow(new StandardItem(tr("Lat: %1").arg(wpt.getLocation().lat(), 0, 'f', 7)));
   it->appendRow(new StandardItem(tr("Lng: %1").arg(wpt.getLocation().lng(), 0, 'f', 7)));
-  if (wpt.getDescription() != QString()) {
+  if (!wpt.getDescription().isEmpty()) {
     it->appendRow(new StandardItem(tr("Desc: %1").arg(wpt.getDescription())));
   }
-  if (wpt.getComment() != QString() && wpt.getComment() != wpt.getDescription()) {
+  if (!wpt.getComment().isEmpty() && wpt.getComment() != wpt.getDescription()) {
     it->appendRow(new StandardItem(tr("Cmt: %1").arg(wpt.getComment())));
   }
   if (wpt.getElevation() > -50000) {

gui/mainwindow.cc

@@ -142,7 +142,7 @@ static QString MakeOptions(const QList<FormatOption>& options)
 static QString MakeOptionsNoLeadingComma(const QList<FormatOption>& options)
 {
   QString str = MakeOptions(options);
-  return (str.length()) != 0 ? str.mid(1) : str;
+  return !str.isEmpty() ? str.mid(1) : str;
 
 }
 
@@ -461,7 +461,7 @@ QString MainWindow::filterForFormat(int idx)
 QString MainWindow::ensureExtensionPresent(const QString& name, int idx)
 {
   QString outname = name;
-  if (QFileInfo(name).suffix().length() == 0) {
+  if (QFileInfo(name).suffix().isEmpty()) {
     QStringList extensions = formatList_[idx].getExtensions();
     if (!extensions.empty() && !extensions[0].isEmpty()) {
       outname += "." + extensions[0];
@@ -524,7 +524,7 @@ void MainWindow::browseInputFile()
 void MainWindow::browseOutputFile()
 {
   int idx = currentComboFormatIndex(ui_.outputFormatCombo);
-  QString startFile = babelData_.outputFileName_.length() == 0 ? babelData_.outputBrowse_ : babelData_.outputFileName_;
+  QString startFile = babelData_.outputFileName_.isEmpty() ? babelData_.outputBrowse_ : babelData_.outputFileName_;
   QFileInfo finfo(startFile);
   if (!finfo.isDir() && (!filterForFormatIncludes(idx, finfo.suffix()))) {
     startFile = finfo.dir().absolutePath();
@@ -534,7 +534,7 @@ void MainWindow::browseOutputFile()
     QFileDialog::getSaveFileName(nullptr, tr("Output File Name"),
                                  startFile,
                                  filterForFormat(idx));
-  if (str.length() != 0) {
+  if (!str.isEmpty()) {
     str = ensureExtensionPresent(str, idx);
     babelData_.outputBrowse_ = str;
     babelData_.outputFileName_ = str;
@@ -805,7 +805,7 @@ bool MainWindow::isOkToGo()
     babelData_.inputFileNames_ << ui_.inputFileNameText->text();
   }
   if ((babelData_.outputType_ == BabelData::fileType_) &&
-      (babelData_.outputFileName_.size() == 0) &&
+      (babelData_.outputFileName_.isEmpty()) &&
       (!ui_.outputFileNameText->text().isEmpty())) {
     babelData_.outputFileName_ = ui_.outputFileNameText->text();
   }
@@ -825,7 +825,7 @@ bool MainWindow::isOkToGo()
     return false;
   }
   if (babelData_.outputType_ == BabelData::fileType_ &&
-      babelData_.outputFileName_.length() == 0) {
+      babelData_.outputFileName_.isEmpty()) {
     QMessageBox::information(nullptr, QString(appName), tr("No output file specified"));
     return false;
   }
@@ -920,7 +920,7 @@ void MainWindow::applyActionX()
 
     // output file or device option
     if (outIsFile) {
-      if (babelData_.outputFileName_ != "") {
+      if (!babelData_.outputFileName_.isEmpty()) {
         args << "-F" << babelData_.outputFileName_;
       }
     } else if (babelData_.outputType_ == BabelData::deviceType_) {

gui/optionsdlg.cc

@@ -60,7 +60,7 @@ FileDlgManager::FileDlgManager(QObject* parent,
 //------------------------------------------------------------------------
 QVariant getOptionValue(QList<FormatOption> opts, int k)
 {
-  if (opts[k].getValue().toString() != "") {
+  if (!opts[k].getValue().toString().isEmpty()) {
     return opts[k].getValue();
   }
   return opts[k].getDefaultValue();
@@ -80,7 +80,7 @@ void FileDlgManager::buttonClicked()
                                        le->text(),
                                        "All Files (*.*)");
   }
-  if (str != "") {
+  if (!str.isEmpty()) {
     le->setText(str);
   }
 }

gui/upgrade.cc

@@ -323,7 +323,7 @@ void UpgradeCheck::httpRequestFinished(QNetworkReply* reply)
     }
   }
 
-  if (response.length() != 0) {
+  if (!response.isEmpty()) {
     QMessageBox information;
     information.setWindowTitle(tr("Upgrade"));