c programming - Extracting string of data from file

Question

I have the following task: I have a file (card) with 5 strings:

U98_25984nhdrwedb \n
U98_5647BGFREdand \n
U98_30984bgtjfYTs \n
U77_76498375nnnnn \n
U98_83645bscdrTRF \n

I need to extract to another file image.txt those strings starting with "U9". The below code without the memory assignment (malloc, calloc) print out the codes correctly to the screen, but it does not print the correct data to the image.txt, where I only get "98_25984nhdrwedb@". I think I am applying the memory allocation incorrectly, but when I use malloc or calloc (before the while loop) it gets worse and print out garbage and I cannot figure out how to set this correctly.

#include <stdio.h>
#include <stdlib.h>
#include <getopt.h>
#include <stdint.h>
    
typedef uint8_t  BYTE;
    
int main()
{
    FILE *input_card = fopen("card","r");    //open the file for reding
    BYTE data[18];
    int i, n = 5;

    FILE* output = fopen("image.txt","w");    //open the output file for writing

    output = malloc(sizeof(data)*18);         //assign memory
    while (!feof(input_card))
    {
        for (i = 1; i <= n; i++)
        {
            fread(data,sizeof(BYTE),18,input_card);
            if(data[i] != 0)
            {
                if (data[0] == 'U' && data[1] == '9')
                {   
                    printf("data: %s",data);
                    fwrite(&data[i],sizeof(BYTE),18,output);
                }
                fclose(output);
            }
        }
    }
    fclose(input_card);
    free(output);
    return 0;
}

Aside from the memory allocation problem you should take a look at Why is “while ( !feof (file) )” always wrong? — anastaciu
– anastaciu, Commented Nov 18, 2020 at 12:21
You close the output file after the first write access. You do not check how many bytes were read at all. Your usage of if(data[i]!=0) is very weird. This checks the second character in first line, third characater in second line, fourth character in third line... Also for each iteration of your while loop you read 5 times from the file. — Gerhardh
– Gerhardh, Commented Nov 18, 2020 at 12:36
OT: if this is a text file you should read it line by line with fgets. — Jabberwocky
– Jabberwocky, Commented Nov 18, 2020 at 12:40

Amin M. Dodin · Accepted Answer · 2020-11-18 12:29:10Z

2

In the following 2 lines from your code, the second line is incorrect:

FILE* output = fopen("image.txt","w");    //open the output file for writing
output = malloc(sizeof(data)*18);         //assign memory <= This is WRONG

The variable output is a FILE pointer. You should not allocate it using malloc. You should only use it if fopen returns it successfully, which means it has already been allocated by fopen.

This means you don't need this:

free(output); // This is also WRONG

Because this already freed the pointer's allocated data:

fclose(output);

answered Nov 18, 2020 at 12:29

Amin M. Dodin

1361 bronze badge

Sign up to request clarification or add additional context in comments.

Comments

cultab · Accepted Answer · 2020-11-18 12:21:52Z

2

fopen() returns a FILE pointer so when you try to allocate memory (using malloc, that also returns a pointer) you're replacing the pointer to the FILE with something that points to memory instead of the file. Removing

output = malloc(sizeof(data)*18);         //assign memory

should make it just work.

answered Nov 18, 2020 at 12:21

cultab

511 silver badge5 bronze badges

Collectives™ on Stack Overflow

c programming - Extracting string of data from file

2 Answers 2

Comments

Comments

Your Answer

Linked

Hot Network Questions

Collectives™ on Stack Overflow

2 Answers 2

Comments

Comments

Your Answer

Sign up or log in

Post as a guest

Linked

Related