Версия для печати темы
Нажмите сюда для просмотра этой темы в оригинальном формате
Форум программистов > C++ Builder > Как посчитать количество строк в файле?


Автор: Yuran89 2.10.2010, 18:27
Я считаю количество строк в текстовом файле следующим образом:
Код

FILE *f;                   
       int count = 0;                 
       char str[256];                    
       f = fopen("config/abonents.txt","r");
       while(!feof(f))                        
         {                                     
          if(fgets(str,10,f)) count++;          
         }                                          
       fclose(f);  


count  показывает количество строк. Странно то что я вытянул этот код из своей старой программки, где он нормально работал. А теперь пытаюсь использовать его в другой программе и часто он показывает на несколько строк больше чем надо. В чем может быть дело?

Добавлено через 7 минут и 58 секунд
сразу же назрел ответ:
вместо
Код

 if(fgets(str,10,f)) count++;

надо писать:
Код

 if(fgets(str,100,f)) count++;


Кажется так. потому что  у меня попадались строчки длиннее 10 символов, а вот длиннее 100 символов точно не попадется.

Автор: EEEEEE 2.10.2010, 19:34
Может стоит читать посимвольно. И искать два символа,  идущих один за одним, в шестнадцаричной системе счисления 0x0D & 0x0A.

Автор: Yuran89 3.10.2010, 00:14
это вот эти символы:" \n " ?

Автор: Platov 4.10.2010, 08:02
А если попробовать вот так:
Код

TStringList *f=new TStringList;
f->LoadFromFile(config\\abonents.txt);
count=f->Count;
delete f;

Если файл не очень большой, то прокатит.

Автор: t4aynik 5.10.2010, 12:53
Цитата(Yuran89 @  2.10.2010,  18:27 Найти цитируемый пост)
Кажется так. потому что  у меня попадались строчки длиннее 10 символов, а вот длиннее 100 символов точно не попадется.

Нельзя делать такое допущение, строка может быть гораздо длинней - текст разный бывает. Бестрее всего и надежнее возможно будет подсчет \n

Powered by Invision Power Board (http://www.invisionboard.com)
© Invision Power Services (http://www.invisionpower.com)