Velocity Reviews - Computer Hardware Reviews

Velocity Reviews > Newsgroups > Programming > C++ > list::begin() iterator comportement after push_back operations

Reply
Thread Tools

list::begin() iterator comportement after push_back operations

 
 
Hizo
Guest
Posts: n/a
 
      04-02-2011
Hi there,

I have a problem with the begin iterator of STL Lists.
Indeed, if we keep the begin iterator of an empty list when we test it
after multiple push_back operations it becomes the end iterator.
Here is my code:

-------------------------------------------
#include <iostream>
using std::cout;
using std::endl;
using std::boolalpha;

#include <list>
using std::list;

int main(int argc, char * argv[])
{
list<int> l;
list<int>::const_iterator it = l.begin();
list<int>::const_reverse_iterator rit = l.rbegin();

l.push_back(1);
l.push_back(2);

cout << boolalpha << (it == l.end()) << endl;
cout << boolalpha << (rit == l.rend()) << endl;

return 0;
}
-------------------------------------------

It actually returns:
true
false

with gcc version 4.3.4 (Gentoo 4.3.4 p1.0, pie-10.1.5)

Is it possible to keep in memory the begin iterator of a list (not
using reverse iterators) which will really point to the begin of the
list after push_back operations on the list (obviously I am not able
to use l.begin() after (because it is an initial state in my algorithm
and I then update the iterator that pointed to the begin iterator
initialy))

Thanks for your help.
 
Reply With Quote
 
 
 
 
Bo Persson
Guest
Posts: n/a
 
      04-02-2011
Hizo wrote:
> Hi there,
>
> I have a problem with the begin iterator of STL Lists.
> Indeed, if we keep the begin iterator of an empty list when we test
> it after multiple push_back operations it becomes the end iterator.
> Here is my code:
>
> -------------------------------------------
> #include <iostream>
> using std::cout;
> using std::endl;
> using std::boolalpha;
>
> #include <list>
> using std::list;
>
> int main(int argc, char * argv[])
> {
> list<int> l;
> list<int>::const_iterator it = l.begin();
> list<int>::const_reverse_iterator rit = l.rbegin();
>
> l.push_back(1);
> l.push_back(2);
>
> cout << boolalpha << (it == l.end()) << endl;
> cout << boolalpha << (rit == l.rend()) << endl;
>
> return 0;
> }
> -------------------------------------------
>
> It actually returns:
> true
> false
>
> with gcc version 4.3.4 (Gentoo 4.3.4 p1.0, pie-10.1.5)
>
> Is it possible to keep in memory the begin iterator of a list (not
> using reverse iterators) which will really point to the begin of the
> list after push_back operations on the list (obviously I am not able
> to use l.begin() after (because it is an initial state in my
> algorithm and I then update the iterator that pointed to the begin
> iterator initialy))
>
> Thanks for your help.


Short answer: No.

All containers start out with c.begin() == c.end(), as that is one way
of seeing that the container is empty.

When you add elements to the container, some or all iterators will be
invalidated. A little different for each container type, but
definitely the begin() iterator will change when you add an element to
the start of the container (which of course happens when you add to an
empty container).

Reverse iterators will not help either, as they will be equally
invalidated.


Bo Persson


 
Reply With Quote
 
 
 
 
Hizo
Guest
Posts: n/a
 
      04-02-2011
On 2 avr, 13:57, "Bo Persson" <b...@gmb.dk> wrote:
> Hizo wrote:
> > Hi there,

>
> > I have a problem with the begin iterator of STL Lists.
> > Indeed, if we keep the begin iterator of an empty list when we test
> > it after multiple push_back operations it becomes the end iterator.
> > Here is my code:

>
> > -------------------------------------------
> > #include <iostream>
> > using std::cout;
> > using std::endl;
> > using std::boolalpha;

>
> > #include <list>
> > using std::list;

>
> > int main(int argc, char * argv[])
> > {
> > list<int> l;
> > list<int>::const_iterator it = l.begin();
> > list<int>::const_reverse_iterator rit = l.rbegin();

>
> > l.push_back(1);
> > l.push_back(2);

>
> > cout << boolalpha << (it == l.end()) << endl;
> > cout << boolalpha << (rit == l.rend()) << endl;

>
> > return 0;
> > }
> > -------------------------------------------

>
> > It actually returns:
> > true
> > false

>
> > with gcc version 4.3.4 (Gentoo 4.3.4 p1.0, pie-10.1.5)

>
> > Is it possible to keep in memory the begin iterator of a list (not
> > using reverse iterators) which will really point to the begin of the
> > list after push_back operations on the list (obviously I am not able
> > to use l.begin() after (because it is an initial state in my
> > algorithm and I then update the iterator that pointed to the begin
> > iterator initialy))

