Boost :: Scoped_Ptr中断了代码

时间:2010-10-11 19:26:01

标签: c++ pointers boost

检查以下代码:

这有效:

T *p = (std::find( this->first(), this->last(), *pPos ));
if( p != last() )
{
    this->push_back(data);

    T *right = (this->last() - 1);
    T *left  = (this->last() - 2);

    while( *pPos != data )
        std::iter_swap( left--, right-- ); 

    return const_cast<T*>(pPos);
}

这不是:

boost::scoped_ptr<T> p(std::find( this->first(), this->last(), *pPos ));
if( p.get() != last() )
{
    this->push_back(data);

    T *right = (this->last() - 1); 
    T *left  = (this->last() - 2); 

    while( *pPos != data ) 
        std::iter_swap( left--, right-- ); 

    return const_cast<T*>(pPos);
}

第二个版本给出

的运行时错误

Expression: _BLOCK_TYPE_IS_VALID_(pHead->nBlockUse)

意味着我的scoped_ptr要么太快超出范围,要么正在做一些使它失效的时髦事物。

我对scoped_ptr做错了什么?

附录:

我无法删除任何指针。这是正常的吗?即使我删除右/左,我也会得到相同的错误,即使它们在返回时不再被引用。

3 个答案:

答案 0 :(得分:7)

boost :: scoped_ptr将删除指针(即boost :: scoped_ptr实例)超出范围。我认为您不想删除指针,它似乎是您班级中的迭代器。

答案 1 :(得分:5)

scoped_ptr用于本地堆分配的内存。它会销毁包含的对象,并在退出作用域时释放它。

尝试在块中间释放内存(例如find返回的内存)是非法的。在不拥有物体的情况下销毁物体将导致双重破坏,这也是非法的。

这里不要使用scoped_ptr

答案 2 :(得分:3)

假设this-&gt; first()\ last()返回一个常规指针,当函数结束时,将调用任何p指向的析构函数。

如果std :: find返回this-&gt; last(),则会调用未初始化变量的析构函数。