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

时间:2015-04-21 15:02:46

标签: c++ multithreading qt qprocess

每次调用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);
}

将导致AFTRER崩溃,所有文件都被删除。

我将我的recordThead作为std::unique_ptr<std::thread>运行。我确实尝试将线程作为QThread运行,但这给了我完全相同的结果。如果调用dir.removeRecursively(),则在finishedProcess()

中完成事件后程序将崩溃

在另一个事件循环中调用removeRecursively()有效。在我的例子中显示的SLOT中使用它时为什么不起作用?

1 个答案:

答案 0 :(得分:1)

vector erase有效地减少了容器大小,删除了被删除的元素数量。

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

你在一个'i'增加的循环中调用它?应该最终尝试擦除超出矢量大小的东西。如果可能的话,我会在循环完成或使用列表后释放整个容器。也许你不需要在这些容器中存储指针而是值(?)。存储指向您分配的对象的指针使您逐个释放,有时,是的,合理但使用C ++ 11并移动语义并不总是如此。