php - 使用类 getter 或类属性？ (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，尽 pipe 类可以重写 getFoo() 返回一个未存储回 _foo 的计算值；这是一个缺陷。

我该怎么办？

1. 将 getter 标记为最终的，在内部使用属性(没有额外的函数调用，强制最终开发人员使用 _foo 作为属性，因为它是 protected )
2. 在内部使用 getFoo()，将 _foo 标记为私有(private)(额外函数调用)

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

另请阅读 http://fabien.potencier.org/article/47/pragmatism-over-theory-protected-vs-private这也建议选项 2 :/

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

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，后者又执行数据库查询。这是浪费。现在考虑以下类似的类:

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，则不会进行任何数据库查找。这将带来巨大的性能提升，因为数据库查找非常昂贵，应尽可能避免。

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