php OOP：在类的构造函数中做某事与调用私有方法做同样的事情[关闭]

Question

在阅读各种教程时，我注意到这两种不同的方法，我很好奇幕后发生了什么。

第一种方法在构造函数中做所有事情——所有事情都是设置变量，或者在这种情况下调用类的方法来做某事。第二种方法在构造函数中调用私有方法来做同样的事情。什么时候一种方式优于另一种方式？我知道微优化是邪恶的，但是在学术层面上调用私有方法会慢一点吗？

这是第一种方法的代码示例（这是一些 zf2 代码）：

class LoginForm extends Form
{
    public function __construct($name = null) {
        // we want to ignore the name passed
        parent::__construct('login-form');

        $this->setAttribute('method', 'post');

        $this->add(array(
            'name' => 'username',
            'attributes' => array(
                'type'=> 'text',
            ),
            'options' => array(
                'label' => 'User Name',
            ),
        )); ...

VS 这个：

class LoginForm extends Form
{
    public function __construct($name = null) {
        // we want to ignore the name passed
        parent::__construct('login-form');

        $this->setAttribute('method', 'post');

        $this->addElements();
    }
    private function addElements() {
        $this->add(array(
            'name' => 'username',
            'attributes' => array(
                'type'=> 'text',
            ),
            'options' => array(
                'label' => 'User Name',
            ),
        )); ...

不明白为什么会被标记。我没有问哪种方式更好我看到了一些我不完全理解的东西，想听听一些可能的原因为什么一个程序员可能会用一种方式来做，而另一个以另一种方式做到了。接受的答案对我很有帮助，因为它还指出了一种方法可能可能出现的设计问题。

Answer 1

构造函数中的私有函数调用是代码异味的症状。要么函数很长——这就是你首先想要函数调用的原因——要么构造函数正在做一些它不应该做的事情。

在这种情况下，setAttribute 和 addElements 似乎正在为对象设置一些默认值。还有其他方法可以实现这一点。

具体来看这个案例，您可能会争辩说LoginForm 不应该是一个类。相反，你可以有一个函数：

function create_login_form() {
    $form = new Form();
    $form->setAttribute("method", "post");
    // etc
    return $form;
}

---

更仔细地查看这段代码：它似乎是某种 HTML 包装器，但......值得怀疑。 HTML 非常擅长它的功能。它是一种 DSL（领域特定语言）。 LoginForm 与 DSL 的原始概念相去甚远。

如果你更进一步，你会得到：

function create_login_form() {
    $form = new HTML_Element("form");
    $form->setAttribute("method", "post");
    // etc
    return $form;
}

这对我来说看起来很合理。