我正在开发一个非常杂乱无章(没有测试，到处都是逻辑)的 PHP 应用程序。我开始看书了Modernizing Legacy Applications In PHP现在我需要替换 global 变量。

在我的例子中，我们在整个应用程序中使用全局变量大量混合了 MySQL 和 MongoDB:$database 用于 MySQL，$mongo 用于 MongoDB。我想知道注入(inject)这两个数据库的最佳方法是什么。大多数例子(在书中和其他地方)都是这样的:

class Database {
    public function __construct() {}
}

class MySQL extends Database {
    private $mysql;

    public function __construct() {
        $default_options = [
            PDO::ATTR_ERRMODE            => PDO::ERRMODE_EXCEPTION,
            PDO::ATTR_DEFAULT_FETCH_MODE => PDO::FETCH_ASSOC,
            PDO::ATTR_EMULATE_PREPARES   => false,
        ];
        // get config from file
        $this->mysql = new PDO(...);
    }

    public function get_instance() { return $this->mysql; }

}

class MongoDB extends Database {

    private $mongodb;  

    public function __construct() {
        // get config from file
        $mongodb = new MongoClient(...);
    }

    public function get_instance() { return $this->mongodb; }
}

class User {

    private $database = null;
    private $table = "users";

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

}

// In another file:
$user = new User(new \Database\MySQL);

这很酷，但我遇到的问题是，当 User 存储其属性，如 name、email、id 等在 MySQL 中，它在 Mongo 中存储类似 user_type 的东西(显然它不止于此，但我只是举一个简单的例子)，而不是依赖注入(inject)如何工作？

我是否必须将两个 Database 类传递给构造函数？

我是否有一个 set_database 函数，当我想调用电话时必须不断切换？

class User {

    private $database = null;
    private $table = "users";

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

    public function get_user() {
        $first_set = $this->database->run('select * from users',[]);
        $this->database->set_database(new \Database\MongoDB);
        $second_set = $this->database->run('users',[]);
    }
}

我正在尝试了解它是如何工作的，如果其他开发人员添加了必须在 User 类中调用的数据库(例如 Redis)，我会继续工作。

最佳答案

_{Warning: the following post contains use of Singleton antipattern as a temporary tool in refactoring of large scale clusterfuck}

我认为您可能错过了重构的一个重要部分(或者至少，关注了错误的部分)。在此过程中，您应该以特定的最终结果为目标。您希望看到的特定架构结构(并且您的所有同事都同意)。现在你似乎只是漫无目的地试图“让它少一些狗屎”。

一开始...

您眼前的里程碑似乎是“摆脱全局变量”。所以，专注于此。尝试同时切换到完全依赖注入(inject)的结构会使事情变得困惑并增加失败的机会。

相反，您应该首先切换到注册表(您可以使用单例作为临时步骤，但以后很难清除)。

所以代码看起来像:

class Foobar {
    function something() {
        global $database; 
        $database->query('SELECT ... blah');
    }
}

将重写为:

class Foobar {
    function something() {
        $database = Registry::getDatabase(); 
        $database->query('SELECT ... blah');
    }
}

这将解决一个特定问题:摆脱全局变量。

第二步

完成后，您可以开始切换到依赖注入(inject)。但不是通过添加类型提示。这将需要具有较大“冲击面”(容易破坏某些东西)的大量重写。

相反，您开始引入 DI 容器，但仅限于以前使用单例或注册表的部分。您创建一个单例，它包含 DI 容器的实例(您基本上将其用作服务定位器 反模式)，并且代码被重写为:

class Foobar {
    function something() {
        $database = Locator::get('database'); 
        $database->query('SELECT ... blah');
    }
}

这就是过程代码(也包括仅静态类)可以在这个方向上重构的方式。

第三步

接下来您需要开始做 OOP 的事情。您需要将那些(以前是全局的)变量移动到与类实例共享:

class Foobar {
    private $database;

    public function __construct() {
        $this->database = Locator::get('database'); 
    }

    public function something() {
        $this->database->query('SELECT ... blah');
    }
}

第四步

只有现在您才能真正转向在构造函数中使用实际依赖项。

class Foobar {
    private $database;

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

    public function something() {
        $this->database->query('SELECT ... blah');
    }
}

您还需要替换所有情况，其中您有:$thing = new Foobar(); 为
$thing = Locator::get('foobar');，然后对使用了 Foobar 的类重复步骤 3 , 直到你到达 Bootstrap 。

循环结束

完成迁移过程后，您将停止使用引导文件中的 Locator 单例包装器，并切换到使用实际的 DI 容器实例。

您可能还注意到，每个步骤都可以执行，同时新功能也被添加到代码库中，因为在任何时候它都不需要在不相关的位置接触大量代码。每个类的迁移可以单独完成，而不会破坏整个应用程序的功能。

但是

(这是一个很大的但是)。

从您展示的这段代码来看，您会遇到另一个需要解决的问题。很有可能，您的 User 类实际上正在实现 active record反模式。因此，当您摆脱全局变量时，下一个可能的步骤是将持久性逻辑与业务逻辑分离并切换到 data mapper。图案。为了进一步阅读，我将只向我的三个旧帖子发送垃圾邮件:1 , 2和 3 .

关于在同一个类中使用多种类型的数据库时的 PHP 依赖注入(inject)，我们在Stack Overflow上找到一个类似的问题： https://stackoverflow.com/questions/50363657/