• 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

Object Storage Arubacloud
0 głosów
299 wizyt
pytanie zadane 4 czerwca 2019 w C i C++ przez Kaper99 Użytkownik (660 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 2019 przez j23 Mędrzec (194,920 p.)
wybrane 6 czerwca 2019 przez Kaper99
 
Najlepsza

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

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

Podobne pytania

0 głosów
1 odpowiedź 207 wizyt
pytanie zadane 26 maja 2018 w C i C++ przez Roman1212 Początkujący (460 p.)
0 głosów
1 odpowiedź 336 wizyt
pytanie zadane 4 maja 2018 w C i C++ przez kikosiak Obywatel (1,010 p.)
0 głosów
1 odpowiedź 1,561 wizyt
pytanie zadane 2 kwietnia 2017 w C i C++ przez chacken Użytkownik (820 p.)

92,555 zapytań

141,403 odpowiedzi

319,557 komentarzy

61,940 pasjonatów

Motyw:

Akcja Pajacyk

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

Oto polecana książka warta uwagi.
Pełną listę książek znajdziesz tutaj.

Akademia Sekuraka

Kolejna edycja największej imprezy hakerskiej w Polsce, czyli Mega Sekurak Hacking Party odbędzie się już 20 maja 2024r. Z tej okazji mamy dla Was kod: pasjamshp - jeżeli wpiszecie go w koszyku, to wówczas otrzymacie 40% zniżki na bilet w wersji standard!

Więcej informacji na temat imprezy znajdziecie tutaj. Dziękujemy ekipie Sekuraka za taką fajną zniżkę dla wszystkich Pasjonatów!

Akademia Sekuraka

Niedawno wystartował dodruk tej świetnej, rozchwytywanej książki (około 940 stron). Mamy dla Was kod: pasja (wpiszcie go w koszyku), dzięki któremu otrzymujemy 10% zniżki - dziękujemy zaprzyjaźnionej ekipie Sekuraka za taki bonus dla Pasjonatów! Książka to pierwszy tom z serii o ITsec, który łagodnie wprowadzi w świat bezpieczeństwa IT każdą osobę - warto, polecamy!

...