Discussion :

[Aspic]

Bonsoir à tous,

Je me suis mis récemment au for_each, et je n'arrive pas à parcourir mon vector 2D pour libérer les ressources en mémoire.
Soit la déclaration suivante :

vector< vector<BITMAP*> > tableau;

Je souhaiterais appliquer la fonction "libererBitmap" à tous les éléments du vector 2D avec des for_each sans boucles for...

Pour libérer la mémoire avec un vector 1D, j'utilise un foncteur de la manière suivante :

Code :

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

1
2
3
4
5
6
7
8
9
10
struct DeleteBitmapVector
{
    template <class T> void operator()(T& p) const
	{
	    libererBitmap(p);
	}
 
};
// plus loin
for_each(tableau1D.begin(), tableau1D.end(), DeleteBitmapVector());

Je voudrais la même chose en 2D

Merci à tous et bonne soirée !

[LittleWhite]

Bonsoir,

Je doute que ce soit vraiment beaucoup mieux qu'un double for ... mais bon.
Dans la théorie (car je n'ai jamais tenté de code de ce genre), cela serait:

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
18
 
struct DeleteBitmap
{
    template <class T> void opertor()(T& p) const
    {
        libererBitmap(p);
    }
}
 
struct DeleteBitmapVector
{
    template <class T> void operator()(T& p) const
    {
        for_each(p.begin(), p.end(), DeleteBitmap();
    }
};
// plus loin
for_each(tableau2D.begin(), tableau2D.end(), DeleteBitmapVector());

Petite explication.
Nous avons un tableau 2D, qui est un std::vector d'un std::vector de Bitmap*. En code: std::vector< std::vector < Bitmap* > > myTab2D;Donc, lors du premier for_each() (celui du tableau 2D), il va parcourir tout les éléments, qui sont des tableau (1D) de Bitmap* -> [codeinline]std::vector < Bitmap* >[codeinline]
Donc, il faut encore parcourir le tableau, afin de liberer tout les pointeurs. Donc j'utilise un nouveau for_each().

Voilà tout, je pense.

[Aspic]

C'est bien ce que je pensais, je voulais l'avis des autres vu mon niveau pathétique en C++

Bah je suis d'accord avec toi, une double boucle for suffirais mais il parait qu'il faut pas avec les vectors d'apfès la faq :
http://cpp.developpez.com/faq/cpp/?p...elete_sequence

Perso, je ne vois pas la différence mais bon

Merci !

[LittleWhite]

Euh, dans cette FAQ, je ne vois pas ce qui dit qu'il faut mieux utiliser un for_each que un for avec des iterateurs.
Je connais un cas, ou les iterateurs sont un peu plus compliqué à gerer (car de la suppression d'element du vecteur lors du parcours). Mais sinon ... le for suffira je pense.

[Aspic]

Euh, dans cette FAQ, je ne vois pas ce qui dit qu'il faut mieux utiliser un for_each que un for avec des iterateurs.
Je connais un cas, ou les iterateurs sont un peu plus compliqué à gerer (car de la suppression d'element du vecteur lors du parcours). Mais sinon ... le for suffira je pense.

Oui on ne dit pas ce qui est le mieux mais vu que c'est cette méthode qui est fournis, voilà pourquoi je l'ai adopté sans savoir vraiment pourquoi

[Goten]

Un double BOOST_FOR_EACH serait plus lisible.

[oxyde356]

Un double BOOST_FOR_EACH serait plus lisible.

+1

[Aspic]

Un double BOOST_FOR_EACH serait plus lisible.

Sauf que je n utilise pas boost
D'ailleurs sincèrement qu'est ce que ça a de mieux cette librairie ? (je sens que je vais me faire lyncher )

[jblecanard]

D'ailleurs sincèrement qu'est ce que ça a de mieux cette librairie ? (je sens que je vais me faire lyncher )

Pour dire si c'est mieux, il faudrait déjà savoir avec quoi tu le compares . Si le compares à rien (la STL seule), c'est mieux dès lors que tu en as l'utilité. Dans ton cas, un boost foreach serait en effet sympa, mais ça te rajoute une dépendance. C'est un choix à faire, comme toujours...

Si le sujet t'intéresse, il y a un débat ici.

[Goten]

Sauf que je n utilise pas boost
D'ailleurs sincèrement qu'est ce que ça a de mieux cette librairie ? (je sens que je vais me faire lyncher )

Mieux que quoi?

[Aspic]

Mieux que quoi?

Je pensais comparer à un conteneur standard de la STL comme un vector voire une map puisque dans mon appli je n'utilise que ce genre de conteneur...

Si le compares à rien (la STL seule), c'est mieux dès lors que tu en as l'utilité. Dans ton cas, un boost foreach serait en effet sympa, mais ça te rajoute une dépendance.

Après on parle d'ajouter des dépendances ? Qu'entendez vous par cela ? (est ce uniquement l'ajout d'une librairie externe ou y'a autre chose...)

[jblecanard]

Je pensais comparer à un conteneur standard de la STL comme un vector voire une map puisque dans mon appli je n'utilise que ce genre de conteneur...

boost foreach s'utilise sur des conteneurs de la STL ...

Après on parle d'ajouter des dépendances ? Qu'entendez vous par cela ? (est ce uniquement l'ajout d'une librairie externe ou y'a autre chose...)

Oui c'est une dépendance car tu as besoin de Boost pour compiler ton projet.

[Kalith]

Sinon tu as un compromis :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
#define foreach(iter, container) for ((iter) = (container).begin(); (iter) != (container).end(); ++(iter))
 
vector< vector<BITMAP*> > tableau;
 
vector< vector<BITMAP*> >::iterator iter1;
vector<BITMAP*>::iterator           iter2;
 
foreach (iter1, tableau)
foreach (iter2, *iter1)
{
    libererBitmap(*iter2);
}

[Aspic]

ok merci pour vos éclairements