3

我有一个名为 Domain 的类，它有一个std::vector名为 itsVec. 该向量可通过 访问getVec()。

当main()我直接通过 getVec()[i]. 但是，当我尝试使用迭代器时，指向第一个元素的指针似乎没有正确初始化。itsVec奇怪的是，在类构造函数中使用迭代器就像一个魅力。

编码

#include<iostream>
#include <vector>

class Domain
{
  public:
    /* ====================  LIFECYCLE     ======================================= */
    Domain  ( ) {};                                    /* constructor        */
    Domain  ( unsigned int , const unsigned int * ) ;  /* custom constructor */
    Domain  ( const Domain &other );                   /* copy constructor   */
    ~Domain ( ) {};                                    /* destructor         */

    /* ====================  ACCESSORS     ======================================= */
    unsigned int              getDim() const {return itsDim;}
    std::vector<unsigned int> getVec() const {return itsVec;}

  protected:
    /* ====================  DATA MEMBERS  ======================================= */
    unsigned int              itsDim;
    std::vector<unsigned int> itsVec;

}; /* -----  end of class Domain  ----- */


Domain::Domain( unsigned int dim, const unsigned int  *externVec)
{
  itsDim = dim;

  for ( size_t j = 0; j < dim ; j++ )
    itsVec.push_back(externVec[j]);

  std::vector<unsigned int>::const_iterator i_vec  = itsVec.begin();

// iterator access works
  for ( ; i_vec != itsVec.end(); i_vec++) 
    std::cout<<"Iterator in constructor: " << (*i_vec) << std::endl;

  std::cout<<std::endl;

} /* -----  end of constructor  ----- */


/*-----------------------------------------------------------------------------
 *  Main
 *-----------------------------------------------------------------------------*/
int main ()
{
  const unsigned int kVec[3] = { 1, 2, 3 };

  Domain k(3,kVec);

  size_t i = 0;

// direct access works
  for( ; i<3 ; i++)
    std::cout << "Vec direct: " << k.getVec()[i] << std::endl;

  std::cout << std::endl;

// iterator access FAILS for the first element
  std::vector<unsigned int>::const_iterator  it_vec  = k.getVec().begin();

  for ( ; it_vec != k.getVec().end(); it_vec++) 
    std::cout<<"Vec iterator " << (*it_vec) << std::endl;

  return 0;
}       /* ----------  end of function main  ---------- */

编译g++ main.cc -o test结果为

$ ./test

Iterator in constructor: 1
Iterator in constructor: 2
Iterator in constructor: 3

Vec direct: 1
Vec direct: 2
Vec direct: 3

Vec iterator 140124160
Vec iterator 2
Vec iterator 3

在另一台机器上测试给出：

Vec iterator 0
Vec iterator 0
Vec iterator 3

是我通过getVec()错误返回向量的方式。迭代器是否处理不正确？我不知道，这里有什么问题或如何解决它。

非常感谢任何帮助，

大弗兰克

4

3 回答 3

2

您正在返回向量的副本而不是引用，因此您没有正确匹配迭代器。您从中获得的迭代器来自begin()您的内部向量的不同副本，与您从中获得的那个不同end()

您可以通过将函数原型更改为以下方式来返回引用，而不是返回内部向量的副本：

const std::vector<unsigned int>& getVec() const {return itsVec;}
于 2012-08-27T12:05:38.060 回答
1

你的getVec()方法按值返回向量，这意味着每次调用getVec()你都会得到一个新的向量副本，所以当你迭代时，你的begin()和end()迭代器是不兼容的。它们属于两个不同的向量。

您可以通过返回对向量的引用来解决此问题：

const std::vector<unsigned int>& getVec() const {return itsVec;}

或扩展您的类以返回向量的开始和结束迭代器。

class Domain
{
 public:
  std::vector<unsigned int>::iterator begin() { return itsVec.begin(); }
  std::vector<unsigned int>::iterator end()  { return itsVec.end(); }
  std::vector<unsigned int>::const_iterator begin() const  { return itsVec.begin(); }
  std::vector<unsigned int>::const_iterator end() const  { return itsVec.end(); }
  std::vector<unsigned int>::const_iterator cbegin() const  { return itsVec.begin(); }
  std::vector<unsigned int>::const_iterator cend() const  { return itsVec.end(); }
 // ... other methods
};
于 2012-08-27T12:05:16.123 回答
1

问题是您正在返回向量的副本，该向量立即被丢弃，给您留下一个不指向任何地方的迭代器。

您可能想要返回一个引用或一个 const 引用。

于 2012-08-27T12:05:44.347 回答