how to read from a file into array in c programming

Question

I have a file called Players.txt which contains

Del Piero|3|Italy|Juventus Ronaldo|0|Portugal|Real Madrit

I want to read each ward into a separated array, like array players to contain
players[NUM_PLAYERS][NAME_LENGTH]={ Del Piero,Ronaldo}
and so on with the other arrays,

I know that it need to use a function called fgets and some string functions.

here is what i tried; My questions are : Is there any other approach to my problem, like to use some different programs, or string programs? And how to get the number of goals form this file and store them into the file

#include <string.h>
#include <stdlib.h>
#include <stdio.h>
#include <stddef.h>
#include <ctype.h>

#define NUM_PLAYERS 20
#define NAME_LENGTH 100
#define COUNTRY_NAME 20


int main (void)    
{    
    FILE *Players;    

    char player_name [NUM_PLAYERS][NAME_LENGTH] = {0};
    char team_name[NUM_PLAYERS][NAME_LENGTH] = {0};
    char country_name[NUM_PLAYERS][COUNTRY_NAME] = {0};

    char temp_buffer[NAME_LENGTH]={0};
    int goals_scored[NUM_PLAYERS] = {0};

    char *ptr1,*ptr2,*ptr3;
    int i;

    Players = fopen("Players.txt", "r");
    if (Players == NULL)
    {
        printf("File not found.\n");
    }
    else
    {
        i=0;
        while ((fgets(temp_buffer,sizeof(temp_buffer), Players) != NULL) && i < NUM_PLAYERS)

        {
            ptr1 = strchr(temp_buffer, '|');
            strncpy(player_name[i], temp_buffer, ptr1 - temp_buffer);
            printf("the name of the player is %s\n.", player_name[i]);
            i ++;

        }       
    }
  fclose(Players);

    return 0;
}

CapelliC · Accepted Answer · 2013-07-09 17:04:17Z

2

You could try to use fscanf, instead of fgets+strchr. Then you will gain simpler code and more safeness (now your code will easily overflow the buffer, with unpredictable results).

 if (fscanf(Players, " %*[^|]|%d|%*[^|]|%*s",
     NAME_LENGTH-1, player_name[i],
     goals_scored + i,
     NAME_LENGTH-1, team_name[i],
     NAME_LENGTH-1, country_name[i]) == 4) {
   ...
 }

note: the pattern is very specific to your data format, and from your question isn't clear to me what delimiter (if any) there is for country_name. Then the last string pattern is the usual %s, that stops at first space

edited Jul 9, 2013 at 17:04

answered Jul 9, 2013 at 16:50

CapelliC

60.1k5 gold badges50 silver badges94 bronze badges

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

3 Comments

user2512806 Over a year ago

Thank you for replying everybody! @CapelliC What do you mean by delimiter ?

CapelliC Over a year ago

@user2512806: it's the character separating fields: apparently from your question, should be |. fscanf, sscanf, scanf all share the same (complex) specification. They are a family.

Jonathan Leffler Over a year ago

Much better to use fgets() and sscanf() than trying to use fscanf().

Shumail · Accepted Answer · 2013-07-09 16:54:39Z

0

I suggest using fscanf instead of using fgets in your code.

For detailed fscanf reference and usage doc, see: http://www.cplusplus.com/reference/cstdio/fscanf/

answered Jul 9, 2013 at 16:54

Shumail

3,1534 gold badges30 silver badges39 bronze badges

1 Comment

Jonathan Leffler Over a year ago

I suggest using sscanf() with fgets() and not using fscanf(); the error reporting is harder, much harder, with fscanf().

cmaster - reinstate monica · Accepted Answer · 2013-07-09 17:16:24Z

I would recommend mmapping: That way you get the entire contents of the file directly into addressable memory, allowing you to examine the data at your leisure.

As an added bonus, you can simply cut it into individual strings (replacing the '|' and ' ' chars with null bytes) without modifying the file, if you specify MAP_PRIVATE in the call to mmap(). This saves you the necessity to copy the strings. The rest is a simple matter of building an index structure into the data.

This is the skeleton of what I use:

const char* mmapInputFile(const char* path, int kernelAdvice, off_t* outLength) {
    //Open the file.
    int file = open(path, O_RDONLY);
    if(file == -1) /* error handling */;
    *outLength = lseek(file, 0, SEEK_END);  //Get its size.

    //Map its contents into memory.
    const char* contents = (const char*)mmap(NULL, *outLength, PROT_READ, MAP_PRIVATE, file, 0);
    if((intmax_t)contents == -1) /* error handling */;
    if(madvise((void*)contents, *outLength, kernelAdvice)) /* error handling */;

    //Cleanup.
    if(close(file)) /* error handling */;
    return contents;
}

Note that it is not necessary to keep the file open while the mapping exists.

Performancewise this is probably the best you can achieve, because the entire process can happen without making a single copy of the data, which saves both CPU time and memory.

Collectives™ on Stack Overflow

how to read from a file into array in c programming

3 Answers 3

3 Comments

1 Comment

Comments

Hot Network Questions

Collectives™ on Stack Overflow

3 Answers 3

3 Comments

1 Comment

Comments

Related