Discussion :

[casafa]

Bonjour,

J'ai ce code qui fonctionne :

Code :

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

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 :

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

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

Possible ? Impossible ? merci d'avance

[Mat007]

Salut,

Tu as essayé :

Code :

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

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

?

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

MAT.

[casafa]

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

[ram-0000]

Moi j'aime bien ce code :

Code :

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

1
2
3
4
5
6
7
while(true)
{
   model=octreeManager->getNextModel();
   if(model==NULL)
      break;
   model->draw();
}

par rapport à :

Code :

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

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.

[koala01]

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 :

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

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

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

Code :

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

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 ), 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

Maintenant, si la question est

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

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 :

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

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 :

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

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 :

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

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 :

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

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 ) 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

[ram-0000]

...

• 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

[koala01]

• 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