MySQL：GET_LOCK锁失效引发重复订单号问题求助

首先，你遇到的重复订单号问题，核心原因是手动维护锁和订单号的流程中可能存在锁未正确生效、异常导致锁未释放，或者max(orderno)读取与UPDATE操作之间的间隙（虽然你用了GET_LOCK，但如果没处理锁获取失败的情况，就会导致多个进程同时执行后续逻辑）。下面是几种更可靠的解决方案，按推荐程度排序：

1. 使用数据库原生自增序列（最优） #

数据库的自增机制是原子性的，能从根源避免重复，而且不需要手动处理锁，性能和可靠性都更好。

MySQL 实现方式 #

方法1：单独的序列表

创建一个专门用于生成自增ID的表：

CREATE TABLE order_sequence (
    id INT UNSIGNED NOT NULL AUTO_INCREMENT PRIMARY KEY
) ENGINE=InnoDB;

然后在代码中这样生成订单号：

// 插入空记录获取自增ID（原子操作）
$DB->Sql("INSERT INTO order_sequence VALUES ()");
$newOrderNo = $DB->insertId(); // 调用你的数据库类获取最后插入ID的方法

// 将生成的订单号更新到目标订单记录
$DB->Sql("UPDATE orders SET orderno=".$newOrderNo." WHERE id=".$actualId);

方法2：单记录计数器表（更节省空间）

创建一个单记录的计数器表，用LAST_INSERT_ID原子性获取递增ID：

CREATE TABLE order_counter (
    counter_name VARCHAR(50) NOT NULL PRIMARY KEY,
    counter_value INT UNSIGNED NOT NULL DEFAULT 0
) ENGINE=InnoDB;

-- 初始化计数器（只需执行一次）
INSERT INTO order_counter (counter_name) VALUES ('ORDERNO') ON DUPLICATE KEY UPDATE counter_value=counter_value;

代码中生成订单号：

-- 原子性递增并获取新值
$DB->Sql("UPDATE order_counter SET counter_value = LAST_INSERT_ID(counter_value + 1) WHERE counter_name = 'ORDERNO'");
$newOrderNo = $DB->insertId();

-- 更新订单记录
$DB->Sql("UPDATE orders SET orderno=".$newOrderNo." WHERE id=".$actualId);

PostgreSQL 实现方式 #

直接创建序列：

CREATE SEQUENCE order_seq START 1;

代码中获取订单号：

$newOrderNo = $DB->resultSql("SELECT nextval('order_seq')");
$DB->Sql("UPDATE orders SET orderno=".$newOrderNo." WHERE id=".$actualId);

2. 改进手动锁流程（次优） #

如果必须保留手动锁的方式，需要完善锁的获取校验和异常处理，确保锁的正确使用：

try {
    // 尝试获取锁，设置超时时间（比如10秒），确保只有一个进程进入后续逻辑
    $lockAcquired = $DB->resultSql("SELECT GET_LOCK('ORDERNO', 10)");
    if (!$lockAcquired) {
        throw new Exception("Failed to acquire lock for order number generation");
    }

    // 处理空表的情况，用COALESCE确保不会返回NULL
    $n = $DB->resultSql("SELECT COALESCE(max(orderno), 0) FROM orders");
    $n += 1;

    // 执行更新操作
    $DB->Sql("UPDATE orders SET orderno=".$n." WHERE id=".$actualId);
} catch (Exception $e) {
    // 处理异常，比如记录日志、返回错误信息
    error_log("Order number generation error: " . $e->getMessage());
} finally {
    // 无论成功还是失败，都释放锁
    $DB->Sql("SELECT RELEASE_LOCK('ORDERNO')");
}

关键改进点： #

• 检查GET_LOCK的返回值，只有拿到锁才继续执行
• 用COALESCE处理orders表为空时max(orderno)返回NULL的情况
• 使用try-finally确保锁一定会被释放，避免死锁

为什么原来的流程会出问题？ #

• 可能未检查GET_LOCK的返回值：如果锁获取失败（比如超时），仍然继续执行后续逻辑，导致多个进程同时读取max(orderno)并生成重复的订单号
• 未处理异常：如果中间的SQL执行出错，RELEASE_LOCK不会被调用，锁会一直被持有，后续进程无法获取锁，可能导致业务阻塞或者绕过锁执行
• max(orderno)本身不是原子操作：虽然有锁，但如果锁未正确生效，多个进程会读到相同的最大值，加1后生成重复订单号

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