0

我知道使用 orm 会更好，我计划在未来使用它。但是现在，我正在使用这样的结构：

具有标题和日期的北极类类数据库操作的类 DataArticle 所以我没有在我的文章类中执行我的数据库操作，而是在一个单独的数据类中。

现在，在我的所有 Data.. 类中，我使用代码执行如下数据库操作：

public function getArticle($id){        
        $query = "SELECT title,date from articles where id = ?";          
        if ($stmt = $this->database->getConnection()->prepare($query)) {            
            $stmt->bind_param('i',$id);
            $stmt->execute();
            $stmt->bind_result($title,$date);   
            $stmt->store_result();
            $stmt->fetch();         
            if(($stmt->num_rows) == 1){
                $article = new Article();
                $article->title = $title;
                $article->date = $date;
                $stmt->close();             
                return $article;
            }else{
                $stmt->close();
                return null;
            }           
        }else{          
            throw new Exception($this->database->getConnection()->error);
        } 
    }

但是以这种方式工作意味着在我的数据类中的每个函数中，我都会连接、执行语句并抛出错误。这是很多可以使用包装器集中的重复代码。

现在我正在遵循有关创建数据库包装器/处理程序以执行所有数据库内容的建议（在函数中抛出异常或如何进行下降错误处理），因此它们都集中在一个类中，从而更易于维护。

所以我创建了这个类来开始使用 PDO：

<?php
class DatabasePDO
{
    private $connection;

    private $host = "";
    private $username = "";
    private $password = "";
    private $dbname = "";

    public function openConnection(){
        $this->connection = new PDO("mysql:host=$this->host;dbname=$this->dbname",$this->username,$this->password);
        $this->connection->setAttribute(PDO::ATTR_ERRMODE, PDO::ERRMODE_EXCEPTION);     
    }

    public function getConnection(){
        return $this->connection;
    }

    public function closeConnection(){
        $this->connection = null;
    }

    public function insert($query, array $data){        
        $this->connection->prepare($query)->execute($data);     
        return $this->connection->lastInsertId();
    }

    public function update($query, array $data) {
        $stmt = $this->connection->prepare($query);
        $stmt->execute($data);
        return $stmt->rowCount();       
    }

    public function delete($query, array $data) {
        $stmt = $this->connection->prepare($query);
        $stmt->execute($data);
        return $stmt->rowCount();       
    }
    public function findOne($query, array $data = null){        
        $sth = $this->connection->prepare($query);
        if($data != null){
            $sth->execute($data);
        }else{
            $sth->execute();
        }       
        if($sth->rowCount() == 1){              
            return $sth->fetchObject();
        }else{
            return null;
        }
    }
    public function find($query, array $data = null){       
        $sth = $this->connection->prepare($query);
        if($data != null){
            $sth->execute($data);
        }else{
            $sth->execute();
        }
        if($sth->rowCount() > 0){
            while($res = $sth->fetchObject()){              
                $results[] = $res;
            }
            return $results;            
        }else{
            return null;
        }
    }
}
?>

但是当阅读一些文章时，我发现这不是一个好的做法，因为 PDO 已经是一个数据库包装器。

但是，通过代码比以前更具可读性。现在只是

public function getArticle($id){       
        $article = $this->database->find("select name, date from articles ?",array($id));       

        $article = new article($article->name, $article->date);
        return $article;
    }

此代码要短得多，并且所有数据库逻辑都在 PDO 包装器类中处理，否则我将不得不在每个函数中重复包装器的代码，并且我的代码将在很多地方而不是一个包装器中。

那么有没有更好的方法来使用我的代码，或者它是我使用它的好方法。

4

2 回答 2

1

不知道你为什么问或你的疑问来自哪里，但是创建一个数据库包装类是处理 SQL 查询的唯一正确方法。

尽管实现需要一些改进，但总体思路和用法几乎都非常出色。是的，将整个屏幕的代码量减少到一行是此类的主要好处之一。

