filestream problem

[Pieter Provoost]

I'm having a little filestream problem. With the code below, I open a file
in order to insert its contents into another file. However, right after the
opening of the file, insertfile.eof() seems to be TRUE. Before the
problematic file is opened, one other file is opened and inserted by the
same piece of code (which is in a loop), without any problems. I'm sure that
the file to be inserted has several lines of contents, and I also checked if
the insertfile stream is closed before opening it.

if(FileExists(insertfilepath))
{
insertfile.open(insertfilepath.c_str());
while(! insertfile.eof())
{
getline(insertfile, insertline);
outfile << insertline << endl;
}
insertfile.close();
}

I would really appreciate some advise here.

Thanks,
Pieter

[Victor Bazarov]

Pieter Provoost wrote:

I'm having a little filestream problem. With the code below, I open a file
in order to insert its contents into another file. However, right after the
opening of the file, insertfile.eof() seems to be TRUE. Before the
problematic file is opened, one other file is opened and inserted by the
same piece of code (which is in a loop), without any problems. I'm sure that
the file to be inserted has several lines of contents, and I also checked if
the insertfile stream is closed before opening it.

if(FileExists(insertfilepath))
{
insertfile.open(insertfilepath.c_str());
while(! insertfile.eof())
{
getline(insertfile, insertline);
outfile << insertline << endl;
}
insertfile.close();
Add

insertfile.clear();

here. The 'eof' bit gets carried over and is not cleared by the 'open'
operation, most likely.
}

I would really appreciate some advise here.

HTH

V

[Pieter Provoost]

"Victor Bazarov" <v.********@comAcast.net> schreef in bericht
news:Ya****************@newsread1.mlpsca01.us.to.v erio.net...

Pieter Provoost wrote:

I'm having a little filestream problem. With the code below, I open a file in order to insert its contents into another file. However, right after the opening of the file, insertfile.eof() seems to be TRUE. Before the
problematic file is opened, one other file is opened and inserted by the
same piece of code (which is in a loop), without any problems. I'm sure that the file to be inserted has several lines of contents, and I also checked if the insertfile stream is closed before opening it.

