fgets не читает полную строку в C

Question

У меня есть файл, который содержит data.csvfloat типа данных:

---

0.22,0.33,0.44

0.222,0.333,0.444

---

Мне нужно прочитать этот файл в двухмерном динамическом массиве. Но я не могу прочитать полную строку с fgets. Не знаете почему?

Вот мой C код, который я использовал на Ubuntu:

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

int main(int argc, char *argv[]) { 
    FILE *fp; 
    float **data;  
    int i,j,rows=2,cols=3; 
    char * token; 
    fp=fopen("data.csv","r"); 
    if(fp==NULL) { 
      fprintf(stderr,"Can't open input file"); 
      exit(1); 
    } 

    data= malloc(rows * sizeof(float*)); 
    char *rowbuffer=malloc(cols * (sizeof(float)+sizeof(char))); 
    i=0; 
    while(fgets(rowbuffer,sizeof(rowbuffer),fp) !=NULL) {  
     data[i] = malloc(cols * sizeof(float));  
     j=0; 
     printf("\n %s",rowbuffer); 
     for (token = strtok(rowbuffer,","); token != NULL; token = strtok(NULL, ",")) { 
      data[i][j++] = atof(token); 
      /*printf("%s",token);*/ 
     } 
     i++; 
    } 
    free(rowbuffer); 
    for(i = 0; i < rows; i++) 
     free(data[i]); 
    free(data); 
    fclose(fp); 
} 

Выход как:

0.22,0.

33,0.44

0.222,0

Ошибка в `./test ': двойной бесплатно или повреждение (уходит): 0x0000000000adf270

Прерывание (сбрасывание ядра)

Может кто-нибудь сказать, почему эта ошибка? :( Или есть лучший способ, чтобы прочитать этот тип файла данных

Answer 1

Ваш кодирования проблема заключается в:

fgets(rowbuffer,sizeof(rowbuffer),fp) 

sizeof(rowbuffer) даст вам только размер указателя, а не размер из памяти, выделенной для указателя.

чтобы решить эту проблему, необходимо поставить правильный размер выделенной памяти [cols * (sizeof(float)+sizeof(char)] для fgets().

Вашей логической проблема заключается в:

Вы предположить, что печататься represntation из значения float будет принимать тот же объем памяти, как это требуется для этого из float переменных. Нет, это не так. В печатном представлении каждая цифра (включая десятичную точку и любую начальную или конечную 0 после десятичной) будет потреблять по одному байту памяти. Это следует учитывать при распределении памяти для целевого буфера.

Answer 2

Один вопрос здесь:

char *rowbuffer=malloc(cols * (sizeof(float)+sizeof(char))); 

sizeof(float) является размер, что поплавок использует в памяти, а не в текстовом представлении. При чтении из файлов вы должны выделить буфер, содержащий целую строку в текстовом формате. В вашем случае хорошая ставка может быть следующее:

int bufsize = cols * (3 + DBL_MANT_DIG - DBL_MIN_EXP + 1) + 1; 

(см это почему это значение и то, что вам нужно #include:. What is the maximum length in chars needed to represent any double value? Задний + 1 должен учитывать символ новой строки, которая fgets() делает читать и включите в буфер.)

Но это предполагает, что во входном файле нет ошибок форматирования, поэтому вам может потребоваться добавить дополнительное значение для этого значения.

После того как вы это значение, использовать его как в malloc() и fgets():

char *rowbuffer=malloc(bufsize); 
i=0; 
while(fgets(rowbuffer,bufsize,fp) !=NULL) { 
... 

На стороне записки, ваш входной файл выглядит, как он может быть лучше читать с помощью scanf().