cs50 恢复，“恢复的图像不匹配”答案

【问题标题】：cs50 recover, "recovered images do not match"cs50 恢复，“恢复的图像不匹配”
【发布时间】：2020-04-22 06:15:23
【问题描述】：

我的源代码编译成功，一共生成了50张图片。
但是，没有一个恢复的图像与原始图像匹配。
所有的 jpeg 如下所示。
如您所见，它们似乎有奇怪的边缘重叠。
其中一些看起来不错，但仍然无法匹配原始图片。

如果您能提供有关如何调试的任何见解，请告诉我。

这是我的代码

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

int main(int argc, char *argv[])
{
    if (argc != 2)
    {
        printf("Usage: ./recover image\n");
        return 1;
    }

    FILE *file = fopen(argv[1], "r");
    if (file == NULL)
    {
        printf("Could not open %s.\n", argv[1]);
        return 1;
    }

    typedef uint8_t BYTE;
    BYTE buffer[512];
    char *filename[8];
    int jpeg_counter = 0;
    bool foundStartOfJPEG = false;
    FILE *img;

    // read memory card until the end of file
    while(fread(buffer, sizeof(BYTE) * 512, 1, file) == 1)
    {
        // if buffer has a signature of JPEG file,
        if (buffer[0] == 0xff && buffer[1] == 0xd8 && buffer[2] == 0xff && ((buffer[3]) & 0xf0) == 0xe0)
        {
            if (foundStartOfJPEG == true)
            {
                fclose(img);
                jpeg_counter += 1;
            }

            foundStartOfJPEG = true;
            // create a file with index
            sprintf(*filename, "%03i.jpg", jpeg_counter);
            // open that file to write into it
            img = fopen(*filename, "w");
            // write the block of memory (buffer), to that file
            fwrite(buffer, sizeof(buffer), 1, img);
        }
        else if (foundStartOfJPEG == true)
        {
            fwrite(buffer, sizeof(buffer), 1, img);
        }
    }

    fclose(file);
    return 0;
}

【问题讨论】：

为什么已经在 while 循环 fread()ing 中再次调用 fread()？
谢谢，我刚刚删除了fread() 的多余行。现在，我恢复了 50 张图片，但它与原始图像不匹配。我会更新问题。 @alk

标签： c cs50 recover

【解决方案1】：

我尝试了您的代码，当您将文件名从指向数组的指针更改时它可以工作（即 char *filename 到 char 文件名）。

【讨论】：

谢谢，将我的文件名更改为字符数组后，它已修复。但是，你知道为什么有一个指针会导致这个问题吗？我的理解是，字符数组和字符指针数组，基本上都是字符串类型。不管怎样，我已经接受了你的回答:)
我不确定为什么图像最终会以它们的方式失真，但我认为指针的使用首先是不正确的，因此错误在某种程度上源于此。我添加了另一个答案来向您展示我的意思。

【解决方案2】：

这也有效并结合了指针的使用。这里 p 是一个指向文件名的指针。但是，我认为它的使用是多余的。完全不使用 p，您可以说文件名是指向文件名的第一个元素的指针，即文件名 [0]。因此，当您使用 char *filename[8] 时，就像说 filename 是一个指向指针的指针......希望这是有道理的！

typedef uint8_t BYTE;
BYTE buffer[512];
char filename[8];
char *p = filename;
int jpeg_counter = 0;
bool foundStartOfJPEG = false;
FILE *img;

// read memory card until the end of file
while(fread(buffer, sizeof(BYTE) * 512, 1, file) == 1)
{
    // if buffer has a signature of JPEG file,
    if (buffer[0] == 0xff && buffer[1] == 0xd8 && buffer[2] == 0xff && ((buffer[3]) & 0xf0) == 0xe0)
    {
        if (foundStartOfJPEG == true)
        {
            fclose(img);
        }

        jpeg_counter += 1;
        foundStartOfJPEG = true;
        // create a file with index
        sprintf(p, "%03i.jpg", jpeg_counter);
        // open that file to write into it
        img = fopen(p, "w");
        // write the block of memory (buffer), to that file
        fwrite(buffer, sizeof(buffer), 1, img);
    }
    else if (foundStartOfJPEG == true)
    {
        fwrite(buffer, sizeof(buffer), 1, img);
    }
}

【讨论】：