Le bêtisier du code source

[GrandGibus]

J'ouvre ici un sujet sur les perles dans le code source. Le but est de retranscrire de manière concise un exemple de code source "stupide" de votre cru ou que vous avez pu lire.

Le langage et la syntaxe ont peu d'importance, soyez le plus clair possible, afin que le maximum d'entre-nous comprenne.

Je commence:

    for (int i=0; i<MAX; i++) {
        switch (i) {
            case 0: .... break;
            case 1: .... break;
            case 2: .... break;
            ...
        }
    }

P.S. c juste histoire de détendre un peu l'atmosphère qui règne en ce moment... peut-être ce sujet aurait-il été plus approprié au Bar, mais c'est bien connu: l'humour d'informaticien...


P.S. 2: c'est interdit de faire du copier/coller des extraits de code à Tatouille :rateau: :love:

 

[tatouille]

c'est marrant je met un bout

int i=0;
while (1)
{
   

   i++;
}

j'espère que tu ne switch pas jusqu'à MAX

 

[GrandGibus]

J'ai ouvert ce post et j'ai immédiatement pensé à toi ...

J'espère au moins que ne sera pas seuls tous les deux !


En réponse: biensûr que le switch va jusqu'à Max !

 

[tatouille]

après la boucle infini
la boucle qui ne commence jamais

 

[Lisaraël]

Si je retrouve les codes javascripts de certains amis que j'ai du corriger pour les aider, j'ai de quoi remplir dix pages de topic...

Mais ce sont des erreurs de débutant, c'est du coup beaucoup moins drôle...

Mais un bout de code qui m'a beaucoup fait rire lors d'une correction, c'est celui-ci (en javascript):

function verifConfirm(param) {
   if (param == true) {
      return true;
   } 
}

Et il l'utilisait tout le temps dans son code, avec des structures genre:

if( verifParam(maVariable) == true ) {
   [actions]
}

Le pire, c'est qu'il m'a fallu des heures pour lui faire comprendre son "erreur"...

 

[PA5CAL]

Il m'est arrivé de rester longtemps, incrédule, devant des bouts de code récalcitrants qui contenaient des erreurs de frappe (parfois du fait d'un copier-coller malheureux), du style :

for (i=0.0; i<1,5; i+=0.01)
{
   ...
}

( "1,5" au lieu de "1.5" )

ou encore :

while (condition);
{
   ...
}

( ";" en trop )

 

[PA5CAL]

Un fois, un stagiaire m'avait pondu une fonction comme celle-ci :

void LibereListeChainee(ITEM *pItem)
{
   // variables locales
   ...

   if (pItem)
   {
      // libération de l'item suivant
      LibereListeChainee (pItem->pNext);

      // libération de l'item courant
      ... 
      free(pItem);
   }
}

La libération de la liste chaînée nécessitait autant de place sur la pile que l'espace occupé par la liste elle-même (plusieurs Mo). De ce fait, on atteignait rapidement l'overflow.

De plus, c'était sous Windows, et les temps de libération devenaient exorbitants. Une liste alloué en deux secondes sur la pile mettait près de cinq minutes à se libérer.

Ça marchait pourtant... sur le papier !

 

[GrandGibus]

Tout un programme en Java (langage Objet par excellence), voici le genre de code type:

public UnObjet {
    public static Object[] les_champs;
}

public UneCollection {
    public static Object[][] une_liste_dobjets;
}

Heuresement, il y avait une classe "dictionnaire":

public LeDico {
    public statc int UnObjetChamp1 = 0;
    public statc int UnObjetChamp2 = 1;
../..
}

On avait (re)baptisé le paradigme de ce logiciel: "Access everything from anywhere"

 

[Didier Guillion]

Bonjour,

Sujet interessant. Il y a quelque temps j'ai porté un soft Mac sur Windows (Reamlz). Le code était truffé d'acces du genre :

short a,b;

void fonction(void)
{
short *p;
short valSupposeeDeA,valSupposeeDeB;

p=&a;
valSupposeeDeA=*(p++);
valSupposeeDeA=*(p++);

}

Bien, sur, plantus assuré, rien ne garantie la succession des données dans le BSS.

Cordialement

 

[tatouille]

On avait (re)baptisé le paradigme de ce logiciel: "Access everything from anywhere"

pour pascal ca aurait pu etre rideau de suite

il m'est arrivé de pas relacher dans un kmod :rateau:
et on en sortait jamais donc ça ds un kernel space
bouf load broumbroum ca grossi ca grossi paf rideau

 

[Didier Guillion]

On avait (re)baptisé le paradigme de ce logiciel: "Access everything from anywhere"

pour pascal ca aurait pu etre rideau de suite

il m'est arrivé de pas relacher dans un kmod :rateau:
et on en sortait jamais donc ça ds un kernel space
bouf load broumbroum ca grossi ca grossi paf rideau

