Количество пустых строк кода (SLOC)

[Holocaust]

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

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

int main()
{
    setlocale(LC_ALL, "Russian");
    int logicstring = 0;
    int emptystring = 0;
    int commentstring = 0;
    float t;
    int sk = 0;
    fstream in("1.txt");
	if (!in)	//проверка на существование файла
	{
		cout << "\nНевозможно открыть файл исходного текста программы!...\n";
		system("pause");
        exit(0);
	}
	string tmp;
    while(!in.eof())
    {
        getline(in,tmp);
        if(tmp.find("//")!= string::npos)
            commentstring++;
        if(tmp.find(" ")!= string::npos)
            emptystring++;
        if(tmp.find(";")!= string::npos)
            logicstring++;
        if(tmp.find("switch")!= string::npos)
            logicstring++;
        if(tmp.find("case")!= string::npos)
            logicstring++;
        if(tmp.find("if")!= string::npos)
            logicstring++;
        if(tmp.find("break")!= string::npos)
            logicstring++;
        sk++;
    }
    in.close();
    t = (sk / 100.0f * commentstring);
    cout<<"Кол-во логических строк кода: "<<logicstring;
    cout<<"\nКол-во пустых строк кода: "<<emptystring;
    cout<<"\nКол-во коментариев: "<<commentstring;
    cout<<"\nпроцент коментариев:"<<t<<"\n\n\n";
}

[Stilet]

if(tmp.find(" ")!= string::npos)

А ты зря решил, что пустая строка равна одному пробелу...
На твоем месте я бы поступал так:
1) Развернул текст в массив строк
2) Подсчитывал очередную строку из массива предварительно TRIM-мировав ее. Если она окажется после подстрижки пустой, значит плюсплюспоймали.

[Holocaust]

Я совсем бигинер , подскажешь как развернуть текст в массив строк ? Дальше сам соображу .

[Stilet]

Дак чего тут подсказывать? Нужно написать некую функцию типа

bool isEmpty(string s){
 for(i=0;i<s.length();i++) if(s[i]!=' ') return false; return true;
}

И соответственно вместо if(tmp.find(" ")!= string::npos) написать if(isEmpty(tmp))

P.S. Кста если ты бигинер, то возьми задачку попроще.

[Holocaust]

Спасибо , всё работает

[Stilet]

смог бы объяснить строки этой части кода:
1)i<s.length()

2)return false; - я понимаю что , возвращает ложь , но я не понимаю смысла в этой строке.

3)return true;

Отвечу на твои вопросы тут, ибо ЛС для персонального срача... пардонте, конструктивного диалога тет-а-тет )))
1) i<s.length() - это условие цикла. Цикл должен пройтись по всей строке, считанной из файла

2) Смысл этого не в возвращении а в условии: if(s[i]!=' ') означает "Если в строке найден непробельный символ". Т.е. если найден такой символ - строка не пустая.

3) Если будет найдет непробел - цикл досрочно завершиться как и функция. Иначе return true укажет, что эта строка не содержит ничего кроме пробелов.