C++ラムダ'this'ポインタ無効移動操作

Question

後、私は私の現在のプロジェクトの次の（簡体字）のコードを有する：

#include <iostream> 
#include <string> 
#include <functional> 
#include <vector> 


class Test{ 

public: 

    Test() = default; 
    Test(const Test& other) = delete; 
    Test& operator=(const Test& other) = delete; 
    Test(Test&& other) = default; 
    Test& operator=(Test&& other) = default; 



    void setFunction(){ 
     lambda = [this](){ 
      a = 2; 
     }; 
    } 

    int callAndReturn(){ 
     lambda(); 
     return a; 
    } 

private: 
    std::function<void()> lambda; 
    int a = 50; 
}; 


int main() 
{ 
    Test t; 
    t.setFunction(); 
    std::vector<Test> elements; 
    elements.push_back(std::move(t)); 
    std::cout << elements[0].callAndReturn() << std::endl; 
} 

を私はそれを実行すると、値50の代わりに期待値2で印刷されています。これは、ラムダ関数が現在のthisポインタを取得しているためです。移動操作後、thisポインターが変更され、関数は間違ったaに書き込みます。

私の質問はラムダのキャプチャされたリファレンスを新しいTestに変更して、値2を表示する方法はありますか？

Answer 1

解決策は、thisを全くキャプチャしません。代わりに、キャプチャした関数タイプを受け入れて変更します。そして、aへの間接アクセスのためのメンバーへのポインタ（値によって取り込まれる）を使用してください。

std::function<void(Test*)> lambda; 

void setFunction(){ 
    auto a = &Test::a; 
    lambda = [=](Test *t){ 
     (t->*a) = 2; 
    }; 
} 

int callAndReturn(){ 
    lambda(this); 
    return a; 
} 

Live Example

---

Galikが指摘したように、あなただけ、あなたもメンバーへのポインタを必要としない、単一のハードコーディングされたメンバーにアクセスする必要がある場合。したがって、ラムダはキャプチャレスである可能性があります。

void setFunction(){ 
    lambda = [](Test *t){ 
     t->a = 2; 
    }; 
}