Euh ? Tu peut décoder ?

Cordialement

 

[p4bl0]

Euh ? Tu peut décoder ?

Cordialement

je vais essayer :
On avait (re)baptisé le paradigme de ce logiciel: "Access everything from anywhere"
-> ça ça veux dire que ça l'a fait rire


il m'est arrivé de pas relacher dans un kmod :rateau:
-> apparemment (je m'y connais pas assez, mais ça commence par k) un truc en rapport avec le noyeau

et on en sortait jamais donc ça ds un kernel space
-> Apparemment là il dit que c'était de pire en pire

bouf load broumbroum ca grossi ca grossi paf rideau

-> La le système à laché...


P.S.: c'est encore une version bêta du traducteur

 

[tatouille]

bah oui rideau bouffe toute la mémoire

j'en ai une belle mais pas de moi
en ce moment sur le nouveau site opensource effort d'Apple

Fatal error: out of dynamic memory in yy_create_buffer() in Unknown on line 0

http://www.macosforge.org/

je ne sais pas qui bidouille du sql foireux mais

 

[p4bl0]

"on line 0"

ah ouais c'est fort là...

J'vous le dis moi le libre c'est de la merde :rateau:

 

[Didier Guillion]

je vais essayer :
On avait (re)baptisé le paradigme de ce logiciel: "Access everything from anywhere"
-> ça ça veux dire que ça l'a fait rire


il m'est arrivé de pas relacher dans un kmod :rateau:
-> apparemment (je m'y connais pas assez, mais ça commence par k) un truc en rapport avec le noyeau

et on en sortait jamais donc ça ds un kernel space
-> Apparemment là il dit que c'était de pire en pire

bouf load broumbroum ca grossi ca grossi paf rideau

-> La le système à laché...


P.S.: c'est encore une version bêta du traducteur

Désolé, pour moi c'est du broussouf. Dommage, c'était un fil interessant.

Cordialement

 

[GrandGibus]

Dans la série VisualBasic, on pouvait trouver des appels du genre:

    goSub ta_mere_en_short

 

[PA5CAL]

J'ai ai aussi eu une bonne, de la part du même stagiaire dont je parlais au post #7.

Dans un outil de simulation, on remplissait une liste de résultats de la manière suivante:

void RemplissageListe(ITEM *debutListe)
{
   ITEM *pListe = debutListe;

   do {
      pListe->data = Calcul();  // remplit l'item avec les données calculées
      pListe = pListe->suivant; // passe au suivant 
   } while ( pList!=NULL );

}

DATAITEM *Calcul()
{
   DATAITEM data;

   // calcule les données et les place dans data
   ...

   return &data; // retourne l'adresse de la structure locale !!!   
}

Comme vous le voyez, &data retourne toujours l'adresse de la structure DATAITEM présente sur la pile, et de surcroît libérée à la sortie de la fonction Calcul().

Au final, toute la liste de résultats contenait (à la relecture) des valeurs souvent identiques, car correspondant au dernier calcul effectué, parfois altéré par un remplissage inopiné et important de la pile (à l'occasion d'une interruption matérielle, par exemple).

Le programme a tourné plusieurs semaines sans qu'on ne s'aperçoive de rien, car les résultats de la simulations étaient assez plausibles. Seule une analyse plus fine lors des tests finaux a démontré le problème. Et il m'a quand même fallu un peu de temps pour retrouver cette ânerie au beau milieu de plusieurs dizaines de milliers de lignes de code.

Il faut dire, on n'a pas idée... hein ?

 

[PA5CAL]

J'en ai vu passer une de chez Microsoft, que vous pouvez retrouver sur cette page Web.

Voici un morceau de code source de Microsoft Office Visual Basic :

//  Function:   RunCommandEx
//  Synopsis:   runs the given command in the current session, more robust
// than RunCommand
//  Arguments:  none
//  Returns:    S_OK if success
//  History:    October 3, 2000 - created [name withheld to protect the guilty]
HRESULT RunCommandEx(LPCWSTR szCmdLine) {
     DWORD dwTry = 0;
     HRESULT hResult = S_OK;

     // try run command 3 times at most
     while (dwTry<4) {
          hResult = RunCommand(szCmdLine);
          if (hResult!=E_FAIL) {
               // we succeeded
               break;
          }
          dwTry++;
     }
     if (dwTry==4) {
          ATLTRACE(L"COuld start the command even we tried 4 times\n");
          ASSERT(FALSE);
     }
     return hResult;
}

Ils réessayent trois fois, au cas où ça ne marcherait pas du premier coup !

Je vous recommande la lecture de la page web en question (et particulièrement du mémo interne destiné à Bill Gates). Ça vaut son pesant de cacahouettes !