我尝试使用PDO类运行查询,并在尝试提交时收到此错误消息:"There is no active transaction"
。
这是我的代码:
public function runExQuery($sql) {
$preparedQuery = $this->connect()->prepare($sql);
$this->connect()->beginTransaction();
$preparedQuery->execute();
$this->connect()->commit();
}
private function connect() {
return new PDO('mysql:host=' . $this->host . ';dbname=' . $this->database . '', $this->username, $this->password);
}
此错误的原因是什么?我探讨了此类问题的先前帖子,但没有找到任何解决方案。
答案 0 :(得分:3)
每次调用时,::connect()
方法都会创建一个新的PDO。
由于事务不能在连接之外生存,因此重新连接会将其擦除。
要更正此问题,请将PDO对象存储为类属性:
class MyPdoClass
{
private $pdo;
// ...
public function connect()
{
if ($this->pdo instanceof PDO) {
return;
}
$this->pdo = new PDO(// ....
}
然后在调用connect:
之后引用它//...
public function runExQuery($query)
{
$this->connect();
$this->pdo->prepare($query);
// ...
}
答案 1 :(得分:2)
每次拨打$this->connect()
时都会创建一个新的PDO对象,所以如果你有:
$stmt1 = $this->connect()->prepare(" ... ");
$stmt2 = $this->connect()->prepare(" ... ");
$stmt1
和$stmt2
实际上是完全不同的PDO对象,因此如果您使用一个对象启动事务,它将不会应用于另一个对象。相反,你应该保存一个PDO对象并引用它,而不是每次都创建一个新对象。
大多数时候,我发现将它传递给类的构造函数更容易,但是如果你想进行最少的编辑,你可以这样做:
class YourClass {
private $dbh;
private function connect() {
if (!isset($this->dbh)) {
$this->dbh = new PDO('mysql:host=' . $this->host . ';dbname=' . $this->database, $this->username, $this->password);
}
return $this->dbh;
}
}
但是,您可能希望将名称connect()
更改为更合乎逻辑的内容,例如getDbh()
。
如果你想把它放在对象的构造函数中,你可以这样做:
class YourClass {
private $dbh;
public function __construct(PDO $dbh) {
$this->dbh = $dbh;
}
}
$dbh = new PDO('mysql:host=' . $host . ';dbname=' . $database, $username, $password);
$yourclass = new YourClass($dbh);
然后在任何其他类方法中,您只需引用$this->dbh
。以您的代码为例:
public function runExQuery($sql) {
$preparedQuery = $this->dbh->prepare($sql);
$this->dbh->beginTransaction();
$preparedQuery->execute();
$this->dbh->commit();
}
就个人而言,这就是我的方式。