bsidy1
Messages postés20Date d'inscriptionmardi 29 août 2006StatutMembreDernière intervention17 août 2012
-
13 août 2012 à 15:39
cs_LDDL
Messages postés33Date d'inscriptiondimanche 27 avril 2003StatutMembreDernière intervention 7 mars 2013
-
17 févr. 2013 à 18:53
Cette discussion concerne un article du site. Pour la consulter dans son contexte d'origine, cliquez sur le lien ci-dessous.
TychoBrahe
Messages postés1309Date d'inscriptionsamedi 31 janvier 2009StatutMembreDernière intervention 5 juin 201312 23 août 2012 à 11:51
(erf, oublié l'étoile dsl ^^)
TychoBrahe
Messages postés1309Date d'inscriptionsamedi 31 janvier 2009StatutMembreDernière intervention 5 juin 201312 23 août 2012 à 11:50
Salut,
+1 pour JLN, il n'y a rien de nouveau, tout le monde peut le faire. Déjà que ça vaut à peine un snipet, alors un code... non quoi. Allez, une petite étoile quand même parce que, bien que ce soit inutile, ce n'est ni mauvais ni dangereux.
cs_emilia123
Messages postés122Date d'inscriptionmercredi 19 décembre 2001StatutMembreDernière intervention 5 janvier 2009 20 août 2012 à 09:44
et pour complément sur l'optimisation:
mettre le résultat d'une fonction dans une variable n'a d'intéret que si ce résultat est utilisé à plusieurs endroits, ce qui évite d'appeler plusieurs fois la même fonction.
Dans le cas du test sur la longueur, ce test n'est effectué qu'une seule fois, et donc le passage par une variable intermédiaire ne fait que "gaspiller" de la mémoire disponible.
---------------
$len_texte = strlen($texte_orig);
if ($len_texte > $max_caracter)
---------------
devrait être simplement :
---------------
if (strlen($texte_orig) > $max_caracter)
---------------
Et enfin, tu devrais permettre de pouvoir modifier les "..." pour pouvoir personnaliser encore plus ton code. Cela pourrait etre un 3eme parametre avec une valeur par défaut:
function troncate($texte_orig, $max_caracter, $texte_ajout='...')
et d'utiliser dans ton code "$texte_ajout" au lieu de "..." en dur.
EM.
cs_emilia123
Messages postés122Date d'inscriptionmercredi 19 décembre 2001StatutMembreDernière intervention 5 janvier 2009 20 août 2012 à 09:38
Bonjour,
Il y a quelques éléments à corriger :
1) dans l'exemple, le 2nd paramètre est "$chr" alors que cette variable n'est pas définie, il faut que cela soit "$max"
2) la "langue" utilisée n'est pas unique dans les noms des variables.
Le nom de tes variables n'est pas constant:
- A des endroits tu mélanges l'anglais et le français ($len_texte),
- A des endroits cela ne correspond à aucune des langues ($max_caracter=> c'est soit character=>EN, soit caractere=>FR),
- et des fois c'est français (texte_orig)
Il faut trouver et employer une seule méthode pour le nom des variables sinon c'est compliqué à relire et à comprendre.
Bonne continuation
EM.
cs_JLN
Messages postés371Date d'inscriptionsamedi 1 juin 2002StatutMembreDernière intervention17 juin 2013 20 août 2012 à 08:06
J'ai beau me creser, je n'arrive pas à trouver la nouveauté dans ce code.
Je pense qu'il n'y a pas de note parce qu'il n'y a rien à dire, sinon que lorsqu'on a besoin de ca, tout un chacun aurait fait de même !
Si tu penses que je n'ai rien compris, dis le moi je serais ravi de comprendre...
bsidy1
Messages postés20Date d'inscriptionmardi 29 août 2006StatutMembreDernière intervention17 août 2012 17 août 2012 à 11:32
Bonjour
Personne ne pense à mettre une note.
Bonne journée
cs_claude77260
Messages postés54Date d'inscriptiondimanche 20 décembre 2009StatutMembreDernière intervention 8 avril 2013 14 août 2012 à 12:47
les noms de variables et fonction ne sont pas les mêmes
a corriger pour la compréhension de tous
cs_claude77260
Messages postés54Date d'inscriptiondimanche 20 décembre 2009StatutMembreDernière intervention 8 avril 2013 14 août 2012 à 12:39
bonjour
pas essayée mais je conserve dans mes tiroirs
Merci et bonne journée
bsidy1
Messages postés20Date d'inscriptionmardi 29 août 2006StatutMembreDernière intervention17 août 2012 13 août 2012 à 15:39
17 févr. 2013 à 18:53
23 août 2012 à 11:51
23 août 2012 à 11:50
+1 pour JLN, il n'y a rien de nouveau, tout le monde peut le faire. Déjà que ça vaut à peine un snipet, alors un code... non quoi. Allez, une petite étoile quand même parce que, bien que ce soit inutile, ce n'est ni mauvais ni dangereux.
20 août 2012 à 09:44
mettre le résultat d'une fonction dans une variable n'a d'intéret que si ce résultat est utilisé à plusieurs endroits, ce qui évite d'appeler plusieurs fois la même fonction.
Dans le cas du test sur la longueur, ce test n'est effectué qu'une seule fois, et donc le passage par une variable intermédiaire ne fait que "gaspiller" de la mémoire disponible.
---------------
$len_texte = strlen($texte_orig);
if ($len_texte > $max_caracter)
---------------
devrait être simplement :
---------------
if (strlen($texte_orig) > $max_caracter)
---------------
Et enfin, tu devrais permettre de pouvoir modifier les "..." pour pouvoir personnaliser encore plus ton code. Cela pourrait etre un 3eme parametre avec une valeur par défaut:
function troncate($texte_orig, $max_caracter, $texte_ajout='...')
et d'utiliser dans ton code "$texte_ajout" au lieu de "..." en dur.
EM.
20 août 2012 à 09:38
Il y a quelques éléments à corriger :
1) dans l'exemple, le 2nd paramètre est "$chr" alors que cette variable n'est pas définie, il faut que cela soit "$max"
2) la "langue" utilisée n'est pas unique dans les noms des variables.
Le nom de tes variables n'est pas constant:
- A des endroits tu mélanges l'anglais et le français ($len_texte),
- A des endroits cela ne correspond à aucune des langues ($max_caracter=> c'est soit character=>EN, soit caractere=>FR),
- et des fois c'est français (texte_orig)
Il faut trouver et employer une seule méthode pour le nom des variables sinon c'est compliqué à relire et à comprendre.
Bonne continuation
EM.
20 août 2012 à 08:06
Je pense qu'il n'y a pas de note parce qu'il n'y a rien à dire, sinon que lorsqu'on a besoin de ca, tout un chacun aurait fait de même !
Si tu penses que je n'ai rien compris, dis le moi je serais ravi de comprendre...
17 août 2012 à 11:32
Personne ne pense à mettre une note.
Bonne journée
14 août 2012 à 12:47
a corriger pour la compréhension de tous
14 août 2012 à 12:39
pas essayée mais je conserve dans mes tiroirs
Merci et bonne journée
13 août 2012 à 15:39