Discussion :

[progfou]

Bonjour tout le monde !

Dans un code, je viens de trouver une structure de l'algorithme qui me semble assez peu efficace...
Qu'en pensez-vous ?

Code :

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

1
2
3
4
5
6
do
{
  if(!ma_fonction1(arg1)) break;
  if(!ma_fonction2(arg2)) break;
  //etc...
}while(0)

ma_fonction1 et ma_fonction2 vont mettre en forme arg1 dans une classe par exemple, et retourner un false en cas d'erreur.

Je trouve cela commode, mais inefficace...
Qu'en est-il ?

Merci d'avance !

[millie]

Bonjour tout le monde !

Dans un code, je viens de trouver une structure de l'algorithme qui me semble assez peu efficace...
Qu'en pensez-vous ?

Code :

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

1
2
3
4
5
6
do
{
  if(!ma_fonction1(arg1)) break;
  if(!ma_fonction2(arg2)) break;
  //etc...
}while(0)

ma_fonction1 et ma_fonction2 vont mettre en forme arg1 dans une classe par exemple, et retourner un false en cas d'erreur.

Je trouve cela commode, mais inefficace...
Qu'en est-il ?

Merci d'avance !

Code :

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

1
2
 
while(mafonction1(arg1) && mafonction2(arg2));

avec évaluation paresseuse des booléens ?

[progfou]

Et dans le cas où ce ne sont pas deux mais 20 ou 30 fonctions (ce qui est le cas) ?

[pseudocode]

Et dans le cas où ce ne sont pas deux mais 20 ou 30 fonctions (ce qui est le cas) ?

Code C :

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

1
2
3
4
5
6
 
if(!ma_fonction1(arg1));
else if(!ma_fonction2(arg2));
else if(!ma_fonction1(arg3));
/* ... */
else if(!ma_fonction20(arg20));

sinon il y a les pointeurs sur fonctions...

[progfou]

Qu'est-ce qui est le mieux ?
Ce qui sera bien optimisé par le compilo, facile à lire, etc. ?

[Lowelace]

Bonjour,

Pour reprendre l'idée de pseudocode, pourquoi ne pas construire une liste chainée de "fonctions" (une implémentation possible du design pattern "chaine de responsabilité") :
Plus :
- code compact (sauf l'initialisation de la liste)
- code évolutif

Moins :
- pas très lisible, ou du moins compréhensible

[pseudocode]

Encore plus sale:

Code C :

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

1
2
 
ma_fonction1(arg1) && ma_fonction2(arg2) && ma_fonction3(arg3);

[progfou]

Oui, ok pour quelques fonctions, mais est-ce que c'est bien de faire cela quand on a des dizaines de fonctions ?