Quel est le pire bout de code que vous ayez jamais écrit ?

[Gordon Fowler]

Quel est le pire bout de code que vous ayez jamais écrit ?
En avez-vous eu honte et pourquoi ?


Un professionnel sérieux se reconnaît au fait qu'il sait évaluer son travail et reconnaître ses erreurs.

Autrement dit, un développeur réfléchi est celui qui sait quand il écrit un code efficace, ou beau et qui répond aux besoins de l'utilisateur ou du client. Mais il sait aussi quand c'est exactement le contraire.

Ihssen, développeur web et membre de la rédaction de Developpez.com, n'a par exemple pas mis longtemps à avouer, non sans une certaine gêne, avoir écrit une application Web communautaire, riche en fonctionnalités... « mais sans avoir la moindre idée sur le concept de jointures en SQL ».

« Pour chaque liste d'affichage, les scripts empilaient donc une ou plusieurs requêtes supplémentaires pour chaque ligne issue de la requête principale. N + 1 était la règle, mais sur plusieurs modules on pouvait trouver des tas de requêtes de type N * M + 1 », explique-t-il.

Au final le site a pourtant fonctionné d'une manière plutôt robuste durant plusieurs années, « mais son serveur dédié a du souffrir. Heureusement qu'il y avait de la pagination ! ».

Autre horreur, Hinault Romaric, lui aussi rédacteur sur Developpez.com, se souvient que « son pire code était pour une petite application de gestion dans le cadre d’un premier stage en entreprise. Le code était bourré des concaténations de chaînes pour les requêtes SQL et de déclarations de variables sans aucune normalisation. Il n’était pas du tout optimisé (utilisation par exemple des if où des switch auraient été suffisants) et encore moins commenté».

Et de conclure « lorsque je tombe parfois sur ce code dans mes archives, il me fait aujourd'hui bien rigoler ». Mais pas à l'époque.

Après avoir abordé le sujet des pires bouts de code que vous ayez vus, et les développements dont vous êtes le plus fier, il nous a paru juste et légitime de vous demander aujourd'hui de confesser vos petites hontes.

Alors, vous ? C'était quoi votre pire bout de code ?

[fregolo52]

Le pire que j'ai fait ? Euhhh ...

C'est lié un bug incompréhensible sur une synchro multithread en C.

Seule solution trouvée : ajouter une trace dans la console (printf).

[mvvvv]

soit positif ou non ...

[elmcherqui]

Le pire code que j'ai fait , c'est pendant mon tous premier stage en .NET y'a 4ans pour une ecole . j'ai du ecrire le code SQL de creation des tables en dure et qui s'execute ligne par ligne ., le tous sans aucune clef primaire ni etrangere . je n'avais pas cette notions mais je savais instinctivement qu'il ne fallais pas faire des operations d'ajout et de suppresion a l'arrache , alors j'ai moi meme cree des classes pour faire le travail du SGBD et verifier ces contrainte par code . et comme j'utilisais MySql pour faire un backup de la table j'ai du ecrire un programme en C++ pour executer des lignes de commandes sur console ( C# ne peut pas le faire a ma connaissance , car j'ai esseye et demandé ) .

c'est sur que y'a un apres , mais sa me reconforte tellement de voir le niveau ou j'etais et comment je pensais par rapport a aujourdui , mais on passe tous par la .

[Hellwing]

Je me souviens d'un petit bout de code plus ridicule que horrible écrit lorsque j'étais encore étudiant.

J'avais écris une méthode qui prenait en paramètre un Booléen et renvoyait un booléen selon le code suivant :

Code :

1
2
3
4
Si le booléen est vrai
  alors je renvoie faux
Si le booléen est faux
  alors je renvoie vrai

Une méthode juste pour ça.

J'ai sans doute dû faire plus horrible comme code depuis, mais celui-ci est mon souvenir le plus mémorable ^^

[Iridian]

En stage de première année, j'ai du ajouter des traces-logs d'informations dans un programme existant ... j'ai du copier/coller des centaines de fois le même bout de code pour écrire dans le log ...

Sinon je fais encore de temps en temps un
si (condition OU vrai)
Pour tester des morceaux de code pendant les développements.

Mais généralement je pense à claquer un //TODO au dessus.

[Lapinpanda]

Code :

1
2
3
4
5
6
 
while(true)
{


}

[elmcherqui]

Envoyé par Lapinpanda

Code :

c'est assez usuelle comme code , faut juste penser a mettre un break ou un return dedans , ou bien mettre le thread qui execute cette partie en background . on utilise souvent une boucle infinie pour ecouter des Client TCP

[dingoth]

Le pire bout de code que j'ai du écrire date de seulement trois mois.

1. Rechercher manuellement un message utilisateur dans toutes les langues et trouver un groupe de lettres commun à tous ces messages mais inexistant dans d'autres messages.
2. Attraper le message lorsqu'il est envoyé afin de mettre éventuellement un booléen à vrai
3. Utiliser le booléen plus tard pour décider d'envoyer ou non un autre message.

Pourquoi ce code ? Parce que nous n'avons (oui, au présent : il n'est pas encore corrigé, ce problème) pas de possibilité de découvrir le type de message recherché par nous même : on nous envoie 3 informations, l'une d'elle est celle qu'on cherche, mais les 3 ne sont jamais dans le même ordre. Les messages, eux, sont correctement placés dans la base de données. On a interdit aux gars du business de changer les messages sans notre accord préalable à cause de ce problème. Heureusement, le projet pour fixer cela arrive. Ouf !

