C：在二进制文件中写入和读取字符串

Question

我想将字符串与许多其他数据一起存储在二进制文件中，我使用下面的代码（当我真正使用它时，字符串将被 malloc ）我可以写入该文件。我在十六进制编辑器中看过它。我不确定是否正确编写了空终止符（或者是否需要）。当我读回时，我得到的字符串长度与我存储的相同，但不是字符串。我做错了什么？

FILE *fp = fopen("mybinfile.ttt", "wb");

char drumCString[6] = "Hello\0";
printf("%s\n", drumCString);    
//the string length + 1 for the null terminator
unsigned short sizeOfString = strlen(drumCString) + 1;
fwrite(&sizeOfString, sizeof(unsigned short), 1, fp);

//write the string
fwrite(drumCString, sizeof(char), sizeOfString, fp);

fclose(fp);

fp = fopen("mybinfile.ttt", "rb");  

unsigned short stringLength = 0;
fread(&stringLength, sizeof(unsigned short), 1, fp);

char *drumReadString = malloc(sizeof(char) * stringLength);
int count = fread(&drumReadString, sizeof(char), stringLength, fp);

//CRASH POINT
printf("%s\n", drumReadString);

fclose(fp); 

Answer 1

我看到了一些问题，有些是有问题的，有些是风格上的。

• 您应该真正测试 malloc、fread 和 fwrite 的返回值，因为分配可能会失败，并且可能不会读取任何数据或书面。
• sizeof(char) 始终 1，无需乘以它。
• 字符数组“Hello\0”实际上有7个字节长。您不需要添加多余的空终止符。
• 我更喜欢习惯用语 char x[] = "xxx"; 而不是指定明确的长度（当然，除非您想要一个比字符串长的数组）。
• 当您fread(&drumReadString ...)时，您实际上覆盖了指针，而不是它指向的内存。这就是崩溃的原因。它应该为 fread(drumReadString ... 。

Answer 2

一些提示：

1

终止 \0 隐含在任何双引号字符串中，通过在末尾添加一个附加字符，最终会得到两个。以下两个初始化是相同的：

char str1[6] = "Hello\0";
char str2[6] = { 'H', 'e', 'l', 'l', 'o', '\0', '\0'};

所以

char drumReadString[] = "Hello";

就足够了，并且在像这样初始化时指定数组的大小是可选的，编译器会计算出所需的大小（6 字节）。

2

写入字符串时，您不妨一次性写入所有字符（而不是一个一个字符地写入 sizeOfString 次）：

fwrite(drumCString, sizeOfString, 1, fp);

3

尽管对于普通台式电脑场景来说并不常见，但 malloc 可以返回 NULL，您将养成总是检查结果的习惯会受益匪浅，因为在嵌入式环境中，获得 NULL 并不是不可能的结果。

char *drumReadString = malloc(sizeof(char) * stringLength);
if (drumReadString == NULL) {
        fprintf(stderr, "drumReadString allocation failed\n");
        return;
}

Answer 3

你不写终止NUL，你不需要，但你必须在阅读时考虑添加它。即malloc stringLength + 1 char，读取stringLength chars并在读取的末尾添加\0。

现在通常的警告：如果您按照此处的方式编写二进制文件，则有很多未声明的假设，这些假设使您的格式难以移植，有时甚至难以移植到同一编译器的另一个版本 - 我已经在编译器版本之间的结构变化。

Answer 4

还有一些要添加到 paxdiablo 和 AProgrammer 中的内容 - 如果您将来打算使用 malloc，只需从一开始就这样做。这是更好的形式，意味着您在切换时无需进行调试。

另外，我没有完全看到 unsigned Short 的用法，如果您计划编写二进制文件，请考虑 unsigned char 类型通常大小为字节，这使得它非常方便用于此目的。

Answer 5

您只需在 fread 函数中删除 &drumReadString 即可。您只需在该函数中使用rumReadString，如 ganesh 提到的那样。因为，drumReadString 是一个数组。数组类似于直接指向内存位置的指针。

Answer 6

你读书的时候做错了。
你已经把 &对于指针变量，这就是它给出分段错误的原因。

我删除了它工作正常并且正确返回 Hello 的信息。

int count = fread(drumReadString, sizeof(char), stringLength, fp);