字符串比较中的十六进制文字问题

Question

我正在读取一个 NES ROM 文件，其中前四个字节是“\x4e\x45\x53\x1a”，或 NES\x1a。在我的实际代码中，给定的文件可以是任意的，因此我想检查以确保此标头位于此处。但是，我遇到了一些麻烦，以下代码演示了这一点：

#include <stdio.h>
#include <string.h>

int main()
{
    FILE *fp;
    fp = fopen("mario.nes", "rb");

    char nes[4];
    char real_nes[4] = "NES\x1a";
    fread(nes, 4, 1, fp);
    printf("A: %x\n", nes[3]);
    printf("B: %x\n", real_nes[3]);
    printf("C: %s\n", nes);
    printf("D: %s\n", real_nes);
    if (strcmp(nes, real_nes) != 0) {
        printf("not a match\n");
    }
    fclose(fp);
    return 0;
}

返回：

A: 1a
B: 1a
C: NES?
D: NES
not a match

其中问号是 \x1a。

我是 C 新手，所以我可能错过了一些关于为什么两个字符串不匹配的微妙（或明显）的东西，以及为什么在打印 D 行时不显示问号，以表示 \x1a 是在字符串的末尾，B 行似乎表明它应该是。

Answer 1

代码中的主要问题是：

char real_nes[4] = "NES\x1a";

这不是一个字符串，因为它不以空终止符字符（'\0'）结尾。
这对于“nes”来说也是同样的问题。

只需这样声明它们即可：

char real_nes[] = "NES\x1a"; /* this is a string, ended by '\0' */
char nes[sizeof real_nes];

确保有足够的位置用于 '\0'。

现在您可以使用 %s 说明符或 strcmp()。无论如何，我建议使用 strncmp() 代替，例如：

if(0 != strncmp(real_nes, nes, sizeof real_nes)) { /* some stuff */ }

HTH.

Answer 2

不要在非零终止字节数组上使用字符串函数。

问题是您有两个 4 字节数组，其中应包含字符串“NES\x1a”（没有为“\0”留下空间，因为它已经是 4 个字节长），但 %s 格式和 strcmp 需要“\0” ' 在末尾终止以知道字符串结束。这就是它无法正常工作的原因。

1.: 不要在此字节数组上使用 %s 格式的 printf。
2.：使用memcmp比较字节。

试试这个：

int i;

printf("Read bytes: 0x");
for(i = 0; i < sizeof(nes); i ++)
  printf("%02X", nes[i]);
printf("\n");

if (memcmp(nes, real_nes, sizeof(nes)) != 0) {
  printf("not a match\n");
}

Answer 3

也许有点太晚了，但我是这样做的：

 // Read the 16 byte iNES header
    char header[16];
    fread( header, 16, 1, file );

    // Search for the "NES^Z" signature
    if( memcmp( header, "NES\x1A", 4 ) )
    {

正如 Xeno 提议的那样，使用 memcmp 你不关心空终止符。毕竟，您并不是真正使用字符串，而更像是 char 数组，由于 null 终止符，这并不相同。由于除了调试之外，您实际上并不需要打印签名，因此您根本不应该关心使用字符串函数。

Answer 4

一些评论和建议：

• 以二进制模式打开文件 - 否则，在非 POSIX 系统上可能会发生有趣的事情（已修复）

  fp = fopen("mario.nes", "rb");
  

• 如果您想打印或空终止您的缓冲区比较它们或使用像 strncmp() 这样的函数，它接受字符串的长度作为额外参数

  printf("C: %.4s\n", nes);
  printf("D: %.4s\n", real_nes);
  if (strncmp(nes, real_nes, 4) != 0) {
  

• ' \x1a' 是非图形替换字符 ^Z

• 检查 io 函数的返回值是否有错误

Answer 5

好吧，一个问题是您对 strcmp 的使用。此函数需要一个以零结尾的字符串（nes 和 real_nes 都不是代码中以零结尾的字符串）。

另一个问题是恐惧。像这样使用它：

fread(nes, 1, 4, fp); // first size_t param is size and second is member count

像这样更改您的代码：

int main()
{
        FILE *fp;
        fp = fopen("mario.nes", "rb");

        char nes[5];
        char real_nes[5] = "NES\x1a";
        fread(nes, 1, 4, fp);
        nes[4] = '\0';
        printf("A: %x\n", nes[3]);
        printf("B: %x\n", real_nes[3]);
        printf("C: %s\n", nes);
        printf("D: %s\n", real_nes);
        if (strcmp(nes, real_nes) != 0) {
            printf("not a match\n");
        }
        fclose(fp);
        return 0;
}

然后看看它是否有效。