c++ - QDir::remove() 在特定 SLOT 中调用时总是导致崩溃

Question

每次我调用QDir::removeRecursively()我的应用程序在正确删除包含文件的文件夹后崩溃。 做了一些测试后，我发现这取决于我如何调用该函数。这是我的代码:

Recorder::Recorder(ParentClass *parent): QObject(parent){

     connect(this,SIGNAL(finishedRec(QString,QString)),this,SLOT(finishedRecording(QString,QString)));
}

void Recorder::start(){
    if (!recording){
        recording=true;
        recThread.reset(new std::thread(&Recorder::recordThread, this));
    }
}

void Recorder::stop(){
    recording = false;
    recThread->join(); recThread.reset();
}
void Recorder::recordThread(){
    QString picDir;
    QString filename;
    //....
    while(recording){
        //writing frames to folder
    }    
    emit finishedRec(picDir,filename);    
}

void Recorder::finishedRecording(QString picDir, QString filename){
    QProcess* proc = new QProcess();
    vecProcess.push_back(proc);
    vecString.push_back(picDir);
    proc->start("C:\\ffmpeg.exe", QStringList() <<"-i"<< picDir << "-r"<< "30" << "-vcodec"<< "ffv1" << filename);
    connect(proc,SIGNAL(finished(int)),this,SLOT(finishedProcess()));
}

void Recorder::finishedProcess(){
    for (int i=0; i<vecProcess.size();i++){
        if(vecProcess.at(i)->state()==QProcess::NotRunning){
            delete vecProcess.at(i);
            vecProcess.erase(vecProcess.begin() + i);

            QString folderToRemove=vecString.at(i);
            folderToRemove.chop(12);
            qDebug() << folderToRemove;
            QDir dir(folderToRemove);
            dir.removeRecursively();
            vecString.erase(vecString.begin() + i);

        }
    }
}

只有当我保留 dir.removeRecursively() 时，我的应用程序才会始终崩溃。没有它一切都按预期工作。甚至用

删除所有文件

QDir dir(path);
dir.setNameFilters(QStringList() << "*.*");
dir.setFilter(QDir::Files);
foreach(QString dirFile, dir.entryList()){
    dir.remove(dirFile);
}

将在删除所有文件后导致崩溃。

我将我的 recordThead 作为 std::unique_ptr<std::thread> 运行.我确实尝试将线程作为 QThread 运行，但这给了我完全相同的结果。如果dir.removeRecursively()被调用程序将在完成 finishedProcess() 中的事件后崩溃

调用 removeRecursively()在不同的事件循环中工作。为什么在我的示例中所示的 SLOT 中使用它时它不起作用？

Answer 1

vector erase通过删除的元素数量有效地减小容器大小，这些元素被销毁。

// one suspect
vecProcess.erase(vecProcess.begin() + i);
// another suspect
vecString.erase(vecString.begin() + i);

然后您在循环中调用它，其中“i”递增？最终应该尝试删除超出 vector 大小的内容。如果可能的话，我会在循环完成或使用列表后释放整个容器。也许您不需要在这些容器中存储指针，而是存储值(？)。存储指向由您分配的对象的指针会让您一个一个地释放，有时，是的，这是合理的，但对于 C++ 11 和移动语义，情况并非总是如此。