Discussion :

[b Oo]

Bonjour je mets une fonction, dite moi laquelle vous préférez. (Ceci est tiré du livre Métier développeur : Kit de survie)

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
int maFonction(const char * filename, int taille)
{
  FILE * fp = fopen(filename, "rb");
  if(fp == 0)
  {
     printf("file not found");
     return ERR1;
  }
 
  char * buffer = new char[taille];
  if(buffer == 0)
  {
     fclose(fp);
     printf("out of memory");
     return ERR2;
  }
 
  int read = fread(buffer, 1, taille, fp);
  if(read != taille)
  {
     delete []buffer;
     fclose(fp);
     printf("read error");
     return ERR3;
  }
 
  delete []buffer;
  fclose(fp);
  return 0;
}

Voici la deuxième fonction écrite différemment :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
 
int maFonction(const char * filename, int taille)
{
  int err=0;
  char * buffer = 0;
  FILE * fp = fopen(filename, "rb");
  if(fp == 0) { err=ERR1; goto Exit; }
 
  buffer = new char[taille];
  if(buffer == 0) { err=ERR2; goto Exit; }
 
  int read = fread(buffer, 1, taille, fp);
  if(read != taille) { err=ERR3; goto Exit; }
 
 Exit:
  if (buffer!=0) delete []buffer;
  if (fp!=0) fclose(fp);
  if(err !=0) printf("Error %d", err); 
 return err;
}

J'aimerais donc avoir vos avis sur les 2 et si l'utilisation de goto est utile et si oui dans quel cas.

Merci d'avance.


PS: je sais il y a du C avec du C++, mais ce code n'est pas de moi.

[Matthieu Brucher]

C'est du pur C, ton truc, y'a pas de C++ dedans - à part le new et le delete -. Avec le C++,on a encore moins besoin du goto grâce à la sémantique par objet, donc perso, jamais.

[Faiche]

que ce soit en C ou en C++ je n'ai jamais utilisé de goto.

De plus, vu que tu ne sais pas d'où tu viens, t'es obligé de refaire des tests, donc en l'occurence, ton goto n'apporte rien.

Enfin, je ne suis pas un expert, mais je dirais que faire des goto peut poser des problèmes d'accès à la mémoire, comme dans le cas des try catch, mais en pire.

Donc non seulement c'est sale mais en plus souvent dangereux.. Mais encore une fois ce n'est que mon avis

[koala01]

Je n'ai jamais mis un seul goto dans n'importe quel programme (meme pas en COBOL)...

Il faut dire que j'ai eu un prof qui nous menaçait (et qui l'aurait fait, j'en suis encore persuadé) de nous jeter par la fenetre si on utilisait goto

[Sylvain Togni]

En C++ il y a notamment l'idiome RAII qui permet de se passer de la plupart des gotos.

[koala01]

En C++ il y a notamment l'idiome RAII qui permet de se passer de la plupart des gotos.

Meme sans passer par l'idiome RAII, il y a parfaitement moyen de se passer des goto...

Le tout, c'est de mettre au point la bonne logique... Et peut etre d'utiliser la bonne technique de mise au point d'algorithmes...

J'en parle dans un autre sujet, mais, il est clair que si l'on crée l'algorithme en utilisant le flowchart, la tentation d'utiliser des goto sera bien plus importante...

Par contre, d'autres techniques (le nassichneiderman en tete) permettent de purement et simplement ignorer l'existence de cette commande "honnie"

[loufoque]

La seule utilité c'est pour combler l'absence d'un break n.

[koala01]

La seule utilité c'est pour combler l'absence d'un break n.

Et encore, dans le cadre d'une boucle, il y a souvent moyen, en la concevant différemment, de ne pas avoir besoin d'un break...

A titre tout à fait personnel, je n'utilise d'ailleurs le break que dans le cadre des switch ...case ... Mais on entre dans un autre débat

[Graffito]

je n'utilise d'ailleurs le break que dans le cadre des switch ...case

+1
et jamais de return en mileu de procedure.