C ++在不同函数中将指针分配给动态内存中的数组

时间:2018-11-08 23:51:25

标签: c++ pointers memory-management dynamic

我刚刚在C ++中完成了有关指针和动态内存的模块,并试图完成个人任务,以便我可以练习这些概念。该程序管理名称的字符串数组。我为自己设置的目标是将列表存储在堆中(以练习使用“ new”),并且在输入新名称时动态调整列表的大小。

免责声明:我意识到使用向量很容易做到这一点,并且在为此苦苦挣扎数小时之后,我重新编写了原始代码,将向量毫无问题地用于列表。但是,我想了解我对指针如何工作的理解被打破了。

程序存在的问题是:我将名称数组初始化为具有零个元素,并且具有添加名称以处理动态大小的功能。首次调用时,似乎可以正确调整数组的大小,并在新数组中添加一个新名称。在函数内添加一个名称,我可以打印新数组的内容。我还可以将旧的数组指针重新分配给堆上新数组的地址。但是,当我在列表中添加名称后从main调用打印功能时,列表中没有名称。根据我的理解,由于我使用的是指针,因此我应该直接更新值,因此在add name函数终止后,这些值应该保持不变。另外,如果我尝试添加第二个名称,程序将崩溃。我在处理内存时出了什么问题?

我搜索了很多,找到的最接近的分辨率是这篇文章:

How to make an array with a dynamic size? General usage of dynamic arrays (maybe pointers too)?

我根据自己的理解修改了代码,但仍然无法正常工作。

    #include <stdio.h>
    #include <vector>
    #include <iostream>

    using namespace std;

    void add_name_to_list(string * my_list, size_t * list_size);
    string get_name();
    void print_names(const string *const my_list, const size_t *const list_size);

    int main()
    {
        string *name_list_ptr {nullptr};
        name_list_ptr = new string [0];
        size_t name_list_size{0};
        size_t *name_list_size_ptr {&name_list_size};

        print_names(name_list_ptr, name_list_size_ptr);
        add_name_to_list(name_list_ptr, name_list_size_ptr);
        print_names(name_list_ptr, name_list_size_ptr);
        return 0;
    }

    void add_name_to_list (string * my_list, size_t *list_size)
    {
        string new_name{get_name()};
        string *new_string_ptr{nullptr};
            new_string_ptr = new string [*list_size+1];
            // copy existing list into new list
            cout << "List size is " << *list_size << " so *list size == 0 is " << (*list_size == 0) << endl;
            if(*list_size == 0)
            {
                new_string_ptr[0] = new_name;
                *list_size = *list_size +1;
                cout << new_string_ptr[0] << " has been added to position " << *list_size << endl;

            }
            else
            {
                print_names(my_list, list_size);
                for(size_t i{0}; i < *list_size; i++)
                {
                    cout << "At position " << i << " original list is " << my_list[i] << endl;
                    new_string_ptr[i] = my_list[i];
                    cout << "The name " << new_string_ptr[i] << " has been added to position " << i << " of the new list" << endl;
                }
                new_string_ptr[*list_size - 1] = new_name;
                *list_size = *list_size + 1;
            }
            print_names(new_string_ptr, list_size);

            string *temp_ptr{nullptr};
            temp_ptr = new string [*list_size-1];
            cout << "temp ptr is " << temp_ptr << " and my list is " << my_list << endl;
            temp_ptr = my_list;
            cout << "temp ptr is " << temp_ptr << " and my list is " << my_list << endl;
            my_list = new_string_ptr;
            delete [] temp_ptr;
            new_string_ptr = nullptr;
            print_names(my_list, list_size);
    }

    string get_name()
    {
        cin.sync();
        cin.clear();
        string new_name{};
        cout << "\nEnter the full name: ";
        getline(cin, new_name);
        cin.sync();
        cin.clear();
        if(new_name.size() <= 1)
            return "0";
        else
            return new_name;    
    }

    void print_names(const string *const my_list, const size_t *const list_size)
    {
        if(*list_size == 0)
            cout << "The list is empty" << endl;
        else
            for(size_t j{0}; j < *list_size; j++)
                    cout << j << ". " << my_list[j] << endl;
    }

根据从搜索中学到的内容,我尝试了一种变体:

    cout << "temp ptr is " << temp_ptr << " and my list is " << my_list << endl;
    //my_list = new_string_ptr;
    //delete [] temp_ptr;
    //new_string_ptr = nullptr;

    delete [] my_list;
    my_list = new string[*list_size];
    my_list = new_string_ptr;
    print_names(my_list, list_size);

不幸的是,结果是相同的。

1 个答案:

答案 0 :(得分:1)

在不检查实现逻辑的情况下,您的列表不会更新,因为您正在分配my_list = new_string_ptr;,但是函数收到了void add_name_to_list (string * my_list, size_t *list_size)

当您刚接触C ++世界时,请让我清楚地解释一下: list_size是指向size_t的指针,因此,如果修改指针存储器,则更改将保留,但是如果修改指针本身,则不会。

list_size = new size_t; // This change doesn't persist
list_size++; // This change doesn't persist

*list_size++; // This change persists and the value of pointed memory was increased.

my_list发生相同的情况下,您正在尝试修改指针本身,而不是指向的内存。

因此,您应该使用:

void add_name_to_list (string * &my_list, size_t *list_size)

或者也许您更舒服

void add_name_to_list (string ** my_list, size_t *list_size)
[...]
*my_list = new_string_ptr;

希望这会有所帮助