Jak usunąć obiekt z listy <list> C++

Question

Witam nie mogę sobie poradzić z funkcją remove stosowaną do list

mam obiekt Klient który przechowywuje zmienne imię, oraz nazwisko

Kilku takich klientów jest zapisanych na listę lista_klientow

nie mam pojęcia jak usunąć klienta o wybranym nazwisku z listy
Za wikipedią: void remove(const T& wartosc)
próbowałem lista_klientow("Kowalski") ale to nie działa bo klient jest obiektem

Answer 1

Cześć,

wydaje mi się, że nie masz przeładowanych operatorów dlatego lista nie może sobie poradzić z usunięciem obiektu. Spróbuj tak. Utwórz klasę:

class  ID
{
public:
    string imie;
    string nazwisko;
    bool operator ==(const ID& dane) const { return imie == dane.imie && nazwisko == dane.nazwisko;}
    bool operator !=(const ID& dane) const{return !operator==(dane); }
};

Tu masz przeładowane dwa operatory == oraz !=. Możesz sobie wprowadzić własne warunki w zależności od problemu.

Następnie tworzysz wartość, która ma być usunięta z listy np. tak

 ID dane_do_usuniecia;
    dane_do_usuniecia.imie = "Jan";
    dane_do_usuniecia.nazwisko = "Kowalski";
    lista.remove(dane_do_usuniecia);

 

Comments

Powiedzmy mam listę klientów, robię kopię jednego klienta z tej listy, i chcę wrzucić go do funkcji remove() to nie działa

void Usun_klienta(string klient_1)
{
    Klient karamba;
for(list<Klient>::iterator i=lista_klientow.begin(); i!= lista_klientow.end(); ++i)
    {  karamba=*i;
       if(karamba.nazwisko==klient_1)
       {
           lista_klientow.remove(karamba);
       }
       }
       }

Coś takiego mi nie działa (brak odpowiedzi)

 

void Usun_klienta(string klient_1)
{
    Klient karamba;
    Klient nowy
for(list<Klient>::iterator i=lista_klientow.begin(); i!= lista_klientow.end(); ++i)
    {  karamba=*i;
       if(karamba.nazwisko==klient_1)
       {
       cin<<nowy.imie;    //podaj imie klienta ktory ma byc usuniety
       cin<<nowy.nazwisko; //podaj nazwisko klienta ktory ma byc usuniety
           lista_klientow.remove(nowy);
       }
       }
       }

A coś takiego już poprawnie usuwa element z listy
 

---

Powiem szczerze, że nie wiem za bardzo co jest złego.

Spróbuj działać bezpośrednio na obiekcie

void Usun_klienta(string klient_1)
{
 
for(list<Klient>::iterator i=lista_klientow.begin(); i!= lista_klientow.end(); ++i)
    {  
       if((*i).nazwisko==klient_1)
       {
           lista_klientow.remove(karamba);
       }
       }
       }

BTW

Po co przechodzić przez całą pętle. Nie łatwiej by Ci było:

void Usun_klienta(string imie, string nazwiso)
{
    Klient karamba;
   karamba.imie = imie;
  karamba.nazwisko = nazwisko;
lista_klientow.remove(karamba);
       }

lub bezpośrednio jako argument użyć obiekt Klient

void Usun_klienta(Klient doUsuniencia) {

lista_klientow.remove(doUsuniencia);

}

Nie ma w ogóle sensu używania pętli for()

---

No tak, na początku chciałem stworzyć kod gdzie podawałbym samo nazwisko klienta, pętla przechodziłaby po całej liście i po znalezieniu klienta o tym nazwisku kopiowałbym go do zmiennej typu Klient którego chciałem użyć do funkcji remove(). Jak przestalo działać to po prostu przeładowałem operatory tak żeby porównywały samo nazwisko w obiektach, i tak to rozwiązałem, a cała pętla już została

Myślałem, że jest jakiś powód dla którego nie mogę wsadzić obiektu z listy do funkcji remove() ale widocznie coś musiałem zepsuć w kodzie, a jest tego kilkaset linijek :D

Dzięki wielkie za ten kod
 

void Usun_klienta(string imie, string nazwiso)
{
    Klient karamba;
   karamba.imie = imie;
  karamba.nazwisko = nazwisko;
lista_klientow.remove(karamba);
       }

Człowiek widzi, ile się musi jeszcze nauczyć bo efekt ten sam, a prostota aż mnie przeraża, żadnych pętli, ani if`ów a robi to co powinno :D

---

Pamiętaj aby dążyć do jak najprostszego kodu. Wtedy bd łatwiej się pracować Tobie oraz ewentualnym wspólnikom. Co do

Myślałem, że jest jakiś powód dla którego nie mogę wsadzić obiektu z listy do funkcji remove() ale widocznie coś musiałem zepsuć w kodzie, a jest tego kilkaset linijek :D

to polecam pisać nie od razu cały program lecz rozbicie go na moduły i testować. Wtedy łatwiej ogarnąć program. Do sprawdzenie poprawności programu służy debugger

https://pl.wikipedia.org/wiki/Debugger

Obczaj go sobie w Twoim IDE powinno to ułatwić testowanie programu.

---

znaczy mam debuggera, i w programie ustawiłem sobie menu na zasadzie uzytkownik wybiera, opcje, a program wykonuje na raz max 1-2 funkcje.
 

Debugera znam, nawet czasami zdarza mi się skorzystać :D

Dzięki za pomoc, i za wszystkie rady :D