PDO:检查用户是否输入了正确的数据库信息(安装脚本)
PDO: Check if user entered correct database info (install script)
我正在为我的项目编写安装脚本。基本上我想做的是检查用户是否输入了正确的数据库信息,如果不正确,return 一个错误。这就是我试图完成的:
<?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!";
}
?>
不幸的是,这总是 return 成功!即使用户输入的数据不正确。有什么我想念的吗?我必须使用不同的方法吗?
这是我用于 PDO 的代码 class:
<?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对象
您应该尝试连接,如果连接失败,那么您就知道信用有误。您可以在您的代码中 运行 这个,或者在您的数据库 class 中创建一个方法来检查这个和 return 真假。 (或者我想你可以修改构造函数以抛出异常,但我认为那不是正确的方法)
$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();
}
看起来如果抛出异常,它会被构造函数捕获。看起来构造函数没有抛出异常。
调用者不会 "see" 异常,因为它在构造函数中 "swallowed"。
在回答你的问题之前,我想我应该指出这种在构造函数内部做事的糟糕设计。
我会去这样的地方:
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());
}
在你的 bootstrap 中:
try {
$db = createDatabase();
} catch(Exception $ex){
echo "Failed!";
}
这样,当你为Database
class编写单元测试时,注入模拟对象会很容易
我正在为我的项目编写安装脚本。基本上我想做的是检查用户是否输入了正确的数据库信息,如果不正确,return 一个错误。这就是我试图完成的:
<?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!";
}
?>
不幸的是,这总是 return 成功!即使用户输入的数据不正确。有什么我想念的吗?我必须使用不同的方法吗?
这是我用于 PDO 的代码 class:
<?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对象
您应该尝试连接,如果连接失败,那么您就知道信用有误。您可以在您的代码中 运行 这个,或者在您的数据库 class 中创建一个方法来检查这个和 return 真假。 (或者我想你可以修改构造函数以抛出异常,但我认为那不是正确的方法)
$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();
}
看起来如果抛出异常,它会被构造函数捕获。看起来构造函数没有抛出异常。
调用者不会 "see" 异常,因为它在构造函数中 "swallowed"。
在回答你的问题之前,我想我应该指出这种在构造函数内部做事的糟糕设计。
我会去这样的地方:
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());
}
在你的 bootstrap 中:
try {
$db = createDatabase();
} catch(Exception $ex){
echo "Failed!";
}
这样,当你为Database
class编写单元测试时,注入模拟对象会很容易