关于我的联系表的问题

Question

我的页面上有一个联系表单，现在大部分都可以正常工作，但是我创建了一些验证，如果用户以某种方式绕过文本输入的常规 HTML maxlength 参数，它会对字符串长度进行一些后端检查。

这是我的两个问题：

1) 如果我在字段中输入数字，那么我会自动收到一条消息，说我已经超过了最大限制，这显然不应该被执行。

2）第二个问题是，当我收到那条消息（接收错误）时，它至少应该停止发送邮件，因为有问题，但它还是发送了电子邮件！！

随意去尝试一下，亲自看看：

http://eclipse-developers.com/v2/eclipse-developers.com/#contactUs

这是我的联系表单的 PHP 脚本：

// here we put in an if statement to check against missing variables (empty values)
        if (isset($_POST['name']) && isset($_POST['email']) && isset($_POST['subject']) && isset($_POST['message'])){

            $name=$_POST['name'];
            $organisation=$_POST['organisation'];
            $reference=$_POST['reference'];
            $email=$_POST['email'];
            $subject=$_POST['subject'];
            $message=$_POST['message'];
            // here we are checking to see if that value anything and not just black.

            if (!empty($name) && !empty($email) && !empty($subject) && !empty($message) ){  
                // this is doing a check for max length, its doing it in php just in case the user
                // cheats and bypasses the html check.

            if (strlen($name>31) || strlen($lname>31) || strlen($organisation>31) || strlen($email>51) || strlen($subject>31) || strlen($reference>31) || strlen($message>2001)){
                echo"sorry, max length for a field has been exceeded. Go back and try again</a><br><br>";
            }
            $to='skyrocketing132@yahoo.com';
            $emailsubject=$subject;
            $body=$name."\nCompany Name: ".$organisation."\nRef: ".$reference."\nMessage: ".$message;
            $headers= 'From: '.$email;

            // mails, if statement so if its true (mail did send)
                if  (mail($to,$emailsubject,$body,$headers)){
                    echo'Thanks for contacting us.';        
                }else{
                    echo'Sorry, an error occurred. Try again later.';
                }
            }
            } else{
            }

Answer 1

你的条件不对，你有：

strlen($name > 31)

这将首先检查$name 的值是否高于31，这将返回一个布尔值true 或false。然后将其转换为字符串并检查其长度，该长度始终高于 0，因此为真。应该是：

strlen($name) > 31

为防止电子邮件被发送，您必须在满足这些条件后中止。你的代码会变成这样：

if (strlen($name) > 31 || strlen($lname) > 31 || strlen($organisation) >31 || strlen($email)>51 || strlen($subject)>31 || strlen($reference)>31 || strlen($message)>2001){
    echo"sorry, max length for a field has been exceeded. Go back and try again</a><br><br>";
} else {
    $to='skyrocketing132@yahoo.com';
    $emailsubject=$subject;
    $body=$name."\nCompany Name: ".$organisation."\nRef: ".$reference."\nMessage: ".$message;
    $headers= 'From: '.$email;

    // mails, if statement so if its true (mail did send)
    if  (mail($to,$emailsubject,$body,$headers)){
        echo'Thanks for contacting us.';        
    }else{
        echo'Sorry, an error occurred. Try again later.';
    }
}

Answer 2

1) 如果我在字段中输入数字，那么我会自动收到一条消息，说我已经超过了最大限制，这显然不应该被执行。

你的条件不对。

strlen($name>31)

改成

strlen($name)>31

2) 第二个问题是，当我收到那条消息（接收错误）时，它至少应该停止发送邮件，因为有问题，但它还是发送了电子邮件！ ！

在失败后放置exit;/die;，或添加if/else逻辑。

强制结束

 if (strlen($name)>31 || strlen($lname)>31 || strlen($organisation)>31 || strlen($email)>51 || strlen($subject)>31 || strlen($reference)>31 || strlen($message)>2001){
     echo"sorry, max length for a field has been exceeded. Go back and try again</a><br><br>";
    exit;
 }

if/else 逻辑

 if (strlen($name)>31 || strlen($lname)>31 || strlen($organisation)>31 || strlen($email)>51 || strlen($subject)>31 || strlen($reference)>31 || strlen($message)>2001){
     echo"sorry, max length for a field has been exceeded. Go back and try again</a><br><br>";
 } else {
    //Attempt email send
 }