1

在他的“Effective STL”中，Meyers 展示了如何正确清理指针向量（std::vector::clear只删除他的指针，而不是它们占用的内存）。

所以他在调用之前建议clear使用for_eachunary_function 调用对象析构函数：

template<typename T> struct DeleteMyObject2: public std::unary_function<const T*, void>
{
    void operator()(const T* ptr);
};

template<> struct DeleteMyObject2<algotest::ImageSelection>
{
   void operator()(const algotest::ImageSelection* ptr)
   {
       delete ptr;
       ptr=0; // this was added by me 
   }
};

void std_clearing_pointers()
{
   std::vector<ImageSelection*> vec;
   vec.reserve(5);
   for(int i=0; i<5; ++i)
       vec.insert(vec.begin(), new ImageSelection());

   std::for_each(vec.begin(), vec.end(), DeleteMyObject2<ImageSelection>());

   // HERE elements of vec are not NULL !!! 
   vec.clear();
}

在书中DeleteMyObject2被称为不带括号，它不编译（问题1 ：为什么？标准改变了？）：

std::for_each(vec.begin(), vec.end(), DeleteMyObject2<ImageSelection>);

无论如何，如果调用，它会编译operator()，DeleteMyObject2但vec.clear()在向量中的对象不是 NULL 之前。我想随着 STL 容器一直在复制它们的元素，ImageSelection指针是按值传递的，所以一切都很好（问题2 ：我正确吗？）。

我尝试通过 ref 传递指针，现在对象在 之后是 NULL for_each，我觉得更安全。 

template<> struct DeleteMyObject1<algotest::ImageSelection>
{
    void operator()(algotest::ImageSelection*& ptr)
    {
        delete ptr;
        ptr=0;
    }
 };

问题3 ：比没有不必要的分配DeleteMyObject2更可取吗？DeleteMyObject1

提前致谢。

4

1 回答 1

1

自从 Effective STL 出现以来，就有了更短、更易读的方法来做到这一点。例如，你现在可以写

vector<int *> a{new int{1}, new int{2}};
for_each(begin(a), end(a), [](int *p){delete p;});

where[](int *p){delete p;}是一个lambda 或匿名函数，它以比您问题中的任何一个类更短的代码对deleteany说。p

顺便说一句，您可能还想考虑一个vector智能指针（例如，vector<shared_ptr<int>>一个指向整数的指针向量）。用于释放资源的显式代码容易出错。

至于你的问题：

  1. 该类应该出现在括号中，用括号表示您想要这个类的（默认构造的）对象。该函数需要一个对象，而不是一个类。

  2. 可以按值传递指针（在这种情况下）。

于 2017-06-11T19:02:36.927 回答