formulaire de contact html et php ????

Bonjour,

je vous fais parvenir mes codes html et php, pouvez vous me dire si c'est bon ? j'essaye sur ma page perso : http://pagesperso-orange.fr/julien.delorme/
le fichier se télécharge ! Je ne comprends pas !!!!

code html :

<form action="formulaire.php" method="post" enctype="multipart/form-data" name="form1">
<table width="100%" border="0">
<tr>
<td width="31%" height="35"><img src="images/message.png" width="112" height="30"></td>
<td width="69%"><span id="votre_sujet">
<input name="votre_sujet" type="text" id="votre_sujet" size="37">
</span></td>
</tr>
<tr>
<td height="46"><img src="images/nom.png" width="113" height="29"></td>
<td><span id="nom">
<input name="votre_nom" type="text" id="votre_nom" size="37">
</span></td>
</tr>
<tr>
<td height="40"><img src="images/prenom.png" width="113" height="30"></td>
<td><span id="prennom">
<input name="votre_prenom" type="text" id="votre_prenom" size="37">
</span></td>
</tr>
<tr>
<td height="40"><img src="images/telephone.png" width="113" height="30"></td>
<td><span id="telephone">
<input name="votre_telephone" type="text" id="votre_telephone" size="37">
</span></td>
</tr>
<tr>
<td height="38"><img src="images/mail.png" width="113" height="30"></td>
<td><span id="email">
<input name="votre_email" type="text" id="votre_email" size="37">
</span></td>
</tr>
<tr>
<td height="50"><img src="images/fichier.png" width="113" height="30"></td>
<td><input type="file" name="fichier" id="fichier"></td>
</tr>
<tr>
<td height="40"><img src="images/messagefin.png" width="113" height="30"></td>
<td>&nbsp;</td>
</tr>
<tr>
<td height="100" colspan="2"><span id="votre_message2">
<textarea name="votre_message" id="votre_message" cols="54" rows="5"></textarea>
<span id="countmessage2">&nbsp;</span></span></td>
</tr>
<tr>
<td height="20"><input name="reset" type="image" id="reset" src="images/reset.png" align="right" value="Envoyer"></td>
<td><input name="reset" type="image" id="reset" src="images/reset.png" align="right" value="Effacer"></td>
</tr>
</table>
</form>
</div>
<div id="google"> <img name="" src="" width="380" height="365" alt="" style="background-color: #00FF00"> </div>
</div>
</div>
</div>
<script type="text/javascript">
<!--
var sprytextfield1 = new Spry.Widget.ValidationTextField("votre_sujet", "none", {hint:"sujet", isRequired:false});
var sprytextfield2 = new Spry.Widget.ValidationTextField("prennom", "none", {isRequired:false, hint:"pr\xE9nom"});
var sprytextfield3 = new Spry.Widget.ValidationTextField("telephone", "phone_number", {format:"phone_custom", pattern:"00 00 00 00 00", hint:"00 00 00 00 00", isRequired:false, useCharacterMasking:true});
var sprytextfield4 = new Spry.Widget.ValidationTextField("nom", "none", {isRequired:false, hint:"nom"});
var sprytextfield5 = new Spry.Widget.ValidationTextField("email", "email", {hint:"votrenom@fournisseur.com", isRequired:false});
var sprytextarea3 = new Spry.Widget.ValidationTextarea("votre_message2", {counterType:"chars_count", counterId:"countmessage2"});
//-->
</script>


code php :


