Utilizzo della variabile membro nell'elenco di acquisizione lambda all'interno di una funzione membro

Il codice seguente viene compilato con gcc 4.5.1 ma non con 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;
}

Questo è l'errore:

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

Così,

1> quale compilatore è giusto?

2> Come posso usare le variabili membro all'interno di una lambda in VS2010?

Credo che VS2010 abbia ragione questa volta, e controllerei se avessi lo standard a portata di mano, ma al momento no.

Ora, è esattamente come dice il messaggio di errore: non è possibile catturare oggetti al di fuori dell'ambito racchiuso della lambda. ^† grid non rientra nell'ambito di applicazione, ma lo thisè (ogni accesso gridavviene effettivamente come this->gridnelle funzioni membro). Per il tuo caso d'uso, catturare thisopere, dal momento che lo userai subito e non vuoi copiarlogrid

auto lambda = [this](){ std::cout << grid[0][0] << "\n"; }

Se tuttavia, desideri archiviare la griglia e copiarla per un accesso successivo, dove il tuo puzzleoggetto potrebbe essere già stato distrutto, dovrai creare una copia locale intermedia:

vector<vector<int> > tmp(grid);
auto lambda = [tmp](){}; // capture the local copy per copy

† Sto semplificando - Google per "raggiungere l'ambito" o vedere §5.1.2 per tutti i dettagli cruenti.

Riepilogo delle alternative:

cattura this:

auto lambda = [this](){};

utilizzare un riferimento locale al membro:

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

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

Credo che tu debba catturare this.

Un metodo alternativo che limita l'ambito del lambda piuttosto che dargli accesso all'intero thisè passare un riferimento locale alla variabile membro, ad es.

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