web-dev-qa-db-fra.com

Utilisation de la variable membre dans la liste de capture lambda dans une fonction membre

Le code suivant est compilé avec gcc 4.5.1 mais pas avec 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;
}

C'est l'erreur:

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

Alors,

1> quel compilateur a raison?

2> Comment utiliser les variables membres dans un lambda dans VS2010?

c++visual-studio-2010lambdac++11
121
vivek

Je crois que VS2010 a raison cette fois-ci et je vérifierais si j'avais la norme en main, mais actuellement je ne l'ai pas.

Maintenant, c'est exactement comme le message d'erreur dit: Vous ne pouvez pas capturer des éléments en dehors de la portée englobante du lambda.† grid n'est pas dans la portée englobante, mais this est (chaque accès à grid se produit réellement comme this->grid dans les fonctions membres). Pour votre cas d'utilisation, capturer this fonctionne, car vous l'utiliserez immédiatement et vous ne voulez pas copier le fichier grid

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

Si, toutefois, vous souhaitez stocker la grille et la copier pour un accès ultérieur, où votre objet puzzle pourrait déjà être détruit, vous devez créer une copie locale intermédiaire:

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

† Je simplifie - Google pour "atteindre la portée" ou voir §5.1.2 pour tous les détails sanglants.

133
Xeo

Résumé des alternatives:

capture this:

auto lambda = [this](){};

utilisez une référence locale au membre:

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

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

74
Trass3r

Je crois que vous devez capturer this.

20
Michael Krelin - hacker

Une autre méthode qui limite la portée du lambda plutôt que de lui donner accès à l'ensemble this consiste à transmettre une référence locale à la variable membre, par ex.

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