<?php
if (isset($_POST["envoyer"])){ // Si le formulaire a été soumis
$etat = "erreur"; // On initialise notre etat à erreur, il sera changé à "ok" si la vérification du formulaire est un succès, sinon il reste à erreur

// On récupère les champs du formulaire, et on arrange leur mise en forme
if (isset($_POST["votre_sujet"])) $_POST["votre_sujet"]=trim(stripslashes($_POST["votre_sujet"])); // trim() enlève les espaces en début et fin de chaine

if (isset($_POST["votre_nom"])) $_POST["votre_nom"]=trim(stripslashes($_POST["votre_nom"]));

if (isset($_POST["votre_prenom"])) $_POST["votre_prenom"]=trim(stripslashes($_POST["votre_prenom"]));

if (isset($_POST["votre_telephone"])) $_POST["votre_telephone"]=trim(stripslashes($_POST["votre_telephone"]));

if (isset($_POST["votre_email"])) $_POST["votre_email"]=trim(stripslashes($_POST["votre_email"])); // stripslashes() retire les backslashes ==> \' devient '

if (isset($_POST["votre_message"])) $_POST["votre_message"]=trim(stripslashes($_POST["votre_message"]));

// Après la mise en forme, on vérifie la validité des champs
if (empty($_POST["votre_sujet"])) { // L'utilisateur n'a pas rempli le champ votre_sujet
$erreur="Vous n'avez pas entr&eacute; votre votre message..."; // On met dans erreur le message qui sera affiché
}
elseif (empty($_POST["votre_nom"])) { // L'utilisateur n'a pas rempli le le champ votre_nom
$erreur="Nous avons besoin de votre nom pour vous r&eacute;pondre...";
}
elseif (empty($_POST["votre_prénom"])) { // L'utilisateur n'a pas rempli le champ votre_prenom
$erreur="Nous avons besoin de votre prénom pour vous r&eacute;pondre...";
}
elseif (empty($_POST["votre_email"])) { // L'utilisateur n'a pas rempli le champ email
$erreur="Nous avons besoin de votre e-mail pour vous r&eacute;pondre...";
}
elseif (!eregi("^[0-9a-z]([-_.]?[0-9a-z])*@[0-9a-z]([-.]?[0-9a-z])*\.[a-z]{2,4}$",$_POST["votre_email"])){ // On vérifie si l'email est bien de la forme messagerie@domaine.tld (cf cours d'expressions régulières)
$erreur="Votre adresse e-mail n'est pas valide...";
}
elseif (empty($_POST["votre_message"])) { // L'utilsateur n'a écrit aucun message
$erreur="Merci de saisir un message...";
}
else { // Si tous les champs sont valides, on change l'état à ok
$etat="ok";
}
}
else { // Sinon le formulaire n'a pas été soumis
$etat="attente"; // On passe donc dans l'état attente
}