if(FileExists(insertfilepath))
{
insertfile.open(insertfilepath.c_str());
while(! insertfile.eof())
{
getline(insertfile, insertline);
outfile << insertline << endl;
}
insertfile.close();

Add

insertfile.clear();

here. The 'eof' bit gets carried over and is not cleared by the 'open'
operation, most likely.

Works fine, thanks! I forgot to mention that I'm using BCB6, I don't know if
this problem also occurs with other compilers...

Pieter

[Karl Heinz Buchegger]

Pieter Provoost wrote:

I'm having a little filestream problem. With the code below, I open a file
in order to insert its contents into another file. However, right after the
opening of the file, insertfile.eof() seems to be TRUE. Before the
problematic file is opened, one other file is opened and inserted by the
same piece of code (which is in a loop), without any problems. I'm sure that
the file to be inserted has several lines of contents, and I also checked if
the insertfile stream is closed before opening it.

if(FileExists(insertfilepath))
{
insertfile.open(insertfilepath.c_str());
while(! insertfile.eof())
{
getline(insertfile, insertline);
outfile << insertline << endl;
}
insertfile.close();
}

I would really appreciate some advise here.

you are using eof() in the wrong way
(I am just waiting for your next posting entitled:
"why does the last line of my file get processed twice?")

--
Karl Heinz Buchegger
kb******@gascad.at

[Victor Bazarov]

Pieter Provoost wrote:

"Victor Bazarov" <v.********@comAcast.net> schreef in bericht
news:Ya****************@newsread1.mlpsca01.us.to.v erio.net...
[..] The 'eof' bit gets carried over and is not cleared by the 'open'

operation, most likely.

Works fine, thanks! I forgot to mention that I'm using BCB6, I don't know if
this problem also occurs with other compilers...

I have no idea about BCB6. AFAICT, it's standard behaviour.

V

[Pieter Provoost]

"Karl Heinz Buchegger" <kb******@gascad.at> schreef in bericht
news:42***************@gascad.at...

Pieter Provoost wrote:

I'm having a little filestream problem. With the code below, I open a file in order to insert its contents into another file. However, right after the opening of the file, insertfile.eof() seems to be TRUE. Before the
problematic file is opened, one other file is opened and inserted by the
same piece of code (which is in a loop), without any problems. I'm sure that the file to be inserted has several lines of contents, and I also checked if the insertfile stream is closed before opening it.

if(FileExists(insertfilepath))
{
insertfile.open(insertfilepath.c_str());
while(! insertfile.eof())
{
getline(insertfile, insertline);
outfile << insertline << endl;
}
insertfile.close();
}

I would really appreciate some advise here.

you are using eof() in the wrong way
(I am just waiting for your next posting entitled:
"why does the last line of my file get processed twice?")

--
Karl Heinz Buchegger

"I would really appreciate some advise here."
The last line is not processed twice, btw.

Pieter

[Karl Heinz Buchegger]

Pieter Provoost wrote:

you are using eof() in the wrong way
(I am just waiting for your next posting entitled:
"why does the last line of my file get processed twice?")

--
Karl Heinz Buchegger
"I would really appreciate some advise here."

See below
The last line is not processed twice, btw.

It is. You just haven't noticed it :-)
Compare the input file closely with the output file and you will
see that the last line is duplicated. In case the last line is an empty
line, you will find, that your output file has one empty line more then
the input file. It is easy to miss, but it is there. If you don't
believe me, make an experiment: In your input file, in the last line,
write some text but don't terminate that line with a 'return'. Then
you will see.

The point is:
eof() turns to true only after you tried *and failed* to read past
the end of file. C++ has no crystal ball, like other languages. It doesn't
try to predict the future.

eof() is ment to be used to figure out why a preceeding read operation
has failed. So the typical read loop in C++ looks like this

while( read operation works ) {
do something with the read data
}

// read operation has failed, but why?
// use eof() to figure out if it was because of EOF
// if it was, then the file was processed entirely.
// if it was not, then something else has happened

if( eof does not return true ) {
alert user of file reading error and take
some actions
}

All of the C++ read operations support this by returning something
that can immediatly be used for checking if the operation has succeeded.
In your case:

while( getline( insertfile, insertline ) )
outfile << insertline << endl;

if( !insertfile.eof() )
cout << "Error during file copy operation";

This also has the advantage, that the reading loop terminates not only
because of eof, but also because of all sorts of errors that can happen
during reading: file corrupt, network connection breakdown, user removing
the floppy from the drive, hard disc failure, etc...

--
Karl Heinz Buchegger
kb******@gascad.at

[Pieter Provoost]

"Karl Heinz Buchegger" <kb******@gascad.at> schreef in bericht
news:42***************@gascad.at...

Pieter Provoost wrote:

you are using eof() in the wrong way
(I am just waiting for your next posting entitled:
"why does the last line of my file get processed twice?")

--
Karl Heinz Buchegger

"I would really appreciate some advise here."

See below

The last line is not processed twice, btw.

It is. You just haven't noticed it :-)
Compare the input file closely with the output file and you will
see that the last line is duplicated. In case the last line is an empty
line, you will find, that your output file has one empty line more then
the input file. It is easy to miss, but it is there. If you don't
believe me, make an experiment: In your input file, in the last line,
write some text but don't terminate that line with a 'return'. Then
you will see.

The point is:
eof() turns to true only after you tried *and failed* to read past
the end of file. C++ has no crystal ball, like other languages. It doesn't
try to predict the future.

eof() is ment to be used to figure out why a preceeding read operation
has failed. So the typical read loop in C++ looks like this

while( read operation works ) {
do something with the read data
}

// read operation has failed, but why?
// use eof() to figure out if it was because of EOF
// if it was, then the file was processed entirely.
// if it was not, then something else has happened

if( eof does not return true ) {
alert user of file reading error and take
some actions
}

All of the C++ read operations support this by returning something
that can immediatly be used for checking if the operation has succeeded.
In your case:

while( getline( insertfile, insertline ) )
outfile << insertline << endl;

if( !insertfile.eof() )
cout << "Error during file copy operation";

This also has the advantage, that the reading loop terminates not only
because of eof, but also because of all sorts of errors that can happen
during reading: file corrupt, network connection breakdown, user removing
the floppy from the drive, hard disc failure, etc...

--
Karl Heinz Buchegger
kb******@gascad.at

Thanks a lot for your reply, I will have a very close look at it. I did the
experiment as you described it before, and in the new file the last line was
not duplicated. But I'm sure you are right if you say I'm using eof() the
wrong way, I'm just starting with C++. I'll rewrite that piece of code...

Cheers
Pieter

[Karl Heinz Buchegger]

Pieter Provoost wrote:

not duplicated. But I'm sure you are right if you say I'm using eof() the
wrong way, I'm just starting with C++. I'll rewrite that piece of code...

Wee see this all to often.
Even book authors get it wrong :-)
--
Karl Heinz Buchegger
kb******@gascad.at

[Victor Bazarov]

Karl Heinz Buchegger wrote:

Pieter Provoost wrote:

not duplicated. But I'm sure you are right if you say I'm using eof() the
wrong way, I'm just starting with C++. I'll rewrite that piece of code...

Wee see this all to often.
Even book authors get it wrong :-)

Book authors are people, aren't they? I mean, most often, that is. :-)

[Panjandrum]

Karl Heinz Buchegger wrote:

Wee see this all to often.
Even book authors get it wrong :-)

Then the library interface is not well designed and the library should
be avoided (at least for real work with files).

[Karl Heinz Buchegger]

Panjandrum wrote:

Karl Heinz Buchegger wrote:

Wee see this all to often.
Even book authors get it wrong :-)

Then the library interface is not well designed and the library should
be avoided (at least for real work with files).

I don't agree.
There are 2 possibilities how eof() can be handled. C and C++
choose one of them for good reasons. One of them beeing the
principle that there should be no difference in streams no
matter where those streams are connected to, be it a file or
be it the console. And for the console (in contrast to a file)
it is impossible to predict if the user is willing to enter
more data or just signals 'end of data'.

People just have to learn that C++ doesn't try to predict
the future. eof() gets true only after a read operation has
failed and not when the current read operation has reached
the end of the stream.

Other languages do it differently. But that is not the fault of
C++.

--
Karl Heinz Buchegger
kb******@gascad.at