Работа со списками методами STL

[JeyKip]

Доброе время суток...)) у меня не большая загвоздка получилась...мне нужно удалить все отрицательные элементы из списка...(это часть задания, остальное я знаю)...использовал функцию erase...я так понял, что она в цикле не работает...(выбивало ошибку на этапе тестирования)...я указал в параметрах erase начало списка и сколько последних его элементов не нужно удалять...и здесь возникает ошибка...кто знает, помогите, пожалуйста...а то и лабу сдавать - и самому интересно...))) Заранее большое спасибо...)))

#include <iostream>
#include <list>
#include <conio.h>
#include <cstdlib>
#include <iomanip>
#include <iterator>

using namespace std;
typedef list<int> CList;

void main ()
{
	setlocale(0,"");
	CList clist, clist2;
	int n;
	cout<<"Введите количество элементов в списке\n";
	cin>>n;
	for (int i=0; i<n; i++){
		clist.push_back(rand()%+20-7);// заполнение списка
	}
	cout<<"Исходный список:\n";
	CList::iterator iter=clist.begin();// указатель на начало списка
	while (iter!=clist.end())// вывод списка на экран
	{
		cout<<setw(5)<<*iter;
		iter++;
	}
	cout<<endl<<endl;
	clist.sort();// сортировка списка
	cout<<"Отсортированный список:\n";
	iter=clist.begin();
	while (iter!=clist.end())// вывод отсортированного спика на экран
	{
		cout<<setw(5)<<*iter;
		iter++;
	}
	cout<<endl<<endl;
    
	CList::iterator iter2=clist2.begin();//копирование отрицательных
	iter=clist.begin();
	for (; iter!=clist.end(); *iter++){
		clist2.push_back(*iter);
	}
	clist2.sort();

	iter=clist.begin();
	int k=0;
	while (iter!=clist.end())
	{
		    if (*iter<0)
				k++;		
	}
	iter=clist.begin();
	clist.erase(clist.begin(), clist.end()--(n-k));

	iter=clist.begin();
	iter2=clist2.begin();
	for (; iter2!=clist2.end(); *iter2++){
		if (*iter2<0)
		clist.push_front(*iter2);
	}

	cout<<"По заданию А:\n";
	iter=clist.begin();
	while (iter!=clist.end()){
		cout<<setw(5)<<*iter;
		iter++;
	}

	getch();
}

[netrino]

Читаем

for(CList::iterator it = clist.begin(); it != clist.end(); ++it)
    if(*it < 0)
        it = clist.erase(it);

[JeyKip]

Спасибо..)))) Ты не знаешь, откуда берутся лишние элементы...?..)))

[JeyKip]

Разобрался...))

В коде:

for(CList::iterator it = clist.begin(); it != clist.end(); ++it)
    if(*it < 0)
        it = clist.erase(it);

вместо if нужен while:

for(CList::iterator it = clist.begin(); it != clist.end(); ++it)
    while (*it < 0)
        it = clist.erase(it);

[netrino]

Даже не вместо if,а вместо for) не учёл я этот момент, пардон)

	iter = clist.begin();
	while(iter != clist.end()) {
		if(*iter < 0)
			iter = clist.erase(iter);
		else
			++iter;
	}

Ну или раз список отсортированный, то:

	iter = clist.begin();
	while(*iter < 0)
		iter = clist.erase(iter);

[ISergeyN]

std::remove_if

#include <functional>
#include <algorithm>
#include <iostream>
#include <iterator>
#include <locale>
#include <list>

int RandomNumber () { return (rand()%15 - 7); }

int main(){

	std::locale::global(std::locale("Russian"));
	std::list<int> list(10);
	std::generate(list.begin(), list.end(), RandomNumber);

	std::cout << "Исходный список:\n";
	std::copy(list.begin(), list.end(), std::ostream_iterator<int>(std::cout, " "));

	std::list<int>::iterator end;

	end = std::remove_if(list.begin(), list.end(), std::bind2nd(std::less<int>(), 0));
	list.erase(end,list.end());

	std::cout << "\nСписок без отрицательных елементов:\n";
	std::copy(list.begin(), list.end(), std::ostream_iterator<int>(std::cout, " "));

	return 0;
}