Les codes les plus hallucinants rencontrés :

[Bovino]

Envoyé par RomainVALERI

Vous connaissez les "solutions de developpement logiciel clefs-en-main"

Hélas, oui...

D'ailleurs, mon patron m'a un jour expliqué que leur base de données est la meilleure, parce qu'avec elle, on peut appeler tous les champs pareil ce qui permet de récupérer les résultats dans un tableau...

[RomainVALERI]

Envoyé par Bovino

mon patron...

En effet, encore un qui a "les cheveux pointus"

[tumoo]

Je viens de rencontrer ça dans le projet que je reprends

Code html :

<hr style="font-weight: bold" />

[RomainVALERI]

Envoyé par tumoo

Je viens de rencontrer ça dans le projet que je reprends

Code html :

<hr style="font-weight: bold" />

Te plains pas ça aurait pu être pire...

Code html :

<br style="font-weight: bold" />

[Bovino]

Tiens... je viens de tomber sur un joli nom de variable...

Code :

lNbLignesTableauVide

Dommage que le code ne soit pas commenté, ça aurait pu valoir le coup !

[RomainVALERI]

Envoyé par Bovino

Tiens... je viens de tomber sur un joli nom de variable...

Code :

lNbLignesTableauVide

Dommage que le code ne soit pas commenté, ça aurait pu valoir le coup !

Mais grave...

Par chez moi la mode serait plutot aux "anciennes versions" (comprendre par-là les tentatives précédentes, en mode shotgun programming), assaisonné d'autres choses plus ou moins sans rapport...

Code javascript :

1
2
3
4
5
6
7
8
// NE PAS MODIFIER ça peut bugger
//lNbLignesTableauZero
// rappeler la MOA > dates validation !!!
//lNbLignesTableauZero = 0
//lNbLignesTableauZero = eval(void(0))
//ne pas oublier d'acheter du pain
//lNbLignesTableau = lNbLignesTableauOld
lNbLignesTableauVide

[Bovino]

Pour la beauté du geste, le contexte dans lequel je l'ai trouvée

Code :

POUR lCpt=1 A lNbLignesTableauVide

[RomainVALERI]

...du basic francisé ou du pseudo-code ?

[Bovino]

WLangage

[RomainVALERI]

Envoyé par Bovino

WLangage

ARF
...le pire c'est que je n'avais pas reconnu... j'en ai déjà fait mais on a le choix des mots-clefs FR ou EN avec Windev/Webdev, et là où je bossais tout était en EN ^^

Mon cerveau a du opérer une auto-lobotomie protectrice pour que je souffre moins

Je me rappelle en revanche de ce genre de déclarations dans le code :

Code :

1
2
maChaine est une chaine.
monAutreChaine et celleCi sont des chaines.

(oui oui ça compile très bien tel quel)

[Ivelios]

maChaine est une chaine.
monAutreChaine et celleCi sont des chaines.

Ce langage pourrais te plaire Shakespeare Programming Language

[RomainVALERI]

Envoyé par Ivelios

Ce langage pourrais te plaire Shakespeare Programming Language

Mais alors... ça permettrait d'écrire des lettres d'amour ??!!

[SamSer]

Je tombe parfois sur des fonctions qui ont des if dont le contenu dépassent les milles lignes... C'est super révélateur. Si je dois faire un changement et que ça fonctionne, ça veut dire que j'ai absolument tout compris au code.

Je tombe également sur du code dont les commentaires sont faux parce qu'ils ont été copié collé d'un autre code. Du coup, je ne lis plus les commentaires et c'est bien mieux comme ça.

Le plus beau reste quand même le code java qui n'a qu'une main ...

[rawsrc]

Envoyé par SamSer

Je tombe parfois sur des fonctions qui ont des if dont le contenu dépassent les milles lignes... C'est super révélateur. Si je dois faire un changement...

M'est d'avis que tu dois sacrément serrer les fesses avant de modifier un truc.
Pour l'avoir vécu, c'est drôlement stressant. Tu tombes sur une classe de 6985 lignes d'un bloc et on te dis y'a un problème dedans...
Oui c'est-à-dire ? Le gars te lance un bon courage et se casse. Et là tu peux aller directement poster sur VDM

[SamSer]

M'est d'avis que tu dois sacrément serrer les fesses avant de modifier un truc.

Je bosse en open space donc serrer les fesses, au sens propre ou figuré, n'est pas une option

Le problème est juste que les codeurs d'il y a dix ans programmaient un peu n'importe comment.D'ailleurs, on me sort souvent comme raison quand je vois un truc débile:
"Hum, c'est historique et vu que pas mal de morceaux de code se reposent la dessus, on peut pas changer."

En gros, le type qui a fait ça était nul mais ça a été remarqué trop tard. Du coup, on fait avec ...

