更新：C++ 指针片段

Question

再次问候，再次感谢为第一个问题提供答案的所有人。以下代码已更新为包含每个分配的两个函数。

要查看原始问题，请点击here。

我非常确信这满足了任务的要求，但我再次非常感谢任何帮助。我是否适当地修改了删除语句？再次感谢。

#include<iostream>
#include<string>

int** createArray(int, int);
void deleteArray(int*[], int);

using namespace std;

int main()
{
    int nRows;
    int nColumns;

    cout<<"Number of rows: ";
    cin>>nRows;

    cout<<"Number of columns: ";
    cin>>nColumns;

    int** ppInt = createArray(nRows, nColumns);

    deleteArray(ppInt, nRows);
}

int** createArray(int nRows, int nColumns)
{
    int** ppInt = new int*[nRows];

    for (int nCount = 0; nCount < nRows; nCount++)
    {
        ppInt[nCount] = new int[nColumns];
    }

    return ppInt;
}

void deleteArray(int** nPointer, int nRows)
{
    for (int nCount = 0; nCount < nRows; nCount++)
    {
        delete[] nPointer[nCount];
    }

    delete[] nPointer;
}

附：这是作业文档本身，以防万一：

(1) 设计并实现一个为二维整数数组分配内存的函数：该函数应该接受两个整数作为参数，一个为行数，一个为列数。您需要在此函数中使用“new”运算符。请记住，我们需要首先创建一个指针数组。然后，对于该数组中的每个指针，我们需要创建一个整数数组。这个函数应该返回一个指向二维整数数组的指针。

(2) 设计并实现一个函数来为这个二维数组释放内存：该函数应该有两个参数（一个指向二维整数数组的指针，另一个是数字数组中的行数）。在函数中，您应该使用“delete”运算符为这个二维数组取消分配内存。您应该首先删除每一行（整数数组），然后删除指针数组。

Answer 1

代码看起来不错。

但是，对于我们人类来说，您可能想要解决一些问题：

1. 您的函数签名（声明）缺少参数名称。更适合：

   int** createArray(int rows, int columns); void deleteArray(int** array, int rows);

2. 您的函数名称对于它们真正创建/删除的内容没有太多描述性。例如，create2DArray 将是一个更明智的选择。

3. 变量的n 前缀伤害了我的眼睛。 numRows 或 rowCount 更具可读性。
4. 同样，ppInt 也很疯狂。尝试array（也适用于nPointer，以保持一致性）。 （很遗憾，你不能写2dArray。）
5. 使用i 作为循环计数器比nCount 或类似的更常见（尤其是对于数组索引）。我建议你改用它。

一些超越和超越的东西，供您个人练习：

1. 创建一个将rows 和cols 作为其构造函数参数的类。确保自动释放数组。
2. 使用std::vector 并为您的类创建一个resize 成员函数。 请注意，这与最初的问题有所不同，后者要求指点。
3. 创建一个copy 函数和一个clone 函数以将数据复制到另一个二维数组（可能具有不同的大小！）或克隆一个现有数组。

Answer 2

没关系。

问题在于您没有考虑代码中的异常安全性。

int** ppInt = new int*[nRows];   // ALLOC 1

for (int nCount = 0; nCount < nRows; nCount++)
{
        ppInt[nCount] = new int[nColumns]; // ALLOC 2
}

说 ALLOC 1 没问题。
但是，如果 ALLOC 2 中的任何一个失败，那么您就会遇到异常和严重的内存泄漏。

例如。
你在第四次调用 ALLOC 2 时失败了。然后你从 ALLOC 1 和前三个调用 ALLOC 2 泄漏内存。现在在你的情况下，代码是如此微不足道，它可能无关紧要。但这是您在编写 C++ 代码时应始终牢记的事情。

如果抛出异常，这里会发生什么，哪些资源会被泄露，哪些资源不会被正确清理。

我认为您应该考虑将二维数组包装在一个类中，这样即使存在异常，您也可以保证正确分配和取消分配内存。

Answer 3

在我看来是合理的。

不过，我不确定是否要提交一个全新的问题。作为对原版的修正，它可能会更好。

Answer 4

对于“deleteArray”，你的原型和定义并不完全相同：

void deleteArray(int*[], int);
void deleteArray(int** nPointer, int nRows)

它们确实具有相同的含义，但为了清楚起见，我认为最好让它们完全相同（使用 `int**' 来强调您正在传递指针的事实）以保持一致性。

另外，在原型中包含参数名称。