JPG не являются правильными, смесь нескольких изображений.Кто-нибудь может сказать мне, что не так?

Question

Я работаю над восстановлением PSET3 в CS50. У меня есть файл "card.raw", в котором файлы jpg располагаются один за другим после первого.

Целью упражнения является «найти первый jpg, сканируя каждый раз 512 байт и проверяя, является ли это началом jpg, затем добавляя следующие байты в файл до тех пор, пока не будет найден следующий запуск jpg, затем открывая другой файл и повторяя до КОНЕЦ ФАЙЛА ".

Изображения не создаются должным образом, и я не уверен, где моя проблема заключается в коде ниже.

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

int main(int argc, char *argv[])
{
    //check if program argument count is correct
    if ( argc != 2)
    {
        fprintf(stderr,"Usage: ./recover image\n");
        return(1);
    }

    char *infile = argv[1];    //Remember file name

    //open file for reading
    FILE *inptr = fopen(infile,"r");
    if (inptr == NULL)
    {
        fprintf(stderr, "Could not open %s.\n", infile);
        return(2);
    }

    typedef uint8_t BYTE;      //new name BYTE

    BYTE buffer[512];          // temporary  storage

    int counter = 0;           // image counter

    while (fread(buffer, 512, 1, inptr) == 1)
    {
        if (buffer[0] == 0xff && buffer[1] == 0xd8 && buffer[2] == 0xff && (buffer[3] & 0xf0) == 0xe0)
        {
            char filename[8];

            start:
            sprintf(filename,"%03i.jpg", counter);
            FILE *outptr = fopen(filename,"w");
            fwrite(buffer, 512, 1,outptr);

            //check next byte
             while (fread(buffer, 512, 1, inptr) == 1)
             {
                  if (buffer[0] != 0xff && buffer[1] != 0xd8 && buffer[2] != 0xff && (buffer[3] & 0xf0) != 0xe0)
                  {
                      fwrite(buffer, 512, 1,outptr);
                      printf("%i",counter);
                  }

                  //if next file starts
                  if(buffer[0] == 0xff && buffer[1] == 0xd8 && buffer[2] == 0xff && (buffer[3] & 0xf0) == 0xe0)
                  {
                      counter++;
                      printf("%i",counter);
                      fclose(outptr);
                      goto start;
                  }
             }
             if (counter > 1)
             {
                 fclose(outptr);
             }
        }
    }
    fclose(inptr);
}

Answer 1

Мог бы написать лучший код, но если это просто логика, то

if (buffer[0] != 0xff && buffer[1] != 0xd8 && buffer[2] != 0xff && (buffer[3] & 0xf0) != 0xe0)
                  {
                      fwrite(buffer, 512, 1,outptr);
                      printf("%i",counter);
                  }

Это неправильно, это должно быть

if (!(buffer[0] == 0xff && buffer[1] == 0xd8 && buffer[2] == 0xff && (buffer[3] & 0xf0) == 0xe0))
                  {
                      fwrite(buffer, 512, 1,outptr);
                      printf("%i",counter);
                  }

Поскольку предыдущий также игнорирует любую часть jpgкоторые имеют один или несколько из первых 4 байтов, равных началу IMG, но не все.Спасибо всем за помощь.