Question

我正在用C编写代码，在文本文件和二进制文件中执行hexdump。我在文本文件中的输出是正确的，但是当我尝试在二进制文件中执行hexdump时，出现了垃圾。如果我的代码的哪一部分是错误的，我该如何纠正我的错误，我想向您寻求帮助。谢谢。

#include <stdio.h>
#define OFFSET 16

main(int argc, char const *argv[]) 
{
    FILE *fp;
    char buff[OFFSET];
    int read;
    int address = 0;
    int i;

    if (argc != 2){
        exit(0);
    }

    fp = fopen(argv[1], "rb");

    while ((read = fread(buff, 1, sizeof buff, fp)) > 0){
        printf("%08x ", address);
        address += OFFSET;

        //print hex values 
        for (i = 0; i < OFFSET; i++){

            if(i >= read){
                buff[i] = 0;
            }
            if(buff[i] >= 33 && buff[i] <= 255 || buff[i] != 00){
                printf("%02x ", buff[i]);
            }
            if(buff[i] == 00){
                printf("   ");
            }
        }

        //print ascii values
        for (i = 0; i < OFFSET; i++) {
            printf("%c", (buff[i] >= 33 && buff[i] <= 255 ? buff[i] : ' '));
        }
        printf("\n");
    }

    fclose(fp);
}

Answer 1

您在逻辑上有两个错误。首先，如注释中所指定，二进制文件中的所有字符都同等重要。不需要（也不应该）测试您的二进制输出if(buff[i] >= 33 && buff[i] <= 255 || buff[i] != 00)。

main的正确声明是int main (void)和int main (int argc, char **argv)（您将看到用等效的char *argv[]编写）。请参阅：C11 Standard §5.1.2.2.1 Program startup p1 (draft n1570)。另请参阅：See What should main() return in C and C++?

接下来是二进制输出，您尝试使用%02x打印 unsigned 值，但是您传递的是 signed 字符。如果char的值为负，则您尝试输出 sign-extended 值，并输出无符号值的完整宽度（02x会将字段填充为2个字符，但不会阻止打印两个以上的字符）。您有几种选择，首先使用hh长度修饰符将类型限制为1个字节，然后简单地将值强制转换为(unsigned char)，例如

            printf("%02hhx ", (unsigned char)buff[i]);

您的逻辑也有点麻烦。您应该使用if ... else if ... else处理二进制情况。此外，当i >= read || buff[i] == 0时，您将输出两个空格，因此您最好将测试合并。

简短的重写可能类似于以下内容（将从作为第一个参数给出的文件中读取-如果没有给出参数，则从stdin中读取）

#include <stdio.h>

#define OFFSET 16

int main (int argc, char const *argv[]) 
{
    char buff[OFFSET] = "";
    int read, address = 0, i;
    FILE *fp = argc > 1 ? fopen (argv[1], "rb") : stdin;

    if (!fp) {
        perror ("fopen");
        return 1;
    }

    while ((read = fread(buff, 1, sizeof buff, fp)) > 0) {
        printf("%08x ", address);
        address += OFFSET;

        for (i = 0; i < OFFSET; i++)    /* print hex values */
            if (i >= read || buff[i] == 0)
                printf("   ");
            else
                printf("%02hhx ", (unsigned char)buff[i]);

        fputs ("| ", stdout); /* optional separator before ASCII */

        for (i = 0; i < OFFSET; i++)    /* print ascii values */
            printf("%c", (buff[i] >= ' ' && buff[i] <= '~' ? buff[i] : ' '));
        putchar ('\n'); /* use putchar to output single character */
    }

    if (fp != stdin)
        fclose (fp);
}

（注意：如果编译器不支持hh前缀，则强制转换本身就足够了）

仔细检查一下，如果还有其他问题，请告诉我。

C中的十六进制转储文本和二进制文件

1 个答案: