为什么在范围结束时销毁临时对象后c ++程序崩溃

时间:2015-01-17 06:59:49

标签: c++ pointers temporary-objects

所以我有点困惑。在调用changeList()之后尝试printList()时,这段代码将失败。但是,当我删除析构函数时,代码运行时没有崩溃。我的问题是,为什么?我理解在这里的代码中,我通过值将对象传递给changeList,因此,临时对象被创建为参数的副本。对这个临时对象所做的任何更改都不会影响我传入的原始对象。那么为什么程序崩溃就像我访问被破坏的对象一样?在完成changeList之后,不应该销毁临时对象,然后printList应该只打印1到10而不是1到45。

void printList(Node*);
void changeList(LinkedList);

int main(){

    LinkedList list;

    for (int i = 0; i < 10; i++)
        list.append(new Node(i+1, nullptr));

    changeList(list);
    printList(list.getHead());
    system("pause");
    return 0;
}

void changeList(LinkedList list){
    list.append(new Node(45, nullptr));
}

void printList(Node* head){
    Node* temp = head;
    if (temp != nullptr){
        cout << temp->value << endl;
        printList(temp->next);
    }//end if
}

1 个答案:

答案 0 :(得分:1)

如果我假设正确,LinkedList的析构函数将销毁列表中的所有节点。我还假设复制构造函数生成列表的浅表副本。 (如果没有显式实现复制构造函数,则这是默认值。)当运行代码时会发生这种情况:

void printList(Node*);
void changeList(LinkedList);

int main(){

    LinkedList list; //first instance is created

    for (int i = 0; i < 10; i++)
        list.append(new Node(i+1, nullptr)); // nodes added to the first instance

    changeList(list); // copy constructor called, second instance is 
                      // created with the same set of node objects


    printList(list.getHead()); // nodes does not exist any more because of
                               // the destructor called at the end of changeList


    system("pause");
    return 0;
}

void changeList(LinkedList list)  // copy constructor called, second instance is created
    list.append(new Node(45, nullptr));  // nodes added to the same chain of nodes
} // destructor is called, all the nodes are destroyed

void printList(Node* head){
    Node* temp = head;
    if (temp != nullptr){
        cout << temp->value << endl;
        printList(temp->next);
    }//end if
}

您可以通过接受changeList函数中的参考参数来解决此问题:

void changeList(LinkedList &list)  // no copy
    list.append(new Node(45, nullptr));  
} // no destructor is called