PHP实现tbl_temp_order整表数据迁移到tbl_perm_order功能失效如何解决

现有代码核心问题 #

• 插入逻辑错误：预处理语句调用execute()执行后，额外调用mysqli_query(getConnection(), $addData)直接执行带占位符?的未传参SQL，这步必然执行失败，导致$done永远为假，后续清空临时表的逻辑永远不会触发。
• 连接管理混乱：每次调用getConnection()都会生成新的数据库连接，既浪费性能，也无法通过事务保证操作原子性，容易出现数据不一致。
• 预编译逻辑冗余：插入语句的预编译操作放在循环体内，每一条数据都重复编译相同的SQL，完全没有利用预处理的性能优势。
• 订单ID逻辑缺陷：所有迁移的数据都共用同一个$id作为orderId，如果orderId是唯一索引，第二条及以后的插入操作会直接报错，仅第一条数据能插入成功。
• 无错误处理机制：没有异常捕获和事务回滚逻辑，迁移过程出现异常时无法快速定位问题，也不能保证数据一致性。

修复后代码示例 #

// 统一获取单条数据库连接，开启事务保证操作原子性
$conn = getConnection();
mysqli_begin_transaction($conn);

try {
    // 查询临时表所有数据
    $selectQry = "SELECT * FROM tbl_temp_order";
    $getData = mysqli_query($conn, $selectQry);

    // 插入语句预编译放在循环外，仅编译一次
    $addData = "INSERT INTO tbl_perm_order(orderId,fId,tId,qty,servantName) VALUES (?,?,?,?,?)";
    $st = $conn->prepare($addData);

    while ($row = $getData->fetch_assoc()) {
        // 若每条记录需要独立orderId，可在这里替换为对应ID生成逻辑
        $orderId = getId();
        $st->bind_param('iiiis', $orderId, $row['fId'], $row['tId'], $row['qty'], $row['servantName']);
        $st->execute();
    }

    // 所有数据插入成功后再清空临时表
    $deleteQry = "DELETE FROM tbl_temp_order";
    mysqli_query($conn, $deleteQry);

    // 提交事务，所有操作生效
    mysqli_commit($conn);
} catch (Exception $e) {
    // 出现异常直接回滚，避免半成功的脏数据
    mysqli_rollback($conn);
    // 生产环境可打开下方注释记录错误日志
    // error_log("订单迁移失败: " . $e->getMessage());
    throw $e;
}

优化说明 #

• 统一使用单条数据库连接，通过事务保证「全部迁移成功才清空临时表」的原子性，不会出现部分迁移部分未迁移的中间状态。
• 预处理语句仅编译一次，循环内仅传参执行，大幅提升批量插入的性能。
• 移除了错误的无参数SQL执行逻辑，只要所有插入操作无异常，才会执行清空临时表的操作。
• 调整了orderId生成逻辑，默认每条记录生成独立ID，你可以根据业务需求灵活调整。
• 增加异常捕获逻辑，出现问题可以快速定位，同时自动回滚避免数据不一致。

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