Le pire bout de code que vous ayez vu

**Just Modeste** · 18/12/2009, 14h11

J'ai droit à des trucs du genre :

Code :

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

1
2
DecodeDate(IncDay(StrToDate(IBQuery->FieldByName("REPROCALC_DatDerIaVel")->AsString),m_DureeGest.ToInt()),year,month,day);

Invité · 18/12/2009, 15h08

Envoyé par Just Modeste

J'ai droit à des trucs du genre :

Code :

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

1
2
DecodeDate(IncDay(StrToDate(IBQuery->FieldByName("REPROCALC_DatDerIaVel")->AsString),m_DureeGest.ToInt()),year,month,day);

c'est vrai que c'est moche mais c'est amusant de faire un max de choses en 1 ligne

ex :

Code :

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

elc = string.Join(";", Array.ConvertAll<char, string>(elc.ToCharArray(), new Converter<char, string>(delegate(char un_char) { return un_char.ToString(); })));

pour insérer un ; entre chaques caractère de elc

**Pill_S** · 18/12/2009, 19h26

Envoyé par seebz

Je suis tombé sur ça récemment, sur une application payante :

(fichier inclus sur toutes les pages)

Code :

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

1
2
3
4
5
6
7
8
if (isset($_GET)) { extract($_GET); }
if (isset($_POST)) { extract($_POST); }
if (isset($_SESSION)) { extract($_SESSION); }
if (isset($_SERVER)) { extract($_SERVER); }
if (isset($_ENV)) { extract($_ENV); }
if (isset($_COOKIE)) { extract($_COOKIE); }
if (isset($_FILES)) { extract($_FILES); }
if (isset($_REQUEST)) { extract($_REQUEST); }

ailleurs :

Code :

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

if($_SESSION['xxxx_admin']) {

et encore plus loin :

Code :

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

1
2
3
$file_type = $_FILES['uploadedfile']['type']; 
$allowed_ext = array('image/pjpeg','image/gif','image/jpeg');
if(in_array($file_type, $allowed_ext)){

très franchement je comprend pas où est le problème... ok c'est pas le meilleur code du monde mais il n'y a ni faute grave ni erreur syntaxique ni quoi que ce soit

**metagoto** · 18/12/2009, 20h07

Envoyé par Pill_S

très franchement je comprend pas où est le problème... ok c'est pas le meilleur code du monde mais il n'y a ni faute grave ni erreur syntaxique ni quoi que ce soit

La faute grave serait la sécurité. Le code est calamiteux de ce point de vue là

**3uclide** · 19/12/2009, 19h44

j'ai moi même fait un truc du genre

Code :

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

1
2
3
4
5
for(i = 0; i <=1; i++)
{
    //traitement
}

Y a un truc que je n'avais pas encore compris :o

**Médinoc** · 19/12/2009, 23h23

Ben quoi? C'est une façon parfaitement OK de faire quelque chose deux fois...

**jabbounet** · 20/12/2009, 18h38

Envoyé par Médinoc

Ben quoi? C'est une façon parfaitement OK de faire quelque chose deux fois...

je suis d'accord ça évite un copier coller d'un code, et en plus tu peux te servir de l'indice pour subtilement modifier le traitement sur la seconde passe

**tresorunikin** · 26/12/2009, 21h10

Un grand développeur de sites webs (sous macromedia) mais un jour, devant le Bloc note il ns a vraiement mis d'accord qu'il était fou avec ces lignes!

Code :

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

1
2
3
4
5
6
7
8
9
10
<html>
<head><title color="red">Une page sous bloc note</title>
</head>
<center>
<body>
Voici que la page est centré
</body>
</center>
</html>

Comme c'est surieux de colorer le titre d'une page!

**Alain Defrance** · 27/12/2009, 11h15

Envoyé par tresorunikin

Un grand développeur de sites webs (sous macromedia) mais un jour, devant le Bloc note il ns a vraiement mis d'accord qu'il était fou avec ces lignes!

Code :

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

1
2
3
4
5
6
7
8
9
10
<html>
<head><title color="red">Une page sous bloc note</title>
</head>
<center>
<body>
Voici que la page est centré
</body>
</center>
</html>

Comme c'est surieux de colorer le titre d'une page!

Rho faudrait lui expliquer ce que c'est de la validation XML

**ILP** · 28/12/2009, 22h10

Envoyé par Alain Defrance

Rho faudrait lui expliquer ce que c'est de la validation XML

D'un point de vue XML, ce qu'il a écrit est correct en "standalone". Mais du point de vue HTML ou xHTML, c'est l'horreur

.

**jabbounet** · 29/12/2009, 01h32

Envoyé par ILP

D'un point de vue XML, ce qu'il a écrit est correct en "standalone". Mais du point de vue HTML ou xHTML, c'est l'horreur

.

d'un certain coté si j'ai bien compris ce développeur a l'habitude d'utiliser un outils pour générer ses pages. il est probable qu'il n'ai pas regardé le code (généré) de près depuis longtemps d'où son erreur. Comme quoi savoir comment marche son outil de travail et ce qu'il y'a en dessous peu aider dans les cas extrême ou tu te retrouve en slip a coder au bloc note à ne pas faire certaines erreurs..

**spidermario** · 01/01/2010, 16h13

Mais tout de même, colorer le titre en rouge, à quoi pensait-il ?

Espérait-il obtenir ça ?

**Haegemonia** · 03/01/2010, 00h34

Oh moi j'en ai une pas mal....

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
unsigned int hachage(char* p){
   unsigned int h=0,g;
   for(;*p;p++){
      h=(h<<4)+*p;
      if((g=h&0xf0000000)){
         h=h^(g>>24);
         h=h^g;
      }
   }
   return 0;
}

Un pote travaillait sur un projet utilisant une fonction de hachage... Il l'ecrit et se rend compte que quelque soit la valeur... elle retournait 0 tout le temps... Et 1 Heure après on arrive dans la salle, on le voit enervé un max, et on lui dit :

Ben c'est logique

Vous avez vu pourquoi

**jabbounet** · 03/01/2010, 02h50

Envoyé par Haegemonia

Oh moi j'en ai une pas mal....

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
unsigned int hachage(char* p){
   unsigned int h=0,g;
   for(;*p;p++){
      h=(h<<4)+*p;
      if((g=h&0xf0000000)){
         h=h^(g>>24);
         h=h^g;
      }
   }
   return 0;
}

cela s'appelle une faute d'étourderie..... un oeil neuf la voit facilement le gars qui l'a développé se fait avoir par son cerveau qui ne veux pas voire la connerie dans le fichier car ce dernier pense que le code est comme il est dans la tête.....

mine de rien elle font souvent perdre du temps celles là

**dvdbly** · 04/01/2010, 08h35

Envoyé par monsieurben

J'ai bossé avec un presta tellement nul que je sais pas trop quel bout de code choisir.

J'ai eu droit à celui-ci, plutôt pas mal :

Dans une page de traitement d'un formulaire :

Code :

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

1
2
3
4
5
<?php 

$nom = post('nom');
$prenom = post('prenom');
[...]

Et dans un fichier inclus, ladite fonction :

Code :

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

1
2
3
4
<?php
function post($var) {
return $_POST['$var'];
}

La fonction ne servait strictement à rien, mais elle était consciencieusement appliquée à tous les champs de tous les formulaires (plus de 500 champs)...

Perso, j'y vois un intérêt certes atténué par le chercher/remplacer :
en cas de modification du nom du tableau, on ne le change qu'une fois sans risque d'oublier une affectation dans un include quelconque... si j'ai bien compris.

**othebault** · 09/01/2010, 01h15

Mode séquentiel en Pascal

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
var
  idx:integer;
begin
  for idx:=1 to 10 do
  begin
    case idx do
    1:begin.....end;
    2:begin.....end;
// et ainsi de suite, un traitement pour chaque valeur de idx
    10:begin...;end;
    end;
  end;

J'ai supposé que c'était conçu pour distraire le mainteneur...

J'ai trouvé aussi quelques techniques que l'on pourrait qualifier, comme dire, de "différentes" :

Exemple 1 : Qu'est ce qu'une variable ?

Une procédure longue (300 lignes environs) qui au début du traitement stocke un premier résultat dans un champ d'une table de paramètres d'un SGBD.
Jusque là, tout va presque bien.
Quelques dizaines de lignes plus loin, la même procédure lit la valeur de ce champ, pour effectuer un nouveau traitement.

Passons sur le fait que cette procédure était monstrueuse. Le plus drôle (??) c'est que ce paramètre en SGBD n'était utilisé que dans cette procédure.

Exemple 2 : Comment faire un ping en delphi 3 sur win 9x ?

1 - ouvrir un fichier texte en écriture
2 - dans ce fichier écrire le source d'un batch qui va appeler la commande ping "adresse" > resultat.txt
3 - lancer le batch par createprocess
4 - charger le fichier resultat.txt
5 - interpréter correctement le texte de resultat.txt

Tout ça au lieu d'utiliser la fonction qui va bien dans icmp.dll

**Arnaud F.** · 09/01/2010, 12h10

Vu récemment :

Code SQL :

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

1
2
3
4
5
6
7
8
9
10
11
12
 
IF w_scan LIKE 'd%' -- w_scan commence par 'd'
BEGIN
	IF substring(w_scan, 1, 1) <> 'd' -- si w_scan ne commence pas par d
		-- Erreur
 
	w_allee =  w_allee2
	w_prof = w_prof2
 
	IF w_allee = w_allee2 AND w_prof = w_prof2 AND w_scan LIKE 'd%'
		-- Suite traitement
END

Code qui pourrait tout simplement s'écrire

Code SQL :

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

1
2
3
4
5
6
7
8
 
IF w_scan LIKE 'd%' -- w_scan commence par 'd'
BEGIN
	w_allee =  w_allee2
	w_prof = w_prof2
 
	-- Suite traitement
END

**Médinoc** · 09/01/2010, 12h57

Wow, le retour du for-case paradigm!

**GoTrUnKo** · 11/01/2010, 12h25

Code PHP :

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

1
2
3
4
5
6
7
8
9
$retour_messages=mysql_query("selet * from matable";
			if(isset($_GET['Sort']))	{			   
		switch($_GET['Sort']){
					case('last') echo " ORDER BY lastname";
								break;					
					default: echo " ORDER BY firstname";
					}
			}
else {echo " ORDER BY firstname";}." LIMIT ".$premiereEntree.", ".$messagesParPage."");