parcours particulier d'un set

:salut:

Je souhaite itérer d'une manière un peu spécial sur un set. Pour des raisons que je ne n'expliquerai pas je dois absolument parcourir deux fois ma boucle si le set ne possède qu'un élément et tout le set sinon.

J'ai codé ceci qui marche :

Code:

---

1 2 3 4 5 6 7 8 9 10 11 12 13

set<int> container; /*...*/   set<int>::const_iterator it; int container_sz=container.size(); const int nb_tours=container_sz==1 ? 2 :container_sz; int i; for(it=container.begin(),i=0;it!=container.end() || i<nb_tours;++it,++i) { if(container_sz<=1) it=container.begin(); /*..*/ }

---

Mais je me demandais s'il existait une technique disons plus propre (je n'aime pas trop le if dans la boucle) de procéder.

Toute idée est la bienvenue!
Merci

Les sous-fonctions sont tes amies

Code:

---

1 2 3

parcourirLeContainer(container); if (container.size() == 1) parcourirLeContainer(container);

---

Mouais sauf que ce que j'estime que ce que je met dans ma boucle n'a rien à faire dans une méthode puisque le code en question (dsl de ne pouvoir être plus précis) ne sera appelé qu'à cet endroit.

Intéressant tout de même.
what else

remplace ParcourirleContainer par ta boucle :)

Citation:

---

Envoyé par bachir006

remplace ParcourirleContainer par ta boucle :)

---

Dupliquer du code Pouaaa :aie:
C'est pas ce que j'appelle coder proprement. :fessee:

Salut,

Citation:

---

Envoyé par befalimpertinent

Mouais sauf que ce que j'estime que ce que je met dans ma boucle n'a rien à faire dans une méthode

---

Mets-le dans une fonction locale, dans un namespace anonyme :

Code:

---

1 2 3 4 5 6

namespace { void update( const std::set<int>& container ) { } }

---

edit : ou sinon tu peux aussi factoriser le traitement sur un élément plutôt que toute la boucle (puisqu'après tout tu n'as pas besoin de faire une boucle quand il n'y a qu'un seul élément)

MAT.

Citation:

---

Envoyé par Mat007

Mets-le dans une fonction locale, dans un namespace anonyme :

Code:

---

1 2 3 4 5 6

namespace { void update( const std::set<int>& container ) { } }

---

---

C'est cette solution que j'utiliserais pour ma part.