восстановление cs50, "восстановленные изображения не совпадают"

Question

Мой исходный код успешно компилируется и создается в общей сложности из 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;
}

Answer 1

Это также работает и включает в себя использование указателей. Здесь p указатель, который указывает на имя файла. Тем не менее, я думаю, что его использование является излишним. Если вообще не использовать p, вы можете сказать, что filename - это указатель, который указывает на первый элемент имени файла, то есть filename [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);
    }
}

Answer 2

Я опробовал ваш код, и он работает, когда вы меняете имя файла с указателя на массив (т. Е. Char * filename на char filename).