Echelle de Goret

Version imprimable

Voir 40 message(s) de cette discussion en une page

29/05/2006, 11h51
Gruik

Echelle de Goret

Demandons ici des explications sur l'echelle de gorettitude de Emdel.

http://emmanuel-delahaye.developpez.com/goret.htm

Force 8 "Plus d'un return par fonction"

Mais pourquoi?
29/05/2006, 11h53
gnto

Il faut plus d'un return par fonction ou il en faut un seul ?
29/05/2006, 11h54
Gruik

Il en faut qu'un
29/05/2006, 11h56
miron

Citation:

Envoyé par Gruik

Demandons ici des explications sur l'echelle de gorettitude de Emdel.

http://emmanuel-delahaye.developpez.com/goret.htm

Force 8 "Plus d'un return par fonction"

Mais pourquoi?

c'est vrai ca...
29/05/2006, 11h57
gnto

ben je vois la philosophie du truc si t'arrete pas de sortir de ta fonction avec des returns c'est pas "naturelle" comme déroulement. Enfin c'est ce que je pense !!

On ma toujours dis que l'utilisation des fonctions goto-like c'est pas le top
29/05/2006, 12h00
miron

Citation:

Envoyé par gnto

ben je vois la philosophie du truc si t'arrete pas de sortir de ta fonction avec des returns c'est pas "naturelle" comme déroulement. Enfin c'est ce que je pense !!

On ma toujours dis que l'utilisation des fonctions goto-like c'est pas le top

???
Donne un exemple...
Parceque si c'est une fonction de test...

Oui mais pour une fonction de ce type

Code:

1
2
3
4
5
6
7
8
9
10
11
12
 
int fonction(void)
{
    char *buffer = NULL;
    buffer = malloc(50);
    if(buffer == NULL) {
        printf("Erreur allocation !\n");
        return -1;
    }
    [...];
    return 0;
}

Je ne vois pas en quoi ca fait goret (surtout Force 8).

Il est vrai que l'on peux faire comme ca.

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
 
int fonction(void)
{
    char *buffer = NULL;
    int ret = 0;
    buffer = malloc(50);
    if(buffer != NULL) {
        [...]
    }
    else {
        printf("Erreur allocation !\n");
        ret = -1;
    }
    return ret;
}

Mais ca implique de mettre tout le code de la fonction dans le if.

29/05/2006, 12h14
gnto

Aprés cette echelle de goret est relative. Moi personnellement je prefere la 2eme fonctions de skyrunner.
29/05/2006, 13h39
Jean-Marc.Bourguet

Citation:

Envoyé par gnto

Aprés cette echelle de goret est relative. Moi personnellement je prefere la 2eme fonctions de skyrunner.

Si la difference est significative, c'est que les ... contiennent trop de choses.

(Note, dans la liste d'Emmanuel, je n'arrive pas a savoir si force 10 est mieux ou pire que force 1).

A+

--
Jean-Marc
29/05/2006, 13h41
Gruik

10 c'est le pire

Jveux dire, dans ce cas par exemple

Code:

1
2
3
4
5
6
7
8
 
int func (const char * search) {
  int k;
  for (k = 0; k < TAB_NB_ELT; k++)
     if (!strcmp(search, my_tab[k].str))
       return k;
  return -1;
}

est-ce si grave?

29/05/2006, 13h49
Gruik

Au fait, je pensais faire un gros topic sur ce forum sur "comment bien programmer en C" où tout le monde pourrait participer et on discuterait pour se mettre d'accord sur certains points.
Ca periphraserait un peu l'echelle de goret mais ça detaillerait le pourquoi du comment, le tout bien présenté avec des couleurs et des sections.
Bien ou pas bien?

Citation:
Envoyé par Gruik
Jveux dire, dans ce cas par exemple
Code:

1 2 3 4 5 6 7 8 int func (const char * search) { int k; for (k = 0; k < TAB_NB_ELT; k++) if (!strcmp(search, my_tab[k].str)) return k; return -1; }
est-ce si grave?
Yeah en plus la tu as couplé avec une variable globale :aie: :aie:

Je ne trouve pas ca grave, mais c'est vrai que l'on peux toujours s'en sortir autrement.
Code:

1 2 3 4 5 6 7 8 9 10 11 int func (const char * search, const struct mystruct *my_tab) { int k; int ret = -1; for (k = 0; k < TAB_NB_ELT; k++) if (!strcmp(search, my_tab[k].str)) { ret = k; break; } return ret; }
Hum je sens que cette discussion a un bel avenir devant elle. :aie:

29/05/2006, 13h53
Skyounet

Citation:

Envoyé par Gruik

