重复使用可变参数函数参数不起作用

Question

我有一个函数尝试将内容记录到控制台和日志文件中，但它不起作用。第二次使用可变长度参数会将垃圾写入控制台。有什么想法吗？

    void logPrintf(const char *fmt, ...) {
        va_list ap;    // log to logfile
        va_start(ap, fmt);
        logOpen;
        vfprintf(flog, fmt, ap);
        logClose;
        va_end(ap);
        va_list ap2;   // log to console
        va_start(ap2, fmt);
        printf(fmt, ap2);
        va_end(ap2);
    }

Answer 1

原始代码失败，因为它尝试在需要使用 vprintf() 的地方使用 printf()。从表面上看像 logOpen 和 logClose 语句这样的可疑点（根据符号，大概它们是打开和关闭 flog 文件的宏流），代码应该是：

void logPrintf(const char *fmt, ...) {
    va_list ap;
    va_start(ap, fmt);
    logOpen;
    vfprintf(flog, fmt, ap);
    logClose;
    va_end(ap);
    va_list ap2;
    va_start(ap2, fmt);
    vprintf(fmt, ap2);
    va_end(ap2);
}

使用两个单独的 va_list 变量没有特殊要求； 在再次使用 va_start() 之前使用 va_end()，就可以使用同一个函数两次。

void logPrintf(const char *fmt, ...) {
    va_list ap;
    va_start(ap, fmt);
    logOpen;
    vfprintf(flog, fmt, ap);
    logClose;
    va_end(ap);
    va_start(ap, fmt);
    vprintf(fmt, ap);
    va_end(ap);
}

只要 code>va_list 值传递给另一个函数（此代码中的 vfprintf() 和 vprintf()），您应该假设它不再可用当前函数。只有调用 va_end() 才是安全的。

此代码中不需要 va_copy()。它有效，但不需要。在其他情况下，您需要 va_copy()，例如当您的函数传递 va_list 时，您需要处理该列表两次：

void logVprintf(const char *fmt, va_list args1)
{
    va_list args2;
    va_copy(args2, args1);
    logOpen;
    vfprintf(flog, fmt, args1);
    logClose;
    vprintf(fmt, args2);
    va_end(args2);
}

请注意，在此代码中，它是调用代码的责任是在 args1 上调用 va_end()。事实上，该标准说：

每次调用 va_start 和 va_copy 宏
应与同一函数中 va_end 宏的相应调用相匹配。

由于 logVprintf() 函数不会调用 va_start 或 va_copy 来初始化 args1，因此它无法合法调用args1 上的 va_end。另一方面，标准要求它为 args2 调用 va_end。

logPrintf() 函数现在可以用 logVprintf() 来实现：

void logPrintf(const char *fmt, ...)
{
    va_list args;
    va_start(args, fmt);
    logVprintf(fmt, args);
    va_end(args);
}

此结构 — 一个带有 va_list 和 cover 的操作函数采用省略号（变量参数）并在转换为 va_list 后将它们传递给操作函数的函数 — 通常是一种很好的工作方式。迟早，您通常会发现需要带有 va_list 参数的版本。

Answer 2

升级你的编译器，这更像是 C++：

template <typename... Args>
void logPrintf(const char *fmt, Args&&... args) {
    logOpen;
    fprintf(flog, fmt, args...);
    logClose;

    printf(fmt, args...);
}

当然，提供 printf 和 fprintf 的类型安全版本会更好。

Answer 3

我认为这种方式更有意义：

void logPrintf(const char *fmt, ...) {
        va_list ap;    // log to logfile
        va_start(ap, fmt);
        logOpen;
        vfprintf(flog, fmt, ap); //logfile
         printf(fmt, ap); //console
        logClose;
        va_end(ap);
    }