PHP, OOP - 表单验证

Question

我创建了一个表单元素类。该类能够

1. 创建 HTML 表单元素和
2. 确认表单是否已提交，如果已提交，则将指定表单元素的提交值加载到页面上处理的对象中。

我已尝试使其尽可能通用，以使其可重用，但由于我是 OOP 方面的新手，如果有人可以检查这个并让我知道这是否也是好的 OOP，那将是非常酷的如果这对于我想要实现的目标来说是一个很好的解决方案。

这是 class_FormControl() 的基本部分

class FormControl{
    var $finalcontrol;
    var $class = "form-control";
    var $form_error;

    protected function StartFormatting($name, $label){
        if (isset($_POST[$name]) AND $_POST[$name] != "") {
            return false;
        }
        $this->finalcontrol = "<label for='$name' >$label</label>";
        return true;
    }

    public function get_control(){
        return $this->finalcontrol;
}
}

class TextBox extends FormControl{

    public function CreateControl($obj, $name, $label, $placeholder, $value = ""){
        if($this->StartFormatting($name, $label)){
            $this->finalcontrol .= "<input type='text' class='$this->class' id='$name' name='$name' placeholder='$placeholder'";

            if ($value != "") { 
                $this->finalcontrol .= " value='$value' "; 
            }

            $this->finalcontrol .= ">";
            return true;
        } 

        $func = "set_" . $name;
        $obj->$func($_POST[$name]);
        return false;

    }
}

这是我在表单页面中使用类的方式:

$r1 = New Recipe();
$tbx = new TextBox();
$ctrl1 = $tbx->CreateControl($r1, "Name", "Nombre", "Nombre", $r1->get_Name());

现在，如果 $ct​​rl1 为真，我继续将对象保存在数据库中。

如果 $ct​​rl1 为假，我继续

echo $tbx->get_control();

在页面的正确位置。

/谢谢!

Answer 1

Note: This is not the only way to do it; it is one of many and it is my personal interpretation of the problem in 10 minutes on my lunch break. Please bear in mind that this would be my implementation with the little information I'd have been given. In the real world, I would find out more about the domain, but I would still adhere to the single responsibility principle in my objects and keep everything loosely coupled. These are the main things to take away from this post.

哎呀

首先，您需要从对象的角度进行思考。每个对象都有其自己的单一职责 wikipedia entry :

In object-oriented programming, the single responsibility principle states that every context (class, function, variable, etc.) should have a single responsibility, and that responsibility should be entirely encapsulated by the context. All its services should be narrowly aligned with that responsibility. [emphasis my own].

您正在做的是将过程代码放入类方法中。这不会使它面向对象!你需要转变心态!

对象

您正在构建一个表单构建器。表单由元素组成。瞬间，我在想:

• 一个表单对象
• 一个 FormElement 对象

上述对象是实体或值对象的特定表示。 Domain Driven Design .

FormElement 可以是所有表单元素(如输入框、按钮等)都必须遵循的接口(interface)。

class Form
{
   /**
    * @var FormElement[]
    */
    protected $elements;

   /**
    * Add a FormElement to the form
    *
    * @param FormElement $element
    */
    public function addFormElement(FormElement $element)
    {
        $this->elements[] = $element;
    }
}

interface FormElement
{
   /**
    * @return The generated html for the given form element
    */
    public function getHtml();
}

现在您需要做的就是确保实现 FormElement 的每个对象在 FormElement::getHtml() 方法中准确返回您想要的内容，并且当您将新元素添加到 Form 时，它们仍然可以正常工作，因为它将是 Form 调用 getHtml()在每个 FormElement 对象上循环，然后将其添加到它自己的 HTML 并输出它。

这是我将使用的 TextBoxFormElement 示例:

class TextBoxFormElement implements FormElement
{ 
   /**
    * @constructor
    *
    * This is where I am declaring that, for this to be a VALID object, it MUST have
    * the following properties passed in
    *
    * @param string $name
    * @param string $label
    * @param string $placeholder
    * @param string $value
    */
    public function __construct($id, $class, $name, $label, $placeholder, $value)
    {
        $this->id          = $id;
        $this->class       = $class;
        $this->name        = $name;
        $this->label       = $label;
        $this->placeholder = $placeholder;
        $this->value       = $value;
    }

   /**
    * Generate the html of this element
    *
    * @return string
    */
    public function getHtml()
    {
        return sprintf(
            "<input type='text' id='%s' class='%s' name='%s' label='%s' placeholder='%s' value='%s'>",
            $this->id, $this->class, $this->name, $this->label, $this->placeholder, $this->value
        );
    }
}

super 全局

您在这些类中使用 $_POST 和 $_GET。 您不应该这样做。您应该执行以下操作:

Each object your write, the public methods of these objects dictate their API. You are writing an API to those object's usages. [My own quote]

有效地，使用任何类型的 $_POST 或 $_GET 将这些对象耦合到这些超全局变量的状态。你将如何测试它们？每次要测试这些超全局变量时，您都必须模拟(或伪造)这些超全局变量的内容。

没有。您应该做的是暂时忘记您自己的应用程序并编码这些对象，以便您可以在任何应用程序中使用它们。然后，您将值传入到对象的构造函数或方法中。不要在此类中直接使用 $_POST 或 $_GET。

结论

您不是在编写 OO 代码。但我不会为你写。但是，我将编写在使用您的库时希望调用的方法，希望您能自己弄清楚自己的实现。将所有内容分开，对每个对象使用单一职责(这不是矫枉过正，它是面向对象的编程)，并不断学习:

// Sanitize and validate input data obviously you would have a RequestValidator used before you even get to this point, making sure exactly what you want can be used by the library
$name = $_POST['name'];

/** Create our form **/
$form = new Form;

/** TextBoxFormElement implements FormElement **/
$textBox1 = new TextBoxFormElement($name, 'Label', 'Placeholder' /** etc **/);

/** Add our FormElement **/
$form->addFormElement($textBox1);

/** Generate our html - the form generates it's own HTML as well as calling getHTML() on the FormElement objects in a loop **/
echo $form->getHtml();

这就是它。这就是您应该创建的对象 API。应该就这么简单。现在带着这些神奇的知识去学习吧。

---

额外信息:另一种方法是查看 Visitor Pattern .基本上，您将生成输出的事物与保存数据的事物分开。这里是 more info如果你想看看这个主题。

永远记住，您的代码应该是可读且易于理解的，如果有帮助，您应该创建一些行业标准的 UML 图来配合您的代码。