为什么会出现运行时错误动态内存分配（堆）？

Question

每次我取消注释 l70-72 时，我的程序都会发生运行时错误。这不是我第一次遇到这个特殊问题。

我创建了一个指向对象数组的指针，将该指针发送到一个函数中，尝试在 main 中执行该函数中定义的命令并得到一个运行时错误，所有这些都是在使用堆的同时进行的。该指针或数组中的指针显然会发生某些事情。

当我使用以下任何一项调用manage_Rectangle_arr(string strCommand, Rectangle * *arr, short *siArrayL) 时会发生错误：“填充数组”、“获取区域”和“空内容”（显然最后两个本质上与“填充数组”相关，因为它们需要它首先被执行才能起作用）

void manage_Rectangle_arr(string, Rectangle * *, short*);

int main()
{
    Rectangle * * arr;
    short siArrayL=1;

    manage_Rectangle_arr("make array", arr, &siArrayL);
    manage_Rectangle_arr("fill array", arr, &siArrayL); //Problem here
    manage_Rectangle_arr("get areas", arr, &siArrayL); //Problem here
    manage_Rectangle_arr("empty contents", arr, &siArrayL); //Problem here
    manage_Rectangle_arr("delete array", arr, &siArrayL);
}

void manage_Rectangle_arr(string strCommand, Rectangle * *arr, short *siArrayL)
{
    if(strCommand=="make array")
    arr = new Rectangle * [ *siArrayL];

    if(strCommand=="fill array") //Problem here
    for(short s=0; s< *siArrayL; ++s)
    arr[s]= new Rectangle(1, 1);

    if(strCommand=="get areas") //Problem here
    for(short s=0; s< *siArrayL; ++s)
    cout << arr[s]->getArea();

    if(strCommand=="empty contents") //Problem here
    for(short s=0; s< *siArrayL; ++s)
    delete arr[s];

    if(strCommand=="delete array")
    delete [] arr;
}

//When I call manage_Rectangle_arr(string strCommand, Rectangle * *arr, short *siArrayL) with any of the following: "fill array", "get areas" and "empty contents" (obviously the last two are related to "fill array" in nature, since they need it to be executed first in order to function)

Full C++ Source Code

Answer 1

从根本上说，这段代码有两个问题，除了 Max 指出的按值调用问题。

第一个错误是你没有正确使用函数，并且滥用字符串来给出命令。

第二个错误是you are using pointers。每门 C++ 课程都犯了这个错误。大多数 C++ 代码中没有指针。不要养成坏习惯。

更正并简化后，代码应如下所示：

int main()
{
    short siArrayL=1;
    std::vector<Rectangle> arr(siArrayL);

    fill_array(arr);
    print_areas(arr);
}

两种方法：

void fill_array(std::vector<Rectangle>& arr) {
    for (std::vector<Rectangle>::iterator i = arr.begin(); i != arr.end(); ++i)
        *i = Rectangle(1, 1);
}

void print_areas(std::vector<Rectangle> const& arr) {
    for (std::vector<Rectangle>::const_iterator i = arr.begin(); i != arr.end(); ++i)
        std::cout << i->getArea();
}

请特别注意“空内容”和“删除数组”如何不再使用。不使用指针只有两个好处。

Answer 2

尽管这似乎得到了很多反对意见，但我会尽力提供一个有用的答案。 :)

您的数组是Rectangle** 类型 - 即指向矩形指针的指针。您按值将其发送到方法中。这意味着您原始方法中的arr 的值将被复制到manage_Rectangle_arr 中的变量arr 中。因此，当您为 arr 分配一个值（new 内存区域/数组的地址）时，arr 将仅在被调用函数中包含该值 - 因为它们是完全不同的变量。

如果您改为将 manage_Rectangle_arr 声明为

void manage_Rectangle_arr(string strCommand, Rectangle * * &arr, short *siArrayL)

该变量/内存区域将通过引用发送，这意味着它将是另一个自动的间接级别。您可以通过使用 Rectangle*** 类型的变量并取消引用变量 arr 来完成此操作，但这很快就会变得愚蠢。

您的代码中还有许多其他“非最佳实践”，但我不会在这里全部讨论。 :) 希望这能帮助您重回正轨！