C：如何以更简洁的方式将空终止符复制到结构成员？

Question

本质上，我对一个字符串进行标记，并将找到的字符串strncpy转换为结构成员，即 stringid。它当然存在缺乏终止的问题，我为它添加了一个额外的数组空间，我不知道如何正确添加它。

我是这样做的：

my_struct[iteration].stringID[ID_SIZE-1] = '\0' //updated

我不确定这是否真的有效，在我看来，它看起来很糟糕。

Str(n) 复制空字符或 0 会导致 GCC 和 MinGW 生成警告：

warning: null argument where non-null required (arg 2)

我是否不知道如何以干净的方式执行此操作？我正在考虑将成员数组设置为全零，然后复制字符串以很好地适应空终止。您有什么建议或做法吗？

Answer 1

有两件事：

1. 请注意 strncpy() 具有非常意外的语义，如果字符串未完全填充，它将始终填充 0 缓冲区，如果字符串完全填充缓冲区，则不会终止字符串。这两个都很奇怪，我建议不要使用它。
2. 永远不要用数组的大小来索引数组，就像 stringID[ID_SIZE] 似乎正在做的那样；这是越界的。

最好的解决方案是编写一个不太奇怪的自定义版本的 strncpy() ，或者（如果您知道输入的长度）只需使用 strcpy() 。

更新：如果输入标记的长度是静态的，但由于标记化过程，它们在源缓冲区中不是以 0 结尾，则只需使用 memcpy()和手动终止：

const char * token = ...; /* Extract from tokenization somehow. Not 0-terminated. */
const size_t token_length = ... /* Perhaps from tokenization step. */
memcpy(my_struct[iteration].stringID, token, token_length);
my_struct[iteration].stringID[token_length] = '\0';

我认为没有必要将上面的内容“包装”在宏中。

Answer 2

实际上，按照您建议的方式进行 null 终止一点也不可怕，我个人非常喜欢它。

在我看来，最好的方法是以类似的方式将其定义为宏：

// for char* blah;
#define TERMINATE_DYNAMIC_STRING(str, len) str[len] = '\0';
// for char mytext[] = "hello";
#define TERMINATE_STRING(str) str[sizeof(str)/sizeof(str[0]) - 1] = '\0';

然后您可以在代码中随心所欲地使用它。

在 Windows 上，Microsoft 为您提供了以下在复制字符串时以 null 终止的函数： StringCchCopy

Answer 3

正如其他人所指出的，strncpy 具有奇怪的语义。进行有界字符串复制的惯用方法是 strncat 到一个空字符串上：

my_struct[iteration].stringID[0] = '\0';
strncat(my_struct[iteration].stringID, src, ID_SIZE-1);

这总是附加一个终止 NUL（并且最多填充 ID_SIZE 字符，包括 NUL）。

Answer 4

我最终编写了一个 strncpyz(char* pszTo, char* pszTo, size_t lSize) 函数来强制 NULL 终止。如果您有一个库可以放入它，那么这会非常有效。使用它还需要最少的代码更改。

我不热衷于宏方法，因为有人会传递指向错误宏的指针。