Php свяжитесь с нами, отправка формы при перезагрузке страницы

Я новичок в PHP и пытаюсь создать простой способ связи с нами для своего веб-сайта. У меня есть форма отправки, но по какой-то причине она также отправляет пустое сообщение каждый раз, когда я перезагружаю страницу. Я разместил немного формы ниже.

Любая помощь здесь / любые советы, которые вы можете предложить, будут оценены.

   <?php

    $name_error = "";
    $name = "";

    if ($_SERVER["REQUEST_METHOD"] == "POST") {
        if (empty($_POST["name"])) {
            $name_error = "Whoops! Please enter your full name";
        } else {
            $name = test_input($_POST["name"]);

            if (!preg_match("/^[a-zA-Z ]*$/",$name)) {
                $name_error = "Whoops! Only letters and white spaces are allowed in a name";
            }
        }

    if ($name_error == ''){
        $message_body = '';
        unset($_POST['submit']);
        foreach ($_POST as $key => $value){
            $message_body .= "$key: $value\n";
        }   

        $to = 'myemail@gmail.com';
        $subject = 'Contact Us Form Submission';

        if (mail($to, $subject, $message_body )){
            $success = "Your message has been sent. A representative will be in contact with you shortly.";
            $name = $email = $phone = $message = $inquiry = $hear = $success = '';
        }
    }   
    function test_input($data) {
        $data = trim($data);
        $data = stripslashes($data);
        $data = htmlspecialchars($data);
        return $data;
    }
?>

вам нужно обернуть проверку почты вокруг всего кода отправки, в настоящее время он закрывается слишком рано ... на самом деле он вообще не закрыт в приведенном выше коде

user10226920 26.10.2018 05:46

Прекратите копировать код из w3sh…, особенно эту бестолковую функцию test_input().

mario 26.10.2018 06:50
2
2
40
1

Ответы 1

Ваша «почтовая» логика должна находиться в методе запроса сервера, простое решение - дублировать вашу логику if, которая будет обернута вокруг вашей логики рассылки. т.е.

if($_SERVER["REQUEST_METHOD"] == "POST"){
  if (mail($to, $subject, $message_body )){
    $success = "Your message has been sent. A representative will be in contact with you shortly.";
    $name = $email = $phone = $message = $inquiry = $hear = $success = '';
  }
}

Конечно, вы также можете просто удалить текущую закрывающую скобку операторов if и закинуть ее в конце кода (поскольку нет необходимости преждевременно прерывать оператор if)

т.е.

<?php

$name_error = "";
$name = "";

if ($_SERVER["REQUEST_METHOD"] == "POST") {
    if (empty($_POST["name"])) {
        $name_error = "Whoops! Please enter your full name";
    } else {
        $name = test_input($_POST["name"]);

        if (!preg_match("/^[a-zA-Z ]*$/",$name)) {
            $name_error = "Whoops! Only letters and white spaces are allowed in a name";
        }
    }

if ($name_error == ''){
    $message_body = '';
    unset($_POST['submit']);
    foreach ($_POST as $key => $value){
        $message_body .= "$key: $value\n";
    }   

    $to = 'myemail@gmail.com';
    $subject = 'Contact Us Form Submission';

    if (mail($to, $subject, $message_body )){
        $success = "Your message has been sent. A representative will be in contact with you shortly.";
        $name = $email = $phone = $message = $inquiry = $hear = $success = '';
    }
}

// <--- ADD CLOSING BRACKET HERE

function test_input($data) {
    $data = trim($data);
    $data = stripslashes($data);
    $data = htmlspecialchars($data);
    return $data;
}

Другие вопросы по теме