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

Question

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

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

501 4

c++c++11 visual-studio-2010 lambda

4 ответов:

Comments

Ничего не найдено.

Xeo · Accepted Answer · 2013-11-27 18:06:18

Я считаю, что VS2010 будет прав на этот раз, и я бы проверил, если бы у меня был стандартный удобный, но в настоящее время я этого не делаю.

теперь, точно так же, как сообщение об ошибке говорит: Вы не можете захватить вещи за пределами области видимости лямбда.^†grid не находится в области видимости, но this is (каждый доступ к grid на самом деле происходит как this->grid в функции-члены). Для вашего usecase, захват 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 для всех кровавых деталей.

Trass3r · Accepted Answer · 2017-02-10 14:54:20

резюме альтернатив:

захват 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

Michael Krelin - hacker · Accepted Answer · 2011-10-26 00:09:33

Я считаю, что вам нужно захватить this.

dlanod · Accepted Answer · 2015-10-29 02:02:10

альтернативный метод, который ограничивает область лямбда, а не дает ему доступ ко всему this - это передача локальной ссылки на переменную-член, например
auto& localGrid = grid;
int i;
for_each(groups.cbegin(),groups.cend(),[localGrid,&i](pair<int,set<int>> group){
            i++;
            cout<<i<<endl;
   });