检查以下代码:
这有效:
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做错了什么?
附录:
我无法删除任何指针。这是正常的吗?即使我删除右/左,我也会得到相同的错误,即使它们在返回时不再被引用。
答案 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(),则会调用未初始化变量的析构函数。