Почему эта программа не показывает первую строку снова и снова?

Question

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

char *readLine(FILE *inFile) //Simply reads line in a text file till "\n" 
{ 
    char *line = realloc(NULL, 1); 
    char c; 
    int i=0; 
    while (!feof(inFile)) 
    { 
     c = fgetc(inFile); 
     if (ferror(inFile)) printf("Error reading"); 
     if (c == 10) 
      { 
       realloc(line,i+1); 
       line[i]= 10; 
       break; 
      } 
     realloc(line, i+1); 
     line[i++] = c; 
    } 
    return line; 
} 

int main(int argc,char **argv) 
{ 
    FILE *inFile; 
    inFile = fopen("testFile","r"); 
    printf("%s",readLine(inFile)); 
    printf("%s",readLine(inFile)); 
    printf("%s",readLine(inFile)); 
    return 0; 
} 

Если содержимое TestFile является: -

abc 
def 
ghi 

Три PRINTF заявления должны показать "ABC" три раза .. Но выход есть: -

abc 
def 
ghi 

Я знаю, что я неправильно в концепции. Помощь Pls.

Answer 1

Но я передаю указатель файла по значению. Поэтому я должен снова и снова получать выходные данные «abc»

А, я понимаю ваше замешательство.

Файл Указатель указывает только на фактическую структуру файла. Состояние, такое как текущее смещение, не является частью указателя, но является частью внутренней структуры.

Другой способ подумать об этом заключается в том, что фактическим объектом, представляющим файл, является FILE. Чтобы получить семантику pass-by-reference, вы передаете указатель на объект. Поскольку вы проходите по ссылке, каждая строка поднимается туда, где последний остановился.

Answer 2

Неверное использование realloc().

realloc(line,i+1); // wrong 

// OK 
void *new_line = realloc(line,i+1); 
if (!new_line) 
{ 
    free(line); 
    return NULL; 
} 
line = new_line; 

Поскольку line передается по значению, это не изменилось. Фактическая перераспределенная память находится в возвращаемом значении. Поэтому line остается неизменным снова и снова, и вы видите одну и ту же линию снова и снова. Редактировать: только что понял, что даже если это ошибка, это не то, что вызовет повторяющиеся строки. Остальные пункты остаются в силе.

Что еще хуже:

1. У вас есть утечка памяти путем потери вновь перераспределены указатель каждый раз.
2. Вы потенциально получаете доступ к освобожденной памяти, потому что старое значение line может стать недействительным после перераспределения, если оно было перераспределено в другой части кучи.
3. Вы перераспределяете память на каждый символ, что потенциально является дорогостоящей операцией.

Answer 3

fgetc() продвигает указатель файла (который находится «где находится следующий символ для чтения»). Вот как вы можете вызвать его в цикле и прочитать целую строку символов.

После того, как он продвигается мимо символа новой строки, он, естественно, переходит к следующему символу, который является началом следующей строки.

Вы можете изменить указатель файла с помощью функции fseek(). Например, вызов fseek(inFile, 0, SEEK_SET) приведет к сбросу его в начало файла, в результате чего следующий вызов fgetc() начнется с первого символа файла.