我有一个电子邮件表单,可以检查三个字段,名称,有效的电子邮件和评论。但是它现在的设置方式,因为名称和注释在一个函数中,它首先检查名称和注释,即使电子邮件无效,我如何重新编写它以便按顺序检查字段。此外,我想重新显示没有错误的字段,因此用户不必再次键入。请帮忙。感谢
<?php
$myemail = "comments@myemail.com";
$yourname = check_input($_POST['yourname'], "Enter your name!");
$email = check_input($_POST['email']);
$phone = check_input($_POST['phone']);
$subject = check_input($_POST['subject']);
$comments = check_input($_POST['comments'], "Write your comments!");
if (!preg_match("/([\w\-]+\@[\w\-]+\.[\w\-]+)/", $email))
{
show_error("Enter a valid E-mail address!");
}
exit();
function check_input($data, $problem='')
{
$data = trim($data);
$data = stripslashes($data);
$data = htmlspecialchars($data);
if ($problem && strlen($data) == 0)
{
show_error($problem);
}
return $data;
}
function show_error($myError)
{
?>
<!doctype html>
<html>
<body>
<form action="myform.php" method="post">
<p style="color: red;"><b>Please correct the following error:</b><br />
<?php echo $myError; ?></p>
<p>Name: <input type="text" name="yourname" /></P>
<P>Email: <input type="text" name="email" /></p>
<P>Phone: <input type="text" name="phone" /></p><br />
<P>Subject: <input type="text" style="width:75%;" name="subject" /></p>
<p>Comments:<br />
<textarea name="comments" rows="10" cols="50" style="width: 100%;"></textarea></p>
<p><input type="submit" value="Submit"></p>
</form>
</body>
</html>
<?php
exit();
}
?>
答案 0 :(得分:5)
首先,我建议您一次验证所有字段,并在表单上显示所有相应的错误消息。主要原因是,如果他们必须提交一大堆表单,那么它可能是糟糕的用户体验,因为他们必须一次解决一个错误。我宁愿一次性更正我的电子邮件地址,密码,评论和选择,而不是一次修复一个,以揭示下一个错误是什么。
也就是说,这里有一些关于验证表单的指示。这通常是我如何处理表单做你想做的事情。这假设您的表单HTML和表单处理器(PHP)在同一个文件中(这就是您现在拥有的)。你可以拆分这两个,但这样做的方法可能有点不同。
$errors = array()
)。当此数组为空时,您知道提交$errors
数组(例如$errors['password'] = 'Passwords must be at least 8 characters long';
)$_POST
数组,或者清理并将$_POST
值分配给个人变量或数组。if (sizeof($errors) > 0) { // show messages } else { // process form }
value=""
属性,并回显用户提交的值。 使用htmlspecialchars()或类似功能转义输出非常重要 有了这些东西,这里有一些表格的重做:
<?php
$myemail = "comments@myemail.com";
$errors = array();
$values = array();
$errmsg = '';
if ($_SERVER['REQUEST_METHOD'] == 'POST') {
foreach($_POST as $key => $value) {
$values[$key] = trim(stripslashes($value)); // basic input filter
}
if (check_input($values['yourname']) == false) {
$errors['yourname'] = 'Enter your name!';
}
if (check_input($values['email']) == false) {
$errors['email'] = 'Please enter your email address.';
} else if (!preg_match('/([\w\-]+\@[\w\-]+\.[\w\-]+)/', $values['email'])) {
$errors['email'] = 'Invalid email address format.';
}
if (check_input($values['comments']) == false) {
$errors['comments'] = 'Write your comments!';
}
if (sizeof($errors) == 0) {
// you can process your for here and redirect or show a success message
$values = array(); // empty values array
echo "Form was OK! Good to process...<br />";
} else {
// one or more errors
foreach($errors as $error) {
$errmsg .= $error . '<br />';
}
}
}
function check_input($input) {
if (strlen($input) == 0) {
return false;
} else {
// TODO: other checks?
return true;
}
}
?>
<!doctype html>
<html>
<body>
<form action="<?php echo $_SERVER['PHP_SELF'] ?>" method="post">
<?php if ($errmsg != ''): ?>
<p style="color: red;"><b>Please correct the following errors:</b><br />
<?php echo $errmsg; ?>
</p>
<?php endif; ?>
<p>Name: <input type="text" name="yourname" value="<?php echo htmlspecialchars(@$values['yourname']) ?>" /></P>
<P>Email: <input type="text" name="email" value="<?php echo htmlspecialchars(@$values['email']) ?>" /></p>
<P>Phone: <input type="text" name="phone" value="<?php echo htmlspecialchars(@$values['phone']) ?>"/></p><br />
<P>Subject: <input type="text" style="width:75%;" name="subject" value="<?php echo htmlspecialchars(@$values['subject']) ?>" /></p>
<p>Comments:<br />
<textarea name="comments" rows="10" cols="50" style="width: 100%;"><?php echo htmlspecialchars(@$values['comments']) ?></textarea></p>
<p><input type="submit" value="Submit"></p>
</form>
</body>
</html>
我有一个更高级的示例,您可以看到here也可以为您提供一些指导。
希望有所帮助。
答案 1 :(得分:1)
使用此脚本结构:
<?php
$errors = array();
if (isset($_POST['send'])) {
// check data validity
if (!mailValid($_POST['email']))
$errors[] = 'Mail is not valid';
...
// send data by email
if (!$errors) {
// send mail and redirect
}
}
?>
<html>
...
<?php
if ($errors) {
// display errors
foreach ($errors as $error) {
echo "$error<br />";
}
}
?>
<form ...>
...
Email: <input type="text" name="email" value="<?php echo isset($_POST['email']) ? htmlspecialchars($_POST['email']) : '' ?>" />
...
</form>
...
</html>
答案 2 :(得分:1)
最简单的选择是使用form validation library。例如,PHP的filter扩展为某些类型提供了验证和清理,但它不是一个完整的解决方案。
如果您坚持自己实施,那么您需要考虑的一个问题就是顺序:表单中元素的顺序或$_POST
中用户输入的顺序。在大多数浏览器中,这些应该是相同的,但是没有标准可以强制执行此操作。如果您想要脱离表单顺序,则需要在一个地方定义表单结构,并使用该信息来执行生成或验证表单(Don't Repeat Yourself(DRY)原则的结果)。迭代适当的结构将为您提供所需的顺序:循环遍历表单为您提供表单顺序,而循环在$_POST
上为您提供用户输入顺序。
看起来你想要的不仅仅是验证数据;你也想准备使用它,这个过程称为“消毒”。
在消毒方面,定义不同类型的消毒剂,而不是单一的check_input
功能。具体的消毒剂可以是函数,也可以是__invoke
方法的对象。创建表单字段到清洁程序的映射(例如,清理程序回调的输入名称数组)。映射中元素的顺序设置了清理的顺序;如果使用单一结构来定义表单信息,则显示顺序和清理顺序将相同。
这是一个非常广泛的大纲:
# $fields could be form structure or user input
foreach ($fields as $name => $data) {
# sanitize dispatches to the appropriate sanitizer for the given field name
$form->sanitize($name, $data);
# or:
//sanitize($name, $data);
# or however you choose to structure your sanitization dispatch mechanism
}
至于将输入值设置为用户提供的数据,只需在输出元素时输出元素值。与所有用户输入(实际上,所有格式化输出)一样,在输出数据时正确地转义数据。对于HTML属性,这意味着使用(例如)htmlspecialchars
。请注意,您应该只转义传出数据。这意味着您的清理功能不应调用htmlspecialchars
。
通过将每个错误放在相应的输入旁边,为元素添加“错误”类并设置“错误”类的样式以使其突出,可以提高可用性。通过在标签文本周围包裹<label>
元素来改善可访问性。
答案 3 :(得分:1)
你总是可以这样做,使用filter_var和in_array检查:
<?php
$myemail = "comments@myemail.com";
//Pre made errors array
$errors=array('name'=>'Enter Your name',
'email'=>'Please enter valid email',
'phone'=>'Please enter valid phone number',
'subject'=>'Please enter valid subject, more then 10 chars',
'comment'=>'Please enter valid comment, more then 10 chars');
//Allowed post params and its validation type
$types = array('name'=>'string',
'email'=>'email',
'phone'=>'phone',
'subject'=>'string',
'comment'=>'string');
//A simple validation function using filter_var
function validate($value,$type){
switch ($type){
case "email":
return ((filter_var($value, FILTER_VALIDATE_EMAIL))?true:false);
break;
case "phone":
return ((preg_match("/^[0-9]{3}-[0-9]{4}-[0-9]{4}$/", $value))?true:false);
break;
case "string":
return ((strlen($value) >=10 )?true:false);
break;
default:
return false;
break;
}
}
//If forms been posted
if(!empty($_POST) && $_SERVER['REQUEST_METHOD'] == 'POST'){
//Assign true, if all is good then this will still be true
$cont=true;
$error=array();
foreach($_POST as $key=>$value){
//if key is in $types array
if(in_array($key,$types)){
//If validation true
if(validate($value, $types[$key])==true){
$$key=filter_var($value, FILTER_SANITIZE_STRING);
}else{
//Validation failed assign error and swithc cont to false
$error[$key]=$errors[$key];
$cont=false;
}
}
}
}
if($cont==true && empty($error)){
//Send mail / do insert ect
}else{
//Default to form
?>
<!doctype html>
<html>
<body>
<form action="" method="post">
<p>Name: <input type="text" name="name" value="<?=@htmlentities($name);?>"/> <?=@$error['name'];?></P>
<P>Email: <input type="text" name="email" value="<?=@htmlentities($email);?>" /> <?=@$error['email'];?></p>
<P>Phone: <input type="text" name="phone" value="<?=@htmlentities($phone);?>"/> <?=@$error['phone'];?></p><br />
<P>Subject: <input type="text" style="width:75%;" name="subject" /> <?=@$error['subject'];?></p>
<p>Comments: <?=@$error['comment'];?><br />
<textarea name="comment" rows="10" cols="50" style="width: 100%;"><?=@htmlentities($comment);?></textarea></p>
<p><input type="submit" value="Submit"></p>
</form>
</body>
</html>
<?php
}?>