C ++当我将promises分配给线程时,“调试错误R6010 -abort()已被调用”

时间:2016-05-18 09:17:41

标签: c++ multithreading promise future

我收到错误:Debug Error R6010 -abort() has been called

在我的代码中:

bool ListFiles(wstring path, wstring mask, vector<wstring>& files) {
HANDLE hFind = INVALID_HANDLE_VALUE;
WIN32_FIND_DATA ffd;
wstring spec;
stack<wstring> directories;

directories.push(path);
files.clear();

while (!directories.empty()) {
    path = directories.top();
    spec = path + L"\\" + mask;
    directories.pop();

    hFind = FindFirstFile(spec.c_str(), &ffd);
    if (hFind == INVALID_HANDLE_VALUE)  {
        return false;
    }

    do {
        if (wcscmp(ffd.cFileName, L".") != 0 && wcscmp(ffd.cFileName, L"..") != 0) {
            if (ffd.dwFileAttributes & FILE_ATTRIBUTE_DIRECTORY) {
                directories.push(path + L"/" + ffd.cFileName);
            }
            else {
                files.push_back(path + L"/" + ffd.cFileName);
            }
        }
    } while (FindNextFile(hFind, &ffd) != 0);

    if (GetLastError() != ERROR_NO_MORE_FILES) {
        FindClose(hFind);
        return false;
    }

    FindClose(hFind);
    hFind = INVALID_HANDLE_VALUE;
}

return true;}

void findText(std::string filename, std::string word , promise<string> &prom) {
std::ifstream f(filename);
std::string s;
std::string notFound = "no";
bool found = false;
if (!f) {
    std::cout << "il file non esiste"<<std::endl;
}
while (f.good()) {
        std::getline(f, s);
        if (s.find(word, 0) != string::npos) {
            found = true;
        }

    }
if (found) {

    //cout << "Parola trovata in -> " << filename << endl;
    prom.set_value_at_thread_exit(filename);
}
else {
    prom.set_value_at_thread_exit(notFound);
}

f.close();}
int main(int argc, char* argv[]){
//vector<std::thread> th;
vector<future<string>> futures;
vector<wstring> files;
string search = "ciao";
string notFound = "no";
if (ListFiles(L"pds", L"*", files)) {

    for (vector<wstring>::iterator it = files.begin(); it != files.end(); ++it) {
        //wcout << it->c_str() << endl;
        wstring ws(it->c_str());
        string str(ws.begin(), ws.end());
        // Show String
        //cout << str << endl;
        //Creo una promise per ogni thread in cui andrò a cercare il risulato
        std::promise<string> prom;
        futures.push_back(prom.get_future());
        std::thread(findText,str,search,std::ref(prom)).detach();

    }
}   

for (int i = 0; i < futures.size(); i++){
    futures.at(i).wait();
    if (futures.at(i).get().compare(notFound)!=0)
    cout << "Parola trovata in ->" <<futures.at(i).get()<<endl;
}
return 0;}

我之前尝试过不使用promises,并且如果找到了单词并使其工作,则每个线程都会打印文件名。 所以我不知道为什么使用promises和future来回溯这个值导致我这个问题...... 我正在使用VS 2013

1 个答案:

答案 0 :(得分:4)

让我们仔细看看这些线条:

prom

首先创建一个局部变量prom,然后将该变量的引用传递给该线程。

这个问题是,一旦循环迭代,变量prom超出范围并且对象被破坏。您曾经拥有的参考文献不再具有它引用的任何内容。使用该引用将导致未定义的行为

因此解决方案是不使用引用(或指向std::thread(findText,str,search,std::move(prom)).detach(); 变量的指针),这会导致问题,因为std::promise无法复制。但是,它可以被移动:

void findText(std::string filename, std::string word , promise<string> &&prom) {
    ...
}

对此你可能需要让线程函数将promise参数作为右值引用:

void findText(std::string filename, std::string word , std::unique_ptr<std::promise<string>> prom) {
    ...
}

如果上述解决方案不起作用,那么您可以使用新的C ++ 11智能指针(如std::unique_ptr)进行动态分配。

然后线程函数应该按值获取智能指针,如

auto prom = std::make_unique<std::promise<std::string>>();
// Or if you don't have std::make_unique
//   std::unique_ptr<std::promise<std::string>> prom(new std::promise<std::string>);
futures.push_back(prom->get_future();
std::thread(findText,str,search,std::move(prom)).detach();

你创建了像

这样的线程
findText

请记住,在你的线程函数(prom)中,变量prom->set_value_at_thread_exit(filename); // ^^ // Note "arrow" operator here 指针,你需要在使用它时使用箭头运算符,例如

file-name.js.erb