4

受另一个主题的启发，我编写了这段代码来模拟一个finally块：

#include <cassert>
#include <iostream>

struct base { virtual ~base(){} };

template<typename TLambda>
struct exec : base 
{
   TLambda lambda;
   exec(TLambda l) : lambda(l){}
   ~exec() { lambda(); }
};

class lambda{
    base *pbase;
public:
    template<typename TLambda>
    lambda(TLambda l): pbase(new exec<TLambda>(l)){}
    ~lambda() { delete pbase; }
};

class A{
    int a;
public:
    void start(){
        int a=1;        
        lambda finally = [&]{a=2; std::cout<<"finally executed";}; 
        try{
            assert(a==1);
            //do stuff
        }
        catch(int){
            //do stuff
        }
    }
};
int main() {
    A a;
    a.start();
}

输出（ideone）：

finally executed

@Johannes 似乎认为它并不完全正确，并评论说：

如果编译器在复制初始化中没有删除临时文件，它可能会崩溃，因为它会使用相同的指针值删除两次

我想知道到底是怎么回事。帮助我理解问题:-)

编辑：

问题修复为：

class lambda{
    base *pbase;
public:
    template<typename TLambda>
    lambda(TLambda l): pbase(new exec<TLambda>(l)){}
    ~lambda() { delete pbase; }

    lambda(const lambda&)= delete;            //disable copy ctor
    lambda& operator=(const lambda&)= delete; //disable copy assignment
};

然后将其用作：

//direct initialization, no copy-initialization
lambda finally([&]{a=2;  std::cout << "finally executed" << std::endl; });

完整代码：http ://www.ideone.com/hsX0X

4

2 回答 2

8

在这个初始化中：

lambda finally = [&]{a=2; std::cout<<"finally executed";};

可以使用隐式定义的复制构造函数 for lambda。这只会复制原始指针pbase，然后将其多次删除。

例如

$ g++ -std=c++0x -Wall -Wextra -pedantic -fno-elide-constructors lambdafun.cc 
$ ./a.out 
a.out: lambdafun.cc:29: void A::start(): Assertion `a==1' failed.
finally executedAborted (core dumped)

实际上，您的断言触发掩盖了双重删除问题，但这表明了我强调的崩溃。

$ g++ -std=c++0x -Wall -Wextra -pedantic -fno-elide-constructors -DNDEBUG lambdafun.cc 
$ ./a.out 
Segmentation fault (core dumped)
于 2011-05-29T11:40:23.663 回答
2

似乎比必要的要复杂得多。为什么不只是：

class finally
{
    std::function<void (void)> const action;
    finally(const finally&) = delete;

public:
    finally(std::function<void (void)> a)
        : action(a)
    {}

    ~finally() { action(); }
};

但总的来说，应该尽量不要将不良的 Java 习惯带到 C++ 中。

于 2011-06-02T05:05:21.587 回答