Форма электронной почты PHP не отправляет все поля - PullRequest
1 голос
/ 22 января 2012

Когда я отправляю форму, я получаю поля last_name и email, но не hte first_name. Сводит меня с ума! Если кто-нибудь может быстро взглянуть на это и сообщить мне, почему я был бы очень признателен. Я новичок в PHP. Спасибо !!

HTML:

<form name="contactform" method="post" action="send_form_email.php">
        <div class="email_section_details"> <span class="msg_icon"><img src="common/images/msg_icon.png" alt="" /></span>
          <div class="row">
                          <div class="col">
            <label for="first_name"><strong>First</strong>NAME:</label>
              <span class="input_bg">
              <input name="first_name" type="text" value="" />
              </span> </div>
            <div class="col">
              <label for="email"><strong>EMAIL</strong>ADDRESS:</label>
              <span class="input_bg">
              <input name="email" type="text" value="" />
              </span> </div>
            <div class="col">
              <label for="last_name"><strong>LAST</strong>NAME:</label>
              <span class="input_bg">
              <input name="last_name" type="text" value="" />
              </span> </div>
            <input type="image" src="common/images/submit_btn.png" class="submit_btn" />
        </div>
          </form>

Вот PHP:

    <?php
if(isset($_POST['email'])) {

    // EDIT THE 2 LINES BELOW AS REQUIRED
    $email_to = "#";
    $email_subject = "Email Update Sign-up Request from #";


    function died($error) {
        // your error code can go here
        echo "We are very sorry, but there were error(s) found with the form you submitted. ";
        echo "These errors appear below.<br /><br />";
        echo $error."<br /><br />";
        echo "Please go back and fix these errors.<br /><br />";
        die();
    }

    // validation expected data exists
    if(!isset($_POST['first_name']) ||
        !isset($_POST['last_name']) ||
        !isset($_POST['email']))


    $first_name = $_POST['first_name']; // required
    $last_name = $_POST['last_name']; // required
    $email_from = $_POST['email']; // required


    $error_message = "";
    $email_exp = '/^[A-Za-z0-9._%-]+@[A-Za-z0-9.-]+\.[A-Za-z]{2,4}$/';
  if(!preg_match($email_exp,$email_from)) {
    $error_message .= 'The Email Address you entered does not appear to be valid.<br />';
  }
    $string_exp = "/^[A-Za-z .'-]+$/";
  if(!preg_match($string_exp,$first_name)) {
    $error_message .= 'The First Name you entered does not appear to be valid.<br />';
  }
  if(!preg_match($string_exp,$last_name)) {
    $error_message .= 'The Last Name you entered does not appear to be valid.<br />';
  }

    $email_message = "Form details below.\n";

    function clean_string($string) {
      $bad = array("content-type","bcc:","to:","cc:","href");
      return str_replace($bad,"",$string);
    }

    $email_message .= "First Name: ".clean_string($first_name)."\n";
    $email_message .= "Last Name: ".clean_string($last_name)."\n";
    $email_message .= "Email: ".clean_string($email_from)."\n";



// create email headers
$headers = 'From: '.$email_from."\r\n".
'Reply-To: '.$email_from."\r\n" .
'X-Mailer: PHP/' . phpversion();
@mail($email_to, $email_subject, $email_message, $headers);  
?>

<!-- include your own success html here -->

Thank you for contacting us. We will be in touch with you very soon. Please use your "Back Button" to return to the Home Page

<?php
}
?>

1 Ответ

2 голосов
/ 22 января 2012

Ваша проблема в следующем коде:

// validation expected data exists
if(!isset($_POST['first_name']) ||
    !isset($_POST['last_name']) ||
    !isset($_POST['email']))


$first_name = $_POST['first_name']; // required
$last_name = $_POST['last_name']; // required
$email_from = $_POST['email']; // required

Это оператор if без следующего кода, поэтому он выбирает следующую строку, где вы присваиваете $ first_name. Так что это никогда не будет выполнено. Это эквивалентно этому:

// validation expected data exists
if(!isset($_POST['first_name']) ||
    !isset($_POST['last_name']) ||
    !isset($_POST['email']))
{
    $first_name = $_POST['first_name']; // required
}

$last_name = $_POST['last_name']; // required
$email_from = $_POST['email']; // required

В соответствующей заметке, в общем, этот код довольно запутан с непоследовательными уровнями отступов и т. Д. Я знаю, что он кажется тривиальным и педантичным, но если вы потратите немного больше времени на написание более чистого кода, вам будет легче устранять ошибки, как это (как и другие люди).

...