Setzen Sie eine While-Schleife fort, bis EOF (Qt)

Question

Ich arbeite an einem Programm, um eine Liste aus einer großen QStringList zu erstellen. Im Grunde wird nach einer String-Übereinstimmung eine While-Schleife beginnen, die nächsten Strings zu einer Liste hinzuzufügen. Dieser Teil funktioniert gut. Das einzige Problem, das ich habe, ist, dass das Programm unerwartet beendet wird, weil ich nicht weiß, wie ich den EOF-Mechanismus hinzufügen sollte.

-Update mit einer detaillierteren Code

Sorry für die nicht mit genug Details über meinen Code bereitstellt. So sieht mein Code jetzt aus. Also, nach dem ersten Mal die Zeichenfolge "PACKAGE TYPE" erkannt wird, verwende ich die Funktion storeLines() Funktion, um die nächsten Strings in einer von drei Listen abhängig zu speichern. Und dies wird bis zum nächsten "PACKAGE TYPE" -Match oder EOF fortgesetzt. Die einzige Sache, die jetzt nicht richtig funktioniert, ist, wenn der Iterator auf der letzten Schnur der QStringList ist. Es erkennt irgendwie nicht, dass der nächste ist inputline.end()

void storeLines(QString department, QStringList::iterator current_line, QStringList::iterator endline){ 
while(QString::compare(*(current_line + 1),"PACKAGE TYPE") && (++current_line != endline)){ //this is not working 
    if(!QString::compare(department, "MDA")) mda_list.push_back(*current_line); 
    else if(!QString::compare(department, "SDA")) sda_list.push_back(*current_line); 
    else mix_list.push_back(*current_line); 
    } 
} 

void void MainWindow::on_pushButton_clicked(){ 
    QString input = ui->listinput->toPlainText().toLatin1(); 
    QStringList inputline = input.split("\n", QString::SkipEmptyParts); 

    for(QStringList::iterator pkg_header(inputline.begin()); pkg_header != inputline.end(); ++pkg_header){ 
     if(!QString::compare(*pkg_header,"PACKAGE TYPE")){ 
      ++pkg_header; 
      if(!QString::compare(*pkg_header,"Department-mda:")) storeLines("MDA", pkg_header, inputline.end()); 
      else if(!QString::compare(*pkg_header,"Department-sda:")) storeLines("SDA", pkg_header, inputline.end()); 
      else storeLines("MIX", pkg_header, inputline.end()); 
     } 
    } 
} 

Vielen Dank im Voraus!

Answer 1

Ihre offensichtliche Absicht besteht darin, die Zeichenfolge "PACKAGE TYPE" in der vorhandenen Liste zu suchen und dann die verbleibenden Elemente in der Liste in eine neue Liste zu kopieren.

Wenn das der Fall ist, warum nicht genau das tun?

for(QStringList::iterator current_line(inputline.begin()); current_line != inputline.end(); ++current_line){ 
    if(!QString::compare(*current_line,"PACKAGE TYPE"){ 
     list.insert(list.end(), ++current_line, inputline.end()); 
     break; 
    } 
} 

Answer 2

Sie schreiben einen Parser - es ist oft am einfachsten, es zu schreiben, wie Sie in der Regel einen schreiben würden, durch die Staaten explizit zu machen und in der Folge über jedes Element des Eingangsstroms laufen. Sie werden auf diese Weise keine Iterator-Fehler hintereinander machen.

Dieser Code entspricht der Absicht in Ihrer Frage und macht es offensichtlich, dass Sie einen Fall verpasst haben: Wenn Sie eine Abteilung erwarten, reagieren Sie nicht auf die PACKAGE TYPE, die vorhanden ist. Sie könnten einen Fehler signalisieren oder im Zustand DEPARTMENT bleiben, aber ich nehme an, Sie sollten damit umgehen und es nicht einfach ignorieren.

QStringList mda_list, sda_list, mix_list; 

void parse(const QString & input) { 
    enum { 
     TYPE, 
     DEPARTMENT, 
     ITEMS 
    } state = TYPE; 
    auto list = &mix_list; 
    auto const kPackageType = QStringLiteral("PACKAGE TYPE"); 

    for (auto const element : input.split("\n", QString::SkipEmptyParts)) { 
     switch (state) { 
     case TYPE: 
      if (element == kPackageType) 
       state = DEPARTMENT; 
      break; 
     case DEPARTMENT: 
      if (element == QStringLiteral("Department-mda:")) 
       list = &mda_list; 
      else if (element == QStringLiteral("Department-sda:")) 
       list = &sda_list; 
      state = ITEMS; 
      break; 
     case ITEMS: 
      if (element == kPackageType) 
       state = DEPARTMENT; 
      else 
       *list << element; 
      break; 
     } 
    } 
} 

Die Verwendung von QStringLiteral gibt Ihnen Kompilierung-gebauten String-Instanzen gegen vergleichen. Der Code würde genauso gut funktionieren, wenn Sie die QStringLiteral(...) Wrapper entfernen, allerdings wegen der unvermeidlichen vorzeitigen Pessimierung.