Discussion :

[tuxlinuxien]

Je suis en école d'informatique, et après 3 mois de C, un gars m'a montré un bout de code qui ne fonctionnait pas.

Code :

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

1
2
3
4
5
6
7
8
9
10
11
void my_putchar(char c)
{
    write(1, &c, 1);
}

int main()
{
    my_putchar('Joyeux noel');
    return (0);
}

Il s'est demandé pourquoi tout le monde a rigolé.

[Louis Griffont]

Juste pour réagir par rapport à ce genre de code que je ne trouve pas si idiot que ca. Genre if(true) ou if(1).

Il m'est déjà arrivé de faire ca dans mon code. La raison c'est qu'il ya effectivement un test qui doit être mis à ce niveau, je fais le test mais il s'avère que ce n’est pas le bon test alors je l'enlève mais sans supprimer l'instruction if (cela me permet de me rappeler cela plus tard en voyant un "if" qui n'a pas de sens dans le code).

Bien entendu pour faire une release, il faut enlever le test ou le corriger. Mais ca peut souvent aider de laisser ca dans un code de façon momentané.

Je préfère, pour ma part, mettre le test en commentaire, et même y ajouter d'autres commentaires afin de rappeler le pourquoi du comment on ne fait plus le test !

[Daestrik]

A la fac j'ai été en binôme sur un projet avec quelqu'un qui avait réécrit la fonction malloc, car celle de base n'était pas assez "optimisée".
Ca m'avait bien fait rire.

C'est pas faux. Malloc est lent. De l'autre côté, il faut être capable de le recoder...

On appréciera :
- le fait de réécrire des fonctions standard
- le fait de les réécrire DEUX fois
- l'ordre des paramètres inversé par rapport à la fonction standard
- cette magnifique façon d'écrire un while dans la première fonction

La seule chose que je peux défendre c'est le fait de réécrire des fonctions standards : certaines ont des défauts (comme par exemple ne pas tester les pointeurs) ce qui est TRES gênant quand tu cherches un segfault que jamais tu n'irai chercher dans une fonction "officielle" à laquelle tu fais confiance.

[jaffael]

Un des bouts codes m'ayant fait le plus rire après reprise d'un soft, c'est un calcul de moyenne (en Pascal):

Code :

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

1
2
3
4
If (pointA.X < pointB.X) then
  moyX:= pointA.X + (pointB.X - pointA.X)/2
else
  moyX:= pointB.X + (pointA.X - pointB.X)/2;

ce qui est évidement la même chose (moyX:= (pointA.X + pointB.X)/2 ), seulement quand c'est répété plusieurs fois (en X et en Y aussi), ça devient vite très lourd à lire.

avec des choses aussi du genre (où maval est un boolean)

Code :

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

1
2
3
4
if (maval=true) then
  maval:= false
else
  maval:= true;

[jabbounet]

La seule chose que je peux défendre c'est le fait de réécrire des fonctions standards : certaines ont des défauts (comme par exemple ne pas tester les pointeurs) ce qui est TRES gênant quand tu cherches un segfault que jamais tu n'irai chercher dans une fonction "officielle" à laquelle tu fais confiance.

Il est souvent plus simple de les encapsuler et de contrôler les paramètres plutôt que de les réécrire.

[Daestrik]

Il est souvent plus simple de les encapsuler et de contrôler les paramètres plutôt que de les réécrire.

Certes, mais dans le cas d'un strcpy ça reste raisonnable.

[Arnaud F.]

Une fois j'ai eu droit à un super :

Code SQL :

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

UPDATE champ1 = champ1 FROM uneTable

Je me suis toujours demandé son utilité (surtout en dehors de toute transaction ...)

[Mac LAK]

Je me suis toujours demandé son utilité (surtout en dehors de toute transaction ...)

Déclencher un trigger et/ou une procédure stockée, peut-être ?

[Arnaud F.]

Déclencher un trigger et/ou une procédure stockée, peut-être ?

J'ai vérifié, t'en fais pas, y a pas

[Mac LAK]

J'ai vérifié, t'en fais pas, y a pas

Alors on va dire qu'à une époque lointaine, il y en a eu une, qu'elle a disparu mais pas le code de déclenchement... L'autre hypothèse étant nettement insultante pour le codeur, c'est forcément la première solution, n'est-ce pas ?

[Tofalu]

Déclencher un trigger et/ou une procédure stockée, peut-être ?

Peut être aussi de vérifier que champ1 respecte toujours une contrainte ou bien de vérifier que le jeu d'enregistrements est updatable. Des raisons, on peut toujours en trouver à posterori, mais est ce qu'il y en avait une sur le moment ?

[jabbounet]

personellement je vais pencher pour l'hypothèse de mac lak.

Ceci dit j'ai aussi vu des procédures stockées lancer des exception (Division par zéro) pour sortir en cas d'erreur.

[Mac LAK]

Ceci dit j'ai aussi vu des procédures stockées lancer des exception (Division par zéro) pour sortir en cas d'erreur.

J'ai vu ça aussi en C++ : des boucles cassées par des levées d'exception volontaires au lieu d'un simple break... Ou des exceptions levées en traitement nominal (= à chaque appel), le code utile étant dans le catch et la gestion des erreurs dans le try...

[inconnu652000]

if (test);
{
code
}

C'est bête mais on peut mettre du temps à le voir

[jabbounet]

if (test);
{
code
}

C'est bête mais on peut mettre du temps à le voir

c'est balot, perso je l'ai vu sur des boucles for

Code :

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

1
2
3
for ( ....) ;
   quelquechose

ou une variante

Code :

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

1
2
3
4
for ( ....) 
   quelquechose;
   quelquechose d'autre qui devrais aussi etre dans la boucle;

comme quoi c'est une bonne habitude de mettre systématiquement des accolades sur les boucles.

[Invité]

je suis tombé là dessus aujourd'hui... personnellement je trouve que ça pique un peu les yeux. Le pire c'est que l'appli est bourré de truc du même genre

Code :

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

1
2
3
4
5
6
7
8
9
10
if (MaClass.maFonction(string1, result) == 0)
	if (MaClass.maFonction(string2, result) == 0)
		if (MaClass.maFonction(string3, result) == 0)
			if (MaClass.maFonction(string4, result) == 0)
				if (MaClass.maFonction(string5, result) == 0)
					if (MaClass.maFonction(string6, result) == 0) == 0)
						if (MaClass.maFonction(string7, result) == 0)
							return false;
