-1

我在Philip Brown的http://culttt.com/2012/10/01/roll-your-own-pdo-php-class/上找到了以下 PDO 类。

我是新来的课程。这运作良好。有没有人发现它有什么问题或可以改进？我想在大规模应用程序中使用它。

class Database
{
private $host = DB_HOST;
private $user = DB_USER;
private $pass = DB_PASS;
private $dbname = DB_NAME;

private $dbh;
private $error;

private $stmt;

public function __construct()
    {
    // Set DSN
    $dsn     = 'mysql:host=' . $this->host . ';dbname=' . $this->dbname;
    // Set options
    $options = array(
        PDO::ATTR_PERSISTENT => true,
        PDO::ATTR_ERRMODE => PDO::ERRMODE_EXCEPTION
    );
    // Create a new PDO instanace
    try
        {
        $this->dbh = new PDO($dsn, $this->user, $this->pass, $options);
        }
    // Catch any errors
    catch (PDOException $e)
        {
        $this->error = $e->getMessage();
        }
    }


public function query($query)
    {
    $this->stmt = $this->dbh->prepare($query);
    }

public function bind($param, $value, $type = null)
    {
    if (is_null($type))
        {
        switch (true)
        {
            case is_int($value):
                $type = PDO::PARAM_INT;
                break;
            case is_bool($value):
                $type = PDO::PARAM_BOOL;
                break;
            case is_null($value):
                $type = PDO::PARAM_NULL;
                break;
            default:
                $type = PDO::PARAM_STR;
        }
        }
    $this->stmt->bindValue($param, $value, $type);
    }


public function execute()
    {
    return $this->stmt->execute();
    }

public function resultset()
    {
    $this->execute();
    return $this->stmt->fetchAll(PDO::FETCH_ASSOC);
    }

public function single()
    {
    $this->execute();
    return $this->stmt->fetch(PDO::FETCH_ASSOC);
    }

public function rowCount()
    {
    return $this->stmt->rowCount();
    }

public function lastInsertId()
    {
    return $this->dbh->lastInsertId();
    }

public function beginTransaction()
    {
    return $this->dbh->beginTransaction();
    }

public function endTransaction()
    {
    return $this->dbh->commit();
    }

public function cancelTransaction()
    {
    return $this->dbh->rollBack();
    }


public function debugDumpParams()
    {
    return $this->stmt->debugDumpParams();
    }

}
4

2 回答 2

1

在我看来，您应该坚持使用本机 PDO 类。本教程中的数据库类不为您提供任何额外的功能，实际上减少了它。例如

public function single()
{
$this->execute();
return $this->stmt->fetch(PDO::FETCH_ASSOC);
}

将返回的数据类型限制为仅数组，而原始 PDO 类有大约 8 种不同的获取样式，您可以根据自己的品味和需求使用。

此外，我在以下代码中看不到任何体面的点

public function beginTransaction()
{
return $this->dbh->beginTransaction();
}

只是包装具有相同名称的完全相同的功能，同样没有额外的功能。

此外，此类限制您创建多个准备好的语句 - 使用原始 PDO 类您可以执行以下操作：

$stmt1 = $db->prepare('SELECT * FROM foo');
$stmt2 = $db->prepare('SELECT * FROM bar');
...
$stmtn = $db->prepare('SELECT * FROM foobar');

您不能使用教程中提供的类来执行此操作 - 语句对象被硬编码为单个类变量。

最后，我真的不赞成为这个类命名函数。函数名称应该像一个简短的描述，但是像“single”和“resultset”这样的名称实际上并没有提供任何类型的信息，即特定函数的作用、返回或它如何影响程序的整体状态。

总而言之，在我看来，这是一个设计相当糟糕的类，与本机 PDO 实现相比没有任何优势。

于 2013-09-08T02:55:33.847 回答
0

有什么问题

一切。即，

private $host = DB_HOST;

拥有一个用于 DB 凭证的类变量绝对没有意义。它们仅在构造函数中使用，仅一次。

catch (PDOException $e)

意义不大。无处使用捕获的错误，这破坏了异常的概念。

其余的课程只是模仿 PDO 功能。

与原始 PDO的唯一区别是它可以让您自动将 PDO NULL 绑定到 DB NULL。但是，我怀疑您是否会有这样做的想法。

因此，像其他人一样，我建议您摆脱这门课和本教程。特别是如果你是新来的课程。这样的例子会破坏你的整个体验。

于 2013-09-08T03:11:03.837 回答