>
> > Thanks for your help.

>
> Short answer: No.
>
> All containers start out with c.begin() == c.end(), as that is one way
> of seeing that the container is empty.
>
> When you add elements to the container, some or all iterators will be
> invalidated. A little different for each container type, but
> definitely the begin() iterator will change when you add an element to
> the start of the container (which of course happens when you add to an
> empty container).
>
> Reverse iterators will not help either, as they will be equally
> invalidated.
>
> Bo Persson


Alright...
But I thought that in lists, it should not be the case since iterators
are not invalidated when adding elements.

And the reverse iterator seems not to be invalidated here ? (cf result
of my code)
But I could not use them anyway...

Thanks.
 
Reply With Quote
 
Hizo
Guest
Posts: n/a
 
      04-02-2011
On 2 avr, 13:57, "Bo Persson" <b...@gmb.dk> wrote:
> Hizo wrote:
> > Hi there,

>
> > I have a problem with the begin iterator of STL Lists.
> > Indeed, if we keep the begin iterator of an empty list when we test
> > it after multiple push_back operations it becomes the end iterator.
> > Here is my code:

>
> > -------------------------------------------
> > #include <iostream>
> > using std::cout;
> > using std::endl;
> > using std::boolalpha;

>
> > #include <list>
> > using std::list;

>
> > int main(int argc, char * argv[])
> > {
> > list<int> l;
> > list<int>::const_iterator it = l.begin();
> > list<int>::const_reverse_iterator rit = l.rbegin();

>
> > l.push_back(1);
> > l.push_back(2);

>
> > cout << boolalpha << (it == l.end()) << endl;
> > cout << boolalpha << (rit == l.rend()) << endl;

>
> > return 0;
> > }
> > -------------------------------------------

>
> > It actually returns:
> > true
> > false

>
> > with gcc version 4.3.4 (Gentoo 4.3.4 p1.0, pie-10.1.5)

>
> > Is it possible to keep in memory the begin iterator of a list (not
> > using reverse iterators) which will really point to the begin of the
> > list after push_back operations on the list (obviously I am not able
> > to use l.begin() after (because it is an initial state in my
> > algorithm and I then update the iterator that pointed to the begin
> > iterator initialy))

>
> > Thanks for your help.

>
> Short answer: No.
>
> All containers start out with c.begin() == c.end(), as that is one way
> of seeing that the container is empty.
>
> When you add elements to the container, some or all iterators will be
> invalidated. A little different for each container type, but
> definitely the begin() iterator will change when you add an element to
> the start of the container (which of course happens when you add to an
> empty container).
>
> Reverse iterators will not help either, as they will be equally
> invalidated.
>
> Bo Persson


Alright...
But I thought that in lists, it should not be the case since iterators
are not invalidated when adding elements.

And the reverse iterator seems not to be invalidated here ? (cf result
of my code and dereferencing it give the expected result)
But I could not use them anyway...

Thanks.
 
Reply With Quote
 
Hizo
Guest
Posts: n/a
 
      04-02-2011
On 2 avr, 13:57, "Bo Persson" <b...@gmb.dk> wrote:
> Hizo wrote:
> > Hi there,

>
> > I have a problem with the begin iterator of STL Lists.
> > Indeed, if we keep the begin iterator of an empty list when we test
> > it after multiple push_back operations it becomes the end iterator.
> > Here is my code:

>
> > -------------------------------------------
> > #include <iostream>
> > using std::cout;
> > using std::endl;
> > using std::boolalpha;

>
> > #include <list>
> > using std::list;

>
> > int main(int argc, char * argv[])
> > {
> > list<int> l;
> > list<int>::const_iterator it = l.begin();
> > list<int>::const_reverse_iterator rit = l.rbegin();

>
> > l.push_back(1);
> > l.push_back(2);

>
> > cout << boolalpha << (it == l.end()) << endl;
> > cout << boolalpha << (rit == l.rend()) << endl;

>
> > return 0;
> > }
> > -------------------------------------------

>
> > It actually returns:
> > true
> > false

>
> > with gcc version 4.3.4 (Gentoo 4.3.4 p1.0, pie-10.1.5)

>
> > Is it possible to keep in memory the begin iterator of a list (not
> > using reverse iterators) which will really point to the begin of the
> > list after push_back operations on the list (obviously I am not able
> > to use l.begin() after (because it is an initial state in my
> > algorithm and I then update the iterator that pointed to the begin
> > iterator initialy))

>
> > Thanks for your help.

