受另一个话题的启发,我编写了这段模拟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();
}输出(意为):
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
发布于 2011-05-29 11:40:23
在这个初始化过程中:
lambda finally = [&]{a=2; std::cout<<"finally executed";};可以使用隐式定义的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-06-02 05:05:21
似乎比必要要复杂得多。为什么不只是:
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++中。
https://stackoverflow.com/questions/6167515
复制相似问题