C语言文件读取

Question

我是 C 新手，我面临的问题是我有一个已解压的二进制文件（即以某种格式）。

我要做的就是打包它并再次解压它，看看它是否与原始解压版本相同。

值得一提的一件事：我被告知打包（即转换为打包）和解包（即转换为解包）功能运行良好..只是想亲自确认一下并学习一点C...

我有两点我认为我犯了错误

1：我读取文件的方式

2：我没有正确考虑打包和解包的变量类型（即对于打包它是unsigned char *而对于解包它是short *）

int main(void) {

FILE *fp;

unsigned char* packed ;
short* unpacked;
size_t result;
int fileSize;

fp = fopen(FILENAME, "rb");

fseek (fp , 0 , SEEK_END);
fileSize = ftell (fp);
rewind (fp);

unpacked = (short*) malloc (sizeof(char)*fileSize);

result = fread(unpacked,1,fileSize,fp);

short *originalUnpacked = unpacked;


convert_to_packed(&unpacked, &packed);

convert_to_unpacked(&unpacked, &packed);

if (originalUnpacked == unpacked)
{
    puts ("Thats it !!");

}

fclose(fp );
return EXIT_SUCCESS;
}

Answer 1

这段代码非常糟糕。

当您应该比较内存时，您正在比较指针（originalUnpacked == unpacked）。为什么要混合使用 short * 和 unsigned char * 指针？如果数据是二进制“blob”，您可能应该只使用后者。

要比较内存，请使用标准 memcmp() 函数。

Answer 2

您需要更好地理解指针是什么。将指针视为您放在字母上的地址，将数据或内存视为房屋。如果我复制地址，现在有两个字母，但只有一所房子，这两个字母都指同一所房子，我可以使用其中一个来到达该房子。现在，复制房子需要做更多的工作，而不仅仅是复制信上的地址。您需要一栋新房子和一封带有新地址的新信。

unpacked = (short*) malloc (sizeof(char)*fileSize);
  ^^                  ^^
the letter          building the house


short *originalUnpacked = unpacked;
         ^^                  ^^
       new letter         old letter = copying the letter (address) not the house!


if (originalUnpacked == unpacked)
        ^^                 ^^
    new letter          old letter - neither changed so are equal (same address)

您将此地址提供给转换函数 convert_to_packed 和 convert_to_unpacked，然后它们会覆盖原始数据，因为您忘记建造新房子来保留原始数据。

要修复代码，你需要建造一座新房子，并用它来存储装箱和拆箱的结果。然后，您需要比较两栋房子的内容，而不是比较它们的字母。

 short newUnpacked = (short*) malloc (sizeof(char)*fileSize);

 convert_to_packed(&unpacked, &packed);
 convert_to_unpacked(&unpacked, &newUnpacked); // put unpacked data into new house!

 if (memcmp (unpacked, newUnpacked, filesize) == 0) // comparing contents of houses!