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

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

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

?>

---

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

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

---

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

---

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

在回答您的问题之前，我想我应该指出，这种糟糕的设计是在构造函数中进行的

我会这样说：

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!";
}

---

这样，当为

数据库

类编写单元测试时，将很容易注入模拟对象

，而这正是该类所做的。看一下

\u结构
：）伙计，放松点。我感谢你的帮助。我只是指出，如果我没有弄错的话，构造函数已经尝试了连接。不过我可能错了。所以我想弄明白。对不起，我的错。无论发生什么，对象都是构造的。如果连接失败，这无关紧要。这就是为什么您需要在代码中尝试连接，或者您可以向类添加一个方法来检查连接并返回true或false。实例化对象不会提供所需的信息。谢谢。让它工作：）将标记你的回答尽快我可以：）个人来说，我会让构造函数抛出一个异常。如果您试图实例化一个数据库对象，但无法连接，那么这是一个非常特殊的情况，更不用说为无效连接返回有效对象的想法听起来很糟糕。但也许那只是我。你在构造中设置
$this->error
。你为什么不检查这个？是的，我知道了。我犯了愚蠢的错误！谢谢是否将标记为answeredWait，以便将PDO对象的创建抽象到另一个函数，然后使用具体的PDOException类型提示进行异常处理？我真的不认为这是更好的设计tbh。我觉得很勉强。还有，没有回答这个问题。我已经改变了捕获部分，虽然在我看来这并不重要。是的，我回答了这个问题。。。有了这段代码，当连接失败时，你就会陷入困境。我已经检查了你的代码，它当然也工作得很好。向上投票是因为我认为向下投票对于一个有效的解决方案是不公平的。
try {
    $db = createDatabase();
} catch(Exception $ex){
    echo "Failed!";
}