什么时候应该在PHP中设置类属性-在构造函数中还是需要时？

Question

我正在学习OOP PHP，并且努力了解何时设置变量。

我的课上有一堆变量，有些（但不是全部）函数使用了这些变量。 下面的代码显示了函数是否需要变量

1）检查当前是否设置

2）如果未设置，它将运行设置该变量的任何函数

是这样吗，还是应该使用__constructor函数设置所有变量？

提前非常感谢-劳拉。

class Person {

private $user_id;
private $eye_color;

public function __construct ($user_id) {

    $this->user_id = $user_id;

}


public function eyeColor () {

    $this->eye_color = get_user_meta($this->user_id, '_eye_color', 'true');

}

public function describeEyes () {

    // If Eye Color Is not set, set it
    if (!isset($this->eye_color)) {
        $this->eyeColor();
    }

    $eye_decription = 'The user has beautiful eyes, they are' . $this->eye_color;

    return $eye_decription;

}

public function describeFace () {

    // If Eye Color Is not set, set it
    if (!isset($this->eye_color)) {
        $this->eyeColor();
    }

    $face_decription = 'The user has a nice face and beautiful eyes, they are' . $this->eye_color;

    return $face_decription;
}

}

Answer 1

我认为没有唯一的方法可以做到这一点，这取决于每种情况。

如果加载数据的函数（在您的情况下为eyeColor() ）在资源或时间上都很昂贵，那么使其仅在需要时运行是一个不错的选择。 如果很简单，则可以在构造函数中运行它。

还请记住，如果您可能会使用eye_color ，那么每次需要时，您都将测试条件子句以查看其是否已经加载。 因此，如果您多次使用此值，最好将其加载到构造函数中并保存这些条件测试。

如果您确定至少要使用它一次，最好将其放入构造函数中。

Answer 2

就您而言，我认为您的Person类的Object Composition已关闭。 如果“眼睛颜色”属于用户，则应保留该位置，并且应仅由Person类公开，而不在两个类中重复。 请参见以下代码更改：

class Person {

    private $user_id;

    public function __construct($user_id)
    {
        $this->user_id = $user_id;
    }

    public function eyeColor()
    {
        return get_user_meta($this->user_id, '_eye_color', 'true');
    }

    public function describeEyes()
    {
        return 'The user has beautiful eyes, they are' . $this->eyeColor();
    }

    public function describeFace()
    {
        return 'The user has a nice face and beautiful eyes, they are' . $this->eyeColor();
    }

}

这里需要注意的几件事：

1. 我们只通过构造函数设置user_id 。 这意味着如果没有user_id ，那么Person对象将不存在，从而实现了您所追求的构图。
2. 我们通过函数eyeColor()公开了get_user_meta($this->user_id, '_eye_color', 'true') ，这意味着Person的眼睛颜色实际上只是对用户眼睛颜色属性的引用。
3. 我们将永远不会在人的眼睛颜色和基础用户的眼睛颜色之间出现不匹配，因为我们不能独立于用户的眼睛颜色来改变人的眼睛颜色。

我将进一步争论说，您应该传递一个User对象，而不是传递user_id ，但这取决于代码的面向对象程度和要采用的程度。