Question

我的PHP脚本有问题，它会检查3个变量（下面的代码）：

$auth（邮件作者）
$subj（邮件主题）
$text（邮件消息）

FORM：
（注意：我使用了“GET”方法，因为出于某种奇怪的原因，“POST”方法不起作用）

                <div id="contact_form">
                        <form method="get" name="contact" action="home.php">
                                <input type="hidden"
                                       name="method"
                                       value="send"/>
                                E-Mail:<br/>
                                <input type="text"
                                       id="author" 
                                       name="author" 
                                       class="require input_field"
                                       value=""/>

                                <br/>
                                Subject:<br/>
                                <input type="text"
                                       id="subject" 
                                       name="subject" 
                                       class="require input_field"
                                       value=""/>

                                <br/>
                                Message:<br/>
                                <textarea id="text"
                                      name="text"
                                      rows="0"
                                      cols="0"
                                      class="required"
                                      value=""></textarea>

                                <br/>
                                <input type="submit"
                                       class="submit_btn"
                                       name="submit"
                                       id="submit"
                                       value="Submit" />
            </form>
        </div>

表格现在运作得很好。

PHP：

<?php // ||HOME.PHP||
$method = $_GET['method'];
$auth = $_GET['author'];
$subj = $_GET['subject'];
$text = $_GET['text'];
$recv = "mymail@stuff.com";


function redirect($location) {
        if($location == "true") {
            header("Location: http://mysite.com/home.php?method=result&status=true");
        } else {
            header("Location: http://mysite.com/home.php?method=result&status=false");
        }
}

//...
//Other methods...
//...

//METHOD SEND
if($method == "send") {


    //HERE IS THE PROBLEM
    //These are apparently not working
    //If i leave the form blank and submit it
    //these won't redirect to "false" (mail not sent),
    //and the script will just continue, send the empty mail
    //and redirect to "true" (mail sent)

    if(empty($auth)) { redirect(""); }
    if(empty($subj)) { redirect(""); }
    if(empty($text)) { redirect(""); }

    if(!strstr($auth, '@')) { redirect(""); }
    if(!strstr($auth, '.')) { redirect(""); }

    if(strlen($auth) < 5) { redirect(""); }
    if(strlen($subj) < 4) { redirect(""); }
    if(strlen($text) < 4) { redirect(""); }

    //From here it should work just fine
    //As i'm not sure the "RESULT" method (below) is working fine, i
    //posted it too.




    $auth =  "From: " . $auth;

    mail($recv,$subj,$text,$auth);
    redirect("true");

    require("template/footer.html");

    exit(0);
}

//METHOD RESULT
if($method == "result") {

    $status = $_GET['status'];

    if($status == "true") {
        echo "mail sent";
    } else {
        echo "mail not sent";
    }
    ?>

    <script language="javascript">
        setTimeout("location.href = 'http://adteam.altervista.org/home.php';", 5000);
    </script>

    <?php

    exit(0);
} ?>

问题在PHP代码中解释（在“SEND”方法下面的注释中）。

你们有什么建议吗？

Answer 1

您需要在设置重定向标头后停止脚本执行。否则，只有在将任何标头发送到浏览器之前，它才会继续发送邮件并设置新的重定向标头。

function redirect($location) {
    if($location) {
        header("Location: http://mysite.com/home.php?method=result&status=true");
    } else {
        header("Location: http://mysite.com/home.php?method=result&status=false");
    }

    die();
}

请注意if( $location == "true" )是一种反模式;最好使用布尔true和false代替字符串。

Answer 2

应该很容易。你说＆＃34;＆＃34;是假的。但它不是因为＆＃34;＆＃34;是真的，但是空的，这是真的。 false未设置或指定为false。所以你应该这样做：

function redirect($location) {
        if($location) {
            header("Location: http://mysite.com/home.php?method=result&status=true");
            exit();
        } else {
            header("Location: http://mysite.com/home.php?method=result&status=false");
            exit();
        }
}


And use: redirect(true) / redirect(false);

Answer 3

字符串将始终评估为true，即使它为空。这正是我们使用empty（）而不是isset（）检查字符串的原因。还有一些事情：

您应该使用POST提交电子邮件。
您应该在验证输入之前检查表单是否实际提交。
您应该创建并显示特定的错误消息，告诉用户他们没有完成哪些必填字段。
您应该在输入上运行一些仔细的验证例程，以避免将您的电子邮件表单用于发送垃圾邮件和恶意软件。

Answer 4

只需在假分支中添加exit：

function redirect($location) {
        if($location == "true") {
            header("Location: http://mysite.com/home.php?method=result&status=true");
        } else {
            header("Location: http://mysite.com/home.php?method=result&status=false");
            exit(0); //simply add it here
        }
}

header函数不会自动停止进一步执行，如果在出现错误时不想发送电子邮件，则必须停止执行其他操作。

实际上，您可以在if语句之后添加exit。

PHP无法检查变量

4 个答案: