Подсчет строк в файле в Linux.

[Алексей Емеля]

Помню в винде писал прогу, где надо было посчитать количество строк в файле (у меня это получилось). Перешел на Linux Mint, переписал алгоритм, но в терминале при запуске выдает Segmentatin Fault. Никак не пойму в чем проблема. Помогите с этим. Скажите что не так. Лично я думаю что это изза EOF. В общем вот исходник:

#include<iostream>
#include<stdio.h>
using namespace std;

int _n( FILE *f ) // Функция, считающая строки в файле
{
int k = 0;
char a;// Каждый символ в файле
do
{
a = fgetc(f);
if( a == '\n') // Если символ перевода строки то
k++; // На одну строку больше
} while( a != EOF ); // До тех пор, пока файл не закончится
return k;
}

int main()
{
int k;
FILE *f = fopen("base.txt", "r");
k = _n(f) / 7;
cout << k << endl;
return 0;
}

Компилировал в IDE Geany.

[Granus]

Попробуй у a сделать тип int. Где-то читал что EOF не влазит в char)

[Алексей Емеля]

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

Попробуй у a сделать тип int. Где-то читал что EOF не влазит в char)

Изменил. То же самое сообщение пишет.

*********************************** *********************

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

[netrino]

Ммм, а файл base.txt есть рядом с программой? Добавьте проверку:

FILE* f = fopen("base.txt", "r");

if (f == NULL) {
  perror("Can't open file");
  return -1;
}
..

[Алексей Емеля]

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

Ммм, а файл base.txt есть рядом с программой? Добавьте проверку:

FILE* f = fopen("base.txt", "r");

if (f == NULL) {
  perror("Can't open file");
  return -1;
}
..

Мне ваш код в программу даже не пришлось вставлять - я сразу вспомнил, что полный вариант программы в другой папке, где лежит этот файл Спасибо

[pproger]

a = fgetc(f);

узнай размер файла, выдели память, равную размеру, считай весь файл разом, потом уже считай кол во \n
либо читай построчно (getline)

а если все-таки пишешь на плюсах, то можно как то так:

#include <iostream>
#include <string>
#include <algorithm>
#include <iterator>
#include <functional>
#include <fstream>
#include <memory>

using namespace std;

int main(int argc, char *argv[])
{
    cout << count_if(istreambuf_iterator<char>(*auto_ptr<fstream>(new fstream(argv[1]))),
            istreambuf_iterator<char>(),
            bind2nd(equal_to<char>(), '\n')) << endl;
}

но так же читает по одному символу