Le pire nom de variable ?

[Sephi-Chan]

La on parlait des variables qui récupère quelque chose pas de la pertinence du nom d'un champ dans une base de donnée ( la encore vous avez juste le nom du champ choisi donc pas d'indication non plus). J'avoue que j'aurais du mettre action! Et pour le switch dsl j'ai abrégé un peu trop le code semblerait! On parle pas de la manière de coder mais bien du nom de la variable non ? Elle récupère un nom de donnée clairement identifiable lui ( même si oui ici il n'est pas bien choisis, mais c'est du code simplifié! ( dsl pas de break pas de " pas de sql query ou autre) Vous avez compris ou j'voulais en venir vous cherchez juste a chipoter!

Merci de ne pas conclure qu'on chipote parce qu'on réfute ton assertion.

On t'explique que dans ton cas il aurait été plus pertinent de nommer ta variable autrement. Alors certe, ça n'est pas illisible, mais ça ne t'aurait pas coûté plus cher d'utiliser un nom approprié et ça aurait été plus lisible encore.

Regarde ce fragment de code :

tmp = User.find(42).role



case tmp

  when 'administrator'

    do_something

  when 'moderator'

    do_something_else

  else

    do_nothing

end

Tu réalises le non-sens ? Pourquoi nommer la variable tmp alors que role ou user_role a plus de sens ?

En faisant ça, j'ai occlulté inutilement de l'information. Comme toi quand tu fais :

$tmp= requete SQL SELECT action_terminee FROM Action WHERE id =?;

Ici, tu sais que tu récupères la valeur de la colonne action_terminee, donc autant appeler la variable de la même façon. C'est tout de même plus clair que tmp.


En somme, ce n'est pas ton utilisation d'une variable mal nommée qui rend ton exemple de code mauvais : c'est le mauvais nommage de la colonne (qui est sujet à trop d'interprétations possibles).

Et pour le code Jquery, oui c'est minifié mais cela arrive souvent de se retrouver avec des codes pareils ( Oui on m'a déjà demandé de modifier ce genre de code sans me filer l'original ... ) donc faut parfois relativiser sur le nom des variables et pas dire forcément c'est mal puis que ca existe aussi et que c'est utilisé!

Désolé mais non. On édite des fichiers propres et on utilise un outil pour packager (réunir plusieurs fichiers en un seul) et minifier (le rendre le plus court possible) lors du déploiement. En aucun cas un développeur devrait modifier du code minifié : ça n'a aucun intérêt.