我正在使用以下代码进行数据库连接
class Database extends PDO{
function __construct(){
try {
parent::__construct(DB_TYPE.':host='.DB_HOST.';dbname='.DB_NAME,DB_USER,DB_PASS);
$this->setAttribute(PDO::ATTR_ERRMODE, PDO::ERRMODE_EXCEPTION);
$this->setAttribute(PDO::MYSQL_ATTR_INIT_COMMAND, "SET NAMES 'utf8'");
} catch(PDOException $e){
Logger::newMessage($e);
logger::customErrorMsg();
}
}
}
每件事如登录,获取数据都运行良好。现在我突然发现异常错误消息
Message: SQLSTATE[08004] [1040] Too many connections
Code: 1040
如何解决此错误?
我有一个模型类,我正在创建新的数据库。
class Model {
protected $_db;
public function __construct(){
//connect to PDO here.
$this->_db = new Database();
}
}
和我制作的每个模型,我都是从模型类扩展而来。
答案 0 :(得分:11)
因为您的Model
类在其构造函数中实例化了一个新的Database
对象,所以每次实例化Model
(或任何扩展它的类)时,您实际上都在打开一个新的数据库连接。如果你创建了几个Model
个对象,那么每个对象都有自己独立的数据库连接,这种情况并不常见,通常是不必要的,不能很好地利用资源,但是由于耗尽了所有服务器,因此也是有害的。可用的连接。
例如,循环创建Model
个对象的数组:
// If a loop creates an array of Model objects
while ($row = $something->fetch()) {
$models[] = new Model();
}
// each object in $models has an independent database connection
// the number of connections now in use by MySQL is now == count($models)
解决方案是使用依赖注入和传递 Database
对象到Model::__construct()
,而不是允许它实例化它自己。
class Model {
protected $_db;
// Accept Database as a parameter
public function __construct(Database $db) {
// Assign the property, do not instantiate a new Database object
$this->_db = $db;
}
}
要使用它,控制代码(将实例化模型的代码)本身应仅调用new Database()
一次。然后必须将控制代码创建的对象传递给所有模型的构造函数。
// Instantiate one Database
$db = new Database();
// Pass it to models
$model = new Model($db);
对于实际需要为模型使用不同的独立数据库连接的用例,您可以将其交给另一个。特别是,这对于用于测试非常有用。您可以替换测试数据库对象或模拟对象。
// Instantiate one Database
$db = new Database();
$another_db = new Database();
// Pass it to models
$model = new Model($db);
$another_model = new Model($another_db);
正如评论中所提到的,使用持久连接可能是一种解决方案,但不是我推荐的解决方案。 PDO将尝试重用具有相同凭据的现有连接(就像您的所有连接一样),但您不一定希望跨脚本执行缓存连接。如果你决定这样做,你需要将属性传递给Database
构造函数。
try {
// Set ATTR_PERSISTENT in the constructor:
parent::__construct(DB_TYPE.':host='.DB_HOST.';dbname='.DB_NAME,DB_USER,DB_PASS, array(PDO::ATTR_PERSISTENT => true));
$this->setAttribute(PDO::ATTR_ERRMODE, PDO::ERRMODE_EXCEPTION);
$this->setAttribute(PDO::MYSQL_ATTR_INIT_COMMAND, "SET NAMES 'utf8'");
}
相关文档位于:http://php.net/manual/en/pdo.connections.php#example-950
使用单例模式(也不推荐),您至少可以将其减少为模型代码中的搜索/替换。 Database
类需要一个静态属性来保持自身的连接。模型然后调用Database::getInstance()
而不是new Database()
来检索连接。您需要在模型代码中进行搜索和替换,以替换Database::getInstance()
。
虽然它运行良好并且难以实现,但在您的情况下,由于您必须使用相同名称的测试类替换整个Database
类,因此会使测试更加困难。您无法在实例的基础上轻松替换测试类。
将单例模式应用于Database
:
class Database extends PDO{
// Private $connection property, static
private static $connection;
// Normally a singleton would necessitate a private constructor
// but you can't make this private while the PDO
// base class exposes it as public
public function __construct(){
try {
parent::__construct(DB_TYPE.':host='.DB_HOST.';dbname='.DB_NAME,DB_USER,DB_PASS);
$this->setAttribute(PDO::ATTR_ERRMODE, PDO::ERRMODE_EXCEPTION);
$this->setAttribute(PDO::MYSQL_ATTR_INIT_COMMAND, "SET NAMES 'utf8'");
} catch(PDOException $e){
Logger::newMessage($e);
logger::customErrorMsg();
}
}
// public getInstance() returns existing or creates new connection
public static function getInstance() {
// Create the connection if not already created
if (self::$connection == null) {
self::$connection = new self();
}
// And return a reference to that connection
return self::$connection;
}
}
现在您只需要更改Model
代码即可使用Database::getInstance()
:
class Model {
protected $_db;
public function __construct(){
// Retrieve the database singleton
$this->_db = Database::getInstance();
}
}