Как сохранить информацию из файла в памяти в C - PullRequest
Новая
       47

Как сохранить информацию из файла в памяти в C

0 голосов
/ 12 апреля 2020

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

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

int main()
{
    printf("Hello world!\n");
    int x = 43; // number of lines in my text file
    FILE *fptr;
    struct values *valuesPtr = malloc(sizeof(struct values) * x);

    if (!valuesPtr)
        return -1;

    if((fptr = fopen("energy.txt", "r")) == NULL) {
        printf ("Error opening file ");
        return 0;
    }

    while (fptr != NULL) {
        for(int i = 0; i < x; i++ ) {
            fscanf(fptr, "%s %s %d", valuesPtr[i].start_vertex, valuesPtr[i].destination_vertex, &valuesPtr[i].num);
            printf("\nStart vertex: %s \nDestination vertex: %s \nWeight: %d\n\n", valuesPtr[i].start_vertex, valuesPtr[i].destination_vertex, valuesPtr[i].num);
        }
        free(valuesPtr);
    }
    fclose(fptr);

    return 0;
}

Мой файл struct.h имеет следующий код: 

struct values{
    int num;
    char start_vertex[250];
    char destination_vertex[250];
} x;

Энергия Файл .txt имеет следующее содержимое: 

York    Hull    60
Leeds   Doncaster   -47
Liverpool   Nottingham  161
Manchester  Sheffield   61
Reading Oxford  -43
Oxford  Birmingham  103
Birmingham  Leicester   63
Liverpool   Blackpool   79
Carlisle    Newcastle   92
Nottingham  Birmingham  77
Leeds   York    39
Glasgow Edinburgh   74
Moffat  Carlisle    65
Doncaster   Hull    76
Northampton Birmingham  90
Leicester   Lincoln 82
Sheffield   Birmingham  122
Lincoln Doncaster   63
Sheffield   Doncaster   29
Bristol Reading 130
Hull    Nottingham  145
Blackpool   Leeds   116
Birmingham  Bristol 139
Manchester  Leeds   64
Carlisle    Blackpool   140
Leicester   Northampton -61
Newcastle   York    135
Glasgow Moffat  -28
Leicester   Sheffield   100
Carlisle    Liverpool   -30
Birmingham  Manchester  129
Oxford  Bristol 116
Leeds   Hull    89
Edinburgh   Carlisle    154
Nottingham  Sheffield   61
Liverpool   Manchester  56
Carlisle    Glasgow 50
Sheffield   Lincoln 74
York    Doncaster   55
Newcastle   Edinburgh   177
Leeds   Sheffield   53
Northampton Oxford  68
Manchester  Carlisle    20

Мне сказали использовать функцию getline() и sscanf() для хранения данных из файла. Но я не уверен, как мне это сделать.

1 Ответ

1 голос
/ 12 апреля 2020

В вашем коде есть несколько проблем

Имея 

    if((fptr = fopen("energy.txt", "r")) == NULL)
    {
        printf ("Error opening file ");
        return 0;
    }
    while (fptr != NULL)

, вы не можете достичь , тогда как , если fptr равно NULL, и значение fptr никогда не изменяется в теле , тогда как , так что каждый является бесконечным l oop

Другая проблема возникает после для вы освобождаете valuesPtr , поэтому со второго хода вы будете писать в освобожденную память с неожиданной памятью. Весьма вероятно, что , тогда как просто бесполезен и может быть удален (но, конечно, для ).

Предположим, что while fini sh вы пытаетесь освободить fptr , но это не способ закрыть описание, вам нужно позвонить fclose (fptr) вместо free , ваш бесплатный имеет неопределенное поведение

Я рекомендую вам проверить возвращаемое значение с помощью fscanf , в настоящее время вы не можете знать, успешно это или нет. Обратите внимание, что fscanf также может записывать из полей, содержащих строку, поскольку вы не ограничиваете количество символов для чтения.

Если вы не можете открыть файл, вы можете указать причину, используя perror : 

perror("Error opening file");

Предложение после редактирования: 

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

typedef struct value {
  int num;
  char start_vertex[250];
  char destination_vertex[250];
} value;  

int main()
{
  const int nLines = 43; // number of lines in my text file
  FILE * fptr;
  value * valuesPtr = malloc(sizeof(value) * nLines);

  if (!valuesPtr) {
    puts("cannot allocate memory");
    return -1;
  }

  if((fptr = fopen("energy.txt", "r")) == NULL)
  {
    perror("Error opening file");
    return -1;
  }

  for(int i = 0; i < nLines; i++ )
  {
      if (fscanf(fptr, "%249s %249s %d",
                 valuesPtr[i].start_vertex, 
                 valuesPtr[i].destination_vertex, 
                 &valuesPtr[i].num) != 3) {
        printf("errored file line %d\n", i);
        break;
      }

      printf("\nStart vertex: %s \nDestination vertex: %s \nWeight: %d\n\n",
             valuesPtr[i].start_vertex, valuesPtr[i].destination_vertex, valuesPtr[i].num);
  }
  free(valuesPtr);
  fclose(fptr);

  return 0;
}

