组织 PHP OOP 类以使用 DB

Question

今天我有一个类结构非常奇怪的 PHP 项目。像这样：

db_base
  `-- db_base_ext extends db_base
        +-- module_1 extends db_base_ext
        +-- module_2 extends db_base_ext
        .   ...
        +-- module_N extends db_base_ext
        `-- db_user extends db_base_ext

class_1
  `-- submodule_1_1 extends class_1

db_base 连接到__construct() 中的db，并有一些辅助方法。

执行示例：

$db = new db_user();
$user = new user($db);
unset($db);

$db = new module_2();

我不喜欢在这段代码中我们两次连接到 DB 并且整个类结构不是很好。

我怎样才能让它变得更好？我的意思是只创建一个数据库实例，然后使用任何类（对应于数据库）？

单例模式对此有好处吗？

Answer 1

你的问题是，你的结构，

module_2 extends db_base_ext
db_user extends db_base_ext
submodule_1_1 extends class_1

它们都破坏了Single Responsibility Principle 和Liskov Substitution Principle。

我怎样才能让它变得更好。我的意思是只创建一个数据库实例，然后 可以与任何类（对应于 DB）一起工作吗？

Dependency injection 是要走的路。您将实例化一个数据库实例，您的所有类将共享相同 $db 实例。

final class MySQLPDO extends PDO
{
    public function __construct(array $params)
    {
      parent::__construct(sprintf('mysql: host=%s; dbname=%s', $params['host'], $params['database']), $params['username'], $params['password']);

      $this->setAttribute(parent::MYSQL_ATTR_INIT_COMMAND, 'SET NAMES UTF8');
      $this->setAttribute(parent::ATTR_ERRMODE, parent::ERRMODE_EXCEPTION);
      $this->setAttribute(parent::ATTR_EMULATE_PREPARES, false);
      $this->setAttribute(parent::ATTR_DEFAULT_FETCH_MODE, parent::FETCH_ASSOC);

    }
}

$db = new MySQLPDO(array(
     'host'     => 'localhost',
     'database' => 'foo',
     'username' => 'root',
     'password' => '',
));
$user = new User($db);

$module = new Module1($user);
$foo = new Foo($db);

那么，你会在这里获得什么？重用能力和测试能力。

---

注意，您应该避免使用 Singleton，因为它们会引入另一种形式的全局状态，这不利于单元测试。

Answer 2

使用正确的类名

首先我注意到项目中的类被命名为函数。如果项目尝试使用 PEAR 命名约定，它应该正确。

• Naming Conventions (Pear Manual)

使模块独立于数据库类

您应该断开模块与db_base_ext 类的连接。如果模块需要访问数据库实例，可以使用构造函数参数或设置器来提供。它不应该直接扩展类。

$module = new Module($database);

Answer 3

您应该考虑为您的 DB 类使用单例模式。

Take a look here for a tutorial。 它使用 PDO 进行通信，但对 mysqli 或其他类型也应有效。