Au fait, je pensais faire un gros topic sur ce forum sur "comment bien programmer en C" où tout le monde pourrait participer et on discuterait pour se mettre d'accord sur certains points.
Ca periphraserait un peu l'echelle de goret mais ça detaillerait le pourquoi du comment, le tout bien présenté avec des couleurs et des sections.
Bien ou pas bien?

Je suis pour, on pourrait deja reprendre des bouts du topic sur le Caymal.
29/05/2006, 13h54
Jean-Marc.Bourguet

Citation:

Envoyé par Gruik

10 c'est le pire

Ah, ne pas indenter -- perso, a ce niveau de non respect des lecteurs je ne prends generalement meme pas la peine de repondre sur Internet, et je n'ai jamais vu dans ma vie professionnelle ou j'ai pourtant vu bien des choses, mais au niveau indentation, le pire que j'ai vu c'est l'utilisation d'un melange de tabulation et d'espaces ne fonctionnant correctement qu'avec une tabulation mise a 5 caracteres pour une partie du code et a 8 caracteres pour une autre dans un fichier qu'on aurait mieux fait de scinder avant que je ne le fasse -- c'est mieux que de tout mettre dans le main(), ce qui m'est deja arrive pour des petits programmes oneshot, et qui arrive sans probleme dans des exercices ou dans du code reduit pour mettre en evidence un probleme. En fait, je ne vois pas de necessite de faire une difference entre main et les autres fonctions, pour lesquelles on a "Fonction de plus d'une page visible" qui est du force 5 d'apres Emmanuel...
29/05/2006, 14h08
Jean-Marc.Bourguet
Citation:
Envoyé par Gruik

Jveux dire, dans ce cas par exemple

Code:

1 2 3 4 5 6 7 8 int func (const char * search) { int k; for (k = 0; k < TAB_NB_ELT; k++) if (!strcmp(search, my_tab[k].str)) return k; return -1; }

est-ce si grave?
Restons sur le probleme du SESE (Single Entry Single Exit) et passons sur la variable globale et le fait que le tableau devrait peut-etre etre trie ou remplace par une table de hachage.

Je ne suis pas un fanatique du SESE meme si j'ai tendance a ecrire du code qui respecte cette contrainte, mais pas au pris d'introduction d'une variable ou de complication d'expression de conditions comme ce le serait ici s'il n'y avait l'alternative subtilement differente:
Code:

1 2 3 4 5 6 7 8 9 int func(char const* niddle) { int k = TAB_NB_ELT - 1; while (k >= 0 && strcmp(niddle, my_tab[k].str) != 0) { --k; } return k; }
Dans du code qui n'est pas SESE, j'aime bien que le code respecte la symetrie et donc qu'il y ait un return soit dans chaque branche du if ou du switch, soit dans aucune.
29/05/2006, 14h09
Gruik

Citation:

Envoyé par Jean-Marc.Bourguet

Ah, ne pas indenter -- perso, a ce niveau de non respect des lecteurs je ne prends generalement meme pas la peine de repondre sur Internet, et je n'ai jamais vu dans ma vie professionnelle ou j'ai pourtant vu bien des choses, mais au niveau indentation, le pire que j'ai vu c'est l'utilisation d'un melange de tabulation et d'espaces ne fonctionnant correctement qu'avec une tabulation mise a 5 caracteres pour une partie du code et a 8 caracteres pour une autre dans un fichier qu'on aurait mieux fait de scinder avant que je ne le fasse -- c'est mieux que de tout mettre dans le main(), ce qui m'est deja arrive pour des petits programmes oneshot, et qui arrive sans probleme dans des exercices ou dans du code reduit pour mettre en evidence un probleme. En fait, je ne vois pas de necessite de faire une difference entre main et les autres fonctions, pour lesquelles on a "Fonction de plus d'une page visible" qui est du force 5 d'apres Emmanuel...

Ouais, ya des trucs en force élevée que je considère comme pas si dramatique
29/05/2006, 14h12
Gruik

Citation:

Envoyé par Skyrunner

Je suis pour, on pourrait deja reprendre des bouts du topic sur le Caymal.

Okay, je m'y attele ce soir
29/05/2006, 14h15
Gruik
Citation:
Envoyé par Jean-Marc.Bourguet

Restons sur le probleme du SESE (Single Entry Single Exit) et passons sur la variable globale et le fait que le tableau devrait peut-etre etre trie ou remplace par une table de hachage.

Je ne suis pas un fanatique du SESE meme si j'ai tendance a ecrire du code qui respecte cette contrainte, mais pas au pris d'introduction d'une variable ou de complication d'expression de conditions comme ce le serait ici s'il n'y avait l'alternative subtilement differente:

Code:

