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

On Apr 24, 10:04 am, lallous <lall...@[EMAIL PROTECTED]
> wrote:
> Hello Group,
>
> Please advise. Does this work with all STL implementations?
>
> Thank you,
> Elias
>
> [code]
> #include <iostream>
> #include <list>
>
> typedef std::list<int> int_list_t;
> typedef std::list<int_list_t::iterator> int_list_iterator_list_t;
>
> void print_list(int_list_t &L)
> {
>    for (int_list_t::iterator it=L.begin();it!=L.end();++it)
>    {
>      std::cout << "value = " << *it << std::endl;
>    }
>
> }
>
> void delete_odd(int_list_t &L)
> {
>    int_list_iterator_list_t it_list;
>    int_list_t::iterator it;
>
>    for (it=L.begin();it!=L.end();++it)
>    {
>      if (*it % 2 != 0)
>        it_list.push_back(it);
>    }
>
>    for (int_list_iterator_list_t::const_iterator di=it_list.begin();di!
> =it_list.end();++di)
>    {
>      L.erase(*di);
>    }
>
> }

It should work with any reasonable STL implementation.

But do you really need that list of iterators?  Why not erase matching
list elements right away? :-)

-- 
Cheers,
Alex

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