snprintf 用于字符串连接

Question

我正在使用 snprintf 将字符串连接到 char 数组：

char buf[20] = "";
snprintf(buf, sizeof buf, "%s%s", buf, "foo");
printf("%s\n", buf);
snprintf(buf, sizeof buf, "%s%s", buf, " bar");
printf("%s\n", buf);

问题是buf 的第二个连接不是添加"bar"，而是用它替换"foo"。输出如下：

foo
bar

第一个%s 应保留buf（在本例中为"foo"）。第二个%s 应该附加"bar" 到它。对吧？

我做错了什么？

Answer 1

您违反了snprintf 上的restrict 合同，该合同规定没有其他参数可以与缓冲区重叠。

无论如何，将输入复制到自身中是一种浪费。 snprintf 返回格式化所需的字符数，所以利用它来追加：

char buf[20] = "";
char *cur = buf, * const end = buf + sizeof buf;
cur += snprintf(cur, end-cur, "%s", "foo");
printf("%s\n", buf);
if (cur < end) {
    cur += snprintf(cur, end-cur, "%s", " bar");
}
printf("%s\n", buf);

Answer 2

虽然接受的答案没问题，但更好的（在我看来）答案是连接字符串是错误的。您应该在对snprintf 的单个 调用中构造整个输出。这就是使用格式化输出函数的全部意义所在，它比指针运算和多次调用更高效、更安全。例如：

snprintf(buf, sizeof buf, "%s%s%s", str_a, str_b, str_c);

Answer 3

试试这个：

char buf[20];
snprintf(buf, sizeof buf, "%s", "foo");
printf("%s\n", buf);
int len = strlen(buf);
snprintf(buf+len, (sizeof buf) - len, "%s", " bar");
printf("%s\n", buf);

输出是“foo bar”。 snprintf 的第一个参数，一个指向 char 的指针，是它开始填充字符的地方。它不注意缓冲区中的内容。不过，函数 strlen 确实很注意。它计算 snprintf 放在那里的 nul (0) 之前的字符数。所以不要传递 buf，而是传递 buf+strlen(buf)。也可以使用 strncat，效率会稍微高一些。

我在您的问题下看到了标签 C++。查找 std::string。好多了。

Answer 4

为什么不使用strncat()？它的设计正是为了做到这一点：

char buf[20] = "";
strncat(buf, "foo", sizeof buf);
printf("%s\n", buf);
strncat(buf, " bar", sizeof buf - strlen(buf));
printf("%s\n", buf);

如果您的系统支持，您可以使用strncat_s() 而不是strncat，因为它具有额外级别的溢出保护，并且无需计算输出缓冲区中剩余的字节数。

如果您必须使用snprintf，则需要创建一个单独的指针来跟踪字符串的结尾。该指针将是您传递给snprintf 的第一个参数。您当前的代码始终使用buf，这意味着它将始终打印到该数组的开头。您可以在每次调用snprintf 后使用strlen 查找字符串的结尾，也可以使用snprintf 的返回值来递增指针。