使用扩展类时的最佳技巧（PHP）

Question

关于我在这里的帖子：扩展 PHP 类时的问题

最好的解决方案是什么？为什么？

$query = $this->Execute("SELECT * FROM $table");
$total = $this->NumRows($query);

或

$query = DBManager::Execute("SELECT * FROM $table");
$total = DBManager::NumRows($query);

或

$query = parent::Execute("SELECT * FROM $table");
$total = parent::NumRows($query);

Answer 1

当您扩展一个类时，除了子类定义的任何方法和属性之外，子类还可以调用父类的所有公共和受保护的方法和属性。如果您不需要重写父类的方法，而只是想在子类中的某个地方使用它们，请使用

public function somethingUsingExecute()
{    
    $query = $this->Execute("SELECT * FROM $table");
    …
}

---

如果您需要在子类中重写execute，但又想使用原始方法调用该方法父级的功能， use

public function execute($table)
{
    // can do something not in parent before parent call
    $query = parent::Execute("SELECT * FROM $table");
    // can do something not in parent after parent call
}

---

静态调用

$query = DBManager::Execute("SELECT * FROM $table");

是您应该忘记它存在的东西，因为您将类名硬编码到 using 类中，并将方法调用耦合到全局范围。两者都很难测试和维护。我不会详细介绍，因为之前已经讨论过这个问题。只需搜索一下即可。

---

关于该 User/DBManager 示例的“最佳技巧”实际上是根本不使用继承。继承创建了一种is-a关系。孩子是父母的特例。但用户不是 DBManager（顺便说一句，我将其称为 DbAdapter）。

DBManager 类的职责是允许访问数据库。您的用户没有该责任。它使用 DBManager 来访问数据库，但它确实需要知道如何详细执行该操作。 DBManager 已经这样做了。

任何使用另一个类的类都应该被注入/聚合。因此，您应该做的是：

class User …

    protected $db;
    protected $isAuthenticated = FALSE;

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

    public function authenticate($username, $password)
    {
        if($this->db->query('SELECT … ')) {
            $this->isAuthenticated = TRUE;
        }
        return $this->isAuthenticated;
    }

    …
}

然后您可以执行

$db = new DBManager;
$john = new User($db);
$john->authenticate('JohnDoe', 'jd/12345');
$jane = new User($db);
$jane->authenticate('JaneDoe', 'jd/67890');
$logger = new DBLogger($db);
$logger->log('something to db');

注入 DB 类的额外好处，即您可以在其他使用类中重用现有的 DBManager 实例。如果扩展 DBManager 类，每个新实例都会创建一个到数据库的新连接，出于性能原因，这是不希望的。

Answer 2

静态调用函数（选项 2 和 3）和作为非静态函数调用它们之间存在差异。这取决于您的整个设置

Answer 3

• $this 指向该类的当前实例。
• DBManager指向父类，但是当然，如​​果你改变它的名字，你就必须改变你的代码。
• parent 指向该类继承的类，无论其名称如何。

因此，在这个示例中，如果没有有关如何使用类和实例的更多详细信息，我认为这并不重要。但是，如果您计划拥有 DB 类的多个实例，您可能应该使用 $this，至少如果您想存储查询或结果而不用下一个查询覆盖它们（如劳特姆建议）。

Answer 4

如果我们仅限于您的选项（否则请阅读下面的第 3 项），我会选择第二个选项，因为：

1）当您编写

$query = $this->Execute("SELECT * FROM $table");
$total = $this->NumRows($query);

它时，就像您有一个类，该类执行查询（所有类型）并处理结果一些特定的查询（在您的情况下是行数。假设您有两个实体，它们都存储在表中（不同），您将有两个类，并且它们都将重复 Execute 方法逻辑，或者一个实体 DAO 将使用另一个实体来执行其 Execute 方法（但这些实体可以并且大多数情况下是不相关的，因此您将损害 OOP 原则）

2）现在您的最后一种情况

$query = parent::Execute("SELECT * FROM $table");
$total = parent::NumRows($query);

比前一种情况更好，但在这种情况下您 DAO继承自一个执行查询执行的类...我不确定这是正确的方法，假设您有两个不同的存储（例如数据库和 xml），用于从您将使用 SQL 的第一个存储中检索信息，例如最后一个可能是 XPath；要使用存储，您将使用相同的接口和两个实现，而不是问题 - 为您的 User 类继承哪一个？没有人，他们没有血缘关系；用户实体不应该知道它存储在哪里。

3）最后一种情况

$query = DBManager::Execute("SELECT * FROM $table");
$total = DBManager::NumRows($query);

听起来像是委托（抱歉，我对PHP不太熟悉），在这里你指定使用哪个类来处理查询，对我来说，当我读到这段代码时，听起来 Execute/NumRows 是静态方法DBManager 类（再次抱歉，我只是使用我的体系结构知识来帮助您和 Java 等基本约定来阅读您的代码）。在这种情况下，我建议您创建 DBManager 实例，然后使用它：

class UserDAO {  
   private $storageManager = new DBManager();

   ...  
   $query = $storageManager::Execute("SELECT * FROM $table");  
   $total = $storageManager::NumRows($query);  
   ....  
}

类似这样的内容（如果我在 PHP 代码中犯了错误，请纠正我；现在只需将其视为“要走的路”，抽象算法描述； ）。在这种情况下，如果必须切换到 DBManager 的另一个实现，您只需更改行 private $storageManager = new DBManager(); 即可。