Компиляция и исполнение: 

pi@raspberrypi:/tmp $ gcc -Wall l.c
pi@raspberrypi:/tmp $ ./a.out

Start vertex: York 
Destination vertex: Hull 
Weight: 60


Start vertex: Leeds 
Destination vertex: Doncaster 
Weight: -47


Start vertex: Liverpool 
Destination vertex: Nottingham 
Weight: 161


Start vertex: Manchester 
Destination vertex: Sheffield 
Weight: 61


Start vertex: Reading 
Destination vertex: Oxford 
Weight: -43


Start vertex: Oxford 
Destination vertex: Birmingham 
Weight: 103


Start vertex: Birmingham 
Destination vertex: Leicester 
Weight: 63


Start vertex: Liverpool 
Destination vertex: Blackpool 
Weight: 79


Start vertex: Carlisle 
Destination vertex: Newcastle 
Weight: 92


Start vertex: Nottingham 
Destination vertex: Birmingham 
Weight: 77


Start vertex: Leeds 
Destination vertex: York 
Weight: 39


Start vertex: Glasgow 
Destination vertex: Edinburgh 
Weight: 74


Start vertex: Moffat 
Destination vertex: Carlisle 
Weight: 65


Start vertex: Doncaster 
Destination vertex: Hull 
Weight: 76


Start vertex: Northampton 
Destination vertex: Birmingham 
Weight: 90


Start vertex: Leicester 
Destination vertex: Lincoln 
Weight: 82


Start vertex: Sheffield 
Destination vertex: Birmingham 
Weight: 122


Start vertex: Lincoln 
Destination vertex: Doncaster 
Weight: 63


Start vertex: Sheffield 
Destination vertex: Doncaster 
Weight: 29


Start vertex: Bristol 
Destination vertex: Reading 
Weight: 130


Start vertex: Hull 
Destination vertex: Nottingham 
Weight: 145


Start vertex: Blackpool 
Destination vertex: Leeds 
Weight: 116


Start vertex: Birmingham 
Destination vertex: Bristol 
Weight: 139


Start vertex: Manchester 
Destination vertex: Leeds 
Weight: 64


Start vertex: Carlisle 
Destination vertex: Blackpool 
Weight: 140


Start vertex: Leicester 
Destination vertex: Northampton 
Weight: -61


Start vertex: Newcastle 
Destination vertex: York 
Weight: 135


Start vertex: Glasgow 
Destination vertex: Moffat 
Weight: -28


Start vertex: Leicester 
Destination vertex: Sheffield 
Weight: 100


Start vertex: Carlisle 
Destination vertex: Liverpool 
Weight: -30


Start vertex: Birmingham 
Destination vertex: Manchester 
Weight: 129


Start vertex: Oxford 
Destination vertex: Bristol 
Weight: 116


Start vertex: Leeds 
Destination vertex: Hull 
Weight: 89


Start vertex: Edinburgh 
Destination vertex: Carlisle 
Weight: 154


Start vertex: Nottingham 
Destination vertex: Sheffield 
Weight: 61


Start vertex: Liverpool 
Destination vertex: Manchester 
Weight: 56


Start vertex: Carlisle 
Destination vertex: Glasgow 
Weight: 50


Start vertex: Sheffield 
Destination vertex: Lincoln 
Weight: 74


Start vertex: York 
Destination vertex: Doncaster 
Weight: 55


Start vertex: Newcastle 
Destination vertex: Edinburgh 
Weight: 177


Start vertex: Leeds 
Destination vertex: Sheffield 
Weight: 53


Start vertex: Northampton 
Destination vertex: Oxford 
Weight: 68


Start vertex: Manchester 
Destination vertex: Carlisle 
Weight: 20

pi@raspberrypi:/tmp $

Я взял можно переименовать значения в значение , потому что struct сохраняет один элемент, а не несколько, в любом случае я рекомендую переименовать его в нечто более точное, значение очень общий. Я также использовал typedef , чтобы не нуждаться в struct везде, где он используется.

Посмотрите детали, например, для строки, которую я использую в формате % 249s , поскольку массивы содержат 250 символов (я удалил 1 для последнего нулевого символа), и, конечно, я проверяю, что он читает 3 элемента.

...