Discussion :

[Magemax]

Bonjour,

j'ai récemment été mis en contact avec un bout de code qui m'a passablement étonné : le programmeur avait en effet défini les chiffres en const :

Code :

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

1
2
3
4
const double zero = 0.0, one = 1.0, two = 2.0, three = 3.0, four = 4.0,
					  five = 5.0, six = 6.0, seven = 7.0, eight = 8.0, nine = 9.0, ten = 10.0,
					  hundr = 100.0, infty = 1.0e100, epsnorm = 1.0e-15, epsnorm2 = 1.0e-30,
					  powerm12 = 1.0e-12;

Ca m'a vraiment étonné, car je ne vois pas du tout l'intérêt de faire ca (le type l'utilise plus tard dans des phases du genre double x = -one; )

Quelqu'un sait-il quel peut être l'utilité de déclarer ce genre de constantes ?

J'ai après moulte réflexion trouvé deux explications :
- Le programmeur ne connait pas les chiffres, parce qu'il est Egyptien, et du coup il préfère les écrire en anglais plutôt que de prendre le risque de confondre les zero et les 5, par exemple
- Cette méthode fait économiser du temps à l'ordinateur : ce dernier a plus de facilité à aller chercher une const, plutôt que de transcrire à chaque fois 1.0 en binaire.


Ai-je raison, ou bien y a t il une autre explication, ou bien n'y a t il aucune explication et le type est juste un peu bizarre ?

Merci d'avance...

[SaumonAgile]

A mon avis, c'est complètement c...
Mais en tout cas le jour où le chiffre 1 s'écrira $£ au lieu de 1, il n'y aura que la constante à changer.

[bossun]

- Le programmeur ne connait pas les chiffres, parce qu'il est Egyptien, et du coup il préfère les écrire en anglais plutôt que de prendre le risque de confondre les zero et les 5, par exemple

A ce que je sache les chiffres s'écrivent pareille en Égypte

Après si tu nous dis ce que fait ton application, peut-être qu'on pourrait trouver une logique.. mais peso je ne suis pas convaincu qu'il y en aie une.

[neptune]

Un jour ce type à lu ou entendu qu'il était de bon ton de transformer ces littéraux en constantes. Du genre const int TVA plutot que 21, etc.

Il a pris le conseil vraiment à la lettre!

Mais si tu as le contexte... envoie, on en sais jamais.

[neptune]

A noter qu'il n'est pas le seul à faire cela. Je viens de passer à travers plusieurs patchs et code source sur le net et pas plus loin que dans un header de GCC on trouve des constantes pour ONE, TWO, TEN...

Il doit peut-être y avoir une astuce, bien que je ne vois pas du tout laquelle. Les constantes sont remplacées à la compilation par la valeur littérale, donc qu'on ne me parle pas d'optimisation! :-)

[hed62]

Le seul intérêt est dans les définitions
"epsnorm = 1.0e-15, epsnorm2 = 1.0e-30," puisque généralement ce sont des valeurs de seuil, de précision minimales requises (donc interressant de pouvoir les modifier vite)
le reste... j'ai du mal à voir...

[Magemax]

A ce que je sache les chiffres s'écrivent pareille en Égypte

Après si tu nous dis ce que fait ton application, peut-être qu'on pourrait trouver une logique.. mais peso je ne suis pas convaincu qu'il y en aie une.

en egypte (et sans doute dans d'autres pays arabes), ils utilisent des chiffres comme ceci



A part ca, le contexte c'est un algorithme de minimisation de fonction que j'ai rencontré dans mon boulot.

Donc y a aucun moment où il ferait des trucs du genre demander à l'utilisateur de rentrer des chiffres en toutes lettres pour les traduire en chiffres.


En plus, le type lâche l'affaire du "mettre tous les nombres en const" et fait des phases du genre :

Code :

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

1
2
3
4
5
low_bound = -one + 1.0e-4;  
 
ou
 
one / Math.Log(ng + 1.1)

Si personne n'a d'explication, je pourrai demander à mes chefs (et même à l'auteur du programme, même s'il ne travaille plus ici), et je vous tiendrai au courant

[neptune]

Oui, demande lui car ca serait très intéressant à savoir.

Et j'ai appris quelquechose au sujet des cabines téléphoniques! Mais c'est le comble qu'ils n'utilisent pas les chiffres qu'ils ont inventés :-)

[Guulh]

Mon expérience m'a prouvé que quand on ne comprend pas du code, on a plus vite fait de surestimer l'intelligence de celui qui l'a codé que de la sous-estimer.
M'étonne qu'il ait pas remplacé 1.1 par one + one / ten

[neptune]

M'étonne qu'il ait pas remplacé 1.1 par one + one / ten

:-D

[tomlev]

Mais c'est le comble qu'ils n'utilisent pas les chiffres qu'ils ont inventés :-)

Ben en fait ils utilisent aussi les chiffres arables, mais il en existe plusieurs formes...
http://fr.wikipedia.org/wiki/Chiffres_arabes

[Magemax]

Après avoir demandé à mes glorieux chefs, j'ai enfin un semblant de réponse que je ne comprends pas vraiment :


apparemment, le programme serait : "traduit de matlab en c#". Je ne vois pas vraiment en quoi ca implique de mettre des trucs du genre "one=1.0;", mais aussi je ne connais pas le matlab, donc peut être qu'en matlab, ca a une utilité (ou peut être qu'ils ont utilisé un traducteur automatique qui fait ce genre de connerie, mais je n'avais jamais entendu parler de traducteurs automatiques d'un langage à un autre)

[Guulh]

Mon expérience (encore ) m'a prouvé que quand on a confirmation qu'une bizarrerie du code ne sert à rien, on a tout intérêt à prendre le temps de la supprimer pour simplifier le code. Histoire d'épargner à soi-même et au potentiel successeur une calvitie prématurée.

Parce que passer d'une classe de 100 Ko sans commentaire à une classe de 60 Ko (cas vécu) dont une bonne part de commentaire, ça prend du temps mais ça simplifie considérablement la maintenance

[Jerome Briot]

peut être qu'en matlab, ca a une utilité

Euhh.. ... on est pas plus tordu que les autres...