Question

所以我花了无数个小时试图找到这个问题的答案。我发现了一些接近它的东西，但不完全是这样我想我会在这里发布。

我正在尝试创建一个二维结构数组。我将调用一个函数来创建结构并将值输入到结构中。这是可能输出的一个例子：

输入：int 5，int 5

1 2 3 4 5
2 4 6 8 10
3 6 9 12 15
4 8 12 16 20
5 10 15 20 25

我能够创建结构但是当我尝试输入值时，我的程序会一直崩溃。任何输入都会很棒！这是我的代码。

struct values{
int mult;
float div;
};

values** create_table(int row, int col){
values** tab = new values*[row];
values* one_row = new values[col];
for (int i = 0; i < row; i++){
    tab[i] = one_row;
}
return tab;
}

void set_m_values(values** tab, int row, int col){
for (int i = 0; i < row; i++){
    for (int j = 0; i < col; j++){
        tab[i][j].mult = (i+1)*(j+1);
    }
}
}

int main() {
int row = 5;
int col = 5;
values** tab = create_table(row, col);
set_m_values(tab, row, col);
for (int i = 0; i < row; i++){
    for (int j = 0; j< col; j++){
        cout <<tab[0][i].mult;
    }
    cout <<endl;
}
return 0;
}

Answer 1

您的初始化错误

values* one_row = new values[col];
for (int i = 0; i < row; i++){
    tab[i] = one_row;

这是创建一行，并将其分配给每一行。

你可能打算这样做：

values** tab = new values*[row];

for (int i = 0; i < row; i++)
{
    tab[i] = new values[col];
}

据说你真的应该使用std::array或std::vector。

另外，这就是导致崩溃的原因，在set_m_values中，您的比较不正确：

for (int j = 0; i < col; j++){ // notice the i in there

应该是：

for (int j = 0; j < col; j++){ // replaced i with j

很可能你复制粘贴并忘记更改它。我想强调一下，你真的应该使用像vector这样的标准库容器。例如，您可以：

// with this you don't need the create_table function
std::vector< std::vector<values> > table(row, std::vector<values>(col));

P.S：您的代码中存在内存泄漏，因为未调用删除。使用std::vector

时，这不是问题

访问2D结构时程序崩溃

1 个答案: