C++ printf 字段宽度说明符‘.*’期望 int 不是 size_t

Question

我正在修复我继承的遗留项目中的编译器警告。新编译器是 gcc 版本 4.8.5 20150623 (Red Hat 4.8.5-4) (GCC)。

他们有很多类似下面的代码：

#include <cstdio>
#include <cstring>

struct foobar
{
    char field1[10];
    char field2[5];
};

int main()
{
    struct foobar foo;
    memset(&foo, ' ', sizeof(foo));
    strncpy(foo.field1, "1234567890", sizeof(foo.field1));

    // Produces warning
    printf("[%.*s]", sizeof(foo.field1), foo.field1);

    return 0;
}

这会产生警告消息：

1_test.c: In function ‘int main()’:
1_test.c:16:49: warning: field precision specifier ‘.*’ expects argument of type ‘int’, but argument 2 has type ‘long unsigned int’ [-Wformat=]
  printf("[%.*s]", sizeof(foo.field1), foo.field1);

这对我来说似乎是错误的，因为 '.*' 应该期望 size_t 但显然它不会......

除了必须执行以下操作之外，还有什么方法可以全局解决此问题：

    // Fixes
    printf("[%.10s]", foo.field1);

    // Fixes
    printf("[%.*s]", static_cast<int>(sizeof(foo.field1)), foo.field1);

Answer 1

正确的解决方案是：

std::cout << std::string( foo.field1, sizeof( foo.field1 ) );

这会输出你想要的并且不会产生任何警告。但是更好的解决方案当然是在struct foobar中使用std::string

Answer 2

你总是有一些不好玩但有时是最好的解决方案来包装宏

#define INT_SIZEOF( x ) static_cast< int >( sizeof( (x) ) )

printf("[%.*s]", INT_SIZEOF(foo.field1), foo.field1);

如果你的代码库对宏过敏，你可以为 static_cast 做一个简单的语法包装，像这样

int AsInt(size_t n) { return static_cast<int>( n ) }
printf("[%.*s]", AsInt(sizeof(foo.field1)), foo.field1);

正如 cmets 中所讨论的，此解决方案只能采用大小值，因此需要单独的 sizeof 运算符。另外，如果sizeof 没有返回适合整数的值（尽管可能性很小），您将失去任何编译时错误检查的机会。

请注意，在这两种情况下，您的代码仍然很丑陋，但至少有一个更短，并通知读者正在发生一些丑陋的事情。

最后，另一种可能的不错的选择。

// constexpr is optional, you simply won't be able to use it in as many places if you don't use it
template<typename T> constexpr int intSizeof( T = T{} /*replace with () if necessary*/ ) 
{ 
    return static_cast<int>( sizeof( T ) ); 
}

printf("[%.*s]", intSizeof<decltype(foo.field1)>(), foo.field1);

这个解决方案绝对是更多的 C++ 和更现代的，没有实际成本