-1

这个程序为什么不能正确读取 hex 文件？

#include <stdio.h>
#include <stdlib.h>

char *buffer;
unsigned int fileLen;
void ReadFile(char *name);


void ReadFile(char *name)
{
        FILE *file;

        file = fopen(name, "rb");
        if (!file)
        {
                fprintf(stderr, "Unable to open file %s", name);
                return;
        }

        //Getting file length
        fseek(file, 0, SEEK_END);
        fileLen=ftell(file);
        fseek(file, 0, SEEK_SET);

        //Allocating memory
        buffer=(char *)malloc(fileLen+1);
        if (!buffer)
        {
                fprintf(stderr, "Mem error!");
                                fclose(file);
                return;
        }

        fread(buffer, fileLen, 1, file);
        fclose(file);


}

int main()
{
   var32 code;
   char filename[20];
   printf("Enter the file name: ");
   scanf("%s", &filename);
   ReadFile(filename);
   printf("FIle contents: %x\n",buffer);

}

如果我打印一个巨大的十六进制文件，它只会打印 5 到 6 位数字。

4

3 回答 3

6 
printf("FIle contents: %x\n",buffer);

"%x" 只打印一个十六进制值。它正在打印缓冲区的内存地址，而不是其内容。

尝试改变：

    fread(buffer, fileLen, 1, file);
    fclose(file);

之后添加：

    ...
    fclose(file);
    size_t i;
    for (i = 0; i < fileLen; i++)
        printf("%02x ", buffer[i];

这将打印整个二进制内容。但是不必读取整个文件来执行此操作，您可以一次输出 1 K 的块...

不同的实现

#include <stdio.h>
#include <stdint.h>
#include <string.h>
#define BUFFER_SIZE     4096
int main(void)
{
     uint8_t  *buffer;  // Explicit 8 bit unsigned, but should equal "unsigned char"
     FILE     *file;
     char     filename[512];
     // We could also have used buffer[BUFFER_SIZE], but this shows memory alloc
     if (NULL == (buffer = malloc(BUFFER_SIZE)))
     {
          fprintf(stderr, "out of memory\n");
          return -1;
     }
     printf("Enter the file name: ");
     // Read a line terminated by LF. Max filename size is MAXPATH, or 512
     fgets(filename, 512, stdin);
     // Trim filename (Windows: CHECK we don't get \r instead of \n!)
     {
         // Being inside a { }, crlf won't be visible outside, which is good.
         char     *crlf;
         if (NULL != (crlf = strchr(filename, '\n')))
              *crlf = 0x0;
     }
     if (NULL == (file = fopen(filename, "rb")))
     {
         fprintf(stderr, "File not found: '%s'\n", filename);
         return -1;
     }
     while(!feof(file) && !ferror(file))
     {
          size_t i, n;
          if (0 == (n = (size_t)fread(buffer, sizeof(uint8_t), BUFFER_SIZE, file)))
               if (ferror(file))
                    fprintf(stderr, "Error reading from %s\n", filename);
                    // Here, n = 0, so we don't need to break: next i-cycle won't run
          for (i = 0; i < n; i++)
          {
               printf("%02x ", buffer[i]);
               if (15 == (i % 16))
                    printf("\n"); // Every 16th byte, a newline
          }
     }
     fclose(file); // file = NULL; // This ensures file won't be useable after fclose
     free(buffer); // buffer = NULL; // This ensures buffer won't be useable after free
     printf("\n");
     return 0;
}
于 2012-07-09T12:50:40.710 回答
4

该printf()语句将以buffer十六进制格式打印 的地址。您需要输入printf()每个字节buffer（并且您必须空终止buffer，以便调用者知道何时停止或将字节数返回buffer给调用者）并确保调用者知道有多少字节buffer（空终止不安全，因为读取二进制文件） .

还：

  • 在取消引用之前确保buffer不为空（这是逐字节打印所必需的）
  • 检查返回值fread()以确保它成功
  • 记住free()在不再需要时缓冲
  • malloc()强制转换 unrequired的返回值（请参阅Do I cast the result of malloc?）
  • 限制读取的字节数scanf()以避免缓冲区溢出：scanf("%19s", filename);
  • code是未使用的局部变量
于 2012-07-09T12:50:45.143 回答
-1

如果使用十六进制表示二进制，则不能直接打印，输出将在文件中的第一个零字节处结束。

此外，如果找不到文件，您先说“无法打开文件”，然后继续说“文件内容：...”

正如其他人指出的那样，您以错误的方式打印缓冲区。

而且您不会释放缓冲区。

于 2012-07-09T12:48:30.470 回答