return true;

[Invité]

ou une variante

Code :

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

1
2
3
4
for ( ....) 
   quelquechose;
   quelquechose d'autre qui devrais aussi etre dans la boucle;

comme quoi c'est une bonne habitude de mettre systématiquement des accolades sur les boucles.

J'ai souvent croise ça aussi il y a quelques mois, sur quelques lignes de code comme c'est le cas ici, c'est assez simple a voir, beaucoup moins quand c'est au milieu de pages et de pages de code du même style (souvent mal indente forcement)... Moi qui ai l'habitude de toujours utiliser les accolades

[deadalnix]

J'ai vu ça aussi en C++ : des boucles cassées par des levées d'exception volontaires au lieu d'un simple break... Ou des exceptions levées en traitement nominal (= à chaque appel), le code utile étant dans le catch et la gestion des erreurs dans le try...

Il y a des gens qui en ont fait un style en java. La programmation par exceptions. Selon eux, c'est génialissime, mais je suis pas franchement convaincu.

[sunseb7]

Le genre de truc que je déteste en PHP, l'utilisation abusive des echo...

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
	<?php
			include ('fonction.inc');
			titre('Pièces justificatives');
		echo "<table border='0' width='95%' CELLPADDING='1' align='center' class='txt_tab_tarif'>";
		echo "<tr><td colspan='2'>";sstitre('Joindre les pièces justificatives suivantes');echo "</td></tr>";
		echo "<tr><td colspan='2'>(<b>photocopie</b> obligatoirement):</td></tr>";
		echo "<tr><td colspan='2'>&nbsp;</td></tr>";
		echo "<tr><td><img src='img/puce2.gif'></td><td>Photocopie du justificatif d'adresse (Eau, EDF...)</td></tr>";
		echo "<tr><td valign='top'><img src='img/puce2.gif'></td><td>Photocopie de l'avis d'imposition ".(date('Y')-2)." ou ".(date('Y')-1)." (si vous êtes imposable)<br> OU photocopie de la dernière notification de la CAF (".date('Y')." avec le quotient familial)</td></tr>";
		echo "<tr><td><img src='img/puce2.gif'></td><td>Cachet de l'établissement scolaire pour les primaires, maternelles et collégiens dans le cadre réservé à cet effet.</td></tr>";
		echo "<tr><td><img src='img/puce2.gif'></td><td>Attestation d'inscription dans l'établissement scolaire pour les élèves fréquentant un lycée</td></tr>";
		echo "<tr><td colspan='2'>&nbsp;</td></tr>";
		echo "<tr><td colspan='2' align='center'><a href='#'><img src='img/b_fermer.gif' border='0' onClick='self.close();'></a></td></tr>";
		echo "</table>";
	?>

Cette page la étant simple, ça va encore... J'en ai de pires :
Celui ci, par exemple, qui, non seulement abuse des echo, mais en plus essaie de sauvegarder des sauts de lignes....
Ah, oui, la technique des tableaux dans des tableaux est un truc de fourbe. On devrait pendre les gens qui osent faire ça !

