Bienvenue sur PEBKAC.fr, le site qui recense les anecdotes où l’on se moque des utilisateurs ne maîtrisant pas l’outil informatique. PEBKAC est un acronyme signifiant « Problem Exists Between Keyboard And Chair ».
Le problème se situe entre la chaise et le clavier : soumettez vos histoires, donnez votre avis !
Ce site n'est pas le site original pebkac.fr. Je publie ici la liste des PEBKAC que j'ai pu sauvegarder avant que le site original ne soit mis hors ligne.
Stagiaire dans le Service Public, l'application que j'utilise passe par une interface Web. Curieux de nature, j'affiche les sources de l'application pour voir à peu près comment tourne la bête. Et j'y vois ceci :

var ogtCloseContext = "Fermer"; // Fermer
var ogtCancel = "Annuler"; // Annuler
var ogtOK = "Ok"; // OK
var ogtNon = "Non"; // Non
var ogtOui = "Oui"; // Oui
var ogtClose = "Fermer"; // Fermer
var ogtReset = "Réinitialiser"; // Réinitialiser
var ogtUrlLogout = ""; // URL logout

Et je peux vous affirmer que des commentaires inutiles comme ça, il y en a quasiment à chaque ligne. PEBKAC.
PEBKAC #5153 proposé par jude le 29/08/2012 | 31 commentaires | 👍🏽 👎🏽 +168
Mieux vaut trop que pas assez.
Commentaire #57130 écrit par ROB le 29/08/2012 à 17h36 | 👍🏽 👎🏽
Voilà, c'est exactement ce que j'allais dire. ^^
Commentaire #57131 écrit par Portanie le 29/08/2012 à 17h37 | 👍🏽 👎🏽
var userComment = "Commentaire"; // Commentaire
Commentaire #57132 écrit par Acné le 29/08/2012 à 17h48 | 👍🏽 👎🏽
En général, quand on me dit trop que je ne fais pas assez de commentaires, ça finit comme ça.

Pas tellement un PEBKAC selon moi...
Commentaire #57133 écrit par Skefrep le 29/08/2012 à 17h48 | 👍🏽 👎🏽
Je me rappelle de mes premiers cours à la fac, ou les enseignants nous incitaient à en faire des caisses comme ca.
Commentaire #57135 écrit par Towac le 29/08/2012 à 17h53 | 👍🏽 👎🏽
J'ai l'impression que c'est plutôt une pseudo-blague du précédent développeur, qui en avait probablement marre qu'on lui dise qu'il fallait qu'il "commente plus".

Quand j'étais étudiant, quand mes profs (qui d'ailleurs devaient avoir moins de lignes de code que moi au compteur, malgré leur âge) me saoulaient avec les commentaires "toutes les 2 lignes au moins" (ouais ! bonne idée ça), alors ça finissait comme ça aussi.
Commentaire #57136 écrit par Clem le 29/08/2012 à 17h54 | 👍🏽 👎🏽
A mon travail, les normes de qualité exigent que 50% des sources soient des lignes de commentaires.
Je ne voit pas comment c'est possible sans écrire des commentaires débiles de ce genre.
Bizarrement, personne ne respecte les normes.

