C++注意的小细节 (二) 不要在vector内部数据上操纵vector

比如，我有一段代码如下：

class cls_vector_item
{
private:
vector<int> data;
public:
cls_vector_item(int a_data_num):data(a_data_num)
{
do_something();
}

void do_it(vector<cls_vector_item>& vct)
{
for(int cnt=0;cnt<data.size();cnt++)
{
cls_vector_item cvi(10);
vct.push_back(cvi);
}
}
}
int main(int argc,char**argv)
{
vector<cls_vector_item> vct(100);
do_something()...

for(int cnt=0;cnt<vct.size();cnt++)
{
vct[cnt].do_it(&vct);
}
}

这段代码，表面生没有什么问题。

当vct.push_back(cvi);时，当vct内部空间不足的时候，将会重新申请一段空间，这个对象就被移走了！

但是，注意，这个对象本来应该移走，但我们现在所在这个对象的内部的这个对象并么有移走，反而会变成垃圾，也就是在vct申请空间时，旧空间被析构，但逻辑上我们的对象仍然是垃圾对象，所以便悲剧了。

所以，以后再使用vct时一定要小心，小心被析构。

解决办法有两种：

（1）给定vector一定的空间，让其有足够的空间。

（2）这种设计模式本身就是有问题的，我们可以调整结构，比如在别处写一个函数do_it(int pos,vector<...> vct);