0

我已经开始学习如何使用 OOP 并创建了一个用户授权类来检查用户是否存在等。目前我使用$dbh作为 PDO 连接的全局变量连接到数据库。我听说以这种方式使用全局变量不是一个好习惯，但我不确定如何改进它，我是否只是将$dbh变量传递给在连接到数据库时需要它的方法，以及为什么不考虑这一点好的做法？

这是我正在使用的一些代码：

调用程序中包含的数据库 PDO 连接：

//test the connection
    try{
        //connect to the database
        $dbh = new PDO("mysql:host=localhost;dbname=oopforum","root", "usbw");

    //if there is an error catch it here
    } catch( PDOException $e ) {
        //display the error
        echo $e->getMessage();
    }

需要数据库连接的类：

class Auth{

        private $dbh;

        function __construct(){

            global $dbh;
            $this->dbh = $dbh;
        }

        function validateLogin($username, $password){

            // create query (placing inside if statement for error handling)
            if($stmt = $this->dbh->prepare("SELECT * FROM oopforumusers WHERE username = ? AND password = ?")){

                $stmt->bind_param(1, $username);
                $stmt->bind_param(2, $password);
                $stmt->execute();


                // Check rows returned
                $numrows = $stmt->rowCount();

                //if there is a match continue
                if( $numrows > 0 ){
                    $stmt->close();
                    return TRUE;
                }else{
                    $stmt->close();
                    return FALSE;
                }

            }else{
                die('ERROR: Could not prepare statement');
            }

        }


        function checkLoginStatus(){

            if(isset($_SESSION['loggedin'])){

                return TRUE;

            }else{

                return FALSE;

            }

        }

        function logout(){

            session_destroy();
            session_start();

        }
    }
4

3 回答 3

6

您应该将 PDO 连接传递给构造函数：

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

该连接称为类的依赖项，因为显然您的类需要它来执行其功能。良好的实践要求你的类应该明确表示存在这种依赖关系；这是通过使其成为强制构造函数参数来实现的。

如果您改为从全局变量中提取依赖项，则会产生几个问题：

  • 您的班级的用户根本不清楚首先存在依赖关系
  • 您的类现在与全局变量耦合，这意味着在不破坏程序的情况下无法删除或重命名该变量
  • 您已经为“远距离操作”创建了先决条件：修改全局变量的值会导致应用程序的另一个（看似无关的）部分改变行为
于 2012-06-30T17:02:14.697 回答
2

您可以简单地将其传递给构造函数。

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

当您创建对象时，您会：

$obj = new Class($dbh);
于 2012-06-30T17:02:12.200 回答
2

通过构造函数传入数据库对象。PHP 的对象模型意味着 = 创建对同一对象实例的新引用。 

class ThingThatDependsOnDatabase
{
    private $db = NULL;

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

    public function selectSomething ($id)
    {
        $sql = 'SELECT * FROM table WHERE id = ?;'
        $this -> db -> prepare ($sql);
        // And so on
    }
}

这是一个名为Dependency Injection的模式示例，因为您的类所依赖的东西是通过方法（setter、构造函数等）注入的。

于 2012-06-30T17:02:34.667 回答