Dépendre ainsi des messages utilisateurs qui peuvent changer à tout moment m'angoisse comme c'est pas possible !

[lastrecrue]

en entrer de méthode j'ai mis

Code :

1
2
if(true)
throw new Excpetion("a");

[Hervé Saladin]

Mon pire code c'est lorsque j'ai débuté la prog en total autodidacte étant ado (il y a entre 15 et 20 ans).
J'avais un Thompson TO8 (il était déjà vieux pour son époque) qui se programmait en BASIC, l'édition du code lui-même se faisait en lignes de commandes car je n'avait pas d'éditeur de texte, je n'avais pas la moindre notion d'algo ni d'informatique en général, j'ai appris tout seul par jeu de façon empirique en modifiant des programmes existant pour voir ce que ça fait et en lisant le manuel utilisateur de l'ordi.
Résultat : je me suis inventé ma propre "méthode" (hum) de programmation et quand j'ai commencé à me sentir un peu plus à l'aise, j'ai créé à partir de 0 un jeu de Jackpot avec une interface graphique représentant une machine à sous avec ses symoles qui défilent, et même une animation du bras du "bandit manchot".

Le tout tenait dans un seul fichier .bas de code BASIC brut qui devait faire 1000 ou 2000 lignes numérotées (de 10 en 10 pour laisser de la place au cas où il faille par la suite insérer une ligne entre les deux), ne contenait pas la moindre fonction ni procédure ni la moindre boucle for ou while, ni de else ... toutes ces structures dont j'ignorais l'existence étaient remplacées par des test simples avec du renvoi brutal à grand renfort de GOTO du genre :

Code :

1
2
3
4
100 A=1
110 PRINT A
120 A=A+1
130 IF A<10 GOTO 100

... en fait c'était tellement "brut" avec des instruction tellement simple et décomposées, c'était presque comme de l'assembleur quand j'y repense.
Pas non plus d'expression booléenne comprenant plus d'un test (pas de ET ni de OU), et toutes les variables étaient nommées A,B,C et ainsi de suite ... et le pire c'est que ça marchait et même plutôt bien et que j'étais capable de m'y retrouver dans mon code !

Avec du recul je me demande comment je faisais pour comprendre cet infâme charabia, si je m'y replongeais aujourd'hui je serais incapable de le relire ou de refaire la même chose.
Cela dit même si c'était une horreur je n'en ai pas honte, je suis même plutôt fier d'avoir commencé comme ça parce que je trouve que j'avais quand même du mérite à avoir découvert ça tout seul sans la moindre aide et quasiment sans la moindre documentation, et à m'être suffisamment acharné pour maîtriser ça "à ma façon".

[nicorama]

En PHP, au lieu de tester si la méthode d'une entité existe, j'ai fait un Try/Catch. Il me restait deux heures avant de livrer le projet et je ne savais pas comment faire ce test dans ce fichu langage. Je ne sais d'ailleurs toujours pas

J'ai pas honte, parce que j'ai gentiment mis une note dans le document de dette technique.

[octal]

Envoyé par Lapinpanda

Code :

Où est le problème dans ce code? mon PIC16F84 il aime bien ce code là

[6-MarViN]

Code :

for (i = 0; i < j; ++i);

au lieu d'un simple

Code :

i = j;

J'étais fatigué quand j'ai écrit ça. Je vous dis pas ma tête quand le lendemain je suis retombé dessus.

[xelab]

Envoyé par nicorama

En PHP, au lieu de tester si la méthode d'une entité existe, j'ai fait un Try/Catch. Il me restait deux heures avant de livrer le projet et je ne savais pas comment faire ce test dans ce fichu langage. Je ne sais d'ailleurs toujours pas

J'ai pas honte, parce que j'ai gentiment mis une note dans le document de dette technique.

http://www.php.net/manual/fr/function.method-exists.php

[pyros]

Un vendredi soir, une fonction de tri sur un std::vector en o(n^3) (sisi, c'est possible...)

Le lundi matin en arrivant je me suis souvenu que la fonction std::sort existait...

[rt15]

Je devais porter une appli C/C++ vers Windows 64 bits.
Le problème est que cette appli nécessitait une XXX.dll propriétaire dispo en 32 bit uniquement (Et exposant une trentaine de fonctions avec la convention d'appel cdecl).

Impossible d'utiliser une dll 32 dans un processus 64. Du coup j'ai fait un wrapper :
appli 64 bit -> dll 64 bit reprenant l'interface de XXX.dll -> Serveur COM en VB6 -> dll 32 bit exposant l'interface de XXX.dll, mais en convention stdcall -> XXX.dll.

Cette usine à gaz est une solution temporaire faite à la va vite en attendant le portage de XXX.dll. Mais depuis je fais des cauchemars en imaginant que cet infâme sandwich puisse malencontreusement partir en production...

[B.AF]

Ma premiére appli pro.

Je n'avais jamais fait de vrai dev pro et donc entre la bd (access évidemment), les tables sans index, les requêtes codées en dur et les écrans multicolors (en dehors des icones bart simpson).

Le pire c'est qu'elle est en prod et fonctionne tous les jours...

[Godestiny]

int pasBien;
int * maladie;
int q = sizeof(maladie) * 5;

if(pasBien){
maladie = (* int) malloc(q);
}

[redbullch]

J'ai lu dans le code d'un enseignant en école d'ingénieur :

while(nombre >= 256)
{

nombre = nombre % 256;

}

Je crois qu'il n'avait pas totalement assimilé le concept "reste de la division entière".