Следующий код компилируется с 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-2010
lambda
c++11
Вивек
источник
источник
pair<const int, set<int> >
, это фактический тип пары карты. Возможно, это также должна быть ссылка на const.Ответы:
Я считаю, что VS2010 будет правильным на этот раз, и я бы проверил, был ли у меня под рукой стандарт, но в настоящее время у меня нет.
Теперь это похоже на сообщение об ошибке: «Вы не можете захватывать вещи вне рамок лямбды. †
grid
не входит в объем, ноthis
есть (каждый доступ кgrid
действительности происходит какthis->grid
в функциях-членах). В вашем случае захватthis
работает, так как вы будете использовать его сразу и не хотите копироватьgrid
Однако, если вы хотите сохранить сетку и скопировать ее для последующего доступа, где ваш
puzzle
объект уже может быть уничтожен, вам нужно будет сделать промежуточную локальную копию:† Я упрощаю - Google для "достижения области" или см. §5.1.2 для всех кровавых деталей.
источник
tmp
быть ,const &
чтобыgrid
сократить копирование? Мы все еще хотим по крайней мере одну копию, копию в lambda ([tmp]
), но нет необходимости во второй копии.grid
хотя, вероятно, оно будет оптимизировано. Короче и лучше:auto& tmp = grid;
и т. Д.[grid = grid](){ std::cout << grid[0][0] << "\n"; }
без дополнительной копииerror: capture of non-variable ‘puzzle::grid’
Краткое изложение альтернатив:
захватить
this
:используйте локальную ссылку на участника:
C ++ 14:
пример: https://godbolt.org/g/dEKVGD
источник
[&grid]
прежнему не работает). Очень рад это знать!Я считаю, вам нужно захватить
this
.источник
grid
напрямую. Проблема в том, что если вы хотите скопировать сетку? Это не позволит вам сделать это.Альтернативный метод, который ограничивает область действия лямбды, а не дает ей доступ ко всему,
this
состоит в передаче локальной ссылки на переменную-член, напримеристочник