if ($etat!="ok"){ // Le formulaire a été soumis mais il y a des erreurs (etat=erreur) OU le formulaire n'a pas été soumis (etat=attente)
if ($etat=="erreur"){ // Cas où le formulaire a été soumis mais il y a des erreurs
echo "<span style=\"color:red\">".$erreur."</span><br /><br />\n"; // On affiche le message correspondant à l'erreur
}
?>

<!-- Formulaire HTML qu'on affiche dans l'état attente ou erreur -->
<form method="post" action="<?php echo $_SERVER["PHP_SELF"]; ?>"> <!-- Les données du formulaire seront récupérée avec la méthode POST, et action correspond à la page contenant le formulaire -->
<p style="text-align:left">
<label for="son_pseudo">Pseudo *</label><br /> <!-- Intitulé du champ pseudo -->
<input type="text" size="40" name="votre_sujet" id="votre_sujet" value="<?php
if (!empty($_POST["votre_sujet"])) {
// le pseudo de l'expéditeur a été saisi --> le réafficher
echo htmlspecialchars($_POST["votre_sujet"],ENT_QUOTES); // htmlspecialchars() convertit les caractères spéciaux en leurs code html, exemple : & devient &amp;
}
?>" />

<br />

<label for="votre_email">E-mail *</label><br /> <!-- Intitulé du champ e-mail -->
<input type="text" size="40" name="votre_email" id="votre_email" value="<?php
if (!empty($_POST["votre_email"])) {
// l'e-mail de l'expéditeur a été saisi --> le réafficher
echo htmlspecialchars($_POST["votre_email"],ENT_QUOTES);
}
?>" />

<br />

<label for="votre_message">Objet *</label><br /> <!-- Intitulé du champ objet -->
<input type="text" size="40" name="votre_message" id="votre_message" value="<?php
if (!empty($_POST["votre_message"])) {
// l'objet du message a été saisi --> le réafficher
echo htmlspecialchars($_POST["votre_message"],ENT_QUOTES);
}
?>
</textarea>

<br />

<input name="reset" type="image" id="reset" src="images/reset.png" align="right" value="Envoyer" /><input name="reset" type="image" id="reset" src="images/reset.png" align="right" value="Effacer" />
</p>
</form>
<!-- FIN du formulaire HTML -->

<?php
}
else { // Sinon l'état est ok donc on envoie le mail
$votre_sujet = $_POST["votre_sujet"]; // On stocke les variables récupérées du formulaire
$votre_nom = $_POST["votre_nom"];
$votre_prénom = $_POST["votre_prénom"];
$votre_téléphone = $_POST["votre_téléphone"];
$votre_email = $_POST["votre_email"];
$votre_message = $_POST["votre_message"];

$mon_email = "rouge.is.design@gmail.com"; // Mise en forme du message que vous recevrez
$mon_pseudo = "rouge.is.design";
$mon_url = "http://rouge.is.design@com";
$msg_pour_moi = "- son sujet : $votre_sujet \n
- Son nom : $votre_nom \n
- Son prenom : $votre_prenom \n
- Son email : $votre_email \n
- Son telephone : $svotre_telephone \n
- Message : \n $votre_message \n\n";

// Mise en forme de l'accusé réception qu'il recevra
$accuse_pour_lui = "Bonjour $votre_nom,\n
Votre message nous a bien été envoyé et nous tâcherons de vous répondre le plus rapidement possible.\n\n
- Votre sujet : $votre_sujet \n
- Votre E-mail : $votre_email \n
- Votre message : \n $votre_message \n\n
Merci et à bientôt sur http://rougeisdesign@gmail.com !";

// Envoie du mail
$entete = "From: " . $votre_sujet . " <" . $votre_email . ">\n"; // On prépare l'entête du message
$entete .= "MIME-Version: 1.0";

if (@mail($votre_email,$votre_sujet,$msg_pour_moi,$entete) && @mail($votre_email,$votre_sujet,$accuse_pour_lui,$entete)){ // Si le mail a été envoyé
echo "<p style=\"text-align:center\">Votre message a &eacute;t&eacute; envoy&eacute;, vous recevrez une confirmation par mail.<br /><br />\n"; // On affiche un message de confirmation
echo "<a href=\"" . $mon_url . "\">Retour</a></p>\n"; // Avec un lien de retour vers l'accueil du site
}
else { // Sinon il y a eu une erreur lors de l'envoi
echo "<p style=\"text-align:center\">Un problème s'est produit lors de l'envoi du message.\n";
echo "<a href=\"".$_SERVER["PHP_SELF"]."\">Réessayez...</a></p>\n"; // On propose un lien de retour vers le formulaire
}
}
?>

Bonjour,

déjà, plusieurs choses :
_ enctype="multipart/form-data"
c'est justement fait pour... télécharger
vous devriez enlever
_
<input name="reset" type="image" id="reset" src="images/reset.png" align="right" value="Envoyer">
<input name="reset" type="image" id="reset" src="images/reset.png" align="right" value="Effacer">
pour effacer : <input type="reset"....
pour envoyer ! <input type="submit"...
( et rien n'empêche la présentation,les
images... en css )
un ID doit être unique dans une page


Cordialement [mon Site] [M'écrire] Bul

merci !!!!

sur dream, j'ai demandé un champ image et m'a écrit <input name=&#8230;

je vous tient au courant
cordialement

Salut,

«pouvez vous me dire si c'est bon ?»
Non ce n'est pas bon du tout. Une masse indigeste illisible n'a jamais été une bonne chose, surtout quand on mélange php, html, css, javascript etc. Là ton truc est très difficilement maintenable car beaucoup trop en bordel, il y a un très gros travail à faire sur l'organisation là. Séparer chaque langage est une très bonne idée pour commencer, par exemple commencer par mettre tout le css dans un fichier css et le javascript dans un fichier js, puis réfléchir sur une manière de ne pas mélanger php et html. Pour ce dernier point les templates sont une option pas mauvaise, mais personnellement je préfère générer du xhtml à partir des bibliothèques de manipulation de xml.