CS50 PSET4恢复：Fread（）不填充缓冲区数组

Question

您好，谢谢您的看看。

我正在通过CS50X工作，并且正在努力恢复。目的是打开一个.raw文件，在512个字节块中读取其内容，检查.jpg标头的最初的四个字节，然后将每个JPEG数据写入新文件。

我写了一个代码的主体，文件编译。调试器告诉我，我的缓冲区[512]变量保持空/零。然后，这意味着如果/其他条件和程序退出，则该程序会跳过。

尽管我的逻辑可能会存在缺陷，但我无法进入程序足够远，无法考虑这一点。

发布之前，我查找了问题。一些来源喜欢使用fread（缓冲区，512，1，输入），但是CS50本身使用fread（Buffer，1，512，Input）。另外，在初始化文件名​​时，我已经尝试了char * filename = malloc（8 * sizeof（char））; and char fileName [8];。对于这两行，我都尝试了每种方法，但仍缺少一些东西。

我的代码在下面。预先感谢您的宝贵时间。

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

typedef uint8_t BYTE;

int main(int argc, char *argv[])
{
    // First check the number of arguments is correct.
    if (argc != 2)
    {
        printf("Correct Usage: ./recover.c [filename]\n");
        return 1;
    }

    // Open the file.
    FILE *inputFile = fopen(argv[1], "r");

    if (inputFile == NULL)
    {
        printf("File not found.\n");
        return 1;
    }

    // Create counter of number of files.
    int counter = 0;
    // Create filename variable
    char *filename = malloc(8 * sizeof(char)); // 7 + 1 for \0
    // Create a 512-size array buffer.
    BYTE buffer[512];
    // Initialise img file for scope access.
    FILE *img = NULL;

    while (fread(buffer, sizeof(BYTE), 512, inputFile))
    {
        // If start of new JPEG:
        if (buffer[0] == 0xff && buffer[1] == 0xd8 && buffer[2] == 0xff && (buffer[3] & 0xf0) == 0xe0)
        {
            if (counter == 0) // If the FIRST JPEG
            {
                // Make new file:
                sprintf(filename, "%03i.jpg", counter);
                img = fopen(filename, "w");
                fwrite(buffer, sizeof(BYTE), 512, img);
            }
            else // If not the first JPEG
            {
                fclose(img); // Close previous file.
                counter++;
                // Make new file:
                sprintf(filename, "%03i.jpg", counter); // Update filename.
                img = fopen(filename, "w");
                fwrite(buffer, sizeof(BYTE), 512, img);
            }
        }
        else if (counter > 0) // buffer is continuation of previous.
        {
            fwrite(buffer, sizeof(BYTE), 512, img);
        }
        else
        {
            printf("I exited with no images.\n");
            return 2;
        }
    }
    free(filename);
    fclose(img);
    fclose(inputFile);
    return 0;
}

Answer 1

读取原始文件中的第一行后，程序退出（返回）（假设它不是 jpeg 标头，发行版原始文件就是这种情况）。 else if (counter > 0) 计算结果为 false，因此执行 else 分支。

Answer 2

谢谢大家的回复。现在问题已解决！

@dinocodersaurus（对不起，尚无法投票）提示我意识到我（错误地）以.jpeg标头开始立即（实际上，看起来数据都以隐藏的消息开头） ， “惊喜”）。

else条件最初是为了避免错误的情况，但是当然，它已过早地退出时循环。几个循环后正确填充缓冲区。

然后，我遇到了第二个问题（由@some_programmer_dude指出），counter ++;在错误的位置，这意味着在第一个新的JPEG之后，没有其他内容。

我还将考虑您的最佳实践评论。