这个delete操作错在哪？（应该不难）

[南京短暂的春天]

t_point是一个类
vector <t_point> data;
for(int j=0;i <8;i++)
{
t_point *pi=new t_point(XX,XX,XX);
data.push_back(*pi);
}
之前用这样的方式操作了8次

现在在释放内存。

for(int i=(BST->data.size())-1;i>=0;i--)
{
delete &(BST->data[i]);
}
编译不报错。运行到delete &(BST->data[i]); 就出问题了。

我把释放内存的代码换成下面


vector <t_point>::iterator ite;
for(ite=data.begin();ite!=data.end();++ite)
delete ite;

delete操作只能在for循环里执行一次。第二次就出错了。

[mengde007]

貌似不能这样吧；
vector <t_point>::iterator ite，temp;
for(ite=data.begin();ite!=data.end();++ite)
{
temp=ite;
delete temp;
}

[goodname]

你的vector里面存放的是对象而不是指针，所以无需delete

程序可以简化一下

for(int j=0;i <8;i++)
{
data.push_back(t_point(XX,XX,XX));
}
对于你的原程序，应该push_back后,应该delete pi;，我认为

[qqwx_1986]

vector <t_point*> data; 
for(int j=0;i <8;i++) 
{ 
   t_point *pi=new t_point(XX,XX,XX); 
   data.push_back(pi); 
} 
...
for(int i=(BST->data.size())-1;i>=0;i--) 
{ 
   delete (BST->data[i]); 
} 

你觉得你vector里面存的是个什么东西

[南京短暂的春天]

[Quote=引用 2 楼 goodname 的回复:]
你的vector里面存放的是对象而不是指针，所以无需delete

程序可以简化一下

for(int j=0;i <8;i++)
{
data.push_back(t_point(XX,XX,XX));
}
对于你的原程序，应该push_back后,应该delete pi;，我认为
[/Quote]
push_back后,delete pi

那我存在data里的数据是不是也没了？

[goodname]

data里面存放的是拷贝，是由vector自己维护的。

比如

void f(vector<int> &v)
{
int a=0;
v.push_back(a);
}
这样，v只是存放的a的一个拷贝的值,是个复制品，当函数执行完毕后，你是否认为a是个局部变量
而导致v中存放的是无效的值啊？

[南京短暂的春天]

[Quote=引用 3 楼 qqwx_1986 的回复:]
C/C++ code
vector <t_point*> data;
for(int j=0;i <8;i++)
{
t_point *pi=new t_point(XX,XX,XX);
data.push_back(pi);
}
...
for(int i=(BST->data.size())-1;i>=0;i--)
{
delete (BST->data[i]);
}



你觉得你vector里面存的是个什么东西
[/Quote]

存的是一些类的对象，不是指针。对delete的操作真的不熟悉。以前做东西几乎没用过。

[qqwx_1986]

不会没有，push_back会把你的数据拷贝进去了

[lingyin55]

应该不会的，push_back(*pi); 之后已经把数据保存到容器里了，
这时delete的只是动态申请的那个空间。



[Quote=引用 4 楼 dcmilan 的回复:]
引用 2 楼 goodname 的回复:
你的vector里面存放的是对象而不是指针，所以无需delete

程序可以简化一下

for(int j=0;i <8;i++)
{
data.push_back(t_point(XX,XX,XX));
}
对于你的原程序，应该push_back后,应该delete pi;，我认为

push_back后,delete pi

那我存在data里的数据是不是也没了？
[/Quote]

[pengzhixi]

for(int j=0;i <8;i++)
{
t_point *pi=new t_point(XX,XX,XX);
data.push_back(*pi);
delete pi;//直接加到这就 OK了。因为data里面存放的 是副本
}

[mabo321]

        vector <t_point> data; 
        for(int j=0;i <8;i++) 
        { 
          t_point pi=t_point(XX,XX,XX); // 为什么非得new ?  
          data.push_back(pi);     // vector 中有默认的 alloc 
        } 

[南京短暂的春天]

