Question

我正在为我的项目编写安装脚本。基本上我想做的是检查用户是否输入了正确的数据库信息，如果没有，则返回错误。这就是我试图实现的目标：

<?php
error_reporting(E_ALL);
ini_set('display_errors',1);

require "classes/pdo.class.php";

// Define configuration
define("DB_HOST", $_POST['db_host']);
define("DB_USER", $_POST['db_name']);
define("DB_PASS", $_POST['db_user']);
define("DB_NAME", $_POST['db_pass']);

try {
    $db = new Database();
    echo "Succes!";
} catch(PDOException $ex){
    echo "Failed!";
}

?>

不幸的是，这总是会回归成功！即使用户输入的数据不正确。有什么我想念的吗？我必须使用不同的方法吗？

这是我用于PDO课程的代码：

<?php

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 column(){
        $this->execute();
        return $this->stmt->fetchAll(PDO::FETCH_COLUMN);
    }

    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();
    }
}

?>

Answer 1

每次获得成功的原因是因为无论连接是否成功，都会创建DB对象。

你应该尝试连接，如果失败，那么你就知道信用是错误的。您可以在代码中运行它，或者在DB类中创建一个方法来检查它并返回true或false。（或者我猜你可以修改构造函数以抛出异常，但我认为这不是正确的方法）

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

Answer 2

看起来如果抛出异常，它就会被构造函数捕获。看起来构造函数不会抛出异常。

调用者不会“看到”异常，因为它在构造函数中被“吞噬”。

Answer 3

在回答你的问题之前，我想我应该指出这个糟糕的设计是在构造函数内部做的。

我会像这样：

function createDbh() 
{    
    $dsn = 'mysql:host=' . DB_HOST . ';dbname=' . DB_NAME;
    // Set options
    $options = array(
        PDO::ATTR_PERSISTENT    => true,
        PDO::ATTR_ERRMODE       => PDO::ERRMODE_EXCEPTION
    );
    // Create a new PDO instanace
    return new PDO($dsn, DB_USER, DB_PASS, $options);
}

function createDatabase()
{
    return new Database(createDbh());
}

在你的引导程序中：

try {
    $db = createDatabase();
} catch(Exception $ex){
    echo "Failed!";
}

这样，当你为Database类编写单元测试时，注入模拟对象会很容易

PDO：检查用户是否输入了正确的数据库信息（安装脚本）

3 个答案: