Посимвольное чтение из файла

@undestroyer · Регистрация: 24.03.2013

Author24 — интернет-сервис помощи студентам

Всем добрый вечер (ночь)! Так сложилась ситуация что нужно мне посимвольно считать текст из файла fin в массив str[i][j], причем считывать надо только ДО символа переноса строки \n(включительно). Скажите пожалуйста почему не работает мой фрагмент (где-то зацикливается):

C++

while (!feof(fin))
    {
        do
        {
            str[i][j]=fgetc(fin);
            j++;
        }
        while (str[i][j]!='\n');
        i++;
        j=0;
    }

Добавлено через 20 часов 24 минуты
По ходу выполнения чтения в str[0][0] записывается ноль и больше ничего не происходит. В чем косяк?

@stima · 09.05.2013, 16:42

C++

    do
        {
            str[i][j]=fgetc(fin);
            j++; //<----------- Подумайте над этим) Подсказка, а именно о местоположении.
        }
        while (str[i][j]!='\n');

@metaluga145 · 09.05.2013, 16:45

C++

while (!feof(fin))
    {
        do
        {
            str[i][j]=fgetc(fin);
            j++;
        }
        while (str[i][j-1]!='\n');
        i++;
        j=0;
    }

@undestroyer · 09.05.2013, 16:57 **[ТС]**

Сообщение от stima

C++

    do
        {
            str[i][j]=fgetc(fin);
            j++; //<----------- Подумайте над этим) Подсказка, а именно о местоположении.
        }
        while (str[i][j]!='\n');

Насколько я понимаю тут все логично. заполнили 1 ячейку -> перешли на другую

Сообщение от metaluga145

C++

while (!feof(fin))
    {
        do
        {
            str[i][j]=fgetc(fin);
            j++;
        }
        while (str[i][j-1]!='\n');
        i++;
        j=0;
    }

аналогично зацикливается

Главная суть: Мне нужно сохранить текст из файла сохраняя форматирование (отдельная строка текста в отдельной строке массива[количество символов в строке меньше 80, но в каждой строке по-разному]). Возможно вы предложите лучшее решение

@metaluga145 · 09.05.2013, 17:04

Вам поможет функция getline

@stima · 09.05.2013, 17:12

Сообщение от undestroyer

Насколько я понимаю тут все логично. заполнили 1 ячейку -> перешли на другую

Какую вы ячейку проверяете? Дууумайте.

@undestroyer · 09.05.2013, 17:22 **[ТС]**

Сообщение от stima

Какую вы ячейку проверяете? Дууумайте.

какую только что заполнил. do ... while сначала выполняет блок do{}, потом проверяет условие в while. Заполнил ячейку очередным символом, если это символ переноса строки, то перехожу на другую строку (i++), если нет - продолжаю получать символы в текущую строку. Правильно?

@metaluga145 · 09.05.2013, 17:25

Сообщение от undestroyer

какую только что заполнил. do ... while сначала выполняет блок do{}, потом проверяет условие в while. Заполнил ячейку очередным символом, если это символ переноса строки, то перехожу на другую строку (i++), если нет - продолжаю получать символы в текущую строку. Правильно?

Нет. Сначала выполнится i++, а потом уже проверится условие

@stima · 09.05.2013, 17:29

1. Прочитали символ. Сохранили его в i(0)j(0)
2. Инкрементируем j. Теперь там 1.
3. Проверили ячейку i(0)j(1).

Вопрос: Ну и что мы проверяем?
Вопрос 2: Это С или C++? Если С++ то http://www.cplusplus.com/refer... eam/tellg/.

@undestroyer · 09.05.2013, 17:37 **[ТС]**

Сообщение от metaluga145

Нет. Сначала выполнится i++, а потом уже проверится условие

Спасибо, понял, но:

C++

while (!feof(fin))
    {
        j=0;
        do
        {
            str[i][j]=fgetc(fin);
            j++;
        }
        while (str[i][j-1]!='\n');
        i++;
    }

тоже где-то зацикливается.
Текст в тестируемом файле:

12354
4575
878

@metaluga145 · 09.05.2013, 17:42

попробуйте вместо feof написать EOF

@stima · 09.05.2013, 17:49

Вы не хотите думать). Ладно:

C

int read_all(FILE* in, char** buff)
{
   if (!in) return 0;
   if (*buff) free(buff);
 
   int count = fseek (in, 0 , SEEK_END);
   *buff= (char*)malloc(count * sizeof(char));
 
   if (!*buff) return 0;
 
   fread (*buff, sizeof(char), count , in);
 
   return count;
}

@undestroyer · 09.05.2013, 18:04 **[ТС]**

Сообщение от metaluga145

попробуйте вместо feof написать EOF

точно не то. ругается при компилировании (expression must have (pointer-to-) function type).
сейчас попробую переписать на getline

Добавлено через 15 минут

Сообщение от stima

Вы не хотите думать). Ладно:

C

int read_all(FILE* in, char** buff)
{
   if (!in) return 0;
   if (*buff) free(buff);
 
   int count = fseek (in, 0 , SEEK_END);
   *buff= (char*)malloc(count * sizeof(char));
 
   if (!*buff) return 0;
 
   fread (*buff, sizeof(char), count , in);
 
   return count;
}

спасибо за код, возможно его и возьму, только не понимаю для чего вы мне возвращаете count...

@stima · 09.05.2013, 18:05

Это стандартная практика. Например если вы дальше будете записывать в файл? Что обратно цикл? или сразу блок в count charов

@metaluga145 · 09.05.2013, 18:13

stima, у меня пару вопросов по Вашему коду.
http://www.cplusplus.com/reference/cstdio/fseek/
fseek возвращает 0, если все прошло удачно. Да? что делает эта строка ?

C++

1	int count = fseek (in, 0 , SEEK_END);

Добавлено через 2 минуты
Кроме этого надо было сохранить каждую строку файла в отдельную строку. Вместо этого все что было в файле, оказалось в буфере. Как разобрать где там какая строка?

@stima · 09.05.2013, 18:16

Вы правы ... используйте это ftell(in);

@undestroyer · 09.05.2013, 18:19 **[ТС]**

Вопрос всем: почему нужно использовать специальные функции для получения строки? почему нельзя посимвольно получать данные из файла и проверять очередной символ на перенос строки?

@stima · 09.05.2013, 18:22

C

for (int i = 0, c = 0; (c = fgetc(fin)) != EOF; ++i)
{
     int j = 0;
     for ( ;str[i][j] != '\n'; ++j)
     {
          str[i][j] = c;
     }
     str[i][j] = 0;
}

п.с. не проверял, но должно работать)

@metaluga145 · 09.05.2013, 18:23

Сообщение от undestroyer

Вопрос всем: почему нужно использовать специальные функции для получения строки? почему нельзя посимвольно получать данные из файла и проверять очередной символ на перенос строки?

Зачем изобретать велосипед?

@undestroyer · 09.05.2013, 18:29 **[ТС]**

Сообщение от metaluga145

Зачем изобретать велосипед?

когда мне нужно было провести какую-то обработку текста я всегда делал это "налету" посимвольно считывая из потока и сразу проводя нужные изменения. сейчас впервые приходится брать текст из файлов. хочется сделать по-привычке)

@metaluga145 244 / 245 / 38 Регистрация: 08.04.2013 Сообщений: 927
	09.05.2013, 17:42	11
	попробуйте вместо feof написать EOF 0

@metaluga145 244 / 245 / 38 Регистрация: 08.04.2013 Сообщений: 927
	09.05.2013, 17:04	5
	Вам поможет функция getline 0

@stima 503 / 352 / 94 Регистрация: 22.03.2011 Сообщений: 1,112
	09.05.2013, 17:12	6
	Сообщение от undestroyer Насколько я понимаю тут все логично. заполнили 1 ячейку -> перешли на другую Какую вы ячейку проверяете? Дууумайте. 0

@undestroyer 0 / 0 / 0 Регистрация: 24.03.2013 Сообщений: 53
	09.05.2013, 17:22 [ТС]	7
	Сообщение от stima Какую вы ячейку проверяете? Дууумайте. какую только что заполнил. do ... while сначала выполняет блок do{}, потом проверяет условие в while. Заполнил ячейку очередным символом, если это символ переноса строки, то перехожу на другую строку (i++), если нет - продолжаю получать символы в текущую строку. Правильно? 0

@metaluga145 244 / 245 / 38 Регистрация: 08.04.2013 Сообщений: 927
	09.05.2013, 17:25	8
	Сообщение от undestroyer какую только что заполнил. do ... while сначала выполняет блок do{}, потом проверяет условие в while. Заполнил ячейку очередным символом, если это символ переноса строки, то перехожу на другую строку (i++), если нет - продолжаю получать символы в текущую строку. Правильно? Нет. Сначала выполнится i++, а потом уже проверится условие 0

@stima 503 / 352 / 94 Регистрация: 22.03.2011 Сообщений: 1,112
	09.05.2013, 17:29	9
	1. Прочитали символ. Сохранили его в i(0)j(0) 2. Инкрементируем j. Теперь там 1. 3. Проверили ячейку i(0)j(1). Вопрос: Ну и что мы проверяем? Вопрос 2: Это С или C++? Если С++ то http://www.cplusplus.com/refer... eam/tellg/. 0

@stima 503 / 352 / 94 Регистрация: 22.03.2011 Сообщений: 1,112
	09.05.2013, 18:05	14
	Это стандартная практика. Например если вы дальше будете записывать в файл? Что обратно цикл? или сразу блок в count charов 0

@stima 503 / 352 / 94 Регистрация: 22.03.2011 Сообщений: 1,112
	09.05.2013, 18:16	16
	Вы правы ... используйте это ftell(in); 0

@undestroyer 0 / 0 / 0 Регистрация: 24.03.2013 Сообщений: 53
	09.05.2013, 18:19 [ТС]	17
	Вопрос всем: почему нужно использовать специальные функции для получения строки? почему нельзя посимвольно получать данные из файла и проверять очередной символ на перенос строки? 0

@metaluga145 244 / 245 / 38 Регистрация: 08.04.2013 Сообщений: 927
	09.05.2013, 18:23	19
	Сообщение от undestroyer Вопрос всем: почему нужно использовать специальные функции для получения строки? почему нельзя посимвольно получать данные из файла и проверять очередной символ на перенос строки? Зачем изобретать велосипед? 0

@undestroyer 0 / 0 / 0 Регистрация: 24.03.2013 Сообщений: 53
	09.05.2013, 18:29 [ТС]	20
	Сообщение от metaluga145 Зачем изобретать велосипед? когда мне нужно было провести какую-то обработку текста я всегда делал это "налету" посимвольно считывая из потока и сразу проводя нужные изменения. сейчас впервые приходится брать текст из файлов. хочется сделать по-привычке) 0