cs50 pset4 recover.c нужна консультация

Question

Я мог бы использовать некоторые рекомендации по этому pset от cs50. Ниже приведен код, который у меня есть. Я застрял в этой точке, где я понятия не имею, как действовать. Когда я запускаю программу, он выводит 16 файлов jpg, но я не могу их просматривать, поэтому что-то должно быть неправильно. Любые советы приветствуются.

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

int main(void) 
{    
// create buffer to store in 
unsigned char buffer[512]; 

// array for filename is 8 for "000.jpg" 
char jpgName[8]; 

// declaring counter for amount of jpegs 
int jpgcounter = 0; 

// open memory card file 
FILE* inptr = fopen("card.raw", "r"); 
FILE* outptr = NULL; 

if (inptr == NULL) 
{ 
    printf("Could not open %s.\n", "card.raw"); 
    return 1; 
} 

    // repeat until end of card 
    while(fread(buffer, sizeof(buffer), 1, inptr) == 1) 
    {         

     // find beginning of jpg 
     if(buffer[0] == 0xff && buffer[1] == 0xd8 && buffer[2] == 0xff && (buffer[3] == 0xe0 || buffer[3] == 0xe1)) 
     { 
      // if allready a outptr file is open close it 
      if (outptr != NULL) 
      { 
        fclose(outptr); 
      } 

      // create filename for jpegs 
      sprintf(jpgName,"%03d.jpg", jpgcounter); 

      // open a new file with name defined by sprintf 
      FILE* outptr = fopen(jpgName, "w");        

      // write into the outfile from buffer 
      fwrite(&buffer, sizeof(buffer), 1, outptr); 

      // increment counter for jpegs 
      jpgcounter++; 
      }          
    } 
    fclose(inptr); 
    return 0; 
} 

Answer 1

Если я правильно понял вашу задачу, вам нужно что-то вроде этого - см. Ниже. Подсказка: что вы планируете делать, если это не начало jpg-файла?

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

int main(void) 
{    

unsigned char buffer[512] = {0}; 
char jpgName[8] = {0}; 
int jpgcounter = 0; 
FILE* outptr = NULL; 

// open memory card file 
FILE* inptr = fopen("card.raw", "rb"); 
if (inptr == NULL) 
{ 
    printf("Could not open %s.\n", "card.raw"); 
    return 1; 
} 

// Start reading  
while(fread(buffer, sizeof(buffer), 1, inptr) == 1) 
{         

     // Find beginning of jpg 
     if(buffer[0] == 0xff && buffer[1] == 0xd8 && buffer[2] == 0xff && (buffer[3] == 0xe0 || buffer[3] == 0xe1)) 
     { 
      if(NULL!=outptr)     
      { 
       fclose(outptr); 
       outptr=NULL; 
      } 

      // Create filename for jpegs 
      sprintf(jpgName,"%03d.jpg", jpgcounter); 

      // Open a new file with name defined by sprintf 
      outptr = fopen(jpgName, "wb");   
      if (NULL == outptr) 
      { 
       printf("Could not open %s.\n", jpgName); 
       return 1; 
      }      

      // Write into the outfile from buffer 
      fwrite(buffer, sizeof(buffer), 1, outptr); 

      // Increment counter for jpegs 
      jpgcounter++; 
     } 
      else 
     { 
      // Continue writing to old file if it is not beginning of file 
      if(NULL!=outptr) 
       fwrite(buffer, sizeof(buffer), 1, outptr);        
     } 
    } 

    if(NULL!=outptr) 
     fclose(outptr); 
    if(NULL!=inptr) 
     fclose(inptr); 

    return 0; 
} 

Answer 2

Это не может работать:

while(fread(buffer, sizeof(buffer), 1, inptr) == 1) 
{         

    // read 512 bytes from inptr and store them in buffer 
    fread(&buffer, sizeof(buffer) , 1, inptr); 

, как вы используете fread дважды и данные считываются первым перезаписаны и вторым. Похоже, что это потеряет первые 512 байт каждого kB в исходном файле.

Вы должны получить дополнительную информацию, если вы удалите второй файл fread().