[Quote=引用 9 楼 pengzhixi 的回复:]
for(int j=0;i <8;i++)
{
t_point *pi=new t_point(XX,XX,XX);
data.push_back(*pi);
delete pi;//直接加到这就 OK了。因为data里面存放的 是副本
}
[/Quote]

我刚写了个测试程序，确实是个副本。之前是我理解错误。不好意思了。太丢人了。现在完全懂了。

分是怎么给的啊？

[pengzhixi]

我的帖子 给分 结贴

[lpf000]

UP

[windsting]

//t_point是一个类 
        vector <t_point> data; 
        for(int j=0;i <8;i++) 
        { 
          t_point *pi=new t_point(XX,XX,XX); 
          data.push_back(*pi); //这里神了，你push_back进入vector的是你new出来的对象的副本，
                                   //这个循环丢掉了8*sizeof(t_point)个字节的内存
        } 
//之前用这样的方式操作了8次 

//现在在释放内存。 

                  for(int i=(BST->data.size())-1;i>=0;i--) 
                { 
                  //这里更强，如果这个data还是上面的data的话，你就在企图delete由vector
                  //分配的内存，而vector很可能根本就并不是使用你的这个分配内存的机制
                     //(vector自己应该有管理内存的机制)
                  //但是因为你给delete的东西确实是一个可以进行操作的指针，编译器当然让它通过
                  delete &(BST->data[i]); 
                } 

        vector <t_point>::iterator ite; 
      for(ite=data.begin();ite!=data.end();++ite) 
        delete ite; //这里强到无以复加了，ite是一个vector<t_point>::iterator类型，
                       //对它进行delete应该是什么结果呢？？
                       //如果data里面存储的是指针，你也该写"delete *ite"
                    //ite指向的对象才是可以被delete的东西啊。

解释完了，但愿你满意，同时请你把基本概念在认真看看，真的。

[jackzhhuang]

delete迭代器居然能通过编译？！但可以肯定这是错的。

楼主似乎还没有理解迭代器。

迭代器是容器类型的一个代理，*iter才是你放入容器的对象。

[nicky_zs]

楼主要搞清楚一个概念。
虽然你最开始是在一个for循环里面不断地new了很多对象，但是你把它push_back到vector中的时候，始终都只是在vector里面保存了一个新的副本而已，而vector中的副本是不需要你去关心如何delete的。你要delete的应该是你在new和push_back之后，把那个new的东西给delete掉。

[ToBeTough]

up

[doudouHuY]

[Quote=引用 18 楼 ToBeTough 的回复:]
up
[/Quote]
vec.push_back(*pPoint); 里头有copy和fill操作会引起拷贝构造函数的多次调用，具体的你跟刀stl代码里头就知道了

[youzhipeng]

[code=C/C++]
#include <vector>
#include <string>
#include <iostream>
using namespace std;
int gCreateTimes = 0;
int gDeleteTimes = 0;
int gCopyTimes = 0;
class CPoint
{
public:
CPoint(){cout<<"D"<<endl;gCreateTimes++; }
CPoint(const CPoint& Copy)
{
gCopyTimes ++;
}
~CPoint(){cout<<"A"<<endl;gDeleteTimes++; }
};

int main()
{
vector<CPoint> vec;
for(int i=0;i<5;i++)
{
CPoint *pPoint = new CPoint;
vec.push_back(*pPoint);
delete pPoint;
}
cout<<"Create"<<gCreateTimes<<endl;
cout<<"Delete"<<gDeleteTimes<<endl;
cout<<"Copy"<<gCopyTimes<<endl;
system("pause");
return 0;
}[code]

*****************************************运行结果**************************************
Create 5
Delete 20
Copy 20
**************************************************************************************

我的分析：
CPoint *pPoint = new CPoint; 肯定调用了五次构造函数。
delete pPoint; 肯定五次析构。

vec.push_back(*pPoint); 这里面就有大文章了， 我觉得就 五次拷贝构造函数+五次析构函数。 但是~~~~~~~


谁能详细帮忙分析下！多谢。

[soft_ruan]

很有可能你的程序用了多台造成的！