Discussion :

[r0d]

Bonjour,

Prenons par exemple le problème suivant: nous voulons remplir un tableau d'entier avec des nombres aléatoires compris entre 0 et un nombre donné. Prenons donc le code suivant qui fais ce qui est demandé:

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
#include <algorithm>
 
// foncteur qui génère des nombre aléatoires compris entre 0 et mod
struct RandMod
{
	RandMod( int mod ) : mod_(mod) {}
	int operator()() { return ( (int) rand() ) % mod_; }
 
private: 
	int mod_;
};
 
main()
{
	int* buffer = new int[10];
	std::generate( buffer, buffer + 10, RandMod( 1000 ) );
}

Ok, le code ci-dessus fonctionne, mais comme je veux être à la mode, je veux utiliser des lambdas. Comment faire pour faire la même chose sans foncteur mais avec un lambda à la place?

[Arzar]

Bonsoir,

Comme ça :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
main()
{
	int* buffer = new int[10];
	std::generate( buffer, buffer + 10, []{ return rand() % 1000; });
}

[JolyLoic]

Il manque la gestion du max :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
main()
{
	int* buffer = new int[10];
	int mod = 123;
	std::generate( buffer, buffer + 10, [=]{ return rand() % mod; });
}

(à noter la présence du = qui indique que tous les éléments capturés le seront par copie, comme c'était le cas dans ton foncteur).

[gbdivers]

En terme de bonne pratique d'écriture, il vaut mieux indiquer explicitement les variables capturées ou pas ?

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
//: capture uniquement la variable mod par copie
std::generate( buffer, buffer + 10, [=mod]{ return rand() % mod; });
// capture tout par copie
std::generate( buffer, buffer + 10, [=]{ return rand() % mod; });

Edit : un peu de lecture sur les lambdas : http://cpp.developpez.com/gbdivers/cpp11/#LIV-A

[cob59]

Il y a aussi la possibilité d'un code un poil plus lisible :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
int* buffer = new int[10];
auto RandGenerator = [] { return rand() % 1000; };
std::generate( buffer, buffer + 10, RandGenerator );

[JolyLoic]

En terme de bonne pratique d'écriture, il vaut mieux indiquer explicitement les variables capturées ou pas ?

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
//: capture uniquement la variable mod par copie
std::generate( buffer, buffer + 10, [=mod]{ return rand() % mod; });
// capture tout par copie
std::generate( buffer, buffer + 10, [=]{ return rand() % mod; });

J'ai encore du mal à parler bonne pratique, car je manque encore de pratique tout court... Je pense pour ma part ne pas expliciter les captures par valeur, mais expliciter les captures par référence, ces dernières demandant à faire attention à la durée de vie des objets capturés par rapport à celle de la lambda.

Il y a aussi la possibilité d'un code un poil plus lisible :

Je ne suis pas convaincu que la lisibilité en soit tant améliorée. Et comme la durée de vie de la lambda est prolongée, si jamais il y a capture par référence, je préfère la première version...

[r0d]

Il y a aussi la possibilité d'un code un poil plus lisible :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
int* buffer = new int[10];
auto RandGenerator = [] { return rand() % 1000; };
std::generate( buffer, buffer + 10, RandGenerator );

Ce code ne prend pas en compte la variable (mod).