标准容器删除指针变量的问题

hwz_119

我有一个deque<A>类型的容器.里面存放的是指针变量.当销毁deque<A>类型的容器变量时,我得先手工删除里面的指针变量.以下方法是否正确?谢谢

emplate <class A>
void Queue<A>::cleara(int i)
{   
    A l_item;
    加锁
    for(int i=0; i<m_que.size(); i++)
    {
        l_item = m_que.pop_front();
        delete *l_item;
    }
    m_que.clear();        
    解锁      
}

hwz_119

谢谢

太平绅士

即使A是指针，也谈不上正确，
这里delete *l_item破坏了和谐。

hwz_119

能不能说详细点?

福瑞哈哥

emplate <class A>
void Queue<A>::cleara(int i)
{   
    A l_item;
    加锁
    for(int i=0; i<m_que.size(); i++)
    {
        l_item = m_que.pop_front();
        delete *l_item;
    }
    m_que.clear();        

自己回忆一下如何通过指针删除对象，把它跟你这段代码比较一下。

phoneix

不如先实现一个可以复制的智能指针，或者用boost::shared_ptr，把这个智能指针作为deque的元素类型，这样就不用这么麻烦了

ivek_troll

原帖由 hwz_119 于 2007-6-29 10:52 发表
我有一个deque<A>类型的容器.里面存放的是指针变量.当销毁deque<A>类型的容器变量时,我得先手工删除里面的指针变量.以下方法是否正确?谢谢

emplate <class A>
void Queue<A>::cleara ...

代码有问题，pop_front 会使size 改变的，建议用iterator 遍历

cookis

1. 
   
2. l_item = *m_que.pop_front()
   

复制代码

如果是 l_item 指针.就应该 delete l_item;  not delete *l_item;
如果是 类变量..那你就错了..根本不用delete

[ 本帖最后由 cookis 于 2007-6-30 00:02 编辑 ]

wolf0403

deque<A> ? deque<A *> ?

converse

LZ用list的size作为循环结束的判断，有没有考虑过pop出来一个之后size发生了变化而i继续递增导致了一些问题呢？如前面的朋友所说，用iterator应该是比较合理的做法。