1

我正在动态创建线程：

auto thr =new std::future<void>(std::async(/*some callable*/));

我将所有这些指针存储在一个std::vector<future<void>*>. 为了释放内存，我正在这样做：

for(int i(0);i<futureVector.size();i++)
{
    if(futureVector.at(i)->valid())
    {
        futureVector.at(i)->get();  // for getting exception,if any
    }
    delete futureVector.at(i);    // the problem is here
}

现在在我的代码中，分配给的内存可能futureVector.at(i)已经被释放（可能在其他线程中，可能由其他函数）。我的问题是如何检测指针futureVector.at(i)是否有效？我的意思是它指向有效std::future与否？

注意：futureVector变量是我的类的静态成员。

假设我不删除该future对象的成本非常大（已经检索到未来）

4

1 回答 1

9

如果由于某种原因您确实需要指针，则不应在向量中使用拥有原始指针。

auto thr = std::make_unique<std::future<void>>(std::async(/*some callable*/)); // C++14

std::unique_ptr<std::future<void>> thr {new auto {std::async(/*some callable*/)}; // C++11

但实际上你可能根本不需要指针。仅仅因为您不知道要创建多少线程并不意味着您需要使用指针。

std::vector<std::future<void>> futureVector;
futureVector.emplace_back(std::async(/*some callable*/));

无论哪种方式，您都不需要手动循环并删除任何内容。

（可能在其他线程中，可能在其他函数中）

如果您仍然使用原始指针，那么您确实需要比这更好地定义所有权策略。允许一些随机的其他代码删除它不拥有的资源不是一个好主意。

于 2013-09-12T14:56:59.767 回答