继续while循环直到EOF (Qt)

Continue a while loop until EOF (Qt)

本文关键字：Qt EOF while 循环 继续      更新时间：2023-10-16

我正在编写一个程序，从一个大的QStringList创建一个列表。基本上在字符串匹配之后，while循环会开始将下一个字符串添加到列表中。这部分工作得很好。我遇到的唯一问题是程序意外退出，因为我不知道应该如何添加EOF机制。

用更详细的代码更新

很抱歉没有提供关于我的代码的足够细节。这是我的代码现在的样子。因此，在第一次检测到字符串"PACKAGE TYPE"之后，我使用函数storeLines()函数将下一个字符串存储到三个相关列表中的一个中。这将持续到下一个"PACKAGE TYPE"匹配或EOF。现在唯一不能正常工作的是当迭代器在QStringList的最后一个字符串上时。它无法检测到下一个是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());
       }
    }
}

提前感谢!

您正在编写一个解析器—通常最简单的方法是像通常那样编写解析器，通过显式地编写状态并按顺序遍历输入流的每个元素。这样就不会产生任何迭代器误差

这段代码与你的问题中的意图相匹配，并且很明显你错过了一个案例:当你期待一个部门时，你没有对PACKAGE TYPE的存在做出反应。您可以发出错误信号，或者保持在DEPARTMENT状态，但我认为您应该处理它，而不是忽略它。

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;
        }
    }
}

使用QStringLiteral为您提供了编译时构建的字符串实例来进行比较。如果您删除QStringLiteral(...)包装器，代码将工作得一样好，但代价是不可避免的过早悲观。

您的意图显然是在现有列表中找到字符串"PACKAGE TYPE"，然后将列表中剩余的元素复制到新列表中。

如果是这样的话，为什么不这样做呢?

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;
    }
}
相关文章：