Как считать массив из файла c

Или нужно как то по другому?

romajke, Раз уж вы выделяете память не для всех элементов массива, то и освобождать нужно только их: добавьте перед free условие:
if(arr[i])

romajke @romajke Автор вопроса

res2001, да. Это понятно, теперь код полностью корректно работает. Но не совсем понятна работа fscanf в данном контексте. Если я, например, не сразу считываю в элемент массива arr[], а предварительно записываю данные в созданную переменную типа char word[SIZE]:

char word[SIZE] while(!feof(fp))

то код уже работает не корректно.
Все элементы массива заполняются последней считанной строкой:

1 ->zet 2 ->zet 3 ->zet 4 ->zet 26 ->zet *** Error in `./tes': double free or corruption (out): 0x00007fffeebbd000 *** Aborted

Если я правильно понимаю, то элементы массива получают ссылку на word, и получают значение которое хранится по адресу переменной word. Но как это изменить - не пойму. Как передать в элемент массива именно значение word?

romajke @romajke Автор вопроса

Проблему решил, с помощью использования strcpy, но понимание не пришло 🙁
Буду очень благодарен, если объясните.

romajke, Операция
arr[k]=word;
Не приводит к копированию строки из одного буфера в другой - этим вы сохраняете адрес буфера word в массив. Поэтому у вас все элементы массива будут ссылаться на один и тот же буфер. Кроме того, что при этом вы получаете не правильный результат, у вас еще образуется утечка памяти, т.к. указатели, на выделенную с помощью malloc память, потеряны и во free вы передаете указатель на word, что при этом освобождается . не известно и поведение не предсказуемо.
Вы правильно решили проблему, явно копируя строку с помощью strcpy. Аналогично нужно делать и с любыми другими массивами только использовать функцию memcpy (с помощью memcpy можно копировать и строки то же, она в этом плане универсальнее чем strcpy).
Ну и на последок - в Си как таковых строк нет, строка - это массив байт, оканчивающийся байтом со значением 0 (нулевой байт то же является частью массива/строки), поэтому и думать нужно о них как о массивах. Различие в том, что под строкой вы понимаете какую-то самостоятельную единую сущность, а это не так - это набор сущностей (байт).

Вам не нужно сразу разбираться с памятью - зачем?

/* читаем из файла требуемое кол. строк ROWS или до конца файла EOF (что наступит раньше), формируем строку по конечному символу '\n', в файле, добавляем завершающий '\0' и полученную строку СИ записываем в двумерный массив по 1-й строке в каждый элемент массива. Затем массив выводим на консоль. Есть подсчет считанных строк. */ #include #include /* prototype exit() */ #include #define ROWS 7 #define COLS 20 void file_open(void); FILE * file_in; //FILE* file_out; char ch, arr_str[ROWS][COLS]; int j, i, count = 0; int main(int agrc, char * argv[]) < file_open(); //ch=getc(file_in); for(j = 0; j < ROWS; ++j)< for(i = 0; (ch=getc(file_in)) != '\n' && (ch != EOF); ++i) < arr_str[j][i] = ch; // заполнение строки >//конец внутреннего for if(ch == EOF) < puts("достигнут конец файла, выход."); //arr_str[j][i] = '\0'; fprintf(stdout, "count is: %i \n", count); break; >// конец if else < arr_str[j][i] = '\0'; ++count; >// конец else > // конец внешнего for fclose(file_in); //fclose(file_out); /* вывод массива на консоль */ fprintf(stdout, "Из файла считано %i строк, которые имеют вид:\n", count); puts(""); // пропуск строки for(j = 0; j < count; ++j)< fprintf(stdout, "%s\n", arr_str[j]); >puts(""); // пропуск строки puts("Программа завершена."); exit(EXIT_SUCCESS); > void file_open(void) < if( (file_in = fopen("file_in", "r")) == NULL)< fprintf(stdout, "%s\n", "не могу отрыть файл для чтения"); exit(EXIT_FAILURE); >/* if( (file_out = fopen("file_out", "w")) == NULL) < fprintf(stdout, "%s\n", "не могу отрыть файл для записи"); exit(EXIT_FAILURE); >*/ > /* Из файла считано 7 строк, которые имеют вид: I am a human 1 I am a 2 human`s 2 I am a 3 human 3 Nmjh## ___ 987 ht! 5555555555555555 66666666666666 77777777777777 Программа завершена. */