>
> Short answer: No.
>
> All containers start out with c.begin() == c.end(), as that is one way
> of seeing that the container is empty.
>
> When you add elements to the container, some or all iterators will be
> invalidated. A little different for each container type, but
> definitely the begin() iterator will change when you add an element to
> the start of the container (which of course happens when you add to an
> empty container).
>
> Reverse iterators will not help either, as they will be equally
> invalidated.
>
> Bo Persson


Alright...
I thought that in lists, it should not be the case since iterators
are not invalidated when adding elements.

Anyway, thanks.
 
Reply With Quote
 
Hizo
Guest
Posts: n/a
 
      04-02-2011
On 2 avr, 13:57, "Bo Persson" <b...@gmb.dk> wrote:
> Hizo wrote:
> > Hi there,

>
> > I have a problem with the begin iterator of STL Lists.
> > Indeed, if we keep the begin iterator of an empty list when we test
> > it after multiple push_back operations it becomes the end iterator.
> > Here is my code:

>
> > -------------------------------------------
> > #include <iostream>
> > using std::cout;
> > using std::endl;
> > using std::boolalpha;

>
> > #include <list>
> > using std::list;

>
> > int main(int argc, char * argv[])
> > {
> > list<int> l;
> > list<int>::const_iterator it = l.begin();
> > list<int>::const_reverse_iterator rit = l.rbegin();

>
> > l.push_back(1);
> > l.push_back(2);

>
> > cout << boolalpha << (it == l.end()) << endl;
> > cout << boolalpha << (rit == l.rend()) << endl;

>
> > return 0;
> > }
> > -------------------------------------------

>
> > It actually returns:
> > true
> > false

>
> > with gcc version 4.3.4 (Gentoo 4.3.4 p1.0, pie-10.1.5)

>
> > Is it possible to keep in memory the begin iterator of a list (not
> > using reverse iterators) which will really point to the begin of the
> > list after push_back operations on the list (obviously I am not able
> > to use l.begin() after (because it is an initial state in my
> > algorithm and I then update the iterator that pointed to the begin
> > iterator initialy))

>
> > Thanks for your help.

>
> Short answer: No.
>
> All containers start out with c.begin() == c.end(), as that is one way
> of seeing that the container is empty.
>
> When you add elements to the container, some or all iterators will be
> invalidated. A little different for each container type, but
> definitely the begin() iterator will change when you add an element to
> the start of the container (which of course happens when you add to an
> empty container).
>
> Reverse iterators will not help either, as they will be equally
> invalidated.
>
> Bo Persson


Alright...
I thought that in lists, it should not be the case since iterators
are not invalidated when adding elements.

I tried this:

-------------------------------------------
#include <iostream>
using std::cout;
using std::endl;
using std::boolalpha;

#include <list>
using std::list;

int main(int argc, char * argv[])
{
list<int> l;
l.push_back(0);

list<int>::const_iterator it = l.begin();
l.pop_back();

l.push_back(1);
l.push_back(2);

cout << *it << endl;

cout << boolalpha << (it == l.end()) << endl;

return 0;
}
-------------------------------------------

The result (with gcc version 4.3.4 (Gentoo 4.3.4 p1.0, pie-10.1.5))
is:
1
false

(i.e. the expected result)

But is it a standard comportement in the STL ?
Can I really rely on this example ?

Thanks
 
Reply With Quote
 
Bo Persson
Guest
Posts: n/a
 
      04-02-2011
Hizo wrote:
> On 2 avr, 13:57, "Bo Persson" <b...@gmb.dk> wrote:
>> Hizo wrote:
>>> Hi there,

>>
>>> I have a problem with the begin iterator of STL Lists.
>>> Indeed, if we keep the begin iterator of an empty list when we
>>> test it after multiple push_back operations it becomes the end
>>> iterator. Here is my code:

>>
>>> -------------------------------------------
>>> #include <iostream>
>>> using std::cout;
>>> using std::endl;
>>> using std::boolalpha;

>>
>>> #include <list>
>>> using std::list;

>>
>>> int main(int argc, char * argv[])
>>> {
>>> list<int> l;
>>> list<int>::const_iterator it = l.begin();
>>> list<int>::const_reverse_iterator rit = l.rbegin();

>>
>>> l.push_back(1);
>>> l.push_back(2);

>>
>>> cout << boolalpha << (it == l.end()) << endl;
>>> cout << boolalpha << (rit == l.rend()) << endl;

>>
>>> return 0;
>>> }
>>> -------------------------------------------

>>
>>> It actually returns:
>>> true
>>> false

>>
>>> with gcc version 4.3.4 (Gentoo 4.3.4 p1.0, pie-10.1.5)

