2011-01-07 91 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請求發送的時間非常接近相同的數據,或反對某種雙擊使用。