与c ++中的删除功能混淆

时间:2017-07-26 18:57:49

标签: c++ c++11 linked-list c++14 delete-operator

所以我用c ++编写了这段代码

#include "ContactBook.hpp"

int main(){

    std::string name;
    ContactList *cl1 = new ContactList();

    while(true){

        std::cout << "Enter a name or press Q to quit" << std::endl;
        std::cin >> name;

        if(name=="Q"||name=="q")
            break;
        else{
            cl1->addToHead(name);
        }
    }

    cl1->print();
    delete cl1;

    return 0;
}

我的标题文件定义 - &gt;

#ifndef ContactBook_hpp
#define ContactBook_hpp

#include <iostream>

class Contact{
    friend std::ostream &operator<<(std::ostream &os, const Contact &c);
    friend class ContactList;
public:
    Contact(std::string name);
private:
    std::string name;
    Contact* next;
};

class ContactList{
public:
    ContactList();
    void addToHead(const std::string &name);
    void print();
private:
    Contact *head;
    static int size;
};

#endif

现在这是我的头文件函数定义。 ContactList和Contact是两个类。联系人列表是联系人的朋友类。

#include "ContactBook.hpp"

Contact::Contact(std::string name){
    this->name = name;
    next = NULL;
}
std::ostream &operator<<(std::ostream &os, const Contact &c){
    os << "Name: " << c.name << std::endl;
    return os;
}

ContactList::ContactList():head(NULL){};
int ContactList::size = 0;

void ContactList::addToHead(const std::string &name){

    Contact *newOne = new Contact(name);

    if(head==NULL){
        head = newOne;
    }
    else{
        newOne->next = head;
        head = newOne;
    }
    ++size;

}

void ContactList::print(){

    Contact *temp;
    temp = head;

    while(temp!=NULL){
        std::cout << *temp;
        temp = temp->next;
    }
}

每当我添加

时都会出现问题
delete newOne;

在addToHead定义的第3个代码段中的++ size之后。

我最终在名称的奇数输入上有一个无限循环(除了1)!我只是不明白为什么会这样!关于这一点的一些知识将非常感激:D!

1 个答案:

答案 0 :(得分:4)

在这里,你的addToHead:

Contact *newOne = new Contact(name);

if(head==NULL){
    head = newOne;
}
else{
    newOne->next = head;
    head = newOne;
}
++size;

可以这样写:

Contact *newOne = new Contact(name);

newOne->next = head; // if (head==NULL) newOne->next=NULL else newOne->next=head;
head = newOne;
++size;

您将newOne的值分配给head。

但是,如果你在++ size之后添加删除,就像你说的那样,head会指向被删除的内容。

您的打印方法中发生的情况是您取消引用已删除的内容。 当您取消引用已删除的内容时,会发生什么是未定义的行为,这可能会导致奇怪的输出或崩溃。

您可能希望使用smart pointers来避免访问已删除的内存和内存泄漏。