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

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

101 голос | спросил vivek 26 +04002011-10-26T01:05:15+04:00312011bEurope/MoscowWed, 26 Oct 2011 01:05:15 +0400 2011, 01:05:15

4 ответа


0

Я считаю, что на этот раз 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 для всех кровавых деталей.

ответил Xeo 26 +04002011-10-26T01:37:35+04:00312011bEurope/MoscowWed, 26 Oct 2011 01:37:35 +0400 2011, 01:37:35
0

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

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

ответил Trass3r 3 FebruaryEurope/MoscowbFri, 03 Feb 2017 19:55:24 +0300000000pmFri, 03 Feb 2017 19:55:24 +030017 2017, 19:55:24
0

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

ответил Michael Krelin - hacker 26 +04002011-10-26T01:09:33+04:00312011bEurope/MoscowWed, 26 Oct 2011 01:09:33 +0400 2011, 01:09:33
0

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

auto& localGrid = grid;
int i;
for_each(groups.cbegin(),groups.cend(),[localGrid,&i](pair<int,set<int>> group){
            i++;
            cout<<i<<endl;
   });
ответил dlanod 29 +03002015-10-29T03:02:10+03:00312015bEurope/MoscowThu, 29 Oct 2015 03:02:10 +0300 2015, 03:02:10

Похожие вопросы

Популярные теги

security × 330linux × 316macos × 2827 × 268performance × 244command-line × 241sql-server × 235joomla-3.x × 222java × 189c++ × 186windows × 180cisco × 168bash × 158c# × 142gmail × 139arduino-uno × 139javascript × 134ssh × 133seo × 132mysql × 132