函数为零数组元素

时间:2017-12-01 08:08:38

标签: c arrays function

我编写了这个检查suduko答案的代码,由于某种原因,我将suduko的平方和数组合并为函数中的零,我把它归零,但是当我在检查函数中使用它时它不是&# 39;全部为零

如果我将我的数组归零函数的完全相同的代码移动到我的其他函数并运行它就可以了。 (我使用的是c99,不知道是否重要)

有什么想法吗?

int squareSum[5][5];
//set array elements to zero
setArrayToZero(squareSize, squareSize, squareSum);
/*for(int i = 0; i<squareSize; i++){
    for(int j = 0; j<squareSize; j++)
        squareSum[i][j] = 0;
}*/
printf("%d, %d\n%d, %d\n\n", squareSum[0][0], squareSum[0][1], squareSum[1][0], squareSum[1][1]);

this is the array is case squareSize is two 如果我在注释中添加for,则数组全部为零,如下所示,它与我调用的函数完全相同。

void setArrayToZero(int rows, int columns, int array[][columns]){
for(int i = 0; i<rows; i++)
    for(int j = 0; j<columns; j++)
        array[i][j] = 0;\\if i print the array in function its all zeros.

P.S 我知道我只使用数组的一部分,它是来自大学的任务,我们不允许使用malloc所以我在最大尺寸--25创建数组。

提前谢谢。

完整的c文件: https://drive.google.com/open?id=1L00L3lvMYNcaz2SswEBnmi9KO-79oaHg

所有打印功能都是课程需求的一部分(用于自动检查)

1 个答案:

答案 0 :(得分:3)

  

对一个出现在一个类型为array-of-T的对象的引用   表达式衰减(有三个例外)到指向它的第一个指针   元件;结果指针的类型是指向T的指针。

所以这意味着在这种情况下(这不是这3个例外中的一个)你传递的数组会衰减成指针而现在你通过访问地址来改变数组,这就是为什么它会保留在被调用者函数中。

你正在以正确的方式做到这一点。问题不在于传递或任何事情。也许你错误地访问它或者你没有正确地初始化它。但是归零没有任何问题。

有关更多信息一切都在C 中传递。在C中没有任何叫做引用传递的东西。指针工作使我们认为在C中有一些叫做引用的东西,但事实并非如此,这里指针变量也被复制到被调用函数中的一些局部变量中。但是因为我们在被调用的函数中有这些地址,我们访问它们并进行更改 - 它们保留在被调用函数中。

OP发布示例代码

之后

从过于复杂的数独检查逻辑来看,存在更多错误。

我只想提一下打印部分。

在2个阵列的C元素中顺序存储。当我们将2d数组传递给函数时,我们需要指定列大小,以便我们可以确定正确的元素。

假设您要访问a[4][7]数组中的10x13。该元素位于地址&a[0][0]+4*13+7。这就是column部分作为参数传递的原因。

现在你做了什么:

int squareSum[5][5], rowColSum[25][2];
//set arrays elements to zero
setArrayToZero(size, 2, rowColSum);
setArrayToZero(squareSize, squareSize, squareSum);

第一个没问题。由于有2列。但是第二个呢? 在这里,您告诉函数您正在传递一个包含列size = 2的数组,但事实并非如此。它仍然是5列的2d数组。

你遇到问题的地方。假设您使用10,21,34,14

初始化数组

假设grid5x5数组(在您的情况下,它是25x25) grid [5] [5] array

你这样做

for(int i = 0; i<squareSize; i++)
  for(int j = 0; j<squareSize; j++)
     scanf("%d",&grid[i][j]);

/*
Input is 13 17 19 23
*/


 +-----+-----+-----+------+-----+-----+-----+----+----+----+--...\
 |  0  |  1  |  2  |   3  |   4 |  5  |  6  |  7 |  8 |  9 |  
 +-----+-----+-----+------+-----+-----+-----+----+----+----+--...\
/      \     \                  /      \     \
  13      17                       19     23

现在您可以像这样访问它

通过此调用print2dArray(int row, int col, int g[][col])

将其传递给该函数

你这样叫`print2dArray(2,2,grid);`` ...

for(int i = 0; i<row; i++)
  for(int j = 0; j<col; j++)
     printf("%d",&grid[i][j]);

现在您将打印这些元素(i) : denotes the order

+-----+-----+-----+------+-----+-----+-----+----+----+----+--...\
|  0  |  1  |  2  |   3  |   4 |  5  |  6  |  7 |  8 |  9 |  
+-----+-----+-----+------+-----+-----+-----+----+----+----+--...\
/      \     \  ^    ^        /      \     \
 13      17     |    |            19     23
 (1)    (2)    (3)  (4)

同样,在你的情况下,那些是未初始化的。这就是为什么你得到那些奇怪的结果。您正在将2d数组的一些元素初始化为0但是当您正在读取数组时,您正在访问一些不同的元素。这就是错误结果的原因。

解决方案:

  • 致电int correctSum = inputGrid(squareSize, 25, grid);请注意,它应为squareSize。相应地更改digitsCounterinputGrid的存储空间。

  • inputGrid的功能签名为inputGrid(size, col, grid[][col]);

  • 调用

    时相同

    setArrayToZero(size, 2, rowColSum);setArrayToZero(squareSize, 5, squareSum);

我根本没有检查逻辑。答案涉及解释2d阵列的打印行为。