C ++ Удаление из связанного списка - PullRequest
Новая
       22

C ++ Удаление из связанного списка

0 голосов
/ 04 мая 2018

Итак, в моей текущей программе мне поручено хранить вектор ветвей. Эти ветви содержат строковое имя и указатель на узел.

В этих узлах хранится книга с именем автора, названием и количеством копий.

В целом, это должно создать структуру данных связанного списка.

В настоящее время я пытаюсь написать программу для удаления или «извлечения» определенной книги из ветки.

После добавления этих книг в ветку АО.

Стэн Мун

Билл Сан

Крис Граунд

Lan Sky

Я пытаюсь посмотреть книгу Криса "Земля". Если книга имеет несколько копий (что в данном случае не имеет), она просто уменьшает количество копий на одну. Если у него есть только одна копия, он удаляет его, устанавливает объект, который ранее указывал на него, на объект, на который указывает Земля, а затем удаляет связь между Землей и ее следующей.

Однако по какой-то причине после выполнения этой функции в моем связанном списке ничего не изменилось. Есть ли причина для этого?

Заранее спасибо! 

#include "Library.h"
#include <vector>
#include <string>
#include <iostream>

using namespace std;
int total;
int index;

Library::Library()
{

}

struct Node //Has everything in it
{
    string author;
    string title;
    int copies;
    Node* next;
};

struct Branch // Stores just the branch, and a point to the node with information in it.
{
    string b_name;
    Node* next;
};

vector<Branch*> lib;

void Library::start()
{
    int choice = 0;
    do
    {
        cout << "Please select a choice." << endl;
        cout << " " << endl;
        cout << "1. Create a branch and insert its books" << endl;
        cout << "2. Given an author name, a title and a branch name, CHECKOUT that book from the branch." << endl;
        cout << "3. Given an author name, title and a branch name, RETURN that book to the branch." << endl;
        cout << "4. Given an author name, title and branch name, FIND the number of copies of that book are available in that branch." << endl;
        cout << "5. PRINT all books contained in a branch." << endl;
        cout << "6. Exit the program." << endl;

        cin >> choice;

        switch (choice)
        {
        case 1:
            insert();
            break;

        case 2:
            checkout();
            break;

        case 3:
            Return();
            break;

        case 4:
            find();
            break;

        case 5:
            printAll();
            break;

            // TODO: other choises
        }
    } while (choice != 6);
}

void Library::insert()
{
    string br;
    string auth;
    string titl;

    cout << "What is the name of the branch?" << endl;
    cin >> br;

    Branch *branch = new Branch();
    branch->b_name = br;
    lib.push_back(branch);
    Node* lastNode = nullptr;

    do
    {
        cout << "What is the author and title of the book?" << endl;
        cin >> auth >> titl;
        if (auth == "NONE" && titl == "NONE") break;

        Node *book = new Node();
        book->author = auth;
        book->title = titl;
        book->copies++;
        if (lastNode == nullptr) {
            branch->next = book;
        }
        else {
            lastNode->next = book;
        }
        lastNode = book;
    } while (auth != "NONE" && titl != "NONE");

    start();
}

void Library::checkout()
{
    string auth;
    string titl;
    string bran;
    bool success = false;

    cout << "Insert author, title and branch" << endl;
    cin >> auth >> titl >> bran;

    for (unsigned int i = 0; i < lib.size(); i++)
    {
            auto* branch = lib[i];
            auto* node = branch->next;
            Node* previous = nullptr;

            while (node)
            {
                if (node->author == auth && node->title == titl && branch->b_name == bran)
                    if (node->copies > 1) node->copies--; success = true;  break;
                if (node->copies == 1)
                {
                    previous->next = node->next;
                    node->next = nullptr;
                    success = true;
                    break;
                }

                if (!success)
                {
                    previous = node;
                    node = node->next;
                }

            }
            if (success)
            {
                cout << "Complete!" << endl;
                cout << "" << endl;
            }


    }
    start();
}

1 Ответ

0 голосов
/ 04 мая 2018

Что по этому поводу: 

    void Library::checkout()
{
    string auth;
    string titl;
    string bran;
    bool success = false;

    cout << "Insert author, title and branch" << endl;
    cin >> auth >> titl >> bran;

    for (unsigned int i = 0; i < lib.size(); i++)
    {
        auto* branch = lib[i];
        auto* node = branch->next;
        Node* previous = nullptr;

        while (node)
        {
            if (node->author == auth && node->title == titl && branch->b_name == bran)
            {
                if (node->copies > 1)
                {
                    node->copies--;
                    success = true; 
                    break;
                }
                else
                {
                    if (previous)
                    {
                        previous->next = node->next;
                    }
                    else
                    {
                        branch->next = node->next;
                    }
                    delete node;
                    success = true;
                    break;
                }
            }
            previous = node;
            node = node->next;
        }
        if (success)
        {
            cout << "Complete!" << endl;
            cout << "" << endl;
        }
    }
    //  no need to call start() as start() has a do/while loop
    // start();

}

Редактировать: исправить неправильное размещение условия удаления

...