Re: "inFile" object cannot read EOF
On Jun 10, 2:23 am, Jerry Coffin <jcof...@taeus.com> wrote:
In article <1181292953.700106.187...@q66g2000hsg.googlegroups.com>,
james.ka...@gmail.com says...
[ ... ]
I'd write this loop something like:
std::vector< donor > donors ;
std::string line1 ;
std::string line2 ;
while ( getline( inFile, line1 ) && getline( inFile, line2 ) ) {
double cash ;
std::istringstream sCash( line2 ) ;
sCash >> cash ;
donors.push_back( donor( line1, cash ) ) ;
}
Personally, I think I'd do it a bit differently:
I'd do it a lot differently in production code as well:-).
Starting with a lot more error handling.
struct donor {
std::string name;
double cash;
// read a single donor's data:
friend std::istream &operator>>(std::istream &i, donor &d) {
std::getline(i, d.name);
std::string line2;
std::getline(line2);
std::istringstream temp(line2);
line2 >> d.cash;
return i;
}
};
Good point. Using a >> operator is definitely the way to go.
std::vector<donor> donors;
std::copy(std::istream_iterator<donor>(infile),
std::istream_iterator<donor>(),
std::back_inserter(donors));
This does not require (or allow) the count at the beginning of the file
though...
Neither did my version:-). In fact, as soon as you use
std::vector, you're freed from this constraint.
--
James Kanze (Gabi Software) email: james.kanze@gmail.com
Conseils en informatique orient=E9e objet/
Beratung in objektorientierter Datenverarbeitung
9 place S=E9mard, 78210 St.-Cyr-l'=C9cole, France, +33 (0)1 30 23 00 34