By using this site, you agree to our updated Privacy Policy and our Terms of Use. Manage your Cookies Settings.
446,233 Members | 1,972 Online
Bytes IT Community
+ Ask a Question
Need help? Post your question and get tips & solutions from a community of 446,233 IT Pros & Developers. It's quick & easy.

Read file problem

P: n/a
I would like to write a program to read a text file (data.txt) on
Windows and Linux machines.

data.txt is formated as follows:

1 10 100 1000
2 20
(a new but empty line)
My C++ program is:

***************************************
ifstream fp_input("data.txt");
int a,b;

while(!fp_input.eof())
{
fp_input>>a;

cout<<"a="<<a<<": ";

while(fp_input.peek()!='\n' && fp_input.peek()!='\r' && !
fp_input.eof())
{
fp_input>>b;
cout<<b<<", ";
}

cout<<endl;
}
***************************************

However, the output of my program is:

1: 10 100 1000
2: 20
2:

I don't know why there is an extra "2" in the end of the output.

Could someone give me a clue to modify my program such that it can
display "data.txt" properly under Windows and Linux?

Thanks.

Mar 23 '07 #1
Share this Question
Share on Google+
2 Replies


P: n/a

<pe*********@gmail.comwrote in message
news:11**********************@e1g2000hsg.googlegro ups.com...
>I would like to write a program to read a text file (data.txt) on
Windows and Linux machines.

data.txt is formated as follows:

1 10 100 1000
2 20
(a new but empty line)
My C++ program is:

***************************************
ifstream fp_input("data.txt");
int a,b;

while(!fp_input.eof())
{
fp_input>>a;

cout<<"a="<<a<<": ";

while(fp_input.peek()!='\n' && fp_input.peek()!='\r' && !
fp_input.eof())
{
fp_input>>b;
cout<<b<<", ";
}

cout<<endl;
}
***************************************

However, the output of my program is:

1: 10 100 1000
2: 20
2:

I don't know why there is an extra "2" in the end of the output.
Because your 'while' loop is incorrect. 'eof()' does not
report true until *after* an attempt to read past end of file.
It does not 'predict' the next read will fail.
>
Could someone give me a clue to modify my program such that it can
display "data.txt" properly under Windows and Linux?
#include <cstdlib>
#include <fstream>
#include <iostream>
#include <sstream>
#include <string>

int main()
{
int ret(EXIT_SUCCESS);
std::ifstream fp_input("data.txt");

if(fp_input)
{
std::string line;

while(std::getline(fp_input, line))
{
std::istringstream iss(line);
int a(0);

if(iss >a)
{
std::cout << a << ": ";
int b(0);

while(iss >b)
std::cout << b << ", ";

if(!iss.eof())
{
std::cerr << "Error reading input\n";
ret = EXIT_FAILURE;
break;
}

std::cout << '\n';
}
else
{
std::cerr << "Error reading input\n";
ret = EXIT_FAILURE;
break;
}

}
}
else
{
std::cerr << "Cannot open input\n";
ret = EXIT_FAILURE;
}

return ret;
}
Note that if the last line of the input file is empty (contains
only a newline character), parsing it will cause an error to
be reported, since your algorithm unconditionally expects an integer
as the first part of a line (the item you read into object 'a').
You'll either need to eliminate this 'requirement', or remove the
blank line.

-Mike


Mar 23 '07 #2

P: n/a
pe*********@gmail.com wrote:
I would like to write a program to read a text file (data.txt) on
Windows and Linux machines.

data.txt is formated as follows:

1 10 100 1000
2 20
(a new but empty line)
My C++ program is:

***************************************
ifstream fp_input("data.txt");
int a,b;

while(!fp_input.eof())
This usually does not do what you want, since eof() is not set until
*after* a read has failed. See:
http://www.parashift.com/c++-faq-lit....html#faq-15.5
and related questions.

[rest of code snipped]
However, the output of my program is:

1: 10 100 1000
2: 20
2:

I don't know why there is an extra "2" in the end of the output.

Could someone give me a clue to modify my program such that it can
display "data.txt" properly under Windows and Linux?
See above.

--
Marcus Kwok
Replace 'invalid' with 'net' to reply
Mar 23 '07 #3

This discussion thread is closed

Replies have been disabled for this discussion.