get integer values using ifstream error.

[zhouchengly]

when I use the following code to get integer values from file:
void getIntegers()
{
vector<ItemType items;
ifstream ifs("test.dat") ;
//¿¼ÂÇΪitemsÒ»´Î ·ÖÅä¿Õ¼ä¡£
ItemType item ;
while( !ifs.eof() )
{
ifs>>item;
items.push_back (item);
}
}
if test.dat is ended with a carriage return, "ifs>>item" will not fail
but doesn't change item's value. this is not I want.
using try...catch having no effect.

So is there an stardard way to get integer(float) from a file? the file
format is like this.
//////test.dat
101

22

35
//end with a carriage return

[Markus Moll]

Hi

zh*********@gma il.com wrote:

while( !ifs.eof() )
{
ifs>>item;
items.push_back (item);
}

This is an faq.
Please go and read http://www.parashift.com/c++-faq-lite/ before posting
questions. Thanks.

Markus

[Ivan Vecerina]

<zh*********@gm ail.comwrote in message
news:11******** **************@ m7g2000cwm.goog legroups.com...

>when I use the following code to get integer values from file:
void getIntegers()
{
vector<ItemType items;
ifstream ifs("test.dat") ;
//¿¼ÂÇΪitemsÒ»´Î ·ÖÅä¿Õ¼ä¡£
ItemType item ;
while( !ifs.eof() )
{
ifs>>item;
items.push_back (item);
}
}
if test.dat is ended with a carriage return, "ifs>>item" will not fail
but doesn't change item's value. this is not I want.
using try...catch having no effect.

So is there an stardard way to get integer(float) from a file?

eof() cannot be set prior to testing whether the next
item can be read or not. Also, failures other than eof
are possible.
And an exception will only be thrown if you configure the
stream accordingly...

Try the following loop:
while( ifs>>item )
items.push_back (item);

