指针向量中的for_each指针

Question

我想在for_each()函数中初始化指针向量：

#include <stdlib.h>
#include <vector>
#include <iostream>
#include <algorithm>
using namespace std;

class Cow{
        public:
                Cow(){ _age = rand()% 20; }
                int get_age() { return _age;}
        private:
                int _age;
};

void add_new(Cow* cowp)
{
        cowp = new Cow;
}

int main()
{
        srand(time(NULL));
        const int herd_size=10;
        vector<Cow*> herd(herd_size);
        for_each(herd.begin(), herd.end(),add_new);
        cout << "Age: " << herd[0]->get_age() << endl; // line 27
}

但是，我在第27行得到一个运行时“分段错误”错误。牛群矢量似乎还没有被初始化。为什么？

Answer 1

您的函数按值接受指针，然后重新分配这些副本。为了影响向量中的指针，您需要引用它们。

void add_new(Cow *& cowp) 

Answer 2

void add_new(Cow* cowp)
{
        cowp = new Cow;
}

您只是在修改Cow指针的本地版本。请记住，一个局部变量在离开它的函数时会被销毁，在这里，您的指针被作为值传递！

您希望修改驻留在向量中的指针，因此需要引用：

void add_new(Cow *& cowp)
{
  cowp = new Cow;
}

这样，您就可以有效地修改驻留在向量中的原始指针。

Answer 3

因为这个std::generate是一个更好的匹配，因为它应该用于生成一个序列。

另一个重要的更改是，您的函数必须引用指针：

void add_new(Cow*& cowp)

实际上，您实际上不需要使用std::for_each甚至std::transform，也不需要使用单独的创建函数，这要感谢新的C++11功能：

for (auto& cow : herd)
    cow = new Cow;

实际上，我建议您不要使用指针，这样就不需要其他初始化了。