メンバー関数内のラムダキャプチャリストでメンバー変数を使用する

次のコードはgcc 4.5.1でコンパイルされますが、VS2010 SP1ではコンパイルされません。

#include <iostream>
#include <vector>
#include <map>
#include <utility>
#include <set>
#include <algorithm>

using namespace std;
class puzzle
{
        vector<vector<int>> grid;
        map<int,set<int>> groups;
public:
        int member_function();
};

int puzzle::member_function()
{
        int i;
        for_each(groups.cbegin(),groups.cend(),[grid,&i](pair<int,set<int>> group){
                i++;
                cout<<i<<endl;
        });
}
int main()
{
        return 0;
}

これはエラーです：

error C3480: 'puzzle::grid': a lambda capture variable must be from an enclosing function scope
warning C4573: the usage of 'puzzle::grid' requires the compiler to capture 'this' but the current default capture mode does not allow it

そう、

1>どのコンパイラが正しいですか？

2> VS2010でラムダ内のメンバー変数を使用するにはどうすればよいですか？

今回はVS2010が正しいと思います。標準が手元にあるかどうかを確認しますが、現在はそうではありません。

これは、エラーメッセージに示されているとおりです。ラムダの外側のスコープからのものをキャプチャすることはできません。^† gridは含まれているスコープではありませんが、メンバー関数のように実際にthisアクセスされます。あなたのユースケースでは、キャプチャは機能します。すぐに使用し、コピーしたくないからです。gridthis->gridthisgrid

auto lambda = [this](){ std::cout << grid[0][0] << "\n"; }

ただし、グリッドを保存して後でアクセスするためにコピーする場合、puzzleオブジェクトが既に破棄されている可能性があるため、中間のローカルコピーを作成する必要があります。

vector<vector<int> > tmp(grid);
auto lambda = [tmp](){}; // capture the local copy per copy

†私は単純化しています-Googleは「範囲に到達」するために、またはすべての悲惨な詳細については§5.1.2を参照してください。

代替案の要約：

キャプチャthis：

auto lambda = [this](){};

メンバーへのローカル参照を使用します。

auto& tmp = grid;
auto lambda = [ tmp](){}; // capture grid by (a single) copy
auto lambda = [&tmp](){}; // capture grid by ref

C ++ 14：

auto lambda = [ grid = grid](){}; // capture grid by copy
auto lambda = [&grid = grid](){}; // capture grid by ref

例：https : //godbolt.org/g/dEKVGD

私はあなたがキャプチャする必要があり、信じていますthis。

ラムダ全体へのアクセスを許可するのではなく、ラムダのスコープを制限する別の方法thisは、メンバー変数へのローカル参照を渡すことです。たとえば、

auto& localGrid = grid;
int i;
for_each(groups.cbegin(),groups.cend(),[localGrid,&i](pair<int,set<int>> group){
            i++;
            cout<<i<<endl;
   });