Code :

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

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
38
39
40
41
42
43
44
			echo "<form name='rech_iti' id='rech_iti' method='GET' action='rech_iti_res.php' target='_self'>";
			echo "<table border='0' width='95%' CELLPADDING='0' align='center' class='txt_tab_tarif'>";
			echo "<tr><td colspan=2 class='txt_sstitre'>Merci de remplir le formulaire suivant</td></tr>";
			echo "<tr><td colspan=2 class='txt_sstitre'>&nbsp;</td></tr>";
			echo "<tr><td>Période de départ souhaité</td><td>";
			echo "<select name='list_per' class='txt_menu_der' id='list_per'>";foreach($tabperiode as $cle=>$valeur){echo "<option class='txt_menu_der' value='".$cle."'>".$valeur."</option>\n";} 
			echo "</select>";
			echo "</td></tr>";
			echo "<tr><td>Heure</td><td>";echo "<select name='heure' class='txt_menu_der' id='heure'>";for($i=0;$i < 24;$i++){echo "<option class='txt_menu_der' value='".$i."'>".$i."</option>\n";}echo "</select> &nbsp; h &nbsp;&nbsp;";echo "<select name='minute' class='txt_menu_der' id='minute'>";for($i=0;$i <= 59;$i{ echo "<option class='txt_menu_der' value='".$i."'>".$i."</option>\n";}echo "</select>&nbsp;&nbsp;mn";
			echo "</td></tr><tr><td colspan='2'>&nbsp;</td></tr>";
			
			//DEPART
			echo "<tr><td colspan='2'>";
			echo "<fieldset style='border:2px solid rgb(191, 191, 191);width: 400px'>";
				echo "<legend class=txt_sstitre>Départ</legend>";
				echo "<table border='0' align='center' width='80%' CELLPADDING='0' class='txt_menu_der'>";
				echo "<tr><td>Commune</td><td>";chmpderoul_noSel_free('list_comm_dep','txt_menu_der',$tabcommune,"onchange=\"go('php_ajax/commune_dep.php','')\"");echo "</td></tr>";
				echo "<tr><td>Rue</td><td>";chmpderoul_mess_free('list_rue_dep','txt_menu_der',$tabrue_dep,'Choisir une commune',"onchange=\"go('php_ajax/rue_dep.php','')\"");echo "</td></tr>";
				echo "<tr><td>Station</td><td>";chmpderoul_mess_free('list_station_dep','txt_menu_der',$tabstation_dep,'Choisir une rue',"");
				echo "<div name='blocdepart' id='blocdepart'><input type='hidden' name='sta_dep' id='sta_dep' value='0'></div>";
				echo "</td></tr>";
				echo "</table>";
			echo "</fieldset></td></tr><tr><td>&nbsp;</td></tr>";
			
			//ARRIVEE
			echo "<tr><td colspan='2'>";
			echo "<fieldset style='border:2px solid rgb(191, 191, 191);width: 400px'>";
				echo "<legend class=txt_sstitre>Arrivée</legend>";
				echo "<table border='0' align='center' width='80%' CELLPADDING='0' class='txt_menu_der'>";
				echo "<tr><td>Commune</td><td>";chmpderoul_noSel_free('list_comm_arr','txt_menu_der',$tabcommune,"onchange=\"go('php_ajax/commune_arr.php','')\"");echo "</td></tr>";
				echo "<tr><td>Rue</td><td>";chmpderoul_mess_free('list_rue_arr','txt_menu_der',$tabrue_dep,'Choisir une commune',"onchange=\"go('php_ajax/rue_arr.php','')\"");echo "</td></tr>";
				echo "<tr><td>Station</td><td>";chmpderoul_mess_free('list_station_arr','txt_menu_der',$tabstation_dep,'Choisir une rue',"");
				echo "<div name='blocarrivee' id='blocarrivee'><input type='hidden' name='sta_arr' id='sta_arr' value='0'></div>";
				echo "</td></tr>";
				echo "</table>";
			echo "</fieldset></td></tr>";
			
			echo "<tr><td colspan='2'>&nbsp;</td></tr><tr><td colspan='2'>&nbsp;</td></tr>";
			echo "<tr><td colspan='2' align='center'><img src='img/b_rechercher.gif' style='cursor:pointer;' border='0'  onClick='testEtValid(\"rech_iti\");'>";
			
			 echo "</td></tr>";
			echo "</table></form>";
		?>

Pour infos, quand on m'a demandé de finaliser le site, j'ai du remettre les pages au propre avant de comprendre ce qui s'y passait...

Quelle solution élégante pour éviter ce problème au fait ? Car c'est vrai qu'on se retrouve parfois vite avec beaucoup de "echo"...

Après pour ce qui est de mélanger traitement et HTML, c'est sûr que c'est juste infernal à maintenir ensuite...

[jabbounet]

Il y a des gens qui en ont fait un style en java. La programmation par exceptions. Selon eux, c'est génialissime, mais je suis pas franchement convaincu.

moi aussi je ne suis pas convaincu