动态分配的数组和堆损坏

Question

我是一个与指针和动态数组分配搏斗的新手程序员。

对于我正在处理的程序，我需要使用用户输入为每个索引处的维度和值分配一个二维。

我后来使用 delete[] 取消分配数组并得到堆损坏错误。

这是我的分配：

    cin>>w>>h;

    int** pond;

    //allocating array
    pond = new int*[w];

    for(int j = 0; j<w; j++)
    {
        pond[j] = new int[h];
    }



    // inputting array values
    for(int k = 0; k < h; k++)
    {
        for(int l = 0; l < w; l++)
        {
            cin>>entry;
            pond[l][w] = entry;
            cout<<"entry is: "<< pond[l][w]<<endl;
        }
    }
    cout<<pond[0][0]<<endl;

最后的 cout 语句是尝试验证数组中的一个值。问题是，它给出了类似于 -8023940... 的随机数，它甚至不接近用户输入，它是简单的整数 (1-9)。但是，for 循环中的 cout 语句会产生正确的值。

我的问题是，我在这里做错了什么？分配看起来很简单，但是一旦退出 for 循环，值就会很疯狂。

这是我的整个程序：

#include <iostream>
#include <string>
using namespace std;


int main()
{

int trials = 0;
int w = 0;
int h = 0;
int entry = 0;
int fishMax = 0;
int holdFishRow = 0;
int holdFishCol = 0;
int fishTotal = 0;
int highRow = 0;
int highCol = 0;

char done = 's';


cin>>trials;

for(int i = 0; i < trials; i++)
{
    cin>>w>>h;

    int** pond;

    //allocating array
    pond = new int*[w];

    for(int j = 0; j<w; j++)
    {
        pond[j] = new int[h];
    }



    // inputting array values
    for(int k = 0; k < h; k++)
    {
        for(int l = 0; l < w; l++)
        {
            cin>>entry;
            pond[l][w] = entry;
            cout<<"entry is: "<< pond[l][w]<<endl;
        }
    }
    cout<<pond[0][0]<<endl;


    // "fishing"...bombing each array index bomberman style.
    for(int z = 0; z < h; z++)
    {
        for(int j = 0; j < w; j++)
        {
            for(int k = 0; k < h; k++)
            {
                holdFishCol = holdFishCol + pond[j][k];
                cout<<pond[j][k]<<endl;
            }

            for(int l = 0; l < w; l++)
            {
                holdFishRow += pond[l][z];
            }

            fishTotal = holdFishCol + holdFishRow;

            if(fishTotal > fishMax)
            {
                fishMax  = fishTotal;
                highRow = j;
                highCol = z;
            }


        }
    }





    cout<< "#"<<i<<": ("<<highRow<<", "<<highCol<<") "<<fishMax<<endl;

    for(int q = 0; q < w; q++)
    {
        delete[] pond[q];
    }

    delete[] pond;

}



cin>>done;

return 0;
}

它遍历每个给定的数组，并以交叉模式（如加号+）添加每个索引的值，总计，并为每个索引给出分数。

Answer 1

问题很可能是这样的说法：

pond[l][w] = entry;

除非w 小于h，否则你写的越界。我怀疑你的意思是

pond[l][k] = entry;

Answer 2

更改此代码

pond = new int*[w];

for(int j = 0; j<w; j++)
{
    pond[j] = new int[h];
}

以下方式

pond = new int*[h];

for(int j = 0; j<h; j++)
{
    pond[j] = new int[w];
}

因为在初始化数组时，您使用的第一个索引在 [0, h) 范围内

// inputting array values
for(int k = 0; k < h; k++)
{
    for(int l = 0; l < w; l++)
    {
        cin>>entry;
        pond[l][w] = entry;
        cout<<"entry is: "<< pond[l][w]<<endl;
    }
}

即使在这个循环中也有一个错字。而不是

pond[k][w] = entry

应该有

pond[l][l] = entry

您总是为循环中的索引指定无效范围。所以你必须更新你所有的循环。