Question

我试图模仿最终的效果。所以我认为我应该进行快速的肮脏测试。

这个想法是使用Most Important const来阻止破坏并将finally块放在lambda中。然而，显然我做错了什么，并在MyFinally（）结束时调用它。我该如何解决这个问题？

#include <cassert>
template<typename T>
class D{
    T fn;
public:
    D(T v):fn(v){}
    ~D(){fn();}
};

template<typename T>
const D<T>& MyFinally(T t) { return D<T>(t); }

int d;
class A{
    int a;
public:
    void start(){
        int a=1;
        auto v = MyFinally([&]{a=2;});
        try{
            assert(a==1);
            //do stuff
        }
        catch(int){
            //do stuff
        }
    }
};
int main() {
    A a;
    a.start();
}

我的解决方案代码（注意：你最终不能在同一个区块中有两个。正如期望的那样。但仍然有点脏）

#include <cassert>
template<typename T>
class D{
    T fn; bool exec;
public:
    D(T v):fn(v),exec(true){}
    //D(D const&)=delete //VS doesnt support this yet and i didnt feel like writing virtual=0
    D(D &&d):fn(move(d.fn)), exec(d.exec) {
      d.exec = false;
    }

    ~D(){if(exec) fn();}
};
template<typename T>
D<T> MyFinally(T t) { return D<T>(t); }


#define FINALLY(v) auto OnlyOneFinallyPlz = MyFinally(v)

int d;
class A{
public:
    int a;
    void start(){
        a=1;
        //auto v = MyFinally([&]{a=2;});
        FINALLY([&]{a=2;});
        try{
            assert(a==1);
            //do stuff
        }
        catch(int){
            FINALLY([&]{a=3;}); //ok, inside another scope
            try{
                assert(a==1);
                //do other stuff
            }
            catch(int){
                //do other stuff
            }
        }
    }
};
void main() {
    A a;
    a.start();
    assert(a.a==2);
}

有趣的是，如果你删除＆amp;在MyFinally的原始代码中它起作用-_-。

Answer 1

// WRONG! returning a reference to a temporary that will be
// destroyed at the end of the function!
template<typename T>
const D<T>& MyFinally(T t) { return D<T>(t); }

你可以通过引入移动构造函数来修复它

template<typename T>
class D{
    T fn;
    bool exec;

public:
    D(T v):fn(move(v)),exec(true){}

    D(D &&d):fn(move(d.fn)), exec(d.exec) {
      d.exec = false;
    }

    ~D(){if(exec) fn();}
};

然后你可以重写你的玩具

template<typename T>
D<T> MyFinally(T t) { return D<T>(move(t)); }

希望它有所帮助。使用auto时，不需要“常量参考”技巧。有关如何在C ++ 03中使用const引用，请参阅here。

Answer 2

您的代码与Sutter不相同。他的函数返回一个值，你的函数返回一个对象的引用，该对象将在函数退出时被销毁。调用代码中的const引用不会维护该对象的生命周期。

Answer 3

这个问题源于使用函数制作者，正如约翰内斯所证明的那样。

我认为你可以通过使用另一个C ++ 0x工具来避免这个问题，即std::function。

class Defer
{
public:
  typedef std::function<void()> Executor;

  Defer(): _executor(DoNothing) {}

  Defer(Executor e): _executor(e) {}
  ~Defer() { _executor(); }

  Defer(Defer&& rhs): _executor(rhs._executor) {
    rhs._executor = DoNothing;
  }

  Defer& operator=(Defer rhs) {
    std::swap(_executor, rhs._executor);
    return *this;
  }

  Defer(Defer const&) = delete;

private:
  static void DoNothing() {}
  Executor _executor;
};

然后，您可以简单地使用它：

void A::start() {
  a = 1;
  Defer const defer([&]() { a = 2; });

  try { assert(a == 1); /**/ } catch(...) { /**/ }
}

Answer 4

问题是由其他人解释的，所以我会建议修复，完全按照Herb Sutter编写代码的方式（顺便说一句，你的代码与他的代码不一样）：

首先，不要通过const引用返回：

template<typename T>
D<T> MyFinally(T t) 
{
   D<T> local(t); //create a local variable
   return local; 
}

然后在呼叫站点写这个：

const auto & v = MyFinally([&]{a=2;}); //store by const reference

这就像Herb Sutter的code一样。

演示：http://www.ideone.com/uSkhP

现在在退出start()函数之前调用析构函数。

不再使用auto关键字的不同实现：

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; }
};

并将其用作：

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

看起来很有意思？

完整演示：http://www.ideone.com/DYqrh

Answer 5

您可以返回shared_ptr：

template<typename T>
std::shared_ptr<D<T>> MyFinally(T t) {
    return std::shared_ptr<D<T>>(new D<T>(t));
}

为什么要调用医生（使用匿名/ lambda函数）

5 个答案:

不再使用auto关键字的不同实现：