Discussion :

[befalimpertinent]

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 :

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

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

[Sylvain Togni]

Les sous-fonctions sont tes amies

Code :

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

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

[befalimpertinent]

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

[bachir006]

remplace ParcourirleContainer par ta boucle

[befalimpertinent]

remplace ParcourirleContainer par ta boucle

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

[Mat007]

Salut,

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 :

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

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.

[Alp]

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

Code :

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

1
2
3
4
5
6
namespace
{
    void update( const std::set<int>& container )
    {
    }
}

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