Par contre (dans les sources d'une autre boîte) j'ai déjà vu des truc comme:
#define VALEUR_1 1
#define VALEUR_2 2
#define VALEUR_3 3
#define VALEUR_4 4
...
Surement une norme de qualité interdisant les valeurs en dur et exigent de passer par des symboles.
Commentaire #57137 écrit par Shirluban le 29/08/2012 à 18h03 | 👍🏽 👎🏽
T'as tout compris...
J'ai bossé sur un projet où il y avait des contrôles automatisés du code : pas de droit aux "Magic Numbers" (nombre dans le code qui sort du chapeau du développeur). Faut déclarer des variables statique pour forcer un minimum à donner un signification à la valeur. Du coup t'en arrive à devoir déclarer toutes les valeurs différentes de 0 -1 et 1. Le plus chiant c'est le 2 qui est très présent dans des calculs géométriques
Commentaire #57144 écrit par Aerius le 29/08/2012 à 20h44 | 👍🏽 👎🏽
Non, justement... trop de commentaires rend le code illisible.
Un code propre doit être lisible et compréhensible sans autre commentaire que la documentation de chaque fonction (/ méthodes et classes)...

On réserve les commentaires pour les subtilités, ce qui est relativement rare dans un code propre.
Commentaire #57145 écrit par OzoneGrif le 29/08/2012 à 20h53 | 👍🏽 👎🏽
Les "Magic Numbers" c'est le mal.
Les "Exceptions Pokemon" (catch 'em all), c'est aussi le mal.
Commentaire #57146 écrit par OzoneGrif le 29/08/2012 à 20h55 | 👍🏽 👎🏽
En même temps si tu mets la doc dans le code. genre au début de chaque fonction et class, à la doxygen ou javadoc c'est bon :)
Commentaire #57147 écrit par but2ene le 29/08/2012 à 21h05 | 👍🏽 👎🏽
Les magics numbers, dans une formule, pas besoin de les declarer trois ans a l'avance, au pire on ecrit la formule en commentaire a coté de l'endroit ou elle est appliquée.

C'est plus gênant quand ils sont utilisé autre part que dans des formule de maths (genre, le taux de regeneration des point de vie d'un personnage, etc)

Mais franchement, les triangles ne vont pas subitement avoir un somme de leurs angles superieure a 180 sur un coup de tete du boss... :p
Commentaire #57149 écrit par palu le 29/08/2012 à 22h05 | 👍🏽 👎🏽
Mouais, ce genre de truc, c'est typiquement imposé par des gens qui ne savent pas coder et qui pensent que personne ne sait coder...

C'est comme le problème des espaces pour l'alignement et les tabulations pour l'indentation :
- Soit c'est bien fait (et le code reste lisible quel-que soit la taille donnée aux tabulation : chez moi c'est l'équivalent de 4 espaces, sauf dans la console (8 espaces quasi-forcés) ou dans une fenêtre rétrécie en largeur (je passe à 2)) ;
- Soit ça se mélange dans le bordel le plus absolu ;
- Soit il y a un facho qui impose de ne mettre que des espaces (genre Python... grrrr !).
Commentaire #57151 écrit par BSK le 29/08/2012 à 22h40 | 👍🏽 👎🏽
A mon humble avis, vu que ce sont les textes qui sont commentés, l'application peut être traduite et le dev parle mieux français qu'anglais alors il a mis les commentaires pour pouvoir s'y retrouver au cas où.
Je ne le fais pas mais, pour le coup, je ne vote pas.
Commentaire #57156 écrit par Cartman34 le 29/08/2012 à 23h19 | 👍🏽 👎🏽
...
#define VALEUR_2147483648 2147483648

"Ayè ! J'ai finit !"
Commentaire #57157 écrit par Cartman34 le 29/08/2012 à 23h21 | 👍🏽 👎🏽
La traduction d'une application, ça ne se fait pas en modifiant les sources comme un bourrin...
Commentaire #57160 écrit par BSK le 29/08/2012 à 23h31 | 👍🏽 👎🏽
Bien d'accord, meme si je serai curieux de connaître des exemples ou le pourcentage des cas où ça a été fait comme ça... Je suis sûr qu'on aurait des surprises :)

