Problem z kodem - projekt z plikiem tekstowym

Question

Cześć, co jest nie tak z tym kodem?

		    file.open("mainFile.txt");
			string str;
			if (file.is_open()) {
				while (getline(file, str))
				{
					stringstream ss(str);
					string name, surname, trade;
					float salary;
					getline(ss, name, ' ');
					getline(ss, surname, ':');
					getline(ss, trade, '-');
					ss >> salary;
					createSimpleEmployer(name, surname, trade, salary);
				}
				file.close();
			}

Główny problem jest taki że kod w while się w ogóle nie wykonuje :(

Plik ma taki format:

Imię Nazwisko:xxxx-2987

 

Comments

Nie widzę reszty kodu, ale z tego co masz to działa. 

#include <iostream>
#include <fstream>
#include <string>
#include <sstream>

using namespace std;
int main() 
{
ifstream file;
file.open("mainFile.txt");
string str;
if (file.is_open()) {
    while (getline(file, str))
    {
        stringstream ss(str);
        string name, surname, trade;
        float salary;
        getline(ss, name, ' ');
        getline(ss, surname, ':');
        getline(ss, trade, '-');
        ss >> salary;
        cout<<name<<' '<<surname<<' '<<trade;
        //createSimpleEmployer(name, surname, trade, salary);
    }
    file.close();
}
}

Jedyny błąd jaki mogłaś zrobić to ustawić ofstream, zamiast ifstream. 

---

cała funkcja wygląda tak

	void getEmployeesFromFile(ifstream& file)
	{
		    file.open("mainFile.txt", ios::in);
			string str;
			if (file.is_open()) {
				while (getline(file, str))
				{ 
					stringstream ss(str);
					string name, surname, trade;
					float salary;
					getline(ss, name, ' ');
					getline(ss, surname, ':');
					getline(ss, trade, '-');
					ss >> salary;
					createSimpleEmployer(name, surname, trade, salary);
				}
				file.close();
			}
	}

 

---

@tkz, zamiast przekazywać ifstream jako argument, to poprostu zadeklarowałam go w ciele funkcji i działa, dzięki za pomoc ;)

---

#include <iostream>
#include <fstream>
#include <string>
#include <sstream>
using namespace std;
void foo(ifstream &);

int main()
{
    ifstream file("mainFile.txt");
    foo(file);
}

void foo(ifstream &file)
{
    
    string str;
    if (file.is_open())
    {
        while (getline(file, str))
        {
            stringstream ss(str);
            string name, surname, trade;
            float salary;
            getline(ss, name, ' ');
            getline(ss, surname, ':');
            getline(ss, trade, '-');
            ss >> salary;
            cout << name << ' ' << surname << ' ' << trade;
            //createSimpleEmployer(name, surname, trade, salary);
        }
        file.close();
    }
}

Tylko, że to też powinno działać. 

Answer 1

void getEmployeesFromFile()
{
    ifstream file("mainFile.txt");
    string str, name, surname, trade;
    float salary;

    while (getline(file, str)) {
        istringstream ss(str);
        if (getline(ss, name, ' ') && getline(ss, surname, ':') && getline(ss, trade, '-') && ss >> salary)
            createSimpleEmployer(name, surname, trade, salary);
    }
}

Nie wiem, po co przekazujesz file w parametrze, skoro i tak używasz ten strumień jakby był lokalny.