使用类吸气剂或类属性？ (PHP)

Question

在内部使用属性时，编写 OOP 类的最佳实践是什么。

考虑下面的类；

<?php
Class Foo
{
    /**
     * @var null|string
     */
    protected $_foo;

    /**
     * @return null|string
     */
    public function getFoo()
    {
        return $this->_foo;
    }

    protected function _doSomething()
    {
        $foo    = $this->_foo;
        $result = null;
        // ...
        return $result;
    }
}

正如你所见，我在 _doSomething() 中使用了属性 _foo，尽管子类可以覆盖 getFoo()，返回一个未存储回 _foo 的计算值；这是一个缺陷。

我该怎么办？

1. 将 getter 标记为 final，在内部使用属性（没有额外的函数调用，强制最终开发者使用 _foo 作为属性，因为它是受保护的）
2. 在内部使用 getFoo()，将 _foo 标记为私有（额外的函数调用）

这两个选项都是防水的，但我非常担心所有额外的函数调用，所以我倾向于使用选项 1，但选项 2 会更“真正的 OOP”恕我直言。

另请阅读http://fabien.potencier.org/article/47/pragmatism-over-theory-protected-vs-private，其中还建议了选项 2：/

另一个相关问题； 如果一个属性有一个设置器，该属性应该是私有的，强制最终开发人员在子类中使用它，还是应该是程序员负责设置有效属性值的不成文规则？

Answer 1

正如您所说，第二种方法是根据 OOP 更正确的方法。您也说对了，调用方法的 CPU 周期成本比将属性作为变量访问的成本更高。但是，在大多数情况下，这将属于微优化的范畴。除非相关值被大量使用（例如在循环的最里面部分），否则它不会对性能产生明显影响。最佳实践往往倾向于正确而不是最高性能，除非性能真的因此而受到影响。

对于简单的变量，在内部使用 getter 并不是很明显，但是如果您正在处理从外部数据源（例如数据库）填充的属性，那么该技术就会发挥作用。使用 getter 可以让您以一种惰性的方式从数据库中获取数据，即按需而不是在需要之前获取数据。例如：

class Foo
{
    // All non-relevent code omitted
    protected $data = NULL;

    public class getData ()
    {
        // Initialize the data property
        $this -> data = array ();
        // Populate the data property with a DB query
        $query = $this -> db -> prepare ('SELECT * FROM footable;');
        if ($query -> execute ())
        {
            $this -> data = $query -> fetchAll ();
        }
        return ($this -> data);
    }

    public function doSomethingWithData ()
    {
        $this -> getData ()
        foreach ($this -> data as $row)
        {
            // Do processing here
        }
    }
}

现在使用这种方法，每次调用 doSomethingWithData 时，结果都会调用 getData，而 getData 又会执行数据库查询。这是浪费。现在考虑以下类似的类：

class Bar
{
    // All non-relevent code omitted
    protected $data = NULL;

    public class getData ()
    {
        // Only run the enclosed if the data property isn't initialized
        if (is_null ($this -> data))
        {
            // Initialize the data property
            $this -> data = array ();
            // Populate the data property with a DB query
            $query = $this -> db -> prepare ('SELECT * FROM footable;');
            if ($query -> execute ())
            {
                $this -> data = $query -> fetchAll ();
            }
        }
        return ($this -> data);
    }

    public function doSomethingWithData ()
    {
        foreach ($this -> getData () as $row)
        {
            // Do processing
        }
    }
}

在这个版本中，您可以随意调用 doSomethingWithData（实际上是 getData），您永远不会触发多个数据库查找。此外，如果从未调用 getData 和 doSomethingWithData，则不会进行任何数据库查找。这将带来巨大的性能提升，因为数据库查找成本很高，应尽可能避免。

如果你在一个可以更新数据库的类中工作，它确实会导致一些问题，但解决起来并不难。如果一个类对其状态进行了更新，那么您的设置器可以简单地进行编码，以便它们在成功时将其关联状态设为空。这样，下次调用 getter 时，数据将从数据库中刷新。