C++动态分配2D数组类

3

我是一个很新的初学者,现在我的任务快要完成了,但是我遇到了一些麻烦。我的问题是如何检索和设置动态分配的二维数组中的值。我定义了一个类来构造二维数组,在这个过程中,我需要有一个选项来设置数组中给定点的值,并从给定点检索值。

当我运行setValue函数时,我遇到了一个段错误。作为一个初学者,越简单的术语越好。请问有人可以帮我理解我做错了什么吗?非常感谢你提前的帮助。

#include <iostream>
using namespace std;


class array2D
{
    protected:
        int xRes;
        int yRes;
        float ** xtable;

    public:
        array2D (int xResolution, int yResolution);
        ~array2D() {}
        void getSize(int &xResolution, int &yResolution);
        void setValue(int x,int y,float val);
        float getValue(int x,int y);
};


array2D::array2D(int xResolution, int yResolution)
{
    xRes=xResolution;
    yRes=yResolution;

    float ** xtable = new float*[yResolution];

    for(int i=0;i < yResolution;i++)
    {
        xtable[i] = new float[xResolution];
    }

    for(int i=0;i < yRes;i++)
    {
        for(int j=0;j < xRes;j++)
        {
            xtable[i][j]=45;
        }
    }
}

void array2D::getSize(int &xResolution, int &yResolution)
{
    xResolution=xRes;
    yResolution=yRes;
    cout << "Size of Array: " << xResolution << ", " << yResolution << endl;
}

void array2D::setValue(int x,int y,float val)
{
    xtable[x][y] = val;
}

float array2D::getValue(int x,int y)
{
    return xtable[x][y];
}

int main()
{
    array2D *a = new array2D(320,240);
    int xRes, yRes;
    a->getSize(xRes,yRes);
    for(int i=0;i < yRes;i++)
    {
        for(int j=0;j < xRes;j++)
        {
            a->setValue(i,j,100.0);
        }
    }

    for(int i=0;i < yRes;i++)
    {
        for(int j=0;j < xRes;j++)
        {
            cout << a->getValue(i,j) << " ";
        }

        cout << endl;
    }

    delete[] a;
}

1
我为你的第一篇帖子中所有的细节鼓掌。 - R Sahu
3
为什么你不使用 std::vector - Ed Heal
很遗憾,我们还没有掌握使用std :: vector的技巧,所以在这个项目中它有点“禁止”使用 :) - JY One
为什么你没有在类内部递增xTable的索引? - user366312
2个回答

6

这条线

float ** xtable = new float*[yResolution];

创建一个函数局部变量。类的成员变量仍然保持未初始化状态,这不是您想要的结果。要分配内存并将其分配给成员变量,请从该行中删除类型说明符。只需使用:
xtable = new float*[yResolution];

此外,您需要在这些行中交换yResolutionxResolution的使用。否则,getValuesetValue将不正确地使用索引。
请交换以下行中xResolutionyResolution的使用,以便您使用:
float ** xtable = new float*[xResolution];
for(int i=0;i < xResolution;i++)
{
    xtable[i] = new float[yResolution];
}

将以下行中xResyRes的使用交换,以便您使用:

for(int i=0;i < xRes;i++)
{
    for(int j=0;j < yRes;j++)
    {
        xtable[i][j]=45;
    }
}

由于您的类使用动态内存分配获取资源,您应该阅读有关“三个法则”的内容,并相应地更新您的类。

参考链接:The Rule of Three


1
请使用 std::vector,更好的方式。 - Ap31
我该如何感谢你?!这解决了我的问题。我已经研究了12个小时可能的解决方案,但是只有通过你的回复现在才明白。真的非常感激你;你是我的救命恩人。 - JY One
1
@KOne,很高兴我能帮到你。只要你从这次经历中学到了东西,它将有助于你未来的编程工作。祝你好运。 - R Sahu

0
int r,c;
cin>>r>>c;
int** p=new int*[r];
for(int i=0;i<r;i++) {
    p[i]=new int[c];
}
for(int i=0;i<r;i++) {
delete [] p[i]; 
}
delete [] p;

使用此代码,您可以在C++中创建一个2D动态数组,该数组将在堆内存中分配其内存。

网页内容由stack overflow 提供, 点击上面的
可以查看英文原文,
原文链接