Использование переменной-члена в списке захвата лямбда внутри функции-члена

Следующий код компилируется с 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?

Ответы

Ответ 1

Я верю, что 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.1 для всех деталей gory.

Ответ 2

Сводка альтернатив:

capture this:

auto lambda = [this](){};

используйте локальную ссылку на элемент:

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

С++ 14:

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

example: https://godbolt.org/g/dEKVGD

Ответ 3

Полагаю, вам нужно захватить this.

Ответ 4

Альтернативный метод, который ограничивает область лямбда, а не дает ей доступ ко всему this, заключается в передаче локальной ссылки на переменную-член, например.

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