说到改进——PDO 有一些很好的技巧来缩短你的代码：

public function find($query, array $data = null){       
    $sth = $this->connection->prepare($query);
    $sth->execute($data);
    return $sth->fetchAll(); 
}

将与您的版本完全相同，但需要的代码少三倍

从关于创建这样一个基于 PDO 的类的敏感性的评论中回答您的问题：
看，PDO 已经是一个半 DAL。可以看到，相比mysqli已经是很大的进步了。虽然 mysqli 显然并且无疑需要在其上创建这样一个类，但 PDO 具有许多开箱即用的必需功能。因此，确实可以使用原始 PDO。许多在 PDO 上创建的类，确实没用。但是，PDO 也可以从一些改进中受益。

看，虽然我确实改进了你的 PDO 代码，将行数减少到只有三行，但基于 mysqli 的版本无法改进，需要全部几十行代码。所以，3 行胜于 12 行，但仍然是 1行的find()功能优于 3 行，不是吗？然而，总有一些更大的改进空间。您可以从我的标签 wiki 中的PDO 失败案例列表中获得一些想法。

于 2013-08-24T18:36:13.163 回答
-1

存储数据库连接的更好方法是使用单例类。

http://php.net/manual/de/language.oop5.patterns.php

<?php
class DatabasePDO
{
    private static $instance;

    private $connection;

    private $host = "";
    private $username = "";
    private $password = "";
    private $dbname = "";

    public static function getInstance()
    {
        if (!isset(self::$instance)) {
            self::$instance = new DatabasePDO();
        }

        return self::$instance;
    }

    public function openConnection(){
        $this->connection = new PDO("mysql:host=$this->host;dbname=$this->dbname",$this->username,$this->password);
        $this->connection->setAttribute(PDO::ATTR_ERRMODE, PDO::ERRMODE_EXCEPTION);     
    }

    public function getConnection(){
        return $this->connection;
    }

    public function closeConnection(){
        $this->connection = null;
    }

    public function insert($query, array $data){        
        $this->connection->prepare($query)->execute($data);     
        return $this->connection->lastInsertId();
    }

    public function update($query, array $data) {
        $stmt = $this->connection->prepare($query);
        $stmt->execute($data);
        return $stmt->rowCount();       
    }

    public function delete($query, array $data) {
        $stmt = $this->connection->prepare($query);
        $stmt->execute($data);
        return $stmt->rowCount();       
    }
    public function findOne($query, array $data = null){        
        $sth = $this->connection->prepare($query);
        if($data != null){
            $sth->execute($data);
        }else{
            $sth->execute();
        }       
        if($sth->rowCount() == 1){              
            return $sth->fetchObject();
        }else{
            return null;
        }
    }
    public function find($query, array $data = null){       
        $sth = $this->connection->prepare($query);
        if($data != null){
            $sth->execute($data);
        }else{
            $sth->execute();
        }
        if($sth->rowCount() > 0){
            while($res = $sth->fetchObject()){              
                $results[] = $res;
            }
            return $results;            
        }else{
            return null;
        }
    }
}
?>

使用此代码，您可以通过调用 DatabasePDO::getInstance()->getConnection() 从任何地方获取数据库连接；

public function getArticle($id){        
    $query = "SELECT title,date from articles where id = ?"; 

    $database = DatabasePDO::getInstance();

    if ($stmt = $database->getConnection()->prepare($query)) {            
        $stmt->bind_param('i',$id);
        $stmt->execute();
        $stmt->bind_result($title,$date);   
        $stmt->store_result();
        $stmt->fetch();         
        if(($stmt->num_rows) == 1){
            $article = new Article();
            $article->title = $title;
            $article->date = $date;
            $stmt->close();             
            return $article;
        }else{
            $stmt->close();
            return null;
        }           
    }else{          
        throw new Exception($database->getConnection()->error);
    } 
}
于 2013-08-24T18:34:37.613 回答