1 2 3 4 5 6 7 8 9 int func(char const* niddle) { int k = TAB_NB_ELT - 1; while (k >= 0 && strcmp(niddle, my_tab[k].str) != 0) { --k; } return k; }

Dans du code qui n'est pas SESE, j'aime bien que le code respecte la symetrie et donc qu'il y ait un return soit dans chaque branche du if ou du switch, soit dans aucune.
Donc, si ya pas de rajout de variable et si ça complexifie pas trop, on se doit d'avoir qu'un seul return?
29/05/2006, 14h19
Skyounet

Force 8 : Passage d'une structure en paramètre...

Non pas que je l'ai deja fait (quoique...), il est vrai qu'il est preferable de passer un pointeur sur la structure. Mais Force 8, ouah...
29/05/2006, 14h22
Gruik

C'est vrai ça, si la structure contient un char * et un entier, est si grave de les recopier?
29/05/2006, 14h33
crocodilex

Citation:

Envoyé par Gruik

Demandons ici des explications sur l'echelle de gorettitude de Emdel.

http://emmanuel-delahaye.developpez.com/goret.htm

Force 8 "Plus d'un return par fonction"

Mais pourquoi?

Mince alors, tu m'as devancé. Je voulais également lancé un topic sur cette echelle suite à une discussion avec Emdel concernant les variables statique.

Très bonne initiative Gruik...
29/05/2006, 15h25
Jean-Marc.Bourguet

Citation:

Envoyé par Gruik

Donc, si ya pas de rajout de variable et si ça complexifie pas trop, on se doit d'avoir qu'un seul return?

L'important c'est que ca reste lisible et qu'il soit facile de raisonner sur le code. Cette derniere contrainte a tendance a favoriser le code SESE.

C'est quelque chose de tres proche du "pas de goto" et entraine parfois une attitude pas tres rationnelles. Certains sont fanatiques et rejettent d'office tout code que ne le fait pas, quitte a favoriser un code qui me semble plus complexe. D'autres, dont je suis proche, considerent que c'est souvent un signe qu'il y a un manque de structuration mais ne rejettent pas a priori cette regle. D'autres enfin la considerent comme stupide.
29/05/2006, 15h33
gnto

pour moi le return c'est presque le goto. On casse le déroulement de la fonction pour ne pas effectuer la suite ?

les breaks existe-t'il en algorithmie ?
29/05/2006, 15h46
Gruik

Ok je comprends bien. Je me demandais juste si yavait des cas où on pouvait avoir plusieurs return.
Si on veut faire bien, qu'un seul return.

Maintenant, en Force 4 : "Usage de TAB au lieu de SPACE"
?

J'ai plutot tendance à configurer mon editeur pour qu'il mette des tabulations reelles au lieu des espaces pour justement qu'il yait pas de probleme si un autre n'utilise pas le meme nombre d'espaces pour son indentation
29/05/2006, 16h26
gnto

Citation:

Envoyé par Gruik

Maintenant, en Force 4 : "Usage de TAB au lieu de SPACE"
?

Ben alors je suis fiers d'etre en force 4 :lol:

Vu la taille des ecrans aujourdhui on peut bien mettre des tabs, pour moi c plus espacé donc c plus lisible.
29/05/2006, 16h30
gnto

est ce que la non optimisation est une force de gorétisation

Mettre un int au lieu d'un short c'est pas force 1 ?
29/05/2006, 16h40
Gruik

Non au contraire, mettre des short au lieu des int est goret si ta variable est algorithmiquement considérée comme un entier.
Je pense qu'il faut utiliser short à la place de int que si l'economie de mémoire est significative, ex un tableau enorme de structures contenant des entiers
29/05/2006, 16h57
Jean-Marc.Bourguet

Citation:

Envoyé par Gruik

Ok je comprends bien. Je me demandais juste si yavait des cas où on pouvait avoir plusieurs return.
Si on veut faire bien, qu'un seul return.

Maintenant, en Force 4 : "Usage de TAB au lieu de SPACE"
?

J'ai plutot tendance à configurer mon editeur pour qu'il mette des tabulations reelles au lieu des espaces pour justement qu'il yait pas de probleme si un autre n'utilise pas le meme nombre d'espaces pour son indentation

Le probleme des tabulations est que generalement, ceux qui les utilisent pour l'indentation le font de sorte que le code soit beau qu'avec une taille de tabulation: la leur qui est differente de 8... Il y a bien moyen d'utiliser convenablement les tabulations pour indenter et les espaces pour aligner -- donc changer la taille de la tabulation n'a pas d'effet sur la beaute du code. Certains arguent pour cette methode; je la connais, mais je n'ai jamais vu quelqu'un l'appliquer de maniere constante car elle n'est pas automatique et necessite de distinguer l'indentation de l'alignement.
29/05/2006, 17h02
Emmanuel Delahaye
Citation:

