OpenCart中MySQL语法错误求助：如何正确执行数据库插入？

首先，咱们先拆解你遇到的致命错误——这完全是SQL语法混淆导致的：你写的INSERT INTO * FROM ... SET ... WHERE ...是彻头彻尾的错误语法，把INSERT（新增记录）和UPDATE（修改已有记录）的用法混在一起了。

从你的代码逻辑来看，你是想修改已登录用户的status字段值，而不是插入一条新的用户记录，所以应该用UPDATE语句，而不是INSERT。

错误原因详解 #

• INSERT语句的作用是向表中添加新行，语法是INSERT INTO 表名 (字段1, 字段2) VALUES (值1, 值2)，不需要WHERE条件（特殊的INSERT ... SELECT场景除外）
• 你要修改已有用户的状态，属于更新操作，必须用UPDATE，语法是UPDATE 表名 SET 字段=值 WHERE 条件

另外，直接把变量拼进SQL语句里还存在SQL注入风险，OpenCart的数据库类已经支持安全的参数处理方式，咱们应该采用更规范的写法。

正确的代码实现 #

把你的代码改成下面这样，既修正了SQL语法，又规避了安全风险：

if ($this->customer->isLogged()) {
    $status = '0';
    $id = $this->customer->getId();
    
    // 使用UPDATE语句修改已有记录，用escape方法处理变量防止注入
    $this->db->query("UPDATE perkulenkijoje_customer SET status = '" . $this->db->escape($status) . "' WHERE customer_id = '" . $this->db->escape($id) . "'");
    
    // 更推荐的预处理绑定写法（OpenCart官方推荐的安全方式）
    // $this->db->query("UPDATE perkulenkijoje_customer SET status = :status WHERE customer_id = :customer_id", [
    //     ':status' => $status,
    //     ':customer_id' => $id
    // ]);
    
    echo ("Info: Status updated successfully");
} else {
    echo ("Info: Please log in");
}

额外补充说明 #

1. 如果你确实需要插入新用户记录（而非修改），正确的INSERT语法示例如下（需根据实际表字段调整）：

$this->db->query("INSERT INTO perkulenkijoje_customer (status, customer_id, 其他字段) VALUES ('" . $this->db->escape($status) . "', '" . $this->db->escape($id) . "', '对应值')");

但结合你的业务逻辑（针对已登录用户操作），显然UPDATE才是正确的选择。

2. 永远记得用$this->db->escape()或者参数绑定处理变量，这是OpenCart开发中防范SQL注入的基本安全规范。

内容的提问来源于stack exchange，提问作者NedasAndrius