Программа на С++ не работает. Что я делаю не так?

Question

[Lililoli]

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

#include <fstream>
#include <iostream>
using namespace std;

int main() {
    char *c=",";
    string str;
    int j,t;
ifstream fin;
fin.open("txt.txt");
t=0;
while (!fin.eof()) {
    j=0;
    getline(fin, str);
        for(int i=0;i<str.length();i++){
            if (*c==str[i]) j++;
            cout<<"Кол-во запятых в строке: "<<t+1<<" :"<<j<<endl;
            }
    t++;
}
fin.close();
return 0;
}

Что я делаю не так?

Comments

[Тимур Покровский]

что значит "не работает"?

[Lililoli]

Тимур Покровский, если в файле записано:

123
,
,,,

программа выдаёт:

Кол-во запятых в строке: 1 :0
Кол-во запятых в строке: 1 :0
Кол-во запятых в строке: 1 :0
Кол-во запятых в строке: 2 :1
Кол-во запятых в строке: 3 :1
Кол-во запятых в строке: 3 :2
Кол-во запятых в строке: 3 :3

[Тимур Покровский]

Lililoli, ну так выводите результат не после просмотра каждого символа, а после просмотра строки

[Lililoli]

Тимур Покровский, ну я и отсталый... спасибо)

Answer 1

[xandox]

#include <iostream>
#include <fstream>
#include <algorithm>

int main(int, char**) {
    char symbol = ',';
    std::ifstream input_file("txt.txt");
    int line_number = 0;
    std::string current_line;
    while (std::getline(input_file, current_line)) {
        auto symbols_in_string = std::count(current_line.begin(), current_line.end(), symbol);
        std::cout << "'" << symbol << "' in line " << line_number + 1 << ": " << symbols_in_string << std::endl;
        line_number += 1;
    }
    return 0;
}