neigedhiver
Messages postés2480Date d'inscriptionjeudi 30 novembre 2006StatutMembreDernière intervention14 janvier 2011
-
13 mars 2008 à 16:49
webdeb
Messages postés488Date d'inscriptionsamedi 5 avril 2003StatutMembreDernière intervention31 mars 2009
-
13 mars 2008 à 17:00
Cette discussion concerne un article du site. Pour la consulter dans son contexte d'origine, cliquez sur le lien ci-dessous.
webdeb
Messages postés488Date d'inscriptionsamedi 5 avril 2003StatutMembreDernière intervention31 mars 20094 13 mars 2008 à 17:00
Heureusement que number_format() existe...
neigedhiver
Messages postés2480Date d'inscriptionjeudi 30 novembre 2006StatutMembreDernière intervention14 janvier 201119 13 mars 2008 à 16:49
Salut,
L'intention est louable, mais... je crains que ta source ne finisse à la poubelle d'ici peu de temps. En effet, elle ne sert à rien... désolé de te le dire... Mais on fait pareil en 1 ligne, et avec une fonction php déjà existante :
echo number_format($nombre, 0, ',', ' ');
Et puis ton algo est bien trop compliqué, même avec une PCRE ça se fait en une ligne.
Et je ne parle même pas de la qualité de ton code...
Par exemple :
for ($i=0;$i<=strlen($variable[0])-1;$i++)
A chaque itération de la boucle, la fonction strlen sera exécutée... Sur une variable qui ne change pas : inutile donc d'exécuter plusieurs fois la fonction si le résultat doit être chaque fois identique.
Bref : y'a déjà une fonction php qui fait ça... Je veux pas être méchant, mais avant de proposer une fonction aussi triviale, il peut être bon de consulter la doc.
13 mars 2008 à 17:00
13 mars 2008 à 16:49
L'intention est louable, mais... je crains que ta source ne finisse à la poubelle d'ici peu de temps. En effet, elle ne sert à rien... désolé de te le dire... Mais on fait pareil en 1 ligne, et avec une fonction php déjà existante :
echo number_format($nombre, 0, ',', ' ');
Et puis ton algo est bien trop compliqué, même avec une PCRE ça se fait en une ligne.
Et je ne parle même pas de la qualité de ton code...
Par exemple :
for ($i=0;$i<=strlen($variable[0])-1;$i++)
A chaque itération de la boucle, la fonction strlen sera exécutée... Sur une variable qui ne change pas : inutile donc d'exécuter plusieurs fois la fonction si le résultat doit être chaque fois identique.
$l = strlen($variable[0]) - 1;
for ($i=0 ; $i <= $l ; $i++) {
}
Bref : y'a déjà une fonction php qui fait ça... Je veux pas être méchant, mais avant de proposer une fonction aussi triviale, il peut être bon de consulter la doc.