[Mail] PHP et envoi d'email avec PHPmailer

**dolf13** · 13/07/2006, 18h44

Bonjour,

J'ai crée un code qui permet aux utilisateurs de mon site d'envoyer des messages à d'autre utilisateur via un formulaire de contact seulement voila les messages ne sont pas toujours envoyés et je vois pas pourquoi ca fonctionne dans certain cas et pas dans d'autres. Si vous avez une idee n'hesitez pas. Voici le code de ma page pour envoyer le message. Je recupere la variable $emailfrom sur la page precedente à l'aide d'un formulaire.

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
 
 
<? include("_connexion.php"); ?>
<?
$message=""; if(!empty($_POST["message"])) $message=$_POST["message"];
$subject=""; if(!empty($_POST["subject"])) $subject=$_POST["subject"];
$emailto=""; if(!empty($_POST["emailto"])) $emailto=$_POST["emailto"];
$idad=""; if(!empty($_POST["idad"])) $idad=$_POST["idad"];
$memberfrom=""; if(!empty($_POST["memberfrom"])) $from=$_POST["memberfrom"];
$memberto=""; if(!empty($_POST["memberto"])) $to=$_POST["memberto"];
$emailfrom=""; if(!empty($_POST["emailfrom"])) $emailfrom=$_POST["emailfrom"];
?>
 
<?
require("c:\php\includes\class.phpmailer.php");
 
 
	$mail = new PHPMailer();
	$mail->IsSMTP();
	$mail->Host = 'monhost';
	$mail->SMTPAuth = true;
	$mail->Username = 'monusername';
	$mail->Password = 'monpassword';
	$mail->From = "$emailfrom";
	$mail->FromName = "$memberfrom";
	$mail->IsHTML(true);
	$mail->Body = "$message";	
	$mail->Subject = "$subject";
    $mail->AddAddress("$emailto");
	$mail->AddReplyTo("$emailfrom");
 
	if (@$mail->Send()){
		$result = "Merci,<br>\r\n message envoyé.\r\n";
	}else{
		$result = "Désolé,<br>\r\n Erreur";
	}
?>
 
 
<!DOCTYPE html PUBLIC "-//W3C//DTD XHTML 1.0 Transitional//EN" "http://www.w3.org/TR/xhtml1/DTD/xhtml1-transitional.dtd">
<html xmlns="http://www.w3.org/1999/xhtml">
<head>
<meta http-equiv="Content-Type" content="text/html; charset=iso-8859-1" />
<title>Untitled Document</title>
</head>
 
<body>
 
 
</body>
</html>
<? mysql_close(); ?>

**gloubi** · 13/07/2006, 20h01

d'abord, j'enleverai les " " autours des nom de variables quand tu les assignes aux propriétés de ta class...

Ensuite, as-tu des exemples de cas ou ça passe et de cas ou ça passe pas ? est-ce que pour ceux pour qui ça passe passe a chaque fois, ou est-ce aléatoire ?

**Yogui** · 13/07/2006, 22h11

Salut

Dans un premier temps, tu ferais effectivement bien de nettoyer le code.

"$variable" et $variable sont identiques mais le 2° est plus intuitif (et plus rapide à exécuter).
?> <? : complètement inutile
remplace <? par <?php
utilise un chemin relatif dans require()
que fait là le mysql_close() alors qu'il n'y a visiblement pas de mysql_open() ?

remplace :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

$message=""; if(!empty($_POST["message"])) $message=$_POST["message"];

par :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
if(!empty($_POST["message"]) and trim($_POST["message"]) != ''){
   $message = trim($_POST["message"]);
}
else{
   $message="";
}

(et fais pareil pour toutes les variables)

Pour terminer, quelques liens :
Envoyer des mails simplement avec PHP et la classe PHPMailer
Apostrophes ou guillemets : lesquels choisir ?

**dolf13** · 13/07/2006, 23h45

Envoyé par Yogui

Salut
[*]que fait là le mysql_close() alors qu'il n'y a visiblement pas de mysql_open() ?[/LIST]remplace :

C'est pour fermer ma connexion à la base. Je debute et j'ai repris ca sur un site sérieux. Pourquoi c pas bon?

remplace :
Code :

$message=""; if(!empty($_POST["message"])) $message=$_POST["message"];

par :
Code :

if(!empty($_POST["message"]) and trim($_POST["message"]) != ''){ $message = trim($_POST["message"]); } else{ $message=""; }

(et fais pareil pour toutes les variables)

Qu'est ce que ce code a de mieux que celui que j'utilise. Comme je t'ai dit je debute et j'aime bien comprendre ce que je fais. Et pourquoi ne pas utiliser tout simplement

$message=$_POST["message"]???

Merci pour ton aide. L'envoi d'email fonctionne tres bien.

Est ce qu'il ne serait pas mieux d'ajouter mes les donnees pour se connecter a mon service mail dans mon fichier _connexion.php?

**Yogui** · 14/07/2006, 00h27

Arf, je n'avais pas fait attention à ton fichier de connexion. Cependant, puisque tu n'utilises pas la base de données de tout le script, il n'est peut-être pas nécessaire de faire cette connexion.

Rapport à mysql_close(), PHP se charge de fermer la connexion à la fin du script : tu n'as vraiment pas besoin de l'appeler mais tu fais comme tu le souhaites. C'est correct mais inutile.

Pour le if(), tu fais comme tu veux aussi...
Avec ta solution, tu commences par affecter la variable puis par remplacer sa valeur si besoin. Avec ma méthode, tu ne donnes à ta variable que la valeur qu'elle a besoin. Ça donne exactement le même résultat, c'est juste organisé différemment.

Ceci est incorrect :
$message=$_POST["message"];
En effet, la variable $_POST["message"] n'existe pas nécessairement, ce qui peut (devrait) conduire à une erreur dans certains cas. C'est pour cette raison que je teste avec !empty().Maintenant, si tu souhaites apprendre davantage de choses sur PHP, je te recommande de jeter un oeil à notre page de cours et à notre FAQ :
http://php.developpez.com/cours/
http://php.developpez.com/faq/

Plus généralement :
http://php.developpez.com/

Bonne continuation

**dolf13** · 14/07/2006, 00h39

merci beaucoup.
Une dernière petite chose si ca te derange pas. Dans les formulaires comment est ce qu'on gère les caractères du style : ' car ca rajoute des \\\\\\\à ne plus savoir quoi en faire et du coup l'info n'est pas toujours enregistree dans la base.
Dolf

**Yogui** · 14/07/2006, 00h51

J'imagine que ton php.ini a mis les magic_quotes_gpc à on, ce qui fait que PHP ajoute \ devant les ' et "... Elle devrait être à off.

La meilleure solution pour toi est de lire la documentation de la fonction mysql_real_escape_string().
Une alternative est la fonction stripslashes().

[Mail] PHP et envoi d'email avec PHPmailer

Langage PHP

Discussions similaires

Partager

Partager