Функция удаления элемента списка

[Shadownx]

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

void delete_element(Node **first,Node **last) {
	Node *current=*first;
	Node *prev=nullptr;
	int delete_element=0;
	cout<<"please enter element value"<<endl;
	cin>>delete_element;
	if(current->data==delete_element) {
		if(current==(*first)) {
			(*first)=(*first)->next;
			delete current;
			return;
		}
	}
	prev=current;
	current=current->next;
	while(current!=nullptr) {
		if(current->data==delete_element && current!=(*last)) {
			prev->next=current->next;
			delete current;
			return;
		} else if(current->data==delete_element && current==(*last)) {
			(*last)=prev;
			(*last)->next=nullptr;
			delete current;
			return;
		}
		prev=current;
		current=current->next;
	}
	cout<<"list don't have this element"<<endl;
}

[Serge_Bliznykov]

когда в списке всего один единственный элемент, first и last указывают на него?
Тогда, если этот единственный хотят удалить, first станет nullptr, а вот last будет указывать на мусор.

[Shadownx]

Сообщение от Serge_Bliznykov

когда в списке всего один единственный элемент, first и last указывают на него?
Тогда, если этот единственный хотят удалить, first станет nullptr, а вот last будет указывать на мусор.

нужно рассмотреть отдельный случай? или можно это как-то в общем виде починить? просто у меня как вы видите написано я полагаю весьма неуклюже и как правильней решить данную проблему я не знаю

[Shadownx]

Вот так я полагаю!?

void delete_element(Node **first,Node **last) {
	Node *current=*first;
	Node *prev=nullptr;
	int delete_element=0;
	cin>>delete_element;
	if(current->data==delete_element) {
		if(current==(*first) && (*first)!=(*last)) {
			(*first)=(*first)->next;
			delete current;
			return ;
		}
		else if(current==(*first) && (*first)==(*last))
		{
			(*first)=nullptr;
			(*last)=(*first);
			delete current;
			return;
		}
	}
	while(current!=nullptr) {
		if(current->data==delete_element && current!=(*last)) {
			prev->next=current->next;
			delete current;
			return;
		} else if(current->data==delete_element && current==(*last)) {
			(*last)=prev;
			(*first)=(*last);
			(*last)->next=nullptr;
			delete current;
			return;
		}
		prev=current;
		current=current->next;
	}
	cout<<"list dont have this element"<<endl;
}

[Serge_Bliznykov]

я бы проще записал:

if(current->data==delete_element) {
		if(current==(*first)) {
			if ((*first)==(*last)){
				(*last)=nullptr;
			}
			(*first)=(*first)->next;
			delete current;
			return ;
		}
}

[Shadownx]

Сообщение от Serge_Bliznykov

я бы проще записал:

if(current->data==delete_element) {
		if(current==(*first)) {
			if ((*first)==(*last)){
				(*last)=nullptr;
			}
			(*first)=(*first)->next;
			delete current;
			return ;
		}
}

спасибо вам огромное,за подсказку и помощь