C++可变参数列表与字符串格式化问题：error函数输出异常

错误根源 #

你的代码核心问题是误用snprintf处理va_list类型参数。snprintf的最后参数是原生可变参数列表（...），而非va_list对象。直接传递va_list会导致函数错误解析参数，读取到随机内存数据，最终输出随机值。

正确处理va_list需要使用对应带v前缀的函数——vsnprintf，它专门用于接收va_list类型的可变参数集合。

具体错误点 #

1. 计算缓冲区大小时，调用snprintf(nullptr, 0, format.c_str(), arguments)是错误的：这里把va_list对象当作普通参数传入，而非展开可变参数，导致格式化解析失败，计算出的size可能不准确。
2. 写入缓冲区时，snprintf(buffer.get(), size, format.c_str(), arg_copy)同样错误：snprintf无法识别va_list类型，会把arg_copy的内存值当成第一个可变参数的内容，因此输出随机数。

修复后的代码 #

#include <string>
#include <cstdint>
#include <memory>
#include <cstdio>
#include <iostream>

using namespace std;

string description = "";

void error(string format, ...){
    va_list arguments;
    va_start(arguments, format);
    va_list arg_copy;
    va_copy(arg_copy, arguments);

    // 使用vsnprintf计算所需缓冲区大小
    const int32_t size = vsnprintf(nullptr, 0, format.c_str(), arguments) + 1;
    va_end(arguments);

    const unique_ptr<char[]> buffer = make_unique<char[]>(size);
    // 使用vsnprintf将格式化内容写入缓冲区
    vsnprintf(buffer.get(), size, format.c_str(), arg_copy);
    va_end(arg_copy);

    description += string(buffer.get(), buffer.get() + size - 1);
}

int main()
{
    int a = 123;
    error("Format %d", a);
    cout << description;

    return 0;
}

补充说明 #

• vsnprintf是snprintf的可变参数适配版本，能够正确解析va_list类型的参数集合，匹配格式化字符串的需求。
• 代码中对va_end的调用是正确的，每个va_start/va_copy都需要对应一个va_end来释放资源。
• 建议添加返回值检查：vsnprintf返回负数表示格式化过程出错，可以根据这个结果增加错误处理逻辑，提升代码健壮性。

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