Re: Deleting items from an std::list , is this code correct?

On Apr 24, 5:43 pm, Ron Natalie <r...@[EMAIL PROTECTED]
> wrote:
> lallous wrote:
> > Hello Group,
>
> > Please advise. Does this work with all STL implementations
>
> Doesn't work with any implementation as far as I know.
>

I think you mis-read the code.  There are *two* lists, one of ints,
the other of interators into the first.


> >    for (int_list_iterator_list_t::const_iterator
di=it_list.begin();di!
> > =it_list.end();++di)
> >    {
> >      L.erase(*di);
> >    }
> > }
>
> An iterator to a deleted item in a list is invalid.  You can't
> apply ++di to it.  This is why sequence erases return the next
> iterator.

The element under di is never erased, only the element under *di is.

> for (int_list_iterator_list_t::const_iterator di=it_list.begin();
> di != it_list.end; )
>      {
>        it = L.erase(*di);
>      }

Fortunately it won't compile due to incompatible iterator types.  If
it would compile and run, it'd be a disaster, since it would be trying
to jump the tracks out of it_list and into L.

--
Chris

-- 
      [ See http://www.gotw.ca/resources/clcm.htm
for info about ]
      [ comp.lang.c++.moderated.    First time posters: Do this! ]