指针向量的迭代器

Question

我阅读了另一篇文章，该文章回答了有关指针向量迭代器的问题。我试图在我的代码中使用相同的概念，但我收到了一些编译错误。我的代码基于的代码示例是：

vector<c*> cvect;
cvect.push_back(new sc);
vector<c*>::iterator citer;
for(citer=cvect.begin(); citer != cvect.end(); citer++) {
    (*citer)->func();
}

我想使用类似的概念为具有两个数据成员的类创建一个深拷贝构造函数，这些数据成员是指向对象的指针向量。我的代码是这样的：

class MyContainer {
    vector<MyStuff*> vecOne;
    vector<MyStuff*> vecTwo;

 public:
    MyContainer(const MyContainer& other);
};

MyContainer::MyContainer(const MyContainer& other) {
    // copy vector one
    vector<MyStuff*>::iterator vec1_itr;
    for (vec1_itr = other.vecOne.begin(); vec1_itr != other.vecOne.end(); vec1_itr++) {
        vecOne.push_back(new MyStuff(vec1_itr));
    }

    // copy vector two
    vector<MyStuff*>::iterator vec2_itr;
    for (vec2_itr = other.vecTwo.begin(); vec2_itr != other.vecTwo.end(); vec2_itr++) {
        vecTwo.push_back(new MyStuff(vec2_itr));
    }
}

我收到一些编译错误，例如：

/path/MyContainer.cpp:38: 错误：'vec1_Itr = other->MyContainer::vecOne. std::vector<_Tp, _Alloc>::begin [with _Tp = MyStuff*, _Alloc = std::allocator<MyStuff*>]()'中的'operator='不匹配

候选人是：__gnu_cxx::__normal_iterator<MyStuff*, std::vector<MyStuff, std::allocator<MyStuff> > >& __gnu_cxx::__normal_iterator<MyStuff*, std::vector<MyStuff, std::allocator<MyStuff> > >::operator=(const __gnu_cxx::__normal_iterator<MyStuff*, std::vector<MyStuff, std::allocator<MyStuff> > >&)

我也收到operator!= 的错误...而另一个向量的另一组相同的错误。

Answer 1

您忘记取消引用迭代器。试试这个：

vecOne.push_back(new MyStuff( **vec1_itr ));

编辑 0：

是的，应该是双重取消引用（上面已修复）。它应该是const_terator，因为您正在处理包含对象的const：

vector<MyStuff*>::const_iterator vec1_itr;

Answer 2

不要将参数作为const 或将vec1_itr 声明为const_iterator。这里的问题是vecOne.begin() 返回一个const_iterator，因为容器是const。如果要更改容器，则必须删除 const 限定符。

附带说明，如果持有一个指针容器意味着您需要管理容器中的指针并且您有两个这样的容器，那么您应该将容器移动到它自己的类中。尽量避免在一个类中管理多个资源。

Answer 3

！！内存泄漏警报！！

您的代码是泄漏的。

复制构造函数 (std::bad_alloc?) 中抛出的任何异常都将导致内存泄漏，因为传递给 vector 的内存将永远不会被清理（不会调用析构函数，因为对象从未首先构建）。

当然，您可以添加所需的 try/catch，但我警告您，代码很快就会变得笨拙（您需要几个）。

这是违反资源管理规则1的直接结果：

一个对象应该管理最多一个资源，在这种情况下它不应该做任何其他事情。

这意味着如果您的对象是一个业务对象（内部包含应用程序逻辑），那么它不应该直接处理资源管理，而是使用已经存在的管理器。

在您的情况下，您有两种解决方案：

1. 推荐：由于这里没有使用多态，所以不要使用指针。 std::vector<MyStuff> 很好
2. 如果您需要多态性，但没有包含在这个玩具示例中，请使用boost::ptr_vector<MyStuff>

加分项：他们两个定义了合理的拷贝构造函数、赋值操作符和析构函数，这样你就不用自己重写了。

编辑：

正如@David 所说，如果您需要多态，则不能使用复制构造，因此需要：

• clone 方法或等效方法
• 指针和动态内存分配

boost::ptr_vector 提供您所需的一切（复制时自动使用clone 方法）。