我有一种情况,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::functionstd::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/

10-17 01:36