Ответы на вопрос 4

Используйте двумерный массив для строк:
char arr[26][SIZE];

И добавьте fclose(fp) после цикла while<. >.

Ответ написан более трёх лет назад
Нравится 1 3 комментария
romajke @romajke Автор вопроса
Все равно получается ошибка сегментации
romajke, а какая максимальная длинна строки в файле?
romajke @romajke Автор вопроса
Входные данные:

abc bcd cde def zet

sergio_nsk @sergio_nsk
fscanf замени на

size_t alloced = 0; getline(&arr[k], &alloced, fp);

getline с сравнением с -1 можно поместить в while.

Ответ написан более трёх лет назад
romajke @romajke Автор вопроса
getline это разве не C++?
sergio_nsk @sergio_nsk
getline - это не C++. Тебя разве забанили в Гугле?
romajke @romajke Автор вопроса
sergio_nsk @sergio_nsk
romajke, это расширение GNU.

team leader, system engineer, master of sports

/* читаем из файла требуемое кол. строк ROWS или до конца файла EOF * (что наступит раньше), формируем строку по конечному символу '\n' * в файле, затем добавляем завершающий '\0' и полученную строку СИ * записываем * в двумерный массив по 1-й строке в каждый элемент массива. * Затем массив выводим на консоль. Есть подсчет считанных строк. * См также ниже мой коммент */ /* file_in находится в каталоге программы и имеет вид: I am a human 1 I am a 2 human`s 2 I am a 3 human 3 Nmjh## ___ 987 ht! 5555555555555555 66666666666666 77777777777777 8888888888888888 */ #include #include /* prototype exit() */ #include #define ROWS 7 #define COLS 20 void file_open(void); FILE * file_in; char ch, arr_str[ROWS][COLS]; int j, i, count = 0; int main(int agrc, char * argv[]) < file_open(); for(j = 0; j < ROWS; ++j)< for(i = 0; (ch=getc(file_in)) != '\n' && (ch != EOF); ++i) < arr_str[j][i] = ch; // заполнение строки >//конец внутреннего for if(ch == EOF) < puts("достигнут конец файла, выход."); fprintf(stdout, "count is: %i \n", count); break; >// конец if else < arr_str[j][i] = '\0'; ++count; >// конец else > // конец внешнего for fclose(file_in); /* вывод массива на консоль */ fprintf(stdout, "Из файла считано %i строк, которые имеют вид:\n", count); puts(""); // пропуск строки for(j = 0; j < count; ++j)< fprintf(stdout, "%s\n", arr_str[j]); >puts(""); // пропуск строки puts("Программа завершена."); exit(EXIT_SUCCESS); > void file_open(void) < if( (file_in = fopen("file_in", "r")) == NULL)< fprintf(stdout, "%s\n", "не могу отрыть файл для чтения"); exit(EXIT_FAILURE); >> /* Вывод на консоль: Из файла считано 7 строк, которые имеют вид: I am a human 1 I am a 2 human`s 2 I am a 3 human 3 Nmjh## ___ 987 ht! 5555555555555555 66666666666666 77777777777777 Программа завершена. */

Ответ написан более трёх лет назад

посимвольная работа со строками имеет более жесткий контроль над процессом. Обратите внимание, что строк в файле больше, чем мы считали. Это задается #define ROWS 7 если задать к примеру 10, то получим:

достигнут конец файла, выход.
count is: 9
Из файла считано 9 строк, которые имеют вид:

I am a human 1
I am a 2 human`s 2
I am a 3 human 3
Nmjh## ___ 987 ht!
5555555555555555
66666666666666
77777777777777
8888888888888888

сообщение о конце файла (EOF тоже отлавливается). И в строке arr_str[j][i] = '\0'; формируется окончательно строка "си" завершающим символом '\0', что тоже важно. Можно использовать динамический массив или массив переменной длины(пропадает нужда следить и использовать free() , но придется точно знать количество строк в файле (высчитывать заранее). Также, при желании для уменьшения времени компиляции можно переписать программку в нотации указателей (компилятор все равно это будет делать сам) и вместо функций использовать указатели на них 🙂 .