Forgottenserver 数据库查询同步问题分析与修复方案

问题背景

在Forgottenserver游戏服务器项目中，近期发现了一个严重的数据库操作问题。当使用executeQuery方法执行SELECT查询时，会导致后续所有SQL查询无法正常执行，服务器会抛出"Commands out of sync; you can't run this command now"的错误。

问题现象

该问题表现为：

1. 当通过db.query(Lua)或Database::executeQuery(C++)执行SELECT查询后
2. 服务器无法继续执行任何SQL操作
3. 所有后续查询都会失败并报错

技术分析

这个问题本质上是MySQL客户端API的使用规范问题。根据MySQL官方文档：

• 执行会产生结果集的查询(SELECT/SHOW/DESCRIBE/EXPLAIN等)后
• 必须调用mysql_store_result()或mysql_use_result()获取结果集
• 使用完毕后必须调用mysql_free_result()释放资源

当前代码中的executeQuery方法仅执行查询，但没有处理可能产生的结果集。当执行SELECT查询时，结果集仍留在连接缓冲区中，导致后续查询无法执行。

解决方案

修复方案是在executeQuery方法中添加结果集处理逻辑：

1. 执行原始查询
2. 无论是否有结果集，都调用mysql_store_result()获取结果
3. 使用智能指针自动管理结果集内存，确保资源释放

核心修复代码如下：

bool Database::executeQuery(const std::string& query)
{
    std::lock_guard<std::recursive_mutex> lockGuard(databaseLock);
    auto success = ::executeQuery(handle, query, retryQueries);
    // 处理可能的结果集
    tfs::detail::MysqlResult_ptr res{mysql_store_result(handle.get())};
    return success;
}

技术细节

1. 智能指针使用：项目中使用自定义的MysqlResult_ptr智能指针管理MySQL结果集，其析构函数会自动调用mysql_free_result()释放资源，确保不会内存泄漏。

2. 线程安全：代码中使用了std::lock_guard保证数据库操作的线程安全。

3. 性能考虑：即使对于不产生结果集的查询，调用mysql_store_result()也是安全的，MySQL文档明确指出"这样做没有害处"。

最佳实践建议

1. 查询方法选择：

   • 对于需要获取结果的查询，应使用storeQuery方法
   • 对于不关心结果的查询(INSERT/UPDATE等)，使用executeQuery

2. 错误处理：所有数据库操作都应检查返回值并适当处理错误

3. 资源管理：始终确保查询结果集被正确释放，避免内存泄漏

总结

这个问题展示了数据库连接状态管理的重要性。通过正确处理查询结果集，我们不仅解决了当前的同步问题，也为服务器提供了更健壮的数据库操作基础。这种修复方式遵循了MySQL官方推荐的最佳实践，同时利用了现代C++的资源管理特性，确保了代码的安全性和可靠性。