在另一个类中使用一个类时的 OOP 效率

Question

我有一个名为 DB (class.pdo.php) 的类，它使用 PDO 处理 mysql 查询的所有处理，还有一个名为 user 的类，我用来管理登录系统。

我的问题涉及总是必须在用户的每个公共函数中实例化 $db，以便我可以使用 DB。这有效率吗？我不应该在用户的 __construct() 中实例化数据库吗？

这是我的代码

 require_once("../../class.pdo.php");

class user {

private $db = null;

public function __construct(){
    /* Empty? */
}

public function find_by_email($email){
    $db = new db();
    $db->query('SELECT * FROM users WHERE email = :email LIMIT 1');
    $db->bind(':email',$email);
    $result = $db->single();
    return $result;
}

public function create($email,$password,$first_name,$last_name){ 
    $db = new db();
    $db->query("INSERT INTO users(email,password,first_name,last_name,created_at) VALUES (:email,:password,:first_name,:last_name,NOW())");
    $db->bind(':email',$email);
    $db->bind(':password',$password);
    $db->bind(':first_name',$first_name);
    $db->bind(':last_name',$last_name);
    $result = $db->execute();
    return $db->lastInsertId();
}

 [more similar functions ommited]

Answer 1

好吧，尽管一些 cmets 建议使用单例模式，但我完全不同意将其用于此目的。

您的应用程序不会总是使用与一个数据库的单一连接。

让我告诉你我会怎么做：

class DbConnector {
    private $dbh;
    private $dsn;

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

    private function connect() {
        if($this->dbh === null) {
            $this->dbh = new PDO($this->dsn);
        }
    }

    public function disconnect {
        if($this->dbh !== null) {
            $this->dbh = null;
        }
    }

    public function query($sql) {
        $this->connect();
        //... do the rest
    }

    public function fetchAll($sql) {
        $this->connect();
        //... do the rest
    }

    public function insert($table, $values) {
        $this->connect();
        //... do the rest
    }

    public function update($table, $values, $cond) {
        $this->connect();
        //... do the rest
    }

    public function delete($table, $cond) {
        $this->connect();
        //... do the rest
    }
}

class User {
    private $dbConn;
    public function __construct(DbConnector $dbConn) {
        $this->dbConn = $dbConn;
    }

    public function create($email,$password,$first_name,$last_name){ 
        $this->dbConn->query("INSERT INTO users(email,password,first_name,last_name,created_at VALUES (:email,:password,:first_name,:last_name,NOW())");
        $this->dbConn->bind(':email',$email);
        $this->dbConn->bind(':password',$email);
        $this->dbConn->bind(':first_name',$email);
        $this->dbConn->bind(':last_name',$email);
        $this->dbConn->execute();
        return $this->dbConn->lastInsertId();
    }

    // ...
}

结果：

• 未使用单例 = 可测试。
• 需要时才打开与数据库的连接
• 您的连接是持久的。如果您在每种方法中打开和关闭连接，您将失去创建事务的能力。

Answer 2

如何使用单例模式为连接创建一个对象并在每次需要时使用它，而不是一直创建新对象？

Answer 3

我会用延迟加载做类似的事情：不要在构造函数中启动，除非你确定每次创建对象时确实需要连接，但绝对不要在每次方法调用时创建新对象。相反，将生成的对象保存到一个对象 var 中，在每次方法调用时都会检查该对象 var，如果丢失则启动连接。

class user {
  protected $_db = null;
  private function _init_db() { $this->_db = new XXX; }
  public function create( $x, $y, $z ) {
    if ( ! $this->_db ) $this->_init_db();
    # use $this->_db ..
  }
  public function find_by_email( $x, $y, $z ) {
    if ( ! $this->_db ) $this->_init_db();
    # etc
  }
}

这具有避免全局静态状态（单例..）的优点，并且仅在最后一刻创建连接/对象，因此您确定确实需要它，而不仅仅是无用的连接。

Answer 4

说到效率，您的代码的主要问题是它为每个调用的方法建立新连接。这确实效率低到会杀死您的数据库服务器。这与您遇到的其他问题无法比拟。

因此，一般而言，您可以采用任何您想要的方式 - 在每个函数中以某种方式获取 db 类的实例或使用类变量 - 但无论哪种方式都必须在整个应用程序中使用 单个 PDO 实例。

另外，从代码量的角度来看，我发现你的函数效率很低，并且会以这种方式优化它们

public function create($email,$password,$first_name,$last_name){ 
    $sql = "INSERT INTO users(email,password,first_name,last_name,created_at) VALUES (?,?,?,?,NOW())";
    $this->db->query($sql);
    $result = $db->execute(func_get_args());
    return $db->lastInsertId();
}

Answer 5

从对象的角度来看，我会让数据库在方法中进行实例化，而不是在整个类中进行。

每个方法应该只看到它需要的变量和数据，以便执行它的功能。 例如，createUser() 方法需要查看变量或属性，例如 $username、$usergroupId 以及 $database 等。

但是，您可能有一个名为randomPassword() 的函数，它会根据数字和字母生成随机密码。

这个 randomPassword() 函数不需要数据库对象，因此，在对象范围内已经初始化的数据库连接将是浪费的。

最好只在需要它的方法中创建新的数据库对象。

另外，在我的应用程序中，我不会在每次使用new database 时创建新的数据库连接。相反，我选择了保持连接处于活动状态的单例 PDO 数据库对象。

然后我可以静态调用数据库对象来检索现有连接。因此，如果在运行我的应用程序的过程中我需要有 20 个数据库对象，那么我的应用程序只返回相同的对象和相同的连接。