Получая ошибку здесь, не уверен, что это, хотя

Question

Получая ошибку здесь, не уверен, что это, хотя

моя сборка здесь полностью успешна, но не выводится в текстовый файл, я знаю, что несколько дней назад задал вопрос об этой программе, и с тех пор я его изменил. Что я делаю не так сейчас?

Заранее спасибо, ребята.
Я пытаюсь ввести данные из файла employeeIn.txt и создать файл employeeOut.txt, состоящий из структур сотрудников.

Вот мой текстовый файл.

123,John,Brown,125 Prarie Street,Staunton,IL,62088
124,Matt,Larson,126 Hudson Road,Edwardsville,IL,62025
125,Joe,Baratta,1542 Elizabeth Road,Highland,IL,62088
126,Kristin,Killebrew,123 Prewitt Drive,Alton,IL,62026
127,Tyrone,Meyer,street,999 Orchard Lane,Livingston,62088

Вывод должен выглядеть так

Рекорд сотрудника: 123
Имя: Джон Браун
Домашний адрес: 125 Prarie Street
Стонтон, IL 62088

Рекорд сотрудника: 124
Имя Мэтт Ларсон
Домашний адрес: …. и так далее

Вот мой код

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

using namespace std;

struct Person {
string first;
string last;
};

struct Address {
string street;
string city;
string state;
string zipcode;
};

struct Employee {
Person name;
Address homeAddress;
int eid;
};

int readEmployee(istream& in, Employee eArray[]);
void displayEmployee(ostream& out,Employee eArray[], int EmployeePopulation);
const int arr=50;
Employee eArray[arr];

ifstream fin;
ofstream fout;

int main(int argc, const char * argv[])
{
fin.open("employeesIn.txt");

if (!fin.is_open()) {
cerr << "Error opening employeesIn.txt for reading." << endl;
exit(1);
}
fout.open("employeesOut.txt");
if (!fout.is_open()) {
cerr << "Error opening employeesOut.txt for writing." << endl;
exit(1);
}

int tingle = readEmployee(fin, eArray);

fin.close();
displayEmployee(fout, eArray, tingle);
fout.close();

exit(0);
}

int readEmployee(istream& in, Employee eArray[])
{
string eidText;
string line;
getline(in, line);

int EmployeePopulation = 0;
while (!in.eof()) {
getline(in, eidText, ',');
eArray[EmployeePopulation].eid = stoi(eidText);
getline(in, eArray[EmployeePopulation].name.first, ',');
getline(in, eArray[EmployeePopulation].name.last, ',');
getline(in, eArray[EmployeePopulation].homeAddress.street, ',');
getline(in, eArray[EmployeePopulation].homeAddress.city, ',');
getline(in, eArray[EmployeePopulation].homeAddress.state, ',');
getline(in, eArray[EmployeePopulation].homeAddress.zipcode);
EmployeePopulation++;
}
return EmployeePopulation;
}

void displayEmployee(ostream& out, Employee eArray[], int EmployeePopulation)
{
for (int i = 0; i <= EmployeePopulation - 1; i++) {
out << "Employee Record: " << eArray[i].eid
<< endl
<< "Name: " << eArray[i].name.first << " " << eArray[i].name.last
<< endl
<< "Home address: " << eArray[i].homeAddress.street
<< endl
<< eArray[i].homeAddress.city << ", " << eArray[i].homeAddress.state << " " <<  eArray[i].homeAddress.zipcode
<< endl
<< endl;
}
}

0

c++file-io inputstream outputstream visual-c++

Решение

Другие решения

У вас было бы меньше проблем, если бы вы использовали vector из Employee,
Вы можете передать его по ссылке на функции.
Функции могут получить количество сотрудников с помощью std::vector::size(),
std::vector автоматически расширяется при использовании push_back метод.

Если бы вы создали методы ввода и вывода для ваших классов, вам не пришлось бы нарушать инкапсуляцию:

class Person // using class to support privacy and encapsulation
{
std::string first_name;
std::string last_name;
public:
friend std::istream& operator>>(std::istream& inp, Person& p);
friend std::ostream& operator<<(std::ostream& out, const Person& p);
};

std::istream& operator>>(std::istream& inp, Person& p)
{
std::getline(inp, p.first_name, ',');
std::getline(inp, p.last_name, ',');
}

std:ostream& operator<<(std::ostream& out, const Peron& p)
{
out << "Name: ";
out << p.first_name;
out << " ";
out << p.last_name;
}

class Employee
{
Person name;
Address addr;
public:
friend std::istream& operator>>(std::istream& inp, Employee& e);
};

std::istream& operator>>(std::istream& inp, Employee& e)
{
inp >> name;
inp >> addr;
};

Отсутствующие форматированные входные и выходные данные оставляются читателю в качестве упражнения.

1

В вашем readEmployee функция, вы прошли isstream& in. Я полагаю, вы должны проверить while (!in.eof()) и нет while (!fin.eof()),
И ваш getline(fin, line); должно быть getline(in, line); тоже.

0

Источник

Accepted Answer

Две вещи:

Вы должны использовать return 0 скорее, чем exit(0) в конце основного.

Проверка на eof после того, как вы выполнили несколько чтений и попытались преобразовать данные неверно. Вы должны проверить на наличие ошибок чтения.

Это исправляет eof вопрос. Программа зависала для меня, потому что stoi выдал исключение, когда чтение не удалось.

int readEmployee(istream& in, Employee eArray[])
{
string eidText;
string line;
//This discards the first line.  Incorrect for the test data you supplied.
getline(in, line);

int EmployeePopulation = 0;
//Check for errors while reading, not eof after the fact.
//This was crashing because stoi failed when no data was
//read due to eof being true after the loop check.
while(  getline(in, eidText, ',') &&
getline(in, eArray[EmployeePopulation].name.first, ',') &&
getline(in, eArray[EmployeePopulation].name.last, ',') &&
getline(in, eArray[EmployeePopulation].homeAddress.street, ',') &&
getline(in, eArray[EmployeePopulation].homeAddress.city, ',') &&
getline(in, eArray[EmployeePopulation].homeAddress.state, ',') &&
getline(in, eArray[EmployeePopulation].homeAddress.zipcode))
{
eArray[EmployeePopulation].eid = stoi(eidText);
EmployeePopulation++;
}
return EmployeePopulation;
}

1