[el_slapper]

Envoyé par SamSer

Le problème est juste que les codeurs d'il y a dix ans programmaient un peu n'importe comment.D'ailleurs, on me sort souvent comme raison quand je vois un truc débile:

Pas plus qu'aujourd'hui. Mais ceux d'aujourd'hui, personne n'a encore remarqué leur code.

Envoyé par SamSer

"Hum, c'est historique et vu que pas mal de morceaux de code se reposent la dessus, on peut pas changer."

Hélas souvent vrai. Parfois c'est de la flemme ou de la trouille, mais souvent il est très délicat de tout casser pour refaire propre. D'autant plus que chacune des 35000 lignes du programme(j'ai pas dit projet, j'ai dit programme.) a été polie, raffinée, corrigée et déboguée pendant des années.

Envoyé par SamSer

En gros, le type qui a fait ça était nul mais ça a été remarqué trop tard. Du coup, on fait avec ...

Pas toujours(même si souvent.....). L'auteur des 35000 lignes a plus d'une fois tiré la sonette d'alarme pour dire qu'il fallait changer d'architecture avant que le monstre ne le dévore. Mais il n'a pas eu le droit de le faire.....

Enfin, les 35000 lignes étaient proprement découpées, avec des découpages cohérents, des paragraphes assez courts, plein de choses centralisées. Mais quand la conception, c'est 72(oui, 72) modules accesseurs à appeler un par un, les résultats des uns étant les paramètres d'appel des autres, et pas toujours dans le même ordre, avec plein d'exceptions fonctionelles dans tous les sens, le monstre devient aussi indécoupable qu'indispensable. L'horreur.

[bernard59139]

Bonjour

Dans un langage de type script, un prog a voulu provoquer une attente de quelques secondes. Sans rire, son code:

Code :

1
2
3
4
DO 10000
 une instruction bidon pour ralentir la boucle
END

La tête des administrateurs quand ils ont constaté la chose.

[kolodz]

Je crois que le plus beau code que j'ai vue pour le moment et qui était destiné à fonctionné c'est celui-ci :

Code java :

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
31
32
33
34
35
36
37
 
 
		public class Main {
 
			public class Couple<Si, Sj> extends java.lang.Object {
 
				private Si a;
				private Sj b;
 
				Couple (Si S01, Sj S02) {
				    a = S01;
				    b = S02;
				}
 
			    }
 
 
	public void Automates(){
 
		int i;
		 String[]Eetat_produit;
		 String[]Xalpha_produit;
		 String[]IItrans_produit;
 
		 Vector List_Etat_trait = new Vector(); 
		 Vector List_Nouv_etat = new Vector();
		 Vector List_Trans_prod = new Vector();
		 Vector Ens_Etatf_prod = new Vector();
 
 
 
 
	}	
 
			public static void main(String argv[]) {
 
		    }

Le plus beau pour moi, c'est le extends java.lang.Object.
Il sait où se trouve cette classe, mais personne ne lui a dit que tout le monde en héritait automatiquement.

Cordialement,
Patrick Kolodziejczyk.

[dark.jabberwock]

Personnellement, le plus hallucinant que j'ai vu doit bien être celui-là en Java.

Ma boîte avait récupéré un projet dont une fonctionnalité était assez lente.

Voici le code incriminé :

Code java :

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
    /**
     * Permet de récupérer l'index courant de la liste
     * 
     * @return index courant sélectionné ou -1 si une erreur c'est produite ( si il y a plus d'un élément sélecvtionné et que la méthode est quand
     *         même appelée
     */
    protected int currentSelectedIndex() {
 
        if (selectedListToUpdate().size() >= 1) {
            for (int i = 0; i < listWhereNavigate().size(); i++) {
 
                if ((selectedListToUpdate().size() == 1)
                    && (listWhereNavigate().get(i).equals(selectedListToUpdate().get(0)))) {
                    return i;
                }
                if ((selectedListToUpdate().size() > 1)
                    && (listWhereNavigate().get(i).equals(selectedListToUpdate().get(selectedListToUpdate()
                            .size() - 1)))) {
                    return i;
                }
            }
        }
        return -1;
    }

Sachant que selectedListToUpdate() est une méthode qui va chercher une liste en cours d'utilisation et que listWhereNavigate() créée une liste triée à partir d'une première liste.

L'amélioration des performances nous a pris beaucoup plus de temps à trouver qu'a effectuer...

[apophyss]

Les pires codes que j'ai trouvé sont des codes trop optimisés (pourquoi faire clair en 15 lignes quand on peut tout faire en une seule ?).

Je préfère un code moins "puriste" mais facile à lire, bien indenter
(surtout que j'ai régulièrement des débutants pour m'aider)...