2

あなたの天才の誰かが、私がphpコンタクトフォームに関して抱えているいくつかの問題を解決してくれるかどうか疑問に思っています.

基本的に、名前、電子メール、コメントという基本的なものを必要とするこの連絡フォームがあります。問題は、フィールドが空白のときに「送信」ボタンを押すと、検証が効果的に開始されますが、ページが「壊れる」という犠牲を払って (つまり、タグ</body>と</html>タグを閉じることができない) ことです。

私はこれが下にあると感じdie();ますfunction died($error).....見てください....どんな助けも大歓迎です!

ありがとう

PHP: (send_email.php ファイル)

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

    // Email addys
    $email_to = "ch@yahoo.co.uk";
    $email_subject = "message";


    function died($error) {
        // ERRORS
        echo "<p class=\"content-text3\">I am very sorry, but there were error(s) found with the form you submitted.</p> ";
        echo "<p class=\"content-text\">These errors are as follows:</p><br /><br />";
        echo "<p class=\"content-text\">$error </p> <br /><br />";
        echo "<p class=\"content-text\">Please <a href=\"contact.php\">go back</a> and fix these errors.</p><br /><br />";
        die();


    }

    // validation 
    if(!isset($_POST['first_name']) ||
        !isset($_POST['last_name']) ||
        !isset($_POST['email']) ||
        !isset($_POST['telephone']) ||
        !isset($_POST['comments'])) {
        died('We are sorry, but there appears to be a problem with the form you submitted.');   
        die();    
    }

    $first_name = $_POST['first_name']; // required
    $last_name = $_POST['last_name']; // required
    $email_from = $_POST['email']; // required
    $telephone = $_POST['telephone']; // not required
    $comments = $_POST['comments']; // 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 />';
  }
  if(strlen($comments) < 2) {
    $error_message .= 'The Comments you entered do not appear to be valid.<br />';
  }
  if(strlen($error_message) > 0) {
    died($error_message);
  }
    $email_message = "Form details below.\n\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";
    $email_message .= "Telephone: ".clean_string($telephone)."\n";
    $email_message .= "Comments: ".clean_string($comments)."\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);  
    ?>
    <!-- sent success -->
    <p class="content-text">Thank you for contacting me. I will be in touch with you very soon.</p>
    <?php
    }
    ?>

お問い合わせフォーム：

 <div id="contact-form">        
          <form id="contactform" method="post" action="send_email.php">
            <table width="450px" border="0">

              <tr>
               <td valign="top">
                <label for="first_name" style="color:#999;">First Name *</label>
               </td>

               <td valign="top">
                <input  type="text" name="first_name" maxlength="50" size="30"/>
               </td>
              </tr>

              <tr>
               <td valign="top">
                <label for="last_name" style="color:#999;">Last Name *</label>
               </td>
               <td valign="top">
                <input  type="text" name="last_name" maxlength="50" size="30"/>
               </td>
              </tr>

              <tr>
               <td valign="top">
                <label for="email" style="color:#999; font-size:14px">Email Address *</label>
               </td>
               <td valign="top">
                <input  type="text" name="email" maxlength="80" size="30"/>
               </td>
              </tr>

              <tr>
               <td valign="top">
                <label for="telephone" style="color:#999; font-size:14px">Telephone Number</label>
               </td>

               <td valign="top">
                <input  type="text" name="telephone" maxlength="30" size="30"/>
               </td>
              </tr>

              <tr>
               <td valign="top">
                <label for="comments" style="color:#999; font-size:14px">Comments *</label>
               </td>

               <td valign="top">
                <textarea  name="comments"  cols="25" rows="6"></textarea>
               </td>
              </tr>

              <tr>
               <td colspan="2" style="text-align:center">
                <input id="sub" type="submit" value="Submit"/>
               </td>
              </tr>
          </table>

          </form>

          </div><!--end contact form-->
4

2 に答える 2

0

はい、あなた自身の問題を特定しました。スクリプトの途中で die() を使用しないでください。おそらく人々はこれを締めくくるために投票するでしょう。私は単純なコードを書き直すつもりはありませんが、正しい方向への実際のプッシュを与える危険を冒して、これがあなたが望むスクリプトの形式であることを提案します:

まず、送信ボタンを変更して name 属性を持たせます:

<input id="sub" type="submit" name="SubmitEmailForm" value="Submit"/>

ここではそれほど重要ではありませんが、複数のフォームを含むページがある場合、どのフォームが送信されたかを判断できます。

検証スクリプトで、errors 配列変数を初期化して空の配列にします。

function validateEmailForm() {
    $errors = array();
    // All validation here
    // if an error occurs...
    $errors[] = 'Error text here.';
    return $errors;
}

このようにして、一度にすべての検証エラーをキャッチできるため、ユーザーが修正する必要があることの完全なレポートを提供できます。

次に、基本的な構造は次のとおりです。

if (isset($_POST['SubmitEmailForm'])) {
    $errors = validateEmailForm();
    if (count($errors) == 0) {
        // Form was fine, go ahead and process it
        // header(..location of your post-processing script or output..);
        // Make sure you exit to end processing of script
        exit; 

    } else {
        //
        // Output a box containing the errors from $errors using a foreach
        //

    }  
}
//
// Output your form markup here.
//

これは、ほとんどの自己投稿フォームが利用する基本的なフレームワークであり、このパターンに装飾を簡単に追加できます。

于 2013-08-13T17:49:41.107 に答える