comparaison+affectation dans une boucle

Bonjour,

J'ai ce code qui fonctionne :

Code:

---

1 2 3 4 5 6 7

while(true) { model=octreeManager->getNextModel(); if(model==NULL) break; model->draw(); }

---

Ma question simple : est-il possible de réduire le nombre ligne de ce code ? Il me semblait de mémoire que l'on pouvait faire une affectation et une comparaison dans la condition d'une boucle mais je n'y arrive pas !

Quelque chose du genre mais qui fonctionne :

Code:

---

1 2	while(model = octreeManager->getNextModel() != NULL) model->draw();

---

Possible ? Impossible ? merci d'avance

Salut,

Tu as essayé :

Code:

---

1 2	while( model = octreeManager->getNextModel() ) model->draw();

---

?

Et sinon tu obtiens quoi comme erreur ? Avec quel compilateur ?

MAT.

Grrr, j'avais déjà testé ton code mais ça ne fonctionnait pas à cause d'une erreur qui se trouvait autre part, désolé du dérangement

Moi j'aime bien ce code :

Code:

---

1 2 3 4 5 6 7

while(true) { model=octreeManager->getNextModel(); if(model==NULL) break; model->draw(); }

---

par rapport à :

Code:

---

1 2	while(model = octreeManager->getNextModel() != NULL) model->draw();

---

Il a l'avantage dêtre plus lisible, plus facile à comprendre, a débugger, à modifier et je suis sûr qu'au bout du compte c'est le même code qui est généré par le compilateur et l'optimiseur.

Salut,

La question me semble très mal posée, pour tout te dire...

En effet, le nombrer de ligne n'importe le plus souvent que peu au compilateur... mais importe énormément à la personne qui le lit.

En effet, si tu écris

Code:

---

fonction1();fonction2();fonction3();

---

cela revient exactement au même, du point de vue du compilateur, que si tu écris

Code:

---

1 2 3

fonction1(); fonction2(); fonction3();

---

Mais la lisibilité du code est bien meilleure dans la seconde version que dans la première.

Or, s'il y a un principe qu'il faut garder en tête en permanence (à vrai dire, il y en a plus d'un, mais il y en a un en particulier qui vaut la peine d'être rappelé dans ce contexte :D), c'est qu'un code source est beaucoup plus souvent lu qu'il n'est écrit/compilé.

Et, sur base de ce principe, on peut t'assurer que la seconde version sera toujours préférable à la première, car la première risque fort d'enduire le lecteur d'horreur... Pardon: de l'induire en erreur ;) :D

Maintenant, si la question est

Citation:

---

Est-il possible d'améliorer l'algorithme, et, partant de là, de diminuer le nombre d'instruction, et donc les performance de cette boucle:question:

---

la réponse est oui, sans aucun doute ;)

Mais, pour garder un minimum de sécurisation de l'exécution, le code correspondant au "meilleur algorithme" prendrait sans doute la forme de

Code:

---

1 2 3 4 5 6 7

model=octreeManager->getFirstModel(); while(model) { model->draw(); model=model->getNextModel(); }

---

voir (étant donné que les boucles sont interchangeables) de

Code:

---

1 2 3 4 5 6

for(model=octreeManager->getFirstModel(); model!=NULL; model=model->getNextModel()) { model->draw(); }

---

en partant des prérequis que:

• getFirstModel() renvoie un pointeur compatible avec model pointant sur le premier élément de la collection
• getNextModel() renvoie un pointeur compatible avec model pointant sur l'élément de la collection qui suit celui en cours ou NULL si l'élément en cours est le dernier élément de la collection

Nous pourrions aussi, si nous avions la certitude que model est un pointeur valide au départ, envisager de travailler sous une forme proche de

Code:

---

1 2 3 4 5

do { model->draw(); model=model->getNextModel(); }while(model);

---

voire, s'il nous est impossible d'avoir cette certitude (car model serait passé en paramètre, par exemple)

Code:

---

1 2 3 4 5 6 7 8

if(model) { do { model->draw(); model=model->getNextModel(); }while(model); }

---

Tous ces exemples fournissent des algorithmes qui sont a priori bien meilleur que celui que tu propose comme code de départ, tout en fournissant un résultat final exactement identique (et, pour certain... nécessitant plus de lignes de code, si on considère les retours à la lignes du fait des accolades... c'est un comble non :question:) mais en préservant malgré tout un niveau de visibilité correct ;)

Maintenant, si tu souhaites que nous t'aidions à choisir celui qui sera le plus adapté à ton cas particuliers, il faudra nous en dire un peu d'avantage ;):D

Citation:

---

Envoyé par koala01

...

• getFirstModel() renvoie un pointeur compatible avec model pointant sur le premier élément de la collection
• getNextModel() renvoie un pointeur compatible avec model pointant sur l'élément de la collection qui suit celui en cours ou NULL si l'élément en cours est le dernier élément de la collection

...

---

• getFirstModel() renvoie un pointeur compatible avec model pointant sur le premier élément de la collection ou NULL si la collection est vide

Sinon, je suis totalement d'accord
+1

Citation:

---

Envoyé par ram_0000

• getFirstModel() renvoie un pointeur compatible avec model pointant sur le premier élément de la collection ou NULL si la collection est vide

Sinon, je suis totalement d'accord
+1

---

Effectivement...

Ces sept mots supplémentaires sont restés dans mon clavier et sont pourtant tres importants :D