par
Hywan » 14 mars 2008, 01:47
Juste au passage, tu peux optimiser ton code de façon évidente.
Premièrement, n'utilises pas les
short_tags (<?) mais plutôt sa syntaxe longue (<?php), ça évite des conflits, des problèmes de distributivité et de sécurité.
Deuxièmement, préfère utiliser les guillemets simples (') plutôt que les doubles ("). C'est plus rapide pour PHP. Un des avantages des guillemets doubles est que l'on n'a pas besoin de concaténer les variables, mais comme tu le fais, tu perds cet avantage et ne récupère que les inconvénients ... Il faut jeter un œil dans la documentation pour mieux comprendre, tu devrais y faire un tour :
les chaînes de caractères (guillemets simples, doubles, et syntaxe
hereDoc au programme).
Troisièmement, la concaténation est pratique, mais lourde, surtout quand on l'utilise un peu comme un cochon de sa queue ! Je m'explique : tu fais une concaténation par retour à la ligne (\n). Ne serait-il pas plus sage de tout mettre dans une seule chaîne ? Je propose :
$content = "\n\n\n\n\n" .
'Identifiant : ' . $DATAS . "\n" .
'Code Allopass : ' . $RECALL . "\n\n" .
'Vous serez sponsor jusqu\'au : ' . date('d/m/Y', $date_fin) .
"\n\n\n\n\n\n\n\n\n\n\n" .
'-------' . "\n\n" .
'E-mail automatique (ne pas répondre à ce message)' . "\n";
Tu gagnes en hauteur, en lisibilité, en temps, en mémoire etc. Certes, ce n'est pas la méthode la plus rapide, mais c'est déjà nettement mieux. J'en ai profité pour corriger une petite faute (« a ce message » vers « à ce message »).
Mais (et j'insiste) ce n'est pas la bonne façon de mettre en page un mail ! Déjà, c'est pas très accessible (le synthétiseur vocal va lire 25 lignes, ultra lourd) et le mail commence 5 lignes plus bas, on ne comprend pas pourquoi, moi je penserais à un bug. Pourquoi ne pas faire un petit paragraphe, un espèce de liste, et 2 lignes plus bas, le pied de mail (« E-mail automatique [...] » ? Ce serait vraiment plus simple ... Ce n'est pas un conseil technique mais ... ergonomique dira-t-on. C'est du côté de la pratique, un conseil donc

. Le fait de mettre un paragraphe peut présenter l'auteur du mail (« Bonjour, vous recevez cet email parce que vous ... »), et permet d'introduire la liste de l'identifiant, code allopass et autre truc. Mieux non ?
Juste au passage, tu peux optimiser ton code de façon évidente.
Premièrement, n'utilises pas les [i]short_tags[/i] (<?) mais plutôt sa syntaxe longue (<?php), ça évite des conflits, des problèmes de distributivité et de sécurité.
Deuxièmement, préfère utiliser les guillemets simples (') plutôt que les doubles ("). C'est plus rapide pour PHP. Un des avantages des guillemets doubles est que l'on n'a pas besoin de concaténer les variables, mais comme tu le fais, tu perds cet avantage et ne récupère que les inconvénients ... Il faut jeter un œil dans la documentation pour mieux comprendre, tu devrais y faire un tour : [url=http://fr.php.net/string]les chaînes de caractères[/url] (guillemets simples, doubles, et syntaxe [i]hereDoc[/i] au programme).
Troisièmement, la concaténation est pratique, mais lourde, surtout quand on l'utilise un peu comme un cochon de sa queue ! Je m'explique : tu fais une concaténation par retour à la ligne (\n). Ne serait-il pas plus sage de tout mettre dans une seule chaîne ? Je propose : [php]$content = "\n\n\n\n\n" .
'Identifiant : ' . $DATAS . "\n" .
'Code Allopass : ' . $RECALL . "\n\n" .
'Vous serez sponsor jusqu\'au : ' . date('d/m/Y', $date_fin) .
"\n\n\n\n\n\n\n\n\n\n\n" .
'-------' . "\n\n" .
'E-mail automatique (ne pas répondre à ce message)' . "\n";[/php]
Tu gagnes en hauteur, en lisibilité, en temps, en mémoire etc. Certes, ce n'est pas la méthode la plus rapide, mais c'est déjà nettement mieux. J'en ai profité pour corriger une petite faute (« a ce message » vers « à ce message »).
Mais (et j'insiste) ce n'est pas la bonne façon de mettre en page un mail ! Déjà, c'est pas très accessible (le synthétiseur vocal va lire 25 lignes, ultra lourd) et le mail commence 5 lignes plus bas, on ne comprend pas pourquoi, moi je penserais à un bug. Pourquoi ne pas faire un petit paragraphe, un espèce de liste, et 2 lignes plus bas, le pied de mail (« E-mail automatique [...] » ? Ce serait vraiment plus simple ... Ce n'est pas un conseil technique mais ... ergonomique dira-t-on. C'est du côté de la pratique, un conseil donc :). Le fait de mettre un paragraphe peut présenter l'auteur du mail (« Bonjour, vous recevez cet email parce que vous ... »), et permet d'introduire la liste de l'identifiant, code allopass et autre truc. Mieux non ?