pętla powoduje zwieszenie programu

Question

void MojaKlasa::usun_ucznia() {
    cout << "Podaj nr dziennika:";
    int numer;
    cin >> numer;
    for (std::vector<Uczen>::iterator temp = tab.begin(); temp < tab.end(); temp++) {
        if (temp->nr_dziennika == numer)
            tab.erase(temp);
    }
    zapisz_z_pliku();
}

Comments

Zwykle warunkiem kontynuacji pętli jest: iterator != kontener.end(). Dla vectora może zadziała <, ale tak bardziej uniwersalnie jest !=.

Po wykonaniu erase() (ogólnie modyfikacji struktury kontenera) iterator jest już nieważny i nie możesz kontynuować. Samo erase natomiast zwraca poprawiony iterator. Można zrobić pętle w której jeśli dany element chcesz usunąć robisz erase zamiast ++.

 

---

for (auto temp = tab.begin(); temp != tab.end();) {
    if (temp->nr_dziennika == numer)
        temp = tab.erase(temp);
    else ++temp;
}

Lub zgodnie z idiomem erase-remove:

auto it = std::remove_if(tab.begin(), tab.end(), [numer] (const Uczen& u) {
        return u.nr_dziennika == numer;
    });

tab.erase(it, tab.end());