1. ホーム
  2. c++

[解決済み] メンバ関数内のラムダ捕捉リストでメンバ変数を使用する

2022-04-21 08:14:05

質問

以下のコードは 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 は、(すべてのアクセスは 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


詳細は§5.1.2 をご覧ください。