PHP PDO更新表[重复]

Question

所以我在学习 Udemy 课程，然后导师做了这个功能

public function update($table, $user_id, $fields = array()){

  $columns = '';
  $i       = 1;

  foreach($fields as $name => $value){

    $columns .= "'{$name}' = :{$name}";
    if($i < count($fields)){
        $columns .= ', ';
    }
    $i++;
  }

  $sql = "UPDATE {$table} SET {$columns} WHERE 'user_id' = {$user_id}";
  if($stmt = $this->pdo->prepare($sql)){
    foreach($fields as $key => $value){
        $stmt->bindValue(':'.$key, $value);
    }
    $stmt->execute();
  }

}

我在他之后写了很多次，但它似乎从来没有工作过，有人能解释一下代码有什么问题吗？

Answer 1

这里

$sql = "UPDATE {$table} SET {$columns} WHERE 'user_id' = {$user_id}";

表名周围的' 应该是反引号，而不是单引号：

$sql = "UPDATE {$table} SET {$columns} WHERE `user_id` = {$user_id}";

这里也一样：

$columns .= "`{$name}` = :{$name}";

Answer 2

你应该用反引号替换单引号：

$columns .= "`{$name}` = :{$name}";

和

$sql = "UPDATE {$table} SET {$columns} WHERE `user_id` = {$user_id}";

只是一个建议：您可以使用数组来构建columns 子句，您可以直接execute 而无需任何bindValue，只需传递$fields 数组即可：

function update($table, $user_id, $fields = array()) {

    $columns = [];

    foreach ($fields as $name => $value) {
        $columns[] = "`{$name}` = :{$name}";
    }

    $sql = sprintf('UPDATE %s SET %s WHERE `user_id` = %s'
            , $table
            , implode(', ', $columns)
            , $user_id
    );

    if ($stmt = $this->pdo->prepare($sql)) {
        $stmt->execute($fields);
    }
}