Question

在我的网站上，我有足够的形式和领域，我有一个完美的系统，但真的很痛苦，并且起床和运行。

这是我的系统所做的，一旦你发送表单，所有的信息都被发送到拥有所有数据并验证它的类。它还存储字段的值，如$_SESSION['userUpdate']['firstName'][1] = $firstName;

如果出现错误，则会创建一个类似于此$_SESSION['userUpdate']['firstName'][0] = 1;的会话变量，1表示该字段为空。如果没有错误，则会话变量将为0。

如果在验证过程中找不到任何错误，则将数据发送到数据库。

之后，表单页面重新加载：

header( 'HTTP/1.1 303 See Other' );
header( 'Location: '.curPageURL().'' );

我使用它，以便在重新加载页面时无法重新发送数据。这就是我使用所有这些会话变量的原因。

然后使用大量/ else if / else我检查会话变量的值和输出错误，并使用先前输入的数据填充表单。

让我给你看一个名字字段的例子。

这是HTML代码：

<label for="firstName" class="block">First Name</label>
<span>Your first name goes here.</span><?php echo $text_first_name ?>
<input type="text" id="firstName" class="mediaText" name="firstName" value="<?php echo $first_name2; ?>" onchange="needToConfirm=true" <?php echo $style_first_name ?> />

这是班级的验证过程：

$_SESSION['ui']['first_name'][1] = $this->first_name; 
if (isset($this->first_name)) 
    {
        if($this->first_name == NULL)
        {
            $_SESSION['ui']['first_name'][0] = 1;
        }
        else if(minRange(3, $this->first_name)) 
        {   
            $_SESSION['ui']['first_name'][0] = 2; 
        }
        else
        {
            array_push($set, "FirstName = '".$db->sql_escape($this->first_name)."'");
        }
    }

这是处理最终错误的php代码：

$error_bg = "style=\"background:#EE5C42\"";

//FIRST NAME
if($_SESSION['ui']['first_name'][0] == 1)
{
    $style_first_name = $error_bg;
    $first_name2 = $_SESSION['ui']['first_name'][1];
}
else if($_SESSION['ui']['first_name'][0] == 2)
{
    $style_first_name = $error_bg;
    $first_name2 = $_SESSION['ui']['first_name'][1];
    $text_first_name = "<span class=\"errorText\">Your first name must consist of at least 3 characters.</span>";
}
else
{
    $first_name2 = $userdetails["FirstName"];
}

在页面的末尾有一个小功能来取消设置会话变量。

我想知道有没有办法让这个更简单，更容易起床和运行？

Answer 1

如果您要求代码优化建议，请按以下步骤操作：

// so you have bunch of error codes
$possible_errors = array(
  '0' => '', // No error
  '1' => '', // Error, but no error message
  '2' => 'Your %s must consists of at least 3 characters',
  ...
);

// then you have form fields stored in session
// fields in session should be named like the keys in $userdetails
// 'formName' => array('FirstName' => array(0, 'German'), 'LastName' => array('1', ''))
$errors = $values = array();
foreach ($_SESSION['formName'] as $field => $element) {
    if ($element[0] > 0) { // Has error
        $error_code = $element[0];
        $error_message = $possible_errors[$error_code];
        if (!empty($error_message)) {
            $errors[$field] = sprintf($error_message, $field);
        }
    } else {
        $values[$field] = $userdetails[$field];
    }
}

// in here you end up with two arrays:
//   - $errors Error messages keyed by field name
//   - $values Values keyed by field name
// You use them like this
<label for="firstName" class="block">First Name</label>
<span>Your first name goes here.</span><?php if (array_key_exists('FirstName', $errors)) echo $errors['FirstName']; ?>
<input type="text" id="firstName" class="mediaText" name="firstName" 
  value="<?php echo $values['FirstName']; ?>"
  onchange="needToConfirm=true"
  <?php if(array_key_exists('FirstName', $errors)):?>style="background:#EE5C42"<?php endif;?> />

Answer 2

我建议使用非常好的Bassistance jQuery Validate plugin添加一些客户端验证。

通过验证创建完美的表单

2 个答案: