why it is wrong?

[kathy]

I have code piece:

....
std::fstream outFile(_T("data.txt"),std::ios::out|std::ios::tru nc);
if(outFile)
{
....
}

When I build it, I got:
"c:\temp\DialogDemoDlg.cpp(202): error C2451: conditional expression of
type 'std::fstream' is illegal."

Why?

[Victor Bazarov]

kathy wrote:

I have code piece:

...
std::fstream outFile(_T("data.txt"),std::ios::out|std::ios::tru nc);
"_T" thing is non-standard. Could it be you're in some non-standard
territory?
if(outFile)
{
...
}

When I build it, I got:
"c:\temp\DialogDemoDlg.cpp(202): error C2451: conditional expression of
type 'std::fstream' is illegal."

Why?

Because the compiler is bad?... Hard to say. This code:

#include <fstream>

int main() {
std::fstream o("blah", std::ios::out);
if (o)
return 0;
else
return 1;
}

Should compile fine. Try it. If it fails, contact the compiler vendor.

V
--
Please remove capital As from my address when replying by mail

[Mike Wahler]

"kathy" <yq*****@yahoo.com> wrote in message
news:11*********************@g47g2000cwa.googlegro ups.com...

I have code piece:

...
std::fstream outFile(_T("data.txt"),std::ios::out|std::ios::tru nc);
Note that the '_T' macro is not part of standard C++ (and
is not germane to your question).
if(outFile)
{
...
}

When I build it, I got:
"c:\temp\DialogDemoDlg.cpp(202): error C2451: conditional expression of
type 'std::fstream' is illegal."

Why?

There's an idiom (and a supporting member function) for checking
the state of a stream, which allows a stream object to be used
in a boolean context. But converting it to a boolean value
doesn't 'just happen'. To do so requires the use of a boolean
logical operator (a stream's 'value' itself is not boolean).
(The member function is 'operator void*', you can research it
if you like).

Try this

if(!outFile)
// stream in is error state
else
// stream is in 'good' state

If you want the first test to be for 'good' state, you
can write:

if(!!outFile)

or:

if(outFile.good())
BTW if you only need output, you should use 'std::ofstream'.
'std::fstream' is intended for streams which are both read
from and written to.

-Mike

[Alf P. Steinbach]

* kathy wrote in [comp.lang.c++]:

I have code piece:

...
std::fstream outFile(_T("data.txt"),std::ios::out|std::ios::tru nc);
if(outFile)
{
...
}

When I build it, I got:
"c:\temp\DialogDemoDlg.cpp(202): error C2451: conditional expression of
type 'std::fstream' is illegal."

Why?

It's a compiler (or rather, standard library implementation) bug;
however, I'm unable to find a relevant bug-list at Dinkumware.

As a workaround you can use "!!", as mentioned by others, or e.g.

std::ofstream outFile( "data.txt", std::ios::out|std::ios::trunc );

There are two differences from what you wrote. The second one is a
correction of a bug that could prevent your code from compiling (adding
in platform-specific details: when you define UNICODE). The first one
fixes the problem you ran into this time; do you see what it is?

CC: P.J.Plauger

--
A: Because it messes up the order in which people normally read text.
Q: Why is it such a bad thing?
A: Top-posting.
Q: What is the most annoying thing on usenet and in e-mail?

[Pete Becker]

Alf P. Steinbach wrote:

It's a compiler (or rather, standard library implementation) bug;

Right the first time: compiler bug.

--

Pete Becker
Dinkumware, Ltd. (http://www.dinkumware.com)

[Pete Becker]

kathy wrote:

I have code piece:

...
std::fstream outFile(_T("data.txt"),std::ios::out|std::ios::tru nc);
if(outFile)
{
...
}

When I build it, I got:
"c:\temp\DialogDemoDlg.cpp(202): error C2451: conditional expression of
type 'std::fstream' is illegal."

Compiler bug. The code is okay, aside from the _T thingy. fstream
doesn't have a constructor that takes wchar_t, so you always want char*.
That doesn't affect this, though.

--

Pete Becker
Dinkumware, Ltd. (http://www.dinkumware.com)

[Pete Becker]

Mike Wahler wrote:

There's an idiom (and a supporting member function) for checking
the state of a stream, which allows a stream object to be used
in a boolean context. But converting it to a boolean value
doesn't 'just happen'. To do so requires the use of a boolean
logical operator (a stream's 'value' itself is not boolean).
(The member function is 'operator void*', you can research it
if you like).

It does just happen when the object is used in a boolean context, such
as if(outFile). The rest of the error message makes the problem clearer.
The compiler goes on to say that there's an ambiguous user-defined
conversion. That's a compiler bug: operator void* is defined in a class
that's a virtual base of both ostream and istream, and fstream is
derived from both (skipping a few steps here and there...)

--

Pete Becker
Dinkumware, Ltd. (http://www.dinkumware.com)

[Mike Wahler]

"Pete Becker" <pe********@acm.org> wrote in message
news:4s********************@giganews.com...

Mike Wahler wrote:

There's an idiom (and a supporting member function) for checking
the state of a stream, which allows a stream object to be used
in a boolean context. But converting it to a boolean value
doesn't 'just happen'. To do so requires the use of a boolean
logical operator (a stream's 'value' itself is not boolean).
(The member function is 'operator void*', you can research it
if you like).

It does just happen when the object is used in a boolean context, such as
if(outFile). The rest of the error message makes the problem clearer. The
compiler goes on to say that there's an ambiguous user-defined conversion.
That's a compiler bug: operator void* is defined in a class that's a
virtual base of both ostream and istream, and fstream is derived from both
(skipping a few steps here and there...)

I stand corrected. Thanks.

-Mike