Je te trouve quand même un peu radical voire excessif... il serait quand même bon entre temps d'essayer de vérifier si des choses peuvent être préservées :
Code:
1
2
3
4 // Puisqu'à partir de là il faut douter de tout... else if(42==42){ ... }
Version imprimable
Vu à l'instant dans du code :
Citation:
// on cache la date de naissance pour ne pas l'afficher (sinon c'est ballot...)
Bon, c'est pas un commentaire, mais je pense que ça rentre bien dans cette catégorie (développé par le manager de ma team actuelle). Simplifié pour la cause :
Petite précisions : Toute réponse anormale (erreur 500, ...) déclenche une errorException
Résultat :Code:
1
2
3
4
5
6 class ErrorException{ public static void ErrorException(String message){ MailAToutesLaTeamDev("Vous n'avez apparemment pas assez testé votre code. Corrigez ça vite fait avant que je vous tombe dessus." + stacktrace) } }
- Spam énorme les premières semaines
- Process de tests suivi les semaines suivantes
Dans un projet microcontroleur embeddé destiné à voler dans la station MIR. Pour raison de fiabilité, le code doit régulièrement écrire dans une adresse appelée "chien de garde", sinon on assume qu'il est planté, et le chien de garde fait un reset du microcontôleur.
Pour cette écriture, le commentaire suivant s'imposait donc:
//porter la pâtée au chien de garde
Ça a volé comme ça :D
Oui, je sais, c'est un affreux anglicisme, mais je ne voyais pas comment traduire. "empieuté" serait pas mal, mais personne n'aurait comprisCitation:
embeddé :mur:Citation:
Citation: Envoyé par Richard Trigaux
:zzz:
:calim2:
Et sinon, "embarqué" éventuellement ? :roll:
(cette micro-société du bout du monde a choisi cette option, entre très très nombreux autres exemples, mais il en fallait bien un... :mrgreen:)
Citation:
Et sinon, "embarqué" éventuellement ?
ah, oui, euuuh.... merci :oops:
Bien sûr je le savais, juste je trouvais pas mes mots :mrgreen:
Vous pouvez arrêter de vous taper la tête au mur, maintenant :-)
Sur un jeu php où je participait au développement, il devait y avoir une IA, faite par un des prog. Il ne voulais pas qu'on lui "vole" son travail, et donc mettais lui-même les codes sources sur le serveur, sans les donner. L'IA était buggé et n'avançais pas
Un jour, j'ai obtenu le code source. Il avais été piqué d'un autre jeu php, open-source lui.
Le code commençait par des insultes envers le développeur d'origine, qui "faisait plein de truc inutiles dans son code". Suivit de commentaire genre "ca sert à quoi ça?", etc
l'IA du jeu open-source marchait très bien, celle du jeu où je développais ne marchait pas. :)
dans la boite ou j'etais, le patron developpait en mode commando, avec des fonctions de min 3-4 ecrans de haut et qqfois plusieurs en largeur aussi..
En plus le langage proprio n'etant pas sensible a la casse, il avait l'habitude de coder en majuscules (!!!)..
Bref, le code commis etait assez incomprehensible, et j'avais vu au detour d'une correction faite par un de mes collegues :
"Le style est inimitable ... "..
Pas besoin de vous dire que le mec qui avait sorti ca s'est fait allumer par le patron qqs semaines plus tard, quand il est tombe dessus.
Une fois, devant une page HTML qui s'affichait mal j'ai machinalement fait un "view source" et j'y ai lu quelque chose du genre:
<!-- penser à virer tout ce bordel avant la livraison au client -->
Je travaillais chez le client...
Tiens, je viens de retrouver un commentaire dans une des procédure d'une des applications que j'ai repris à la maintenance..
suivit d'une exécution de procédure..Citation:
--Rustine permettant de ne pas intégrér de doublons emails
ça, c'est de l'efficacité !
Retrouvé dans une vieille application développée en mode pompier
Code:
1
2
3
4
5
6
7
8 try{ //Dieu nous garde !! ... du code de m... } catch(Exception $e){ // Ce salopard de Dieu nous a lâchés !! mail($adresseAdmin,"C'est arrivé !!","Avant d'aller pointer à l'ANPE => ".$e->getMessage()); }
Grâce aux TODO qu'Eclipse détecte, j'ai trouvé ça :
OMG..... Ce truc tourne comme ça depuis plusieurs années, j'ai peur :mrgreen:Code:
1
2
3
4
5 /* * Cela ne nous concerne pas TODO c'est louche on devrait * avoir detecté la fin de la valeur avant ... */
Un truc que j'avais vu qui m'avait fait sourire.
Je me suis toujours demandé pourquoi ce else ? Et pourquoi un case ! :?Code:
1
2
3
4
5
6
7
8
9
10
11
12
13 //VarBool est déclarer comme un Booléen pouvant donc prendre 2 valeurs uniquement (True et False) Case VarBool Of True : Begin ... du code end False : Begin ... du code end Else Begin ... du code End End;
Sinon, un truc que j'ai fait, ici. Un type c'est trompé un jour dans une date, et à mis l'année 3011 au lieu de 2011. Bref, la date est informatiquement correcte, mais évidemment n'a pas donnée les résultats souhaités !
On m'a dit que je devais sécuriser la saisie de date... Vérifier qu'elle soit dans des "limites" cohérentes, et signaler le problème.
Alors, dans le cas ou la date dépasse de plus d'un mois, la date en cours, j'affiche un message
Si la date dépasse l'annéeCitation:
Hola, t'es sur de ton coup ?
Si la date dépasse le siècleCitation:
He bin, ça c'est de la prévision !
Ça a beaucoup fait rire, car évidemment, ça c'est reproduit ! :mouarf:Citation:
He, Ho ! N'anticipons pas, veux-tu ? Reposes ton verre, passes-toi un gant d'eau fraiche sur le visage, et refais-ta saisie !
Un jour je devais écrire du code qui utilisait une librairie qu'un de nos fournisseurs nous avait vendu, et je tombais systématiquement sur un bug au même endroit...
Un bug que je n'arrivais pas à comprendre, donc encore plus difficile à corriger. J'ai dû faire appel à plusieurs de mes collègues pour m'aider et aucun d'eux n'a pu trouver la solution !
Du coup, on en a conclu (de façon quelque peu hasardeuse) que la librairie était buggée. Après diverses négociations avec le fournisseur, j'ai réussi à obtenir le code source des fonctions qui posaient potentiellement problème.
Et je suis tombé sur un commentaire mémorable !! Je le partage avec vous :
Mémorable !!Code:// This should work but it doesn't