MAIL EN HTML

Commentaire de miata le 17/09/2006 02:38:01

C'est un de mes premier code PHP ...
Merci de faire des critique ...

Commentaire de kankrelune le 18/09/2006 14:24:30

Je rejoins Audayls sur ce qu'il (elle?) a dit

- Niveau : débutant
- le copyright n'en parlons pas
- utiliser les simples quotes et concaténer les variables pour plus de clarté et des performances (légèrement) meilleurs
- <br /> et non pas <br>

"# # En cas d'erreur inconnue ...
# echo '<< ERREUR DANS LE FICHIER PHP >>';"

Cette ligne est totalement inutile... ça sert d'ailleur à rien de faire des exit()... soit ça passe soit ça passe dans tout les cas ta condition gérera le résultat et tes exit() t'empeche de faire une mise en pae plus complète... ex...

echo '<fieldset>
<ledend>Envoi d\'email</legend>
<div style="text-align:center"><br />';

if(!mail($to_add, $sujet, $message, $html_generate))
    echo '<span class="Style4">Erreur lors de l'envoi du Mail !</span>';
        else
            echo '<span class="Style3">Message envoyé !</span>'

echo '<br />
           <br />
     <span class="Style2">Récapitulatif :</span>
        <br />
     <span class="Style2">De : </span><span class="Style1">'.$from.' ( '.$from_add.' )</span>
        <br />
     <span class="Style2">A : </span><span class="Style1">'.$to.' ( '.$to_add.' )</span>
        <br />
     <span class="Style2">Sujet :</span> <span class="Style1">'.$sujet.'</span>
</div>
</fieldset>';
        

"Si vous avez un formulaire avant, ajouter simplement :

extract($_POST);

puis assigné au variable le nom des zone de texte du formulaire ..."

Surtout pas... on utilise $_POST['maVar'] (après avoir pensé à néttoyé son contenu) mais surtout pas extract($_POST)... .. .

Moi ce qui me gène dans ta source c'est qu'a part un mail() il n'y a rien... il faut éditer le message,nom/mail de l'expéditeur et nom/mail du destinataire à la main... tu aurais au moins pu faire un formulaire... de ce fait je met 4/10 car l'utilisation de mail() n'a rien de complexe le seul bon point c'est qu'il y a des headers de définit... .. .

@ tchaOo°

Commentaire de Therion007 le 18/09/2006 21:22:53

T'en fais du remuménage pur pas grand chose. Tu te compliques la vie sur un envoie de mail avec php. Pourquoi tu ne sépares pas le css du reste dans un fichier.css ?
Le copyright sert il a grd chose quand tu proposes une sources qui de tte façon vu les erreures et le niveau sera bidouillé, remanié voir meme amélioré ?
Cdt,

Commentaire de kankrelune le 25/09/2006 14:44:01

Extract peut être très utile sur un tableau normal... mais il ne faut surtout pas l'utiliser sur une super global tel que $_POST... ça revient à mettre register_globals à on et c'est un coup à se faire plomber son site... .. .

@ tchaOo°

Commentaire de miata le 21/07/2007 12:46:23

Je viens relire mon code 10 mois plus tard ... C'est clair que j'ai foutu des trucs totalement inutiles :D Mais bon, on progresse au fil du temps ;) J'edit cela ...