Объединить два отсортированных связанных списка на месте с помощью оператора + - PullRequest
Новая
       14

Объединить два отсортированных связанных списка на месте с помощью оператора +

0 голосов
/ 08 ноября 2019

Я пытался объединить два отсортированных связанных списка без использования дополнительной памяти, и я пытался перегрузить оператор +. Я думаю, что я, возможно, не понимал, что оператор перегружен настолько хорошо, или что, возможно, я возился с некоторыми указателями, с которыми я не должен связываться. Я также включил перегрузку операторов << и >>, потому что, может быть, я что-то там балую, хотя я в этом сильно сомневаюсь. 

#include <iostream>

using namespace std;
struct node{

    int value;
    node* next;

};
class LinkedList{

public:
    node *head, *tail;
    LinkedList();
    void AddElement(int);
    LinkedList& operator + (const LinkedList&);
    friend ostream& operator << (ostream&, const LinkedList&);
    friend istream& operator >> (istream&, LinkedList&);

};
LinkedList& LinkedList::operator + (const LinkedList& b){

    LinkedList c;
    node* temp_head;
    node* temp_a = head;
    node* temp_b = b.head;
    if(temp_a == NULL){
        temp_head = temp_b;
    }
    if(temp_b == NULL){
        temp_head = temp_a;
    }
    if(temp_a->value < temp_b->value){
        temp_head = temp_a;
    }else{
        temp_head = temp_b;
        temp_b = temp_a;
        temp_a = temp_head;
    }
    while(temp_a->next != NULL){
        if(temp_a->next->value > temp_b->value){
            node* temp = temp_b;
            temp_b = temp_a->next;
            temp_a->next = temp;
        }
        temp_a = temp_a->next;
    }
    temp_a->next = temp_b;
    while(temp_b->next != NULL){
        temp_b = temp_b->next;
    }
    c.head = temp_head;
    c.tail = temp_b;
    cout << c;
    return c;
}
LinkedList::LinkedList(){

    head = NULL;
    tail = NULL;

}
istream& operator >> (istream& in, LinkedList& l){

    cout << "New List" << endl;
    cout << "Number of elements in the list:" << endl;
    int n;
    cin >> n;
    for(int i = 0; i < n; i++){
        int new_value;
        cin >> new_value;
        l.AddElement(new_value);
    }

    return in;

}
ostream& operator << (ostream& out, const LinkedList& l){

    node* p = l.head;
    while(p){
        cout << p->value << " ";
        p = p->next;
    }
    cout << endl;

    return out;

}
void LinkedList::AddElement(int new_value){

    // function that adds a new element at the end of the list
    node* q =  new node;
    q->value = new_value;
    q->next = NULL;
    if(head == NULL){
        head = q;
        tail = q;
        q = NULL;
    }else{
    tail->next = q;
    tail = q;
    }

}
int main()
{
    LinkedList a, b;
    cout << "List 1." << endl;
    cin >> a;
    cout << a;
    cout << "List 2." << endl;
    cin >> b;
    cout << b;
    cout << (a + b);
    return 0;
 }

1 Ответ

1 голос
/ 08 ноября 2019

Ваш оператор + перегрузка возвращает ссылку на объект, который вы создали в перегрузке. Когда вы возвращаетесь из-за перегрузки, объект уничтожается, оставляя вам висящую ссылку. Включите предупреждения, так как это должно было быть обнаружено вашим компилятором.

...