Even easier, if your file contains a simple array of values,
you can use istream_iterato r ( #include <iterator):

typedef std::istream_it erator<ItemType InItem;
std::vector<Ite mTypeconst items( (InItem(ifs)), InItem() );
// directly initializes the collection from the file
Note
- the collection can then (optionally) be 'const', which
is a good idea for safety if you don't edit the vector.
- the extra parenthesis around the first parameter is somehow
required for syntactic disambiguation with a function
declaration ( see "C++ most vexing parse" or a FAQ...)

hth -Ivan
--
http://ivan.vecerina.com/contact/?subject=NG_POST <- email contact form
Brainbench MVP for C++ <http://www.brainbench.com

[ninapeng728]

thanks!
an extra question:
how to read the first n elements of the file using the second solution?
"
typedef std::istream_it erator<ItemType InItem;
std::vector<Ite mTypeconst items( (InItem(ifs)), InItem() );
"

"Ivan Vecerina дµÀ£º
"

<zh*********@gm ail.comwrote in message
news:11******** **************@ m7g2000cwm.goog legroups.com...

when I use the following code to get integer values from file:
void getIntegers()
{
vector<ItemType items;
ifstream ifs("test.dat") ;
//¿¼ÂÇΪitemsÒ»´Î ·ÖÅä¿Õ¼ä¡£
ItemType item ;
while( !ifs.eof() )
{
ifs>>item;
items.push_back (item);
}
}
if test.dat is ended with a carriage return, "ifs>>item" will not fail
but doesn't change item's value. this is not I want.
using try...catch having no effect.

So is there an stardard way to get integer(float) from a file?

eof() cannot be set prior to testing whether the next
item can be read or not. Also, failures other than eof
are possible.
And an exception will only be thrown if you configure the
stream accordingly...

Try the following loop:
while( ifs>>item )
items.push_back (item);

Even easier, if your file contains a simple array of values,
you can use istream_iterato r ( #include <iterator):

typedef std::istream_it erator<ItemType InItem;
std::vector<Ite mTypeconst items( (InItem(ifs)), InItem() );
// directly initializes the collection from the file
Note
- the collection can then (optionally) be 'const', which
is a good idea for safety if you don't edit the vector.
- the extra parenthesis around the first parameter is somehow
required for syntactic disambiguation with a function
declaration ( see "C++ most vexing parse" or a FAQ...)

hth -Ivan
--
http://ivan.vecerina.com/contact/?subject=NG_POST <- email contact form
Brainbench MVP for C++ <http://www.brainbench.com

[emil]

zh*********@gma il.com wrote:

when I use the following code to get integer values from file:
void getIntegers()
{
vector<ItemType items;
ifstream ifs("test.dat") ;
//¿¼ÂÇΪitemsÒ»´Î ·ÖÅä¿Õ¼ä¡£
ItemType item ;
while( !ifs.eof() )
{
ifs>>item;
items.push_back (item);
}
}
if test.dat is ended with a carriage return, "ifs>>item" will not fail
but doesn't change item's value. this is not I want.
using try...catch having no effect.

So is there an stardard way to get integer(float) from a file? the file
format is like this.
//////test.dat
101

22

35
//end with a carriage return

you can use fscanf func

[ninapeng728]

It's a solution, but considering the efficiency, I prefer to use stream
than fscanf.
"emil дµÀ£º
"

zh*********@gma il.com wrote:

when I use the following code to get integer values from file:
void getIntegers()
{
vector<ItemType items;
ifstream ifs("test.dat") ;
//¿¼ÂÇΪitemsÒ»´Î ·ÖÅä¿Õ¼ä¡£
ItemType item ;
while( !ifs.eof() )
{
ifs>>item;
items.push_back (item);
}
}
if test.dat is ended with a carriage return, "ifs>>item" will not fail
but doesn't change item's value. this is not I want.
using try...catch having no effect.

So is there an stardard way to get integer(float) from a file? the file
format is like this.
//////test.dat
101

22

35
//end with a carriage return

you can use fscanf func

[Markus Moll]

Hi

ni*********@gma il.com wrote:

It's a solution, but considering the efficiency, I prefer to use stream
than fscanf.
"emil 写�：

[...]

>you can use fscanf func

Again, _please_ read the FAQ ( http://www.parashift.com/c++-faq-lite/ )
before posting. In this case especially [5.4], where it says: "do not
top-post". Thanks.

Markus

[Ivan Vecerina]

<ni*********@gm ail.comwrote in message
news:11******** **************@ e3g2000cwe.goog legroups.com...

>thanks!

NB: please don't top-post, see
http://www.parashift.com/c++-faq-lit...t.html#faq-5.4

>an extra question:
how to read the first n elements of the file using the second solution?
"
typedef std::istream_it erator<ItemType InItem;
std::vector<It emTypeconst items( (InItem(ifs)), InItem() );
"

Sadly, I can't think of an obvious way to read a fixed number
of elements using the above approach - not using the standard
library alone.
Regrettably, I think in this case I would revert to using a loop:
std::vector<Ite mTypeitems(n);
for( unsigned i = 0 ; i<n ; ++i )
ifs >items[i];

Various C++ library implementations provide extensions that would also
help with this, for example: http://www.sgi.com/tech/stl/copy_n.html
std::vector<Ite mTypeitems;
copy_n( istream_iterato r<ItemType>(ifs ), n, back_inserter(i tems) );

It would also be possible to write an iterator wrapper to allow
writing:
typedef count_iterator< std::istream_it erator<ItemType InItem;
std::vector<Ite mTypeconst items( (InItem(ifs)), InItem(n) );

I don't know if the planned/official C++ library extensions (TR1, TR2,
or C++0x) include a facility that would support any of the above...

--
http://ivan.vecerina.com/contact/?subject=NG_POST <- email contact form
Brainbench MVP for C++ <http://www.brainbench.com