你会如何改进这个表单验证代码？

Question

我需要验证一个表单。这是一些服务器端验证，检查是否填写了必填字段，电子邮件是有效的，数字和字母数字字段不包含流氓字符。

我想就如何提高代码的可靠性和简洁性提出一些反馈意见。

<?php 

// current state 
$valid = true; 

// post data collection 
$name = "John Doe"; 
$email = "user@gmail.com"; 
$age = "19"; 

// select data that needs validation 
$required = array($name, $email); 
$validEmail  = array($email); 
$validNumber = array($age); 
$validAlpha  = array($name); 

// check required fields 
for ($i=0; $i<count($required); $i++) { 
    if (strlen($required[$i]) == 0) { 
     echo "Please fill out all required fields"; 
     $valid = false; 
     break; 
    } 
} 

// check for valid email field 
for ($i=0; $i<count($validEmail); $i++) { 
    if (preg_match('/^([A-Za-z0-9_\-\.])+\@([A-Za-z0-9_\-\.])+\.([A-Za-z]{2,4})$/', $validEmail[$i])) { 

    } else { 
     echo '"' . $validEmail[$i] . '"' . ' is an invalid email address <br />'; 
     $valid = false; 
    } 
} 

// check numeric fields 
for ($i=0; $i<count($validNumber); $i++) { 
    if (preg_match('/^[0-9 ]+$/', $validNumber[$i])) { 

    } else { 
     echo '"' . $validNumber[$i] . '"' . ' is an invalid number <br />'; 
     $valid = false; 
    } 
} 

// check alpha 
for ($i=0; $i<count($validAlpha); $i++) { 
    if (preg_match('/^[a-zA-Z ]+$/', $validAlpha[$i])) { 

    } else { 
     echo '"' . $validAlpha[$i] . '"' . ' contains invalid characters. This field only accepts letters. <br />'; 
     $valid = false; 
    } 
} 

// return "Your form was successfully sent" 
if ($valid) { 
    echo 'Your form was successfully sent. <br />Back to the site <form><input type="button" value="back" onclick="history.go(-1);return true;"></form>'; 
} 

?> 

Answer 1

减少重复代码通常可以以避免过滤器，用于验证清单：

• 回路（学习foreach代替人工for计数）
• 功能
• 数组规则集

你的情况，你需要采用更好的输入变量处理，即保持了名称，使过滤这样的：

$rules = array(
    "email" => FILTER_VALIDATE_EMAIL, 
    "number1" => '/^[0-9 ]+$/', 
    "alpha2" => '/^[a-zA-Z ]+$/',   
); 

foreach ($rules as $varname => $verify) { 
    if (is_int($verify) ? !filter_var($_REQUEST[$varname], $verify) 
         : !preg_match($verify, $_REQUEST[$varname])) 
    { 
     echo "The field '$varname' contains invalid whatevers..."; 
     $valid = false; 
    } 
} 

来自校验规则，你也可以有自己的变量名的额外的间接当然，更好的错误信息。但这是这里的基本方法。同样，使用一些自定义函数进一步帮助可读性和减少代码。

Answer 2

验证电子邮件（和其他一些领域），您可以（必须？）使用filter_var代替正则表达式，例如：

filter_var('bob@example.com', FILTER_VALIDATE_EMAIL) 

将返回TRUE。

这里是http://www.php.net/manual/en/filter.filters.validate.php