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


Saving my PDO connection as a global variable

在询问有关PDO查询的另一个问题时,我被告知将PDO连接对象保存为全局对象以在调用数据库查询的各种函数中使用它通常是不好的做法。

以下是我通常如何使用我的PDO对象:

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

我读过的参数更多是关于代码维护和调试的,很难追踪谁修改了PDO对象以及它在代码中的位置。其他人只是简单地拒绝使用全局变量来存储PDO对象,但无法真正解释为什么全局变量是一种糟糕的方法。

但是,对于只有一个数据库的中小型项目,使用全局变量真的有缺点吗?我通常分别拥有我的连接脚本和函数脚本,其中函数脚本将require_once()连接脚本,其中创建了我的PDO对象。这样,我的连接始终建立,对 PDO 对象的所有修改都在我的连接脚本中完成。

使用这种方法是否存在任何根本缺陷?

使用这种方法是否存在任何根本缺陷?

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

让我们看看你的代码,

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

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

对于每个数据库供应商,您必须创建一个具有不同变量的单独文件。就像这样

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

通过使用全局状态,最终会导致大量代码重复,因为您无法传递参数,因此无法在运行时更改函数的行为。

现在让我们看看这个,

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 操作)。

  • data access abstractioncomputation logic结合在一起的桥梁通常称为服务

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

  • 你最好不要再担心你的代码,开始考虑抽象层。

  • 最后,在
  • 你开始做任何事情之前,你首先在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);
}

对于一个很小的项目,使用global不会有太大的伤害。 当项目规模开始增长时,事情就会开始变得糟糕。

如果你的somefunction()中有 300 行使用 5 个全局变量,那么查看你的代码的人不会知道该函数使用外部变量,除非他们通过它寻找全局变量。

另外,不使用全局变量非常容易...那么为什么要这样做

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

编辑:

show_profile.php是您的控制器。 控制器收集视图的所有数据并加载视图。这是一个非常简化的版本。

require( 'functions.php' );
$db = new PDO();
$user = get_user( $db, $user_id );
require( 'view.php' );