PHP pdo 实例作为私有静态属性

Question

我正在尝试基于 OOP 设计我的网站，但我在如何设计数据库连接方面遇到了麻烦。目前，我正在一个抽象类 Connector 中创建一个私有静态 PDO 对象。显然，任何需要与数据库交互的东西都会扩展这个类。我一直在反复讨论如何确保脚本中只有一个连接或 PDO 对象，因为有些页面需要多个扩展连接器的类。许多人似乎为此目的推荐单例模式，但我目前的做法似乎完成了同样的事情。

这是我当前的代码。

abstract class Connector
{
    private static $dbh;

    public function __construct()
    {
        try
        {
            self::$dbh = new PDO(...);
            self::$dbh->setAttribute(PDO::ATTR_ERRMODE, PDO::ERRMODE_EXCEPTION);
        }
        catch(PDOException $e)
        {
            die($e->getMessage());
        }
    }

    public function getDB()
    {
        return self::$dbh;
    }
}

那么任何子类都会像这样使用它。

class Subclass extends Connector
{
    public function interactWithDB()
    {
        $stmt = $this->getDB()->prepare(...);
        // etc...
    }
}

我认为，从理论上讲，子类的每个实例都应该始终访问同一个 PDO 实例。这段代码是否真的有意义，或者我是否以某种方式误解了静态属性？是不好的设计/实践，还是 Singleton 有更多的优势？

如果有不清楚的地方请评论，谢谢！

编辑：

连接器类的存在并不是为了保存 PDO 对象。它的析构函数关闭连接（使其为空）并包含诸如 isValueTaken 之类的函数，用于检查数据库中是否已存在值。它具有以下抽象功能

abstract function retrieveData();
abstract function setData();

例如，我有一个扩展连接器的用户类。它定义了 setData() 在数据库中注册用户。我不知道这是否会对响应产生影响。

Answer 1

如果有人正在阅读本文，请注意，如果有人使用 Phil 的上述代码 sn-p，请记住在 getDriverOptions() 中的 PDO 前面使用黑色斜线，以便引用全局命名空间。它应该看起来像这样。

public function getDriverOptions() {
      return [
          \PDO::ATTR_ERRMODE => \PDO::ERRMODE_EXCEPTION,
          \PDO::ATTR_EMULATE_PREPARES => false,
          \PDO::ATTR_DEFAULT_FETCH_MODE => \PDO::FETCH_ASSOC
      ];
  }

Answer 2

前言

单例方法通常不受欢迎。你将成为set upon by raptors。

---

您实际上要问的是如何配置连接并使其全局可用。这通常被称为全局状态。您可以使用容器类和静态方法来实现这一点。

这是一个例子

namespace Persistence\Connection\Config;

interface PDOConfig {
    public function getDSN();
    public function getUsername();
    public function getPassword();
    public function getDriverOptions();
}

class MySqlConfig implements PDOConfig {
    private $username;
    private $password;
    private $db;
    private $host = 'localhost';
    private $charset = 'utf8';

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

    // getters and setters, etc

    public function getDSN() {
        return sprintf('mysql:host=%s;dbname=%s;charset=%s',
            $this->host, $this->db, $this->charset);
    }

    public function getDriverOptions() {
        return [
            PDO::ATTR_ERRMODE => PDO::ERRMODE_EXCEPTION,
            PDO::ATTR_EMULATE_PREPARES => false,
            PDO::ATTR_DEFAULT_FETCH_MODE => PDO::FETCH_ASSOC
        ];
    }
}

namespace Persistence\Connection;

use Persistence\Connection\Config\PDOConfig;

class Registry {
    private static $connection;
    private static $config;

    public static function setConfig(PDOConfig $config) {
        self::$config = $config;
    }

    public static getConnection() {
        if (self::$connection === null) {
            if (self::$config === null) {
                throw new RuntimeException('No config set, cannot create connection');
            }
            $config = self::$config;
            self::$connection = new \PDO($config->getDSN(), $config->getUsername(),
                $config->getPassword(), $config->getDriverOptions());
        }
        return self::$connection;
    }
}

然后，在您的应用程序执行周期的某个时间点（早期）

use Persistence\Connection\Config\MySqlConfig;
use Persistence\Connection\Registry;

$config = new MySqlConfig('username', 'password', 'dbname');
Registry::setConfig($config);

然后，你可以使用

Registry::getConnection();

在代码中的任何位置检索 PDO 实例。

Answer 3

显然，任何需要与数据库交互的东西都会扩展这个类。

从 OOP 的角度来看，这确实没有意义。当某个类扩展另一个类时，这意味着“是”关系。如果你走这条路，你将很难不违反OCP，这是SOLID中的字母之一。

我一直在反复讨论如何确保脚本中只有一个连接或 PDO 对象，因为有些页面需要多个扩展连接器的类。

简单！只需创建一个实例。

许多人似乎为此目的推荐单例模式，但我目前这样做的方式似乎完成了同样的事情。

许多这样的人对 OOP 原则一无所知。使用单例只会引入“花哨”global instance / state。

这段代码是否真的有意义，或者我是否以某种方式误解了静态属性？

说实话这更像是对OOP的误解。

这是糟糕的设计/实践，还是单例有更多优势？

见上文。

---

你应该做的（在 OOP 中）是 inject the database connection 进入需要它的类。这使您的代码松散耦合，从而使您的代码具有更好的可维护性、可测试性、可调试性和灵活性。

另外，我真的不明白为什么需要为 pdo 连接创建数据库类，因为 PDO API 本身已经是 OOP。因此，除非您有真正的理由为 PDO 编写适配器（可能是这种情况，因为有一些），否则我会放弃它。

我的 €0.02

--

响应您的编辑：

连接器类的存在并不是为了保存 PDO 对象。它的析构函数关闭连接（使其为空）。

通常根本不需要关闭连接。处理请求后，连接将自动关闭（除非我们谈论的是持久连接）。

它包含诸如 isValueTaken 之类的函数，用于检查一个值是否已经在数据库中。它具有以下抽象功能

这听起来像是另一个班级的工作。

例如，我有一个扩展连接器的用户类。它定义了 setData() 在数据库中注册用户。我不知道这是否会对响应产生影响。

不，我的观点仍然成立。用户无需从数据库继承。这听起来是不是很奇怪。从数据库继承的用户（我不想见到那个人）。如果需要，您应该将数据库连接注入到用户中。