Envoyé par Gruik

Demandons ici des explications sur l'echelle de gorettitude de Emdel.

http://emmanuel-delahaye.developpez.com/goret.htm

Force 8 "Plus d'un return par fonction"

Mais pourquoi?
- Pour respecter les principes de la programmation structurée (ben oui, le C, c'est pas le BASIC...)
- Pour être sûr de bien faire ce qu'on a à faire (les sorties de secours, c'est le meilleur moyen d'oublier une ressource...
- Pour avoir du code lisible et maintenable
- Pour pouvoir mettre un point d'arrêt à un endroit connu
La vrai question est "pourquoi pas en force 9 ou 10" ? Parce que dans des cas tordus ça peut permettre de gagner quelque nanosecondes... Je le passerais bien en 9 quand même...

Genre, pour aligner ceci?

Code:

1
2
3
4
 
    if (condition 1
        && condition 2
        && condition 3)

29/05/2006, 17h04
Emmanuel Delahaye

Citation:

Envoyé par Skyrunner

Mais ca implique de mettre tout le code de la fonction dans le if.

Et ? C'est grave ?
29/05/2006, 17h06
Jean-Marc.Bourguet
Citation:
Envoyé par Gruik

Genre, pour aligner ceci?

Code:

1 2 3 4 if (condition 1 && condition 2 && condition 3)
Je supose qu'il s'agit de l'utilisation des tabulations. Pour une utilisation correcte, il
faut que la premiere ligne n'ait que des tabulations avant le if, les deux autres le meme nombre de tabulations suivies par 4 espaces.
29/05/2006, 17h08
Emmanuel Delahaye
Citation:

Envoyé par gnto

pour moi le return c'est presque le goto. On casse le déroulement de la fonction pour ne pas effectuer la suite ?

les breaks existe-t'il en algorithmie ?

En C, ça existe :
Code:

1 2 3 4 5 6 do { break; } while (0);
Mais je ne vais pas créer un niveau 11 pour ça...

OK. Je vais expliquer que mon echelle inclue le respect de la programmation structurée. Ca fera le ménage dans les questions.

Pour moi, c'est tellement évident, que je ne peux pas envisager de coder autrement...
29/05/2006, 17h10
Emmanuel Delahaye

Citation:

Envoyé par Jean-Marc.Bourguet

(Note, dans la liste d'Emmanuel, je n'arrive pas a savoir si force 10 est mieux ou pire que force 1).

*** TROLL ALERT ***
29/05/2006, 17h13
Gruik

Citation:

Envoyé par Emmanuel Delahaye

*** TROLL ALERT ***

Vous battez pas les djeunz
29/05/2006, 17h15
gnto

j'ai envie de dire qui seme le vent récolte la tempête
29/05/2006, 17h15
gege2061

Citation:

Envoyé par Emmanuel Delahaye

Mais je ne vais pas créer un niveau 11 pour ça...

Tu peux y ajouter les typedef tordus :

Code:

typedef char UINT64
29/05/2006, 17h17
Emmanuel Delahaye

Citation:

Envoyé par Gruik

Maintenant, en Force 4 : "Usage de TAB au lieu de SPACE"
?

J'ai plutot tendance à configurer mon editeur pour qu'il mette des tabulations reelles au lieu des espaces pour justement qu'il yait pas de probleme si un autre n'utilise pas le meme nombre d'espaces pour son indentation

8O 8O ? Si je demande à l'éditeur de mettre 4 espaces, il y aura 4 espaces dans le fichier. Point. Il n'y aura pas de TAB et le code aura strictement la présentation que j'ai voulue quelque soit l'éditeur ou l'imprimante ou le mode <pre></pre> en HTML ou la fenêtre du debugger. Portabilité maximale. C'est le but.

Les TABs peuvent être interprétées différements selon les éditeurs, imprimantes etc. Pas portable.-> Force 4 (je suis gentil..., mais ça ne me dérange pas, car de toutes façons, je repasse tout à l'indenteur qui met le code au carré...indentation de 3 espaces etc.)
29/05/2006, 17h22
Gruik

Ok, parce que j'ai une fonctionnalité dans mon editeur (PSPad) c'est qu'on peut lui dire si les tabulations insérées doivent être converties en espace ou rester en tabs, et si elles restent en tabs on peut lui dire combiein de caracteres elles doivent prendre
Donc j'ai pensé que la tabulation était mieux qu'un nombre fixe d'espace qd on travaille à plusieurs

Voir 40 message(s) de cette discussion en une page

Fuseau horaire GMT +2. Il est actuellement 12h18.