php - 将我的 PDO 连接保存为全局变量

Question

<分区>

Answer 1

Is there any fundamental flaws in using this approach?

您首先要了解的是，$pdo 是存储 逻辑的一部分。这意味着，它应该只在执行抽象数据访问的类中使用，无论是 SQL 表还是集合。

让我们看看你的代码，

function somefunction(){
    global $pdo;

    $statement = $pdo->prepare("some query");
    $statement->execute();
}

如果您将来想从 MySQL 切换到 Mongo/MSSQL/PgSQL 怎么办？ 然后你将不得不重写很多代码。

对于每个数据库供应商，您都必须创建一个包含不同变量的单独文件。就这样

function somefunction(){
    global $mongo;
    return $mongo->fetch(...);
}

_{By using a global state, you end up with mass code duplication, because you cannot pass parameters and thus cannot change function's behavior at runtime.}

现在让我们看看这个，

function somefunction($pdo){
    $statement = $pdo->prepare("some query");
    $statement->execute();
}

此处，$pdo 作为参数传递，因此没有全局状态。但是问题依然存在，你最终违反了单一职责原则

如果您真的想要可维护、干净且可读性强的东西，您最好坚持使用 DataMappers .这是一个例子，

$pdo = new PDO(...);

$mapper = new MySQL_DataMapper($pdo);
$stuff = $mapper->fetchUserById($_SESSION['id'])    

var_dump($stuff); // Array(...)

// The class itself, it should look like this
class MySQL_DataMapper
{
    private $table = 'some_table';

    private $pdo;

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

    public function fetchUserById($id)
    {
        $query = "SELECT * FROM `{$this->table}` WHERE `id` =:id";
        $stmt = $this->pdo->prepare($query);

        $stmt->execute(array(
           ':id' => $id
        ));

        return $stmt->fetch();
    }
}

结论

• 不管你的项目是大是小，你应该始终避免所有形式的全局状态(全局变量、静态类、单例)-为了代码的可维护性

• 您必须记住，$pdo 不是您业务逻辑的一部分。它是存储逻辑的一部分。这意味着，在您开始使用业务逻辑(例如繁重的计算)之前，您应该真正抽象表访问(包括 CRUD 操作)

• 将您的数据访问抽象和计算逻辑结合在一起的桥梁通常称为服务

• 您应该始终将事物函数的需要作为参数传递

• 您最好停止担心您的代码并开始考虑抽象层。

• 最后，在开始做任何事情之前，您首先要在 bootstrap.php 中初始化所有服务，然后根据用户的输入开始查询存储 ($ _POST 或 $_GET)。

就像，

public function indexAction()
{
    $id = $_POST['id']; // That could be $this->request->getPost('id')
    $result = $this->dataMapper->fetchById($id);

    return print_r($result, true);
}