 De BackInBiz - le 27-11-2004
Niveau qualité de codage en PHP j'ai vu mieux ...
D'ailleurs je ne sais pas pas ce que ce code vient faire sur ce site ...
Si vous voulez que je vous file des bouts de code pour faire des trucs interessant en PHP c'est jouable mais c'est un peu hors contexte ...
Donc voila eviter le echo("") ca fait blaireau.
Amicalement, un amateur de PHP | De kiki92 - le 01-12-2004
Salut BackInBiz,
Peux tu expliquer pourquoi c'est pas top le echo("") ??
Parce que toi tu as l'air d'être plus fort que celui qui a fait le script mais tu n'explique pas le pourquoi de ton commentaire, et il serait interressant de nous faire partager ton savoir, car moi tu me dis que <le echo("") > c'est pas top mais si tu me dis que le <0°=+==}}> n'est pas top non plus c'est pareil pour moi, mais surement pour d'autres !
Donc ce qui serait sympa c'est d'expliquer aux moins bons que vous non ?
Merci d'avance pour la réponse
Cordialement. | De Pascal - le 02-12-2004
Le résultat est que ça fonctionne non ? moi je suis pour qu'il y est du php | De Xavier - le 02-12-2004
Bonjour,
Comme dit BackinBiz, ce script est une horreur PHP, sans vouloir te vexer mercier134.
La plus grosse erreur étant la position de la fonction mail(). Dans ton exemple, mail() est placée de telle façon qu'à chaque chargement de la page, un mail sera envoyé. Ce qui veut dire qu'un mail sera envoyé avant même que l'utilisateur ne puisse remplir les champs.
La bonne méthode est de mettre dans le action de ton form une autre page. Une page PHP qui elle enverra le mail et donnera un accusé de réception.
formulaire.htm entre <body> et </body>
<form name="form1" method="post" action="email.php">
<table border="0">
<tr>
<td width="125">À :</td>
<td width="567"><input name="a1" type="text" id="a12">@<input name="a2" type="text" id="a22"></td>
</tr>
<tr>
<td>De : </td>
<td><input name="de1" type="text" id="de1">@<input name="de2" type="text" id="de2">
</td>
</tr>
<tr>
<td> Sujet du message : </td>
<td><input name="sujet" type="text" id="sujet2" size="95"></td>
</tr>
<tr>
<td>Message : </td>
<td><textarea name="message" cols="90" rows="5"></textarea></td> </tr>
<tr>
<td> </td>
<td><input type="submit" name="Submit" value="Envoyer">
<input type="reset" name="Submit2" value="Rétablir">
</td>
</tr>
</table>
<p> </p>
</form>
|
|
email.php
<?php
$from = $_POST['de1']."@".$_POST['de2'];
$to = $_POST['a1']."@".$_POST['a2'];
$object = $_POST['sujet'];
$msg = $_POST['message'];
$msg.= "\n\n $from vous invite a aussi envoyer vos mail grace a ...";
mail ($to, $object, $msg, "From: $from");
print("Merci d'avoir envoyer le mail suivant a $to : <br>$msg<br>");
?>
|
|
Et hop, un petit tour sur phpdebutant.com ;)
Xav le québécois | De Brett - le 02-12-2004
Essaie de faire des critiques constructives en proposant un bon script par ex, ça fait progresser tout le monde.
Le script est peut être simplet ou mal codé, mais bon, un peu d'humilité de tps en tps fait pas de mal. Le seul truc à redire est la faute d'orthographe à "Merci d'avoir envoyÉ le mail..." avec un e accent aigu.
Bye ! | De ademain - le 06-12-2004
Bonjour,
Le code avec le formulaire et le traitement php de l'autre est pas top non plus car il t'ouvre une fentre blanche tel que tu l'as traite.
Mettre une close dans une boucle if est plus utile;
if(!isset( var dur formulaire)){
traitement de la fonction mail()
}
else{
chants manquants
} | De benske - le 27-12-2004
alors quesqu'il faut mettre pour finir? | De mercier134 - le 10-11-2005
Merci pour toutes vos critiques ...
...Vous avez raison mon script est très mal codé ! Mais je débutait en tant que Webmaster quand j'ai mis ce script !
J'ai chercher tellement longtemps avant de trouver qu'on pouvait envoyer un mail que dés que j'ai trouvé je l'ai mis sur l'éditeurjavascript (même si c'est pas du javascript ! )
Bye | De commealamaison - le 08-10-2006
Vous battez pas, bon ya une erreur y a une erreur, sympa de proposer une solution, mais c'est vrais que pour nous débutant c'est pas évident, surtout quand cela touche la sécurité ou bug possible. Merci quand même pour l'effort de tous !!! | De Dédé25 - le 20-12-2006
La solution de Xavier est simple et efficace.
J'ai essayé plein de script proposés qui avaient l'air sympa, mais tous étaient trop compliqués pour un novice, ou ne fonctionnaient pas bien.
Celui-ci est le plus simple que j'ai pu voir.
Merci. | De esibal - le 10-01-2007
Mais il faut mettre quoi a la fin???^^' | De alex55 - le 06-02-2007
Ce script n'est pas sécurisé contre les injections de codes malveillants. En outre il est un peu désuet et très mal codé.
Et je trouve cela un peu idiot de poster un script PHP dans l'Editeur Javascript, qui n'est en aucun cas l'Editeur PHP !
Sauf bien-sûr si JC nous ouvre une petite catégorie PHP... :-)
A+ | De wwe--undertaker - le 13-02-2007
Bonjours je suis entrain de teste mon scrit mais je ne reçois pas d'email pourrier vous m'indiquer ou placer ladresse email sur se scrip php SVP
formulaire.htm entre <body> et </body>
<form name="form1" method="post" action="email.php">
<table border="0">
<tr>
<td width="125">À :</td>
<td width="567"><input name="a1" type="text" id="a12">@<input name="a2" type="text" id="a22"></td>
</tr>
<tr>
<td>De : </td>
<td><input name="de1" type="text" id="de1">@<input name="de2" type="text" id="de2">
</td>
</tr>
<tr>
<td> Sujet du message : </td>
<td><input name="sujet" type="text" id="sujet2" size="95"></td>
</tr>
<tr>
<td>Message : </td>
<td><textarea name="message" cols="90" rows="5"></textarea></td> </tr>
<tr>
<td> </td>
<td><input type="submit" name="Submit" value="Envoyer">
<input type="reset" name="Submit2" value="Rétablir">
</td>
</tr>
</table>
<p> </p>
</form>
email.php
<?php
$from = $_POST['de1']."@".$_POST['de2'];
$to = $_POST['a1']."@".$_POST['a2'];
$object = $_POST['sujet'];
$msg = $_POST['message'];
$msg.= "\n\n $from vous invite a aussi envoyer vos mail grace a ...";
mail ($to, $object, $msg, "From: $from");
print("Merci d'avoir envo |
Home Page