>>
>>> Is it possible to keep in memory the begin iterator of a list (not
>>> using reverse iterators) which will really point to the begin of
>>> the list after push_back operations on the list (obviously I am
>>> not able to use l.begin() after (because it is an initial state
>>> in my algorithm and I then update the iterator that pointed to
>>> the begin iterator initialy))

>>
>>> Thanks for your help.

>>
>> Short answer: No.
>>
>> All containers start out with c.begin() == c.end(), as that is one
>> way of seeing that the container is empty.
>>
>> When you add elements to the container, some or all iterators will
>> be invalidated. A little different for each container type, but
>> definitely the begin() iterator will change when you add an
>> element to the start of the container (which of course happens
>> when you add to an empty container).
>>
>> Reverse iterators will not help either, as they will be equally
>> invalidated.
>>
>> Bo Persson

>
> Alright...
> I thought that in lists, it should not be the case since iterators
> are not invalidated when adding elements.
>
> I tried this:
>
> -------------------------------------------
> #include <iostream>
> using std::cout;
> using std::endl;
> using std::boolalpha;
>
> #include <list>
> using std::list;
>
> int main(int argc, char * argv[])
> {
> list<int> l;
> l.push_back(0);
>
> list<int>::const_iterator it = l.begin();
> l.pop_back();
>
> l.push_back(1);
> l.push_back(2);
>
> cout << *it << endl;
>
> cout << boolalpha << (it == l.end()) << endl;
>
> return 0;
> }
> -------------------------------------------
>
> The result (with gcc version 4.3.4 (Gentoo 4.3.4 p1.0, pie-10.1.5))
> is:
> 1
> false
>
> (i.e. the expected result)
>
> But is it a standard comportement in the STL ?
> Can I really rely on this example ?
>
> Thanks


Dereferencing an invalid iterator is undefined behavior, so we can't
test for it - anything could happen, like the list allocator reusing
the deleted node for one of the new nodes.

But we can't rely on that.


Bo Persson


 
Reply With Quote
 
James Kanze
Guest
Posts: n/a
 
      04-03-2011
On Apr 2, 1:32 pm, Hizo <hizostu...@gmail.com> wrote:
> On 2 avr, 13:57, "Bo Persson" <b...@gmb.dk> wrote:


[...]
> I thought that in lists, it should not be the case since iterators
> are not invalidated when adding elements.


They aren't. When the list is empty, begin() returns the same
iterator as end(), i.e. an iterator which points to one past the
last element. Regardless of what you do after that, that
iterator will point to one past the last element.

> I tried this:


> -------------------------------------------
> #include <iostream>
> using std::cout;
> using std::endl;
> using std::boolalpha;


> #include <list>
> using std::list;


> int main(int argc, char * argv[])
> {
> list<int> l;
> l.push_back(0);


> list<int>::const_iterator it = l.begin();


it points to the first element, i.e. the element with 0 in it.

> l.pop_back();


This invaliates it, since it removes the element it points to.

> l.push_back(1);
> l.push_back(2);


> cout << *it << endl;


And this is undefined behavior.

> cout << boolalpha << (it == l.end()) << endl;
> return 0;}
> -------------------------------------------


> The result (with gcc version 4.3.4 (Gentoo 4.3.4 p1.0, pie-10.1.5))
> is:
> 1
> false


> (i.e. the expected result)


On what grounds is it expected?

Try compiling with -D_GLIBCXX_CONCEPT_CHECKS -D_GLIBCXX_DEBUG
-D_GLIBCXX_DEBUG_PEDANTIC. (These options really should be the
default, but like most compilers, g++'s defaults are rather
worthless.)

--
James Kanze
 
Reply With Quote
 
 
 
Reply

Thread Tools

Posting Rules
You may not post new threads
You may not post replies
You may not post attachments
You may not edit your posts

BB code is On
Smilies are On
[IMG] code is On
HTML code is Off
Trackbacks are On
Pingbacks are On
Refbacks are Off


Similar Threads
Thread Thread Starter Forum Replies Last Post
stand-alone JMS, other JDBC operations, and transactions ( ActiveMQ + JOTM + JDBC operations ) Jesus M. Salvo Jr. Java 2 02-11-2006 06:33 PM
Difference between Java iterator and iterator in Gang of Four Hendrik Maryns Java 18 12-22-2005 05:14 AM
applet's call comportement Emmanuel Freund Java 10 11-04-2004 12:15 PM
Iterator Operations Michael Mellor C++ 0 01-20-2004 01:58 PM
Iterator doubts, Decision on Iterator usage greg C++ 6 07-17-2003 01:26 PM



Advertisments
 



1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19 20 21 22 23 24 25 26 27 28 29 30 31 32 33 34 35 36 37 38 39 40 41 42 43 44 45 46 47 48 49 50 51 52 53 54 55 56 57