web-dev-qa-db-ja.com

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

次のコードは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のラムダ内でメンバー変数を使用するにはどうすればよいですか？

c++visual-studio-2010lambdac++11
121
vivek

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

さて、それはまさにエラーメッセージが言っているのと同じです：ラムダの外側のスコープの外にあるものをキャプチャすることはできません。† gridは囲まれたスコープ内にありませんが、thisは含まれています（gridへのアクセスはすべて、実際にはメンバー関数のthis->gridとして発生します）。ユースケースでは、thisをキャプチャすると機能します。すぐに使用し、gridをコピーしたくないためです。

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を参照してください。

133
Xeo

代替案の要約：

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

74
Trass3r

thisをキャプチャする必要があると思います。

20
Michael Krelin - hacker

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

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