Facepalm code

Forum pour parler de prog. et d'informatique questions/réponses/trucs sont les bienvenus.

Facepalm code

Messagede Oli » Sam Sep 18, 2010 1:07 pm

Postez des trucs que vous voyez dans du code qui vous découragent. Pas besoin nécessairement d'être du copy/paste (après tout partager du code de la job c'est pas l'idéal).

Je viens de voir une classe avec deux variantes de constructeurs (avec chacune une liste d'arguments différents). Ce qui est louche, c'est que la première initialise ses membres, et l'autre fait des assignations sur ses membres. Très beau.
Récamment on a vraiment le don de créer des problèmes ou il n'y en a jamais eu... - face de crane Ouelette
Avatar de l'utilisateur
Oli
le violoneux ronald
 
Messages: 11556
Inscription: Sam Sep 09, 2006 1:22 pm
Localisation: ( ͡° ͜ʖ ͡°)

Re: Facepalm code

Messagede OliDeFordTaurus » Sam Sep 18, 2010 1:19 pm

Un truc qui me gosse toujours c'est le monde qui oublient qu'un booléen, ben c'est déjà un booléen :|.

Code: Tout sélectionner
if(VariableBool1 == true && VariableBool2 == true)


ou

Code: Tout sélectionner
if(VariableBool1)
{
  return true;
}
else
{
  return false;
}



Mais le pire c'est le merge des deux :
Code: Tout sélectionner
if(VariableBool1 == true)
{
  return true;
}
else
{
  return false;
}


ASTI.
"It runs on the world most powerful graphic chip : imagination." - Sheldon Cooper
Avatar de l'utilisateur
OliDeFordTaurus

 
Messages: 2014
Inscription: Jeu Déc 18, 2008 1:08 am

Re: Facepalm code

Messagede Mat » Dim Sep 19, 2010 8:20 pm

Bin ca te rappelle que c'est un boolean.
coco
Avatar de l'utilisateur
Mat
Le refus de friandiser
 
Messages: 4830
Inscription: Jeu Sep 04, 2008 5:39 pm
Localisation: Le forum

Re: Facepalm code

Messagede Mike plus gros que jamais » Dim Sep 19, 2010 10:12 pm

Chez CAE, stait dans le coding standard de l'équipe d'interface où j'étais de faire ça. Pour la clarté.

...
"I would bury my dick so far inside her ass that whoever could pull it out would be crowned king Arthur."
Avatar de l'utilisateur
Mike plus gros que jamais
Pedobear
 
Messages: 3205
Inscription: Lun Sep 18, 2006 12:16 am
Localisation: Dans un apparte à moitié roux

Re: Facepalm code

Messagede ben » Dim Sep 19, 2010 10:28 pm

Moi ce qui me tue, c'est n'importe quel style de boîte de commentaire fancy qui prend 10 minutes à ajuster.

Exemples:

Code: Tout sélectionner
/********************************************************************
**                    BOITE DE COMMENTAIRE                        **
**  Nom:       MaFonction                                         **
**  Arguments: int a : valeur A                                   **
**             int b : valeur B                                   **
********************************************************************/


Code: Tout sélectionner
/////////////////////////////////////////////////////////////////////
//                                                                 //
//   JE CHERCHE VRAIMENT A ATTIRER L'ATTENTION AVEC MA BOITE DE    //
//   COMMENTAIRES PARCE QUE MON EDITEUR EST TROP A me crosser POUR //
//   DONNER UNE COULEUR DIFFERENTE AUX COMMENTAIRES                //
//                                                                 //
/////////////////////////////////////////////////////////////////////


Crisse. Si je voulais du ASCII art, j'irais sur http://www.ascii-art.de/.
ben

 
Messages: 2686
Inscription: Sam Déc 16, 2006 9:53 am
Localisation: Montreal, QC

Re: Facepalm code

Messagede Oli » Dim Sep 19, 2010 11:14 pm

Au cégep on nous disait de faire des boîtes comme ça. :(

[spoiler]Moi non plus j'aime pas ça, j'en ai jamais fait à part si on était obligés.[/spoiler]
Récamment on a vraiment le don de créer des problèmes ou il n'y en a jamais eu... - face de crane Ouelette
Avatar de l'utilisateur
Oli
le violoneux ronald
 
Messages: 11556
Inscription: Sam Sep 09, 2006 1:22 pm
Localisation: ( ͡° ͜ʖ ͡°)

Re: Facepalm code

Messagede Phil » Dim Sep 19, 2010 11:59 pm

moi j'en fais en esti dans mon code lololol
Black Label... Criss moué sa sur la tabel
Avatar de l'utilisateur
Phil
Administrator
 
Messages: 11158
Inscription: Jeu Sep 07, 2006 4:33 pm
Localisation: Fleurimont

Re: Facepalm code

Messagede ben » Lun Sep 20, 2010 12:23 am

C'est pas tellement un problème à *lire*, c'est un problème à *maintenir*.

La complexité de la boîte de commentaire est inversement proportionelle à la probabilité qu'un autre développeur prenne la peine de l'updater quand le nom de la fonction, un argument ou sa description change dans le futur.

À ce point-ci dans la discussion, un développeur Emacs souligne habituellement que la-dite boite de commentaire peut être faite par une macro écrite en Lisp. Et c'est là que je fais mon facepalm.
Dernière édition par ben le Lun Sep 20, 2010 12:29 am, édité 1 fois.
ben

 
Messages: 2686
Inscription: Sam Déc 16, 2006 9:53 am
Localisation: Montreal, QC

Re: Facepalm code

Messagede Poelonne » Lun Sep 20, 2010 12:27 am

Commencer ses noms de méthodes par un M majuscule, c'est affreux. Jusqu'à quel point faut être insécure avec son code, avoir un design obscure ou nommer mal ses affaires pour sentir le besoin de différencier les méthodes des fonctions ou des classes?

À la limite, tagger les propriétés avec un m_var ou mVar, je comprends; tagger les paramètres avec un param_ ou peu importe, je le comprends; tagger les pointeurs avec un pPointeur, je peux le comprendre... Mais différencier les noms de méthode des noms de fonction, come on! Ça va même à l'encontre de certaines "règles" de bases du design en C++...
En cas de doute, souris.
Avatar de l'utilisateur
Poelonne
einstein
 
Messages: 2465
Inscription: Lun Sep 03, 2007 2:43 pm
Localisation: Ascot, Sherbrooke

Re: Facepalm code

Messagede Phil » Lun Sep 20, 2010 2:21 am

ben a écrit:C'est pas tellement un problème à *lire*, c'est un problème à *maintenir*.

maintenir du code, c'est pour ceux qui ont pas écris du code parfait.
Black Label... Criss moué sa sur la tabel
Avatar de l'utilisateur
Phil
Administrator
 
Messages: 11158
Inscription: Jeu Sep 07, 2006 4:33 pm
Localisation: Fleurimont

Re: Facepalm code

Messagede ben » Lun Sep 20, 2010 3:28 am

Phil a écrit:
ben a écrit:C'est pas tellement un problème à *lire*, c'est un problème à *maintenir*.

maintenir du code, c'est pour ceux qui ont pas écris du code parfait.

Si je le maintiens, ca va de soi que c'est pas moi qui l'a écrit :geek1:
ben

 
Messages: 2686
Inscription: Sam Déc 16, 2006 9:53 am
Localisation: Montreal, QC

Re: Facepalm code

Messagede Phil » Lun Sep 20, 2010 4:08 am

indeed, c'était vraiment mal formulé. Ca sert surment a rien que j'essaie de vous faire croire que c'est pas le cas pour mon code...
Black Label... Criss moué sa sur la tabel
Avatar de l'utilisateur
Phil
Administrator
 
Messages: 11158
Inscription: Jeu Sep 07, 2006 4:33 pm
Localisation: Fleurimont

Re: Facepalm code

Messagede Oli » Lun Sep 20, 2010 8:11 am

Poelonne a écrit:Commencer ses noms de méthodes par un M majuscule, c'est affreux.

Ou commencer ses classes par "C" (allô MFC).

Par convention je mets toujours un "m" (ou "mp" si c'est un pointeur) juste avant les variables membre, et un "a" en avant des arguments mais that's it. Je vois souvent des "f" comme préfixe pour la même chose dans notre code par contre ("field" sûrement, mais ça jure avec tout le reste).
Récamment on a vraiment le don de créer des problèmes ou il n'y en a jamais eu... - face de crane Ouelette
Avatar de l'utilisateur
Oli
le violoneux ronald
 
Messages: 11556
Inscription: Sam Sep 09, 2006 1:22 pm
Localisation: ( ͡° ͜ʖ ͡°)

Re: Facepalm code

Messagede Dieu » Lun Sep 20, 2010 11:03 am

Lorsque le "precompiled header" contiens un paquet d'includes juste parce que le monde voulait accéder à ces fichiers par plus qu'une seule classe. Après ça ils modifient une fois un fichier comme difficulty.h et ça recompile tout le projet... Gooood Jooooob!
Talleyrand à un retour des Etats Unis d’Amérique dira « J’y ai trouvé un pays avec trente-deux religions mais une seule sauce. », résumant ainsi l’importance des sauces dans la cuisine française.
Avatar de l'utilisateur
Dieu
NSFW
 
Messages: 7072
Inscription: Dim Sep 10, 2006 6:03 pm
Localisation: (ノಥ益ಥ)ノ彡┻━┻

Re: Facepalm code

Messagede AntonN » Lun Sep 20, 2010 11:37 am

ben a écrit:
Phil a écrit:
ben a écrit:C'est pas tellement un problème à *lire*, c'est un problème à *maintenir*.

maintenir du code, c'est pour ceux qui ont pas écris du code qui compile.

Si je le recompile, ca va de soi que c'est pas moi qui l'a écrit :geek1:
Des fois, je sors du forum pi je me dis WOW!
Avatar de l'utilisateur
AntonN
Tas de marde
 
Messages: 4997
Inscription: Ven Sep 15, 2006 3:15 pm
Localisation: À Laval!

Suivante

Retourner vers Programmation et Informatique

Qui est en ligne

Utilisateurs parcourant ce forum: Aucun utilisateur enregistré et 2 invités

cron