Re: When operator>> can throw...

From:

=?UTF-8?B?RXJpayBXaWtzdHLDtm0=?= <Erik-wikstrom@telia.com>

Newsgroups:

comp.lang.c++

Date:

Sat, 19 Jan 2008 17:31:14 GMT

Message-ID:

<Ctqkj.2986$R_4.2262@newsb.telia.net>

On 2008-01-19 16:15, Daniel T. wrote:

This is basically what I have now...

class Foo { /* definition irrelevant */ };

istream& operator>>( istream& is, Foo& foo ); // could throw

int main() {
   ifstream file( "file.txt" );
   Foo foo;
   int i = 0;
   try {
      while ( file >> foo ) {
         //process foo;
         ++i;
      }
   }
   catch ( exception& e ) {
      cout << "error at: " << i << '\n';
   }
}

The problem I'm having is that op>> could throw. If it does, I don't
want to process the foo object that threw, but I want the program to
report an error and continue extracting foo objects.

I could make op>> a no-throw and give Foo some sort of error or invalid
state, but that doesn't feel right. I can't report the error inside the
op>> because I don't have enough information to make a full report.

Does anyone have a better idea, or do I give Foo and error state and
query it after the read?

Something like this perhaps?

int main() {
   ifstream file( "file.txt" );
   Foo foo;
   int i = 0;
   while ( file.eof() == false ) {
      try {
         file >> foo;
      } catch ( exception& e )
         cout << "error at: " << i << '\n';
         continue;
      }
      ++i;
   }
}

--
Erik Wikstr?m