我有一种情况,boost::function 和 boost::bind(实际上是 std::tr1::function 和 bind)在仍在使用时被删除。这安全吗?我通常会避免它,但有问题的代码有点根深蒂固,我唯一的其他选择是添加一个新线程。
typedef function<int(int)> foo_type;
foo_type* global_foo = NULL;
int actual_foo( int i, Magic* m )
{
delete global_foo;
return m->magic(i);
}
int main()
{
Magic m;
global_foo = new foo_type( bind( &actual_foo, _1, &m );
return (*global_foo)(10)
}
绑定(bind)参数始终是普通整数类型(实际代码中的 int 和指针),而不是引用。
最佳答案
boost::function
或 std::tr1::functions
是可复制的对象。因此,通常绝对没有理由分配它们——只需按值传递它们。
它们针对大多数实际情况进行了很好的优化......所以只需按值传递它们:
typedef function<int(int)> foo_type;
foo_type global_foo;
int actual_foo( int i, Magic* m )
{
delete global_foo;
return m->magic(i);
}
int main()
{
Magic m;
global_foo = bind( &actual_foo, _1, &m );
return global_foo(10)
}
您建议的代码很危险,请运行以下代码:
#include <boost/function.hpp>
#include <boost/bind.hpp>
#include <iostream>
using namespace std;
boost::function<void()> *glb;
struct Data {
int x;
Data(int _x = 0) : x(_x) { cout<<"ctor:"<<this<<endl; }
~Data() { cout<<"dtor:"<<this<<endl; }
Data(Data const &p) {x=p.x; cout<<"ctor:"<<this<<endl; }
Data const &operator=(Data const &p) { x=p.x; cout<<this<<"="<<&p<<endl; return *this; }
};
void func(Data const &x)
{
delete glb;
cout<<&x<<endl;
}
int main()
{
glb=new boost::function<void()>(boost::bind(func,Data(3)));
(*glb)();
return 0;
}
您会发现您尝试在
func
中访问已被调用的 cout<<&x<<endl
中显示的销毁对象(具有相同指针值的 dtor)。因为当你销毁你的函数对象时,你也销毁了你的绑定(bind)参数
have 和
Data const &x
变得不可用,因为它被 global_function
破坏了编辑: 评论说明:
如果你有类似的东西
map<string,function<void()> > calls;
void delete_key(){
calls.erase("key");
}
main()
{
calls["key"]=delete_key;
// Wrong and dangerous
// You delete function while in use
calls["key"]();
// Correct and safe
// You create a copy of function so it would not
// be deleted while in use.
function<void()> f=calls["key"];
f();
}
关于c++ - 使用时删除 boost 功能,我们在Stack Overflow上找到一个类似的问题:https://stackoverflow.com/questions/1273900/