2011-01-07 95 views
0

我在http://www.brianrhea.com有一个PHP/AJAX的形式在我的网站(点击申请项目)PHP邮件发送重复使用不同的时间戳

当我测试多台计算机的形式,它工作正常,我。但是,我会零星地收到一封重复的电子邮件,并且至少听到一位试图提交的用户的消息,告诉他们一个错误,我无法复制。

这是我正在使用的PHP。有什么突出的潜在问题?

<?php 

//Retrieve form data. 
//GET - user submitted data using AJAX 
//POST - in case user does not support javascript, we'll use POST instead 
$name = ($_GET['name']) ? $_GET['name'] : $_POST['name']; 
$email = ($_GET['email']) ?$_GET['email'] : $_POST['email']; 
$subject = ($_GET['subject']) ?$_GET['subject'] : $_POST['subject']; 
$comments = ($_GET['comments']) ?$_GET['comments'] : $_POST['comments']; 

//flag to indicate which method it uses. If POST set it to 1 
if ($_POST) $post=1; 

//Simple server side validation for POST data, of course, you should validate the email 
if (!$name) $errors[count($errors)] = 'Please enter your name.'; 
if (!$email) $errors[count($errors)] = 'Please enter your email.'; 
if (!$subject) $errors[count($errors)] = 'Please choose a subject.'; 
if (!$comments) $errors[count($errors)] = 'Please enter your comments.'; 

//if the errors array is empty, send the mail 
if (!$errors) { 

    //recipient 
    $to = '[email protected]';  
    //sender 
    $from = $name . ' <' . $email . '>'; 

    //subject and the html message 
    $subject = 'Comment from ' . $name; 
    $message = ' 
    <!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></head> 
    <body> 
    <table> 
     <tr><td>Name</td><td>' . $name . '</td></tr> 
     <tr><td>Email</td><td>' . $email . '</td></tr> 
     <tr><td>Subject</td><td>' . $subject . '</td></tr> 
     <tr><td>Comments</td><td>' . nl2br($comments) . '</td></tr> 
    </table> 
    </body> 
    </html>'; 

    //send the mail 
    $result = sendmail($to, $subject, $message, $from); 

    //if POST was used, display the message straight away 
    if ($_POST) { 
     if ($result) echo 'Thank you! We have received your message.'; 
     else echo 'Please verify that you have entered a valid email address.'; 

    //else if GET was used, return the boolean value so that 
    //ajax script can react accordingly 
    //1 means success, 0 means failed 
    } else { 
     echo $result; 
    } 

//if the errors array has values 
} else { 
    //display the errors message 
    for ($i=0; $i<count($errors); $i++) echo $errors[$i] . '<br/>'; 
    echo '<a href="form.php">Back</a>'; 
    exit; 
} 


//Simple mail function with HTML header 
function sendmail($to, $subject, $message, $from) { 
    $headers = "MIME-Version: 1.0" . "\r\n"; 
    $headers .= "Content-type:text/html;charset=iso-8859-1" . "\r\n"; 
    $headers .= 'From: ' . $from . "\r\n"; 

    $result = mail($to,$subject,$message,$headers); 

    if ($result) return 1; 
    else return 0; 
} 

?> 
+0

可能会更容易使用$ _REQUEST:默认情况下包含$ _GET,$ _POST和$ _COOKIE内容的关联数组。 – 2011-01-07 17:59:10

+0

而FYI没有验证你$ _GET/$ _ POST,你应该验证用户输入 – 2011-01-07 18:03:29

回答

0

此代码似乎未发送邮件两次。

  • 尝试在应用程序中搜索其他可能调用此代码/函数的地方。
  • 检查是否有刷新预防 - 如果您检测到ajax请求发送的时间非常接近相同的数据,或反对某种双击使用。