"Tu veux la version FR ? Re-download le soft entièrement pardi !" (ou rachète-le tant qu'on y est)
Commentaire #57163 écrit par Clem le 30/08/2012 à 00h08 | 👍🏽 👎🏽
Windows ?
/me s'éloigne en siflottant...
Commentaire #57165 écrit par BSK le 30/08/2012 à 02h02 | 👍🏽 👎🏽
Ben heu... Le multi-langues doit être prévu dès le départ sinon le dev devra tout recoder pour l'intégrer. Je dirais que 90% des applis sont traduites directement dans le soft.
Je me souviens d'un jeu ou même les noms des variables ont été traduites (genre b_open_window => b_fenetre_ouverte)
Commentaire #57166 écrit par psykauze le 30/08/2012 à 07h21 | 👍🏽 👎🏽
Le nom des variables comme ça, personnellement je trouve ça vraiment très con...
Pour moi, tout ce qui touche à l'applicatif (tables SQL, noms de champs, noms de classes, méthodes, etc...) doivent être en anglais (et sur un pattern bien défini qui sera toujours le même).

Y'a rien qui m'énerve plus que de voir des méthodes de classe qui s'appellent " private function affichageDesDonnees() "...
Commentaire #57173 écrit par Clem le 30/08/2012 à 10h15 | 👍🏽 👎🏽
Photoshop, y'a un support de langue, mais si tu veux changer de langue faut le racheter...
Commentaire #57177 écrit par palu le 30/08/2012 à 10h53 | 👍🏽 👎🏽
C'est marrant, je dis exactement la même chose, deux commentaires avant, je me prends deux votes négatifs, et toi quatre positifs. C'est ton avatar, ta photo a plus la classe je pense.
Commentaire #57190 écrit par Skefrep le 30/08/2012 à 11h45 | 👍🏽 👎🏽
Non, il est admin.
Commentaire #57218 écrit par -K- le 30/08/2012 à 13h03 | 👍🏽 👎🏽
@palu
Ou alors y a la technique de récupérer les fichiers de localisation d'une ancienne version de toshop.
(ce que j'ai fait juste hier parceque je m'étais trompé à l'install...)
Commentaire #57221 écrit par r-dc le 30/08/2012 à 13h18 | 👍🏽 👎🏽
Et pourtant j'ai terminé mon commentaire par "ça finit comme ça AUSSI", en référence à la précédente réponse que tu avais apporté :-/
Commentaire #57260 écrit par Clem le 30/08/2012 à 15h46 | 👍🏽 👎🏽
Je te l'ai dit, ton avatar, c'est une belle photo en noir et blanc, tandis que moi j'ai un vieux jpg colorié sous paint que je trimbale depuis plus de six ans. Forcément, du coup, j'ai moins la classe. Tiens, je vais en changer.
Commentaire #57269 écrit par Skefrep le 30/08/2012 à 16h45 | 👍🏽 👎🏽
@psykauze : Ils n'auraient pas passé le code source dans google translate ?
@clem : ouais dans le meilleurs des mondes. Mais avoues quand même que private function affichageDesDonnees() c'est mieux que private function a().
Commentaire #57271 écrit par but2ene le 30/08/2012 à 16h47 | 👍🏽 👎🏽
@but2ene : Et que dire de code du genre "int MaVariableQuiRenvoieUnALaFinDeLaFonctionSiAEstPlusGrandQueBOuQuiRenvoieMoinsUnSinon;" ?
C'était la spécialité d'un camarade de promotion en première année d'école d'ingénieur...
Commentaire #57402 écrit par Acorah le 31/08/2012 à 14h17 | 👍🏽 👎🏽
:-D
Pour la légende, cette photo a été prise alors que j'avais 3 grammes dans chaque doigt de pied.
Classe, il est vrai.
Commentaire #57424 écrit par Clem le 31/08/2012 à 17h57 | 👍🏽 👎🏽
Personnellement, en cours, j'aimais bien juste pour emmerder les profs, faire des trucs du genre :

var bool deFeu;
var int ErroSurprise;
var float EurDeBarque;

C'était naze, c'est vrai. Mais bon, à 19 ans, ça me faisait marrer...
J'adopte à présent les conventions de nommage Zend :-P
Commentaire #57426 écrit par Clem le 31/08/2012 à 17h59 | 👍🏽 👎🏽
@BSK : Genre Python qui n'impose pas une seconde de mettre des espaces pour l'indentation. Les tabulations marchent très bien. Tu peux même commettre l'action dégueulasse de mélanger espaces et tabulations pour un même bloc, dans quel cas une tabulation vaudra huit espaces, comme le veut la convention.
Commentaire #77758 écrit par Hart le 06/02/2013 à 01h09 | 👍🏽 👎🏽