【问题标题】:Prepared statement returning False准备好的语句返回 False
【发布时间】:2016-12-10 20:35:35
【问题描述】:

我知道有很多问题与我的问题类似。但我真的想不出这里的问题。我有一个名为“UsersClass”的类,它负责与用户相关的每一项任务。而我使用mysqli prepare 语句从数据库中插入或选择数据,问题是在很多情况下prepare 语句返回false。我通过每隔一段时间建立新连接解决了这个问题,但这在其他功能“2006:Mysql server has gone away”中引起了另一个问题,请参见下面的代码: 在这里和其他函数中,prepare 返回 false。

function isPostingAllowed() {
  $this->setPsSelectUsers($this->_connection->prepare("Select * from userpermissions where userpermissions.UserId = ? and userpermissions.PermissionId = 1"));
 $this->_psSelectUsers->bind_param('i',$this->UserId);
 $this->_psSelectUsers->execute();
        if ( false===$this->_psSelectUsers ) {
        die('prepare() failed: ' . htmlspecialchars($this->_connection->error));}
 if ($this->_psSelectUsers->fetch()){
     return true;
 }else{
     return false;
     }}


function setPsSelectUsers($stmt) {
    $this->unsetPsSelectUsers();
           // mysqli_close($this->_Connection);
           // $this-> __construct();

    $this->_psSelectUsers= $stmt;}

当我取消注释这两行时,第一个函数将起作用并且准备状态不会返回 false,但在这种情况下,以下函数将抛出错误 2006:

 function checkUserAuthentication() {
 $this->setPsSelectUsers($this->_connection->prepare("SELECT UserLogInName FROM Users WHERE UserLogInName=? AND UserPassword=?"));
 $this->_psSelectUsers->bind_param('ss',$this->UserLogInName, $this->UserPassword);
 $this->_psSelectUsers->execute();
 if ($this->_psSelectUsers->fetch()){
     return true;
 }else{
     return false;
     }}

那么如何在不产生新问题的情况下解决第一个问题?

【问题讨论】:

  • 这个unsetPsSelectUsers() 方法有什么作用?我怀疑这里有问题。另外,不要调用setPsSelectUsers() 方法,为什么不直接这样做:$this->_psSelectUsers = $this->_connection->prepare(...);
  • @RajdeepPaul 私有函数 unsetPsSelectUsers() { if (isset($this->_psSelectUsers)) { $this->_psSelectUsers->close();未设置($this->_psSelectUsers);}
  • @RajdeepPaul 是的,我明白,但它们是相同的。 unsetPsSelectUsers() 仅取消设置 _psSelectUsers。
  • 我在下面更新了我的答案。我已经解释了导致此问题的原因以及解决方案。希望这能回答您的所有疑问。

标签: php mysql mysqli prepared-statement


【解决方案1】:

问题:

...我使用mysqli prepare 语句从数据库中插入或选择数据,问题是在很多情况下prepare 语句返回false。

那是因为您以 乱序 方式运行查询,即您在关闭前一个语句对象之前执行->prepare()。给定您当前的代码,在您准备好的语句中添加以下错误报告代码,

if(!$this->_connection->prepare(...)){
    printf('errno: %d, error: %s', $this->_connection->errno, $this->_connection->error);
    die();
}

如果您查看$this->_connection->error,您会看到以下错误,

命令不同步;你现在不能运行这个命令

此问题已在许多论坛中记录,例如:

  • 来自MySQL documentation

    如果您的命令不同步;你现在不能在你的客户端代码中运行这个命令,你以错误的顺序调用客户端函数。

  • 来自这个SO thread

    您不能同时进行两个查询,因为 mysqli 默认使用无缓冲查询(对于准备好的语句;...

解决方案:

以正确的顺序执行命令,(See this example code)

  1. 打开连接(一次:在最开始时,而不是在每个查询执行期间)
  2. 创建准备好的语句
  3. 绑定参数
  4. 执行查询
  5. 绑定结果变量
  6. 将值提取到这些绑定变量中
  7. 关闭语句对象
  8. 关闭连接(一次:在最后,而不是在每个查询执行期间)

(按照步骤 2 到 7 执行所有查询,但根据您的查询,一个或多个步骤可能是可选的)

所以解决方法是,在再次调用->prepare()之前关闭之前的语句对象。将此方法调用 $this->unsetPsSelectUsers();setPsSelectUsers() 方法中取出,并将其放在 isPostingAllowed()checkUserAuthentication() 方法的 if ($this->_psSelectUsers->fetch()){...}else{...} 块之前。此外,将$this->_psSelectUsers->fetch() 方法调用的状态保存在一个变量中,并在随后的if 块中使用它。所以你的代码应该是这样的:

public function isPostingAllowed() {
    $this->setPsSelectUsers($this->_connection->prepare("Select * from userpermissions where userpermissions.UserId = ? and userpermissions.PermissionId = 1"));
    if(!$this->_psSelectUsers){
        printf('errno: %d, error: %s', $this->_connection->errno, $this->_connection->error);
        die();
    }
    $this->_psSelectUsers->bind_param('i',$this->UserId);
    $this->_psSelectUsers->execute();
    $status = $this->_psSelectUsers->fetch();
    $this->unsetPsSelectUsers();
    if ($status){
        return true;
    }else{
        return false;
    }
}

private function setPsSelectUsers($stmt){
    $this->_psSelectUsers= $stmt;
}

private function unsetPsSelectUsers() { 
    if (isset($this->_psSelectUsers)) { 
        $this->_psSelectUsers->close(); 
        unset($this->_psSelectUsers);
    }
}

public function checkUserAuthentication() {
    $this->setPsSelectUsers($this->_connection->prepare("SELECT UserLogInName FROM Users WHERE UserLogInName=? AND UserPassword=?"));
    if(!$this->_psSelectUsers){
        printf('errno: %d, error: %s', $this->_connection->errno, $this->_connection->error);
        die();
    }
    $this->_psSelectUsers->bind_param('ss',$this->UserLogInName, $this->UserPassword);
    $this->_psSelectUsers->execute();
    $status = $this->_psSelectUsers->fetch();
    $this->unsetPsSelectUsers();
    if ($status){
        return true;
    }else{
        return false;
    }
}

而且,你不必再使用这个setPsSelectUsers() 方法,你可以直接在你的方法中使用$_psSelectUsers 属性,如下所示:

$this->_psSelectUsers = $this->_connection->prepare(...);

【讨论】:

  • 首先感谢您的大力帮助:),其次问题已解决,注册页面现在运行良好,因此我认为这是已接受的答案,尽管不知从哪里弹出了新错误:) 它似乎有无限循环。我想我会测试和重写代码的很多部分。无论如何,非常感谢您的帮助:) :)
  • @ParcRoi 不客气! ;-) 很高兴我能帮上忙。 干杯!
猜你喜欢
  • 2015-07-13
  • 2012-01-31
  • 1970-01-01
  • 2018-10-05
  • 1970-01-01
  • 2011-02-02
  • 1970-01-01
  • 2020-11-05
  • 1970-01-01
相关资源
最近更新 更多