Poniższy kod kompiluje się z gcc 4.5.1, ale nie z 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;
}
To jest błąd:
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
Więc,
1> który kompilator ma rację?
2> Jak mogę używać zmiennych składowych wewnątrz lambdy w VS2010?
c++
visual-studio-2010
lambda
c++11
vivek
źródło
źródło
pair<const int, set<int> >
, to jest rzeczywisty typ mapy. Prawdopodobnie powinno to być również odniesienie do stałej.Odpowiedzi:
Uważam, że tym razem VS2010 ma rację i sprawdziłbym, czy mam pod ręką standard, ale obecnie nie mam.
Teraz jest dokładnie tak, jak w komunikacie o błędzie: Nie możesz przechwytywać rzeczy poza otaczającym zakresem lambda. †
grid
nie znajduje się w zakresie obejmującym, alethis
jest (każdy dostęp dogrid
faktycznie odbywa się tak, jakthis->grid
w przypadku funkcji składowych). Przechwytywaniethis
działa w Twoim przypadku , ponieważ będziesz go używać od razu i nie chcesz kopiować plikugrid
Jeśli jednak chcesz zachować siatkę i skopiować ją do późniejszego dostępu, gdzie Twój
puzzle
obiekt może już zostać zniszczony, musisz wykonać pośrednią, lokalną kopię:† Upraszczam - Google dla „osiągnięcia zakresu” lub zobacz §5.1.2, aby poznać wszystkie krwawe szczegóły.
źródło
tmp
byćconst &
dogrid
wyrąbać na kopiowanie? Nadal chcemy mieć co najmniej jedną kopię, kopię do lambda ([tmp]
), ale nie potrzebujemy drugiej kopii.grid
chociaż prawdopodobnie zostanie zoptymalizowane. Krótsze i lepsze jest:auto& tmp = grid;
itd.[grid = grid](){ std::cout << grid[0][0] << "\n"; }
aby uniknąć dodatkowej kopiierror: capture of non-variable ‘puzzle::grid’
Podsumowanie alternatyw:
przechwytywanie
this
:użyj lokalnego odniesienia do członka:
C ++ 14:
przykład: https://godbolt.org/g/dEKVGD
źródło
[&grid]
nie działa). Bardzo się cieszę, że to wiem!Uważam, że musisz złapać
this
.źródło
grid
bezpośrednio. Problem polega na tym, a co jeśli chcesz skopiować siatkę? To nie pozwoli ci tego zrobić.Alternatywną metodą, która ogranicza zakres lambdy zamiast dawania jej dostępu do całości,
this
jest przekazanie lokalnego odwołania do zmiennej składowej, np.źródło