удаление элемента из списка с условием[c++]

[luibrain]

функция удаления зацикливается
не могу понять в чем дело

удалить надо те элементы, которые меньше заданного

void udalenie()

#include "stdafx.h"
#include "iostream"
using namespace std;

class muzei
{
public:
	char nazvanie[30];
	int exponati;
	int zapexponati;
	muzei *next;
	muzei();
};

muzei::muzei()
{
	next = NULL;
}

class Elina
{
	muzei *first, *last, *tek;
public:
	Elina();
	~Elina();
	void sozd();
	void vivod();
	void vivod2();
	void dobavlenie();
	void udalenie();
};

Elina::Elina()
{
	first = last = tek = NULL;
}

Elina::~Elina()
{
	while (first)
	{
		last = first->next;
		delete first;
		first = last;
	}
}

void Elina::sozd()
{
	last = first = tek = NULL;
	cout << "признак конца ввода - точка ";
	cout << endl;
	do
	{
		tek = last;
		last = new muzei;
		cout << "Ввведите название музея - ";
		cin >> last->nazvanie;
		cout << endl;
		if (strcmp(last->nazvanie, ".") == 0) { delete last; break; };
		cout << "Введите количество экспонатов - ";
		cin >> last->exponati;
		cout << endl;
		cout << "Введите количество запасных экспонатов - ";
		cin >> last->zapexponati;
		cout << endl;
		last->next = NULL;
		if (first == NULL) first=last;
		else tek->next = last;
	} while (true);
	tek->next = first;
	cout << "список создан";
}
void Elina::vivod()
{
	tek = first;
	do
	{
		cout << " Название музея - " << tek->nazvanie;
		cout << endl;
		cout << " Количество экспонатов  - " << tek->exponati;
		cout << endl;
		cout << " Количество запасных экспонатов - " << tek->zapexponati;
		tek = tek->next;
		cout << endl;
	} while (tek != first);
}
void Elina::vivod2()
{
	int k = 0;
	tek = first;
	do
	{
		if (tek->zapexponati > (0.3*tek->exponati))
		{
			cout << " Название музея - " << tek->nazvanie;
			cout << endl;
			cout << " Количество экспонатов  - " << tek->exponati;
			cout << endl;
			cout << " Количество запасных экспонатов - " << tek->zapexponati;
			cout << endl;
			k++;
		}
		tek = tek->next;

	} while (tek != first);
	if (k == 0) cout << "Нет таких";
	cout << endl;
}

void Elina::dobavlenie()
{
	cout << endl;
	int i = 0; muzei *p;
	char z[30];
	p = new muzei;
	cout << "vv. название, добавить - ";
	cin >> z;
	tek = first;

	do
	{
		if (strcmp(tek->nazvanie, z) == 0)
		{
			p->next = tek->next;
			tek->next = p;
			cout << "vv. название музея - ";
			cin >> p->nazvanie;
			cout << "vv. Количество экспонатов - ";
			cin >> p->exponati;
			cout << "vv. Количество запасных экспонатов - ";
			cin >> p->zapexponati;
		}
		tek = tek->next;
	} while (tek != first);
	
}


void Elina::udalenie()
{
	int n;
	cout << " vv. n - ";
	cin >> n;
	muzei *p;
	p = first;
	do
	{
		if (p->exponati < n)
		{
			tek = p;
			p=p->next;
			delete (tek);
		}
	} while (p != first);
}

[p51x]

функция удаления зацикливается
не могу понять в чем дело

p сдвигается с first и маленькая вероятность, что он им вдруг станет... может там сравнение с last имелось в виду?

[luibrain]

не знаю, у меня задание такое:
Удалить сведения о музеях, в которых количество экспонатов меньше заданного.

[p51x]

Причем тут задание? Вы по списку двигаетесь с начала в начало или все-таки в конец? Это простейшая логика же.

[luibrain]

у меня кольцевой список
последней элемент указывает на первый
двигаюсь с начала