0

我已经在这里搜索了答案，但找不到适合我的情况。

我有一个函数 make_array()，它生成一个二维数组，并且必须用零填充它。在 main() 中调用此函数并打印值时，会出现一个奇怪的输出，这在其他机器上是不可重现的......

[...]    
Valuearray[0][13]: 0
Valuearray[0][14]: 65
Valuearray[0][15]: 0
[...]

数组的其余部分包含零。寻找我随机更改的答案

Array[i][j] = 0;

到

Array[i][j] = 123;

并得到

Valuearray[0][0]: 0
Valuearray[0][1]: 0
Valuearray[0][2]: 0
Valuearray[0][3]: 0
Valuearray[0][4]: 0
Valuearray[0][5]: 0
Valuearray[0][6]: 0
Valuearray[0][7]: 0
Valuearray[0][8]: 0
Valuearray[0][9]: 0
Valuearray[0][10]: 0
Valuearray[0][11]: 0
Valuearray[0][12]: 0
Valuearray[0][13]: 0
Valuearray[0][14]: 65
Valuearray[0][15]: 0
Valuearray[0][16]: 123
[...]

其余的包含 123。所以只有前 16 个元素不会被 for 循环更改。

我的程序代码：

//#include <iostream>
#include <stdio.h>
#include <string.h>
#include <stdlib.h>

const int global_file_count = 10;
const int global_max_x_coord = 309;
const int global_min_x_coord = 111;


int** make_array(int SizeY, int SizeX){ // SizeY=global_file_count+1, 
                                        // SizeX=global_max_x_coord-global_min_x_coord

    //http://pleasemakeanote.blogspot.de/2008/06/2d-arrays-in-c-using-malloc.html

    int** Array;  
    Array = (int**) malloc((SizeX)*sizeof(int*)); 
    for (int i = 0; i < SizeX; i++) {
       Array[i] = (int*) malloc((SizeY)*sizeof(int));
   }

    for (int i=1;i<=SizeY;i++){ // should fill everything element with 0 (????)
        for(int j=0;j<=SizeX;j++){
                Array[i][j] = 123;
        }
    }

return Array;
    free(Array);
}

int main(){

    int** Valuearray = make_array(global_file_count+1, (global_max_x_coord-global_min_x_coord));

    for (int i=0;i<=global_file_count+1;i++){
        for(int j=0;j<=(global_max_x_coord-global_min_x_coord);j++){
                printf("Valuearray[%i][%i]: %i\n", i, j, Valuearray[i][j]);
        }
    }
    free(Valuearray);
}

另外，当我种植额外的

Array[0][14] = 0;

前

return Array;

在我的 make_array() 函数中，一切都很好，并且值发生了变化。但我真的宁愿让我的 for 循环这样做......

我的问题是：由于它是不可重现的，我能做些什么呢？这是什么原因？我使用 Ubuntu 12.10.，Sublime Text 2 作为 IDE，gcc (Ubuntu/Linaro 4.7.2-2ubuntu1) 4.7.2 作为编译器，如果这些信息有用的话。在此先感谢并为这一大堆文字感到抱歉。

4

3 回答 3

2

试试这个

for (int i=0;i<SizeX;i++){ // should fill everything element with 0 (????)
    for(int j=0;j<SizeY;j++){
            Array[i][j] = 0;
    }
}

或替换malloc()为calloc()而不是上面的 for 循环。

于 2013-05-10T15:38:11.080 回答
1

您交换SizeX并SizeY在填充数组时，这是正确的：

for ( i=0;i<SizeX;i++){ // should fill everything element with 0 (????)
    for( j=0;j<SizeY;j++){

此外，您在循环中索引了太多次，例如：

for (int i=1;i<=SizeY;i++)
             ^^^^^^^^

应该：

for (int i=1;i<SizeY;i++)

因此，您将越界，这是未定义的行为。此外，这是关于释放 2D 数组的先前线程。

于 2013-05-10T15:30:05.180 回答
1

填充数组时，您需要交换使用SizeXand SizeY，因为SizeX是第一个索引的范围，而SizeY第二个索引的范围。

此外，您需要从 开始索引0，并在size - 1（无论size是什么）结束。

所以 ：

for (int i = 0; i < SizeX; i++) {
    for (int j = 0; j < SizeY; j++) {
        Array[i][j] = 123;
    }
}

（请注意，您还必须修复打印数组内容的循环中的索引）

于 2013-05-10T15:41:12.017 回答