• Najnowsze pytania
  • Bez odpowiedzi
  • Zadaj pytanie
  • Kategorie
  • Tagi
  • Zdobyte punkty
  • Ekipa ninja
  • IRC
  • FAQ
  • Regulamin
  • Książki warte uwagi

Błąd przy wypisywaniu | lista dwukierunkowa

0 głosów
189 wizyt
pytanie zadane 4 czerwca w C i C++ przez Kaper99 Użytkownik (530 p.)

Witam,
Przy wypisywaniu elementów z listy dwukierunkowej program psuje się zawsze przy wypisywaniu ostatniego elementu tzn. wypisuje wszystkie elementy z listy, a gdy ma wypisać ostatni element wyrzuca błąd.
Podejrzewam, że gdzieś źle ustawiam wskaźnik na następny element, ale nie wiem gdzie. Wrzucam kod, proszę  o przeglądnięcie i pomoc 
 

#include <iostream>
#include <cstdlib>
using namespace std;
struct element {
    int date;
    element* next;
    element* previous;
};
struct lista {
    element* head;
    element* tail;
    int liczba_el = 0;
};
void add_last(lista& l);
void add_first(lista& l);
void show_list(lista l);
int main()
{
    lista l;

    add_first(l);
    add_last(l);
    add_first(l);
    add_first(l);

    show_list(l);

    return 0;
}
void add_last(lista& l)
{

    int a;
    element* nowy = new element;
    cout << "Podaj wartosc do dodania  na koniec\n";
    cin >> a;
    nowy->date = a;
    nowy->next = nullptr;
    nowy->previous = nullptr;
    if (l.tail != nullptr) {
        l.tail->next = nowy;
        nowy->previous = l.tail;
    }
    l.tail = nowy;
    if (l.head == nullptr) {
        l.head = nowy;
    }
    l.liczba_el = l.liczba_el + 1;
}
void add_first(lista& l)
{

    int a;
    element* nowy = new element;
    cout << "Podaj liczbe do dodania na poczatek\n";
    cin >> a;
    nowy->date = a;
    nowy->next = nullptr;
    nowy->previous = nullptr;
    if (l.head != nullptr) {

        l.head->previous = nowy;
        nowy->next = l.head;
    }
    l.head = nowy;
    if (l.tail == nullptr) {
        l.tail = nowy;
    }
    l.liczba_el = l.liczba_el + 1;
}
void show_list(lista l)
{
    element* temp = l.head;

    for (int i = 1; i <= l.liczba_el; i++) {
        cout << "Liczba nr to " << i << " ";
        cout << temp->date << endl;
        temp = temp->next;
    }
}

 

1 odpowiedź

0 głosów
odpowiedź 4 czerwca przez j23 VIP (105,900 p.)
wybrane 6 czerwca przez Kaper99
 
Najlepsza

head i tail w lista powinieneś ustawić na nullptr.

komentarz 6 czerwca przez Kaper99 Użytkownik (530 p.)
Pomogło, dziękuję!

Podobne pytania

0 głosów
1 odpowiedź 61 wizyt
pytanie zadane 26 maja 2018 w C i C++ przez Roman1212 Początkujący (370 p.)
0 głosów
1 odpowiedź 92 wizyt
pytanie zadane 4 maja 2018 w C i C++ przez kikosiak Obywatel (1,010 p.)
0 głosów
1 odpowiedź 295 wizyt
pytanie zadane 2 kwietnia 2017 w C i C++ przez chacken Użytkownik (820 p.)
Porady nie od parady
Komentarze do pytań nie służą do odpowiadania, od tego jest wydzielona sekcja odpowiedzi. Funkcją komentarzy jest natomiast możliwość uzyskania dodatkowych informacji na temat samego posta.Komentarze

65,794 zapytań

112,440 odpowiedzi

237,547 komentarzy

46,729 pasjonatów

Przeglądających: 232
Pasjonatów: 10 Gości: 222

Motyw:

Akcja Pajacyk

Pajacyk od wielu lat dożywia dzieci. Pomóż klikając w zielony brzuszek na stronie. Dziękujemy! ♡

Oto dwie polecane książki warte uwagi. Pełną listę znajdziesz tutaj.

...