Comment remplacer trop de "if" ?

**Clad3** · 26/03/2005, 15h05

Coucou a tous !
Dans une partie de mon programme, je vais avoir pas mal de "if" se suivant les uns les autres ( janalyse le contenu d'un string et selon celui ci je fait quelquchose . )
C'est une facons correcte de faire? ou est-ce vraiment horrible ?
Si oui qu'y a t-il de mieux ?

**Aurelien.Regat-Barrel** · 26/03/2005, 15h24

En général faut éviter d'avoir plus de 3 niveaux d'imbrications et 1.5 page de code. Tu peux regrouper l'intérieur de tes blocs dans une nouvelle fonction.

**Clad3** · 26/03/2005, 15h57

Hum, le probleme n'est le niveau d'imbrication, ni la code a l'intérieur des if ( je l'est deja mis dans des fonctions spécifique )
Mais, il va y avoir hum .... une bonne 10aine de if ( une 20aine maxi je pense, mais rien n'est sur ) ce suivant les un les autres , sachant que je vais passer dans cette fonction plusieur fois par frame ...
enfin bon, ca se trouve je m'en fait pour rien

EDIT : remplacer mes if successif par ds

Code :

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

1
2
3
4
5
6
7
8
 
if{}
else{
      if{}
      else{
             if{]
             }
       }

est-til une bonne idée?

**salseropom** · 26/03/2005, 18h21

Salut, et as-tu pensé au switch ?

**Clad3** · 26/03/2005, 19h07

heu ma syntaxe du dessus est trés laide .... car trés fausse m'enfin ; il faut que je revoi ca ...

ui j'ai epnsé au switch ; mais ca ne me parait pas trés simple pour ce que je veut faire :/

voila le type de fonction que j'aurai ( ici avec le traitement d'une seule balises ... plus tard il y en aura bien d'autre )

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
void cTexteDialogue::TraiterBalise(std::string balise)
{
	if ( balise.find("/Color") != std::string::npos )
	{
		BaliseColor(balise,0);
	}
	if ( (balise.find("Color") != std::string::npos)  && (balise.find('/') ==  std::string::npos ) )
	{
		BaliseColor(balise,1);
	}
 
}

Enfin c'est peu être ma facons d'analyse le pb qui est fausse ..je sais pas trop ...

**Mehdi Feki** · 26/03/2005, 19h17

je ne crois pas qu c'est une bonne idée les if et les else comme tu les a presenté .

Code :

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

1
2
if (){}
          else if (){}

est plus approprié mais ca dependra toujours de tes conditions et comment elle sont structurées.

Ne te prends pas trop la tete pour ces trucs essaie juste de bien ecrire ecrire ton code d'une maniere claire et structuré

Code :

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

// /* et n'oblie pas de commenter chaque condition de ton code */

Pour le switch qui t'as qu'il va gerer des variables entieres ou une promotion de int !! En plus les variables de case doivent etre constantes !!!

EDIT : j'ai mis comme meme 10 minutes à ecrire ce poste
J'ai pas vu ta reponse Clad3

**Mehdi Feki** · 26/03/2005, 19h23

Il fallait dire plutot que tu parsais un fichier XML ( c'est bien ca non

)
Bein oui moi aussi j'utilise des dizaines de boucles imbriquées !!
On a pas le choix

**Clad3** · 26/03/2005, 20h31

je parse un .txt

je n'ai aps encore regardé du coté xml, trop de taff ^^ mais un jour peut-etre .... je n'aime pas me lancé dans trop de choses nouvelle a la fois ;. c'est le meilleur moyen de se planter sur toute la ligne ^^
merci de reponses, je vais méditer ca.

**Laurent Gomila** · 26/03/2005, 20h36

[chiant]

Je persiste à dire que tu t'embêtes pour rien : un fichier XML + la lib qui va bien, ou alors expressions régulières voire analyseur lexical, et le tour serait joué en moins de deux.

[/chiant]

**Clad3** · 26/03/2005, 20h44

hum .... de toute evidence si mon projet prend de l'ampleur, il va me faloir un format de fichier un peu plus évolué ...

Je ne doute pas du tout que le xml soit un mauvais format ... mais je ne l'ai jamais utilisé ( tout au plus, j'ai lu un exemple simple une fois ) et j'ai un peu peur de m'y mettre ... mais bon, je vais essayer de me motiver

allez courage adrien , courage ^^

**Laurent Gomila** · 26/03/2005, 20h49

Le XML en soi c'est très simple : c'est simplement une hiérarchie de balises avec ou non des attributs et des données quelconques entre. Comme le HTML, sauf que là c'est toi qui définit tes balises.

Après tu as bien sûr des parsers XML, qui te renverront directement le contenu de ton fichier dans une forme facilement lisible, il ne te restera plus qu'à effectuer les actions correspondant à chaque balise.

Les expressions régulières c'est également très simple : une fois que tu as écrit l'expression régulière qui va bien, tu envoies ça à ta biliothèque, un coup de moulinette et hop tu récupères ce qu'il faut comme il faut.