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;
}

这对我来说似乎很合理。