如何将两个2d数组中的整数添加到用户指定长度的新2d数组中？

Question

我在这里很陌生，今年是我第一次学习c。 我遇到了有关2D阵列等的问题。 问题是： 编写一个找到两个2D矩阵之和的程序。

我可以很容易地做到这一点，但是我遇到了一个问题。 例如，我将第一组数组的长度设为3x3。

如果我的第一个2D数组和第二个数组具有：

• {1,2,3; 4,5,6; 7,8,9}（第一个数组）
• {0,0,0; 0,0,0; 0,0,0}（第二个数组）

我还提供了用户的行数和列数。 （用户输入3x2），则应显示为

• {1,2; 3,4; 5,6}（输出）

但我越来越

• {1,2; 3,5; 6,8}

另一个例子

• 用户输入2x4 OUTPUT应该为{1,2,3,4; 5,6,7,8}

我在这里做错了什么？

#include <stdio.h>
#include <stdlib.h>

#define MAXROW 3
#define MAXCOL 3

int main() {
    int ray1[MAXROW][MAXCOL], ray2[MAXROW][MAXCOL];
    int r, c;
    printf("Enter the number of ROWS: ");
    scanf("%d", &r);
    printf("Enter the number of COLUMNS: ");
    scanf("%d", &c);

    int sumRay[r][c];

    printf("\n");

    printf("Input integers for Array %d.\n", 1);
    arrayIN(ray1);

    printRay(ray1);
    printf("Input integers for Array %d.\n", 2);
    arrayIN(ray2);

    printRay(ray2);
    arraySUM(ray1, ray2, r, c, sumRay);
    printSumRay(r, c, sumRay);

    //printRay(sumRay);
}

void arrayIN(int ray[MAXROW][MAXCOL]) {
    int r, c;
    for (r = 0; r < MAXROW; r++) {
        for (c = 0; c < MAXCOL; c++) {
            printf("Enter Number for [ROW:%d COL:%d]: ", r, c);
            scanf("%d", &ray[r][c]);
        }
    }
}

void arraySUM(int ray1[MAXROW][MAXCOL], int ray2[MAXROW][MAXCOL],
              int r, int c, int sumRay[r][c]) {
    int i, j;
    int x, y;
    i = j = 0;
    int sum;
    for (x = 0; x <= r; x++, i++) {
        if (i < MAXROW) {
            for (y = 0; y <= c; y++, j++) {
                if (j < MAXCOL) {
                    sum = ray1[i][j] + ray2[i][j];
                    sumRay[x][y]= sum;
                } else {
                    j = 0;
                }
            }
        } else {
            i = 0;
        }
    }
    printf("\n");
}

void printSumRay(int r, int c, int sumRay[r][c]) {
    int i, j;
    for (i = 0; i < r; i++) {
        printf("\n");
        for (j = 0; j < c; j++) {
            printf("%d\t", sumRay[i][j]);
        }
    }
}

void printRay(int ray[MAXROW][MAXCOL]) {
    int i, j;
    for (i = 0; i < 3; i++) {
        printf("\n");
        for (j = 0; j < 3; j++) {
            printf("%d\t", ray[i][j]);
        }
    }
    printf("\n");
}

Answer 1

首先，您需要将函数原型放置在main()之前，或者至少将函数定义移到main()之前。 对我来说，为什么要使用VLA来解决sumRay[][]完全不清楚，但是对于ray1[][]和ray2[][] ，使用不变尺寸的数组是不ray1[][] 。 除非您有令人信服的理由，否则最好使用VLA。

您应该将size_t类型用于保存数组索引的变量。 然后必须修改处理size_t变量的scanf()和printf()语句以使用%zu转换说明符。

由于不清楚的原因， arraySum()函数对两对重复的数组索引进行迭代。 这里的逻辑很复杂，而且过于复杂。 您报告的虚假输出可以追溯到此函数。 很难阅读和理解，这表明它需要重写。 而且，如果您打算仅部分添加输入数组，则名称不反映此意图。 我简化了此功能，加强了逻辑并删除了重复项。 有关执行部分数组添加的版本，请参见下面的更新。

printSumRay()函数似乎是多余的，因为printRay()函数可以完成相同的工作，因此我将其删除，重写printRay()以使用VLA并收紧代码。 原始代码在这里的循环控制表达式中使用了幻数3 ，而不是利用MAXROW和MAXCOL 。 但是，即使不使用VLA，也最好将维度传递给将与数组一起使用的任何函数。

这是原始代码的修改版本：

#include <stdio.h>
#include <stdlib.h>

void arrayIN(size_t r, size_t c, int ray[r][c]);
void arraySUM(size_t r, size_t c, int ray1[r][c], int ray2[r][c], int sumRay[r][c]);
void printRay(size_t r, size_t c, int ray[r][c]);

int main(void)
{

    size_t r,c;

    printf("Enter the number of ROWS: ");
    scanf("%zu", &r);
    printf("Enter the number of COLUMNS: ");
    scanf("%zu", &c);

    int ray1[r][c], ray2[r][c], sumRay[r][c];

    printf("\n");

    printf("Input integers for Array %d.\n", 1);
    arrayIN(r, c, ray1);
    putchar('\n');

    printRay(r, c, ray1);
    putchar('\n');

    printf("Input integers for Array %d.\n", 2);
    arrayIN(r, c, ray2);
    putchar('\n');

    printRay(r, c, ray2);
    putchar('\n');

    arraySUM(r, c, ray1, ray2, sumRay);
    printRay(r, c, sumRay);
    putchar('\n');

    return 0;
}

void arrayIN(size_t r, size_t c, int ray[r][c])
{
    for(size_t i = 0; i < r; i++) {
        for(size_t j = 0; j < c; j++) {
            printf("Enter Number for [ROW:%zu COL:%zu]: ", i, j);
            scanf("%d", &ray[i][j]);
        }
    }
}

void arraySUM(size_t r, size_t c, int ray1[r][c],int ray2[r][c], int sumRay[r][c])
{
    for(size_t i = 0; i < r; i++) {
        for(size_t j = 0; j < c; j++) {
            sumRay[i][j] = ray1[i][j] + ray2[i][j];
        }
    }
}

void printRay(size_t r, size_t c, int ray[r][c])
{
    for(size_t i = 0; i < r; i++) {
        for(size_t j = 0; j < c; j++) {
            printf("%8d",ray[i][j]);
        }
        putchar('\n');
    }
}

下一步，您可以在输入代码中添加一些错误检查，检查对scanf()的调用中的返回值。 而且，按原样输入每个元素的数组编号很麻烦。 您可能会尝试改善此问题的方法。

更新资料

如果您的真正目标是仅合并数组的初始行和列，则上面的代码仅需进行少量修改即可工作。 您仍然应该使用VLA，但不要在main()主体中定义const size_t maxrow和const size_t maxcol而不是定义全局常量MAXROW和MAXCOL 。 无论如何，您都应该将这些数组维传递给函数，而不要依赖于全局值。

添加了一个函数partArraySUM() ，其名称更接近其用途。 此函数与arraySUM()之间的唯一区别是，输入数组ray1[][]和ray2[][]维数与保存结果的数组sumRay[][]维数不同。 无需为此保留单独的索引。

#include <stdio.h>
#include <stdlib.h>

void arrayIN(size_t r, size_t c, int ray[r][c]);
void arraySUM(size_t r, size_t c, int ray1[r][c], int ray2[r][c], int sumRay[r][c]);
void partArraySUM(size_t r_sz, size_t c_sz, int ray1[r_sz][c_sz], int ray2[r_sz][c_sz], size_t r, size_t c, int sumRay[r][c]);
void printRay(size_t r, size_t c, int ray[r][c]);

int main(void)
{
    const size_t maxrow = 3;
    const size_t maxcol = 3;
    size_t r,c;

    printf("Enter the number of ROWS: ");
    scanf("%zu", &r);
    printf("Enter the number of COLUMNS: ");
    scanf("%zu", &c);

    int ray1[maxrow][maxcol], ray2[maxrow][maxcol], sumRay[r][c];

    printf("\n");
    printf("Input integers for Array %d.\n", 1);
    arrayIN(maxrow, maxcol, ray1);
    putchar('\n');

    printRay(maxrow, maxcol, ray1);
    putchar('\n');

    printf("Input integers for Array %d.\n", 2);
    arrayIN(maxrow, maxcol, ray2);
    putchar('\n');

    printRay(maxrow, maxcol, ray2);
    putchar('\n');

    partArraySUM(maxrow, maxcol, ray1, ray2, r, c, sumRay);
    printRay(r, c, sumRay);
    putchar('\n');

    return 0;
}

void arrayIN(size_t r, size_t c, int ray[r][c])
{
    for(size_t i = 0; i < r; i++) {
        for(size_t j = 0; j < c; j++) {
            printf("Enter Number for [ROW:%zu COL:%zu]: ", i, j);
            scanf("%d", &ray[i][j]);
        }
    }
}

void arraySUM(size_t r, size_t c, int ray1[r][c],int ray2[r][c], int sumRay[r][c])
{
    for(size_t i = 0; i < r; i++) {
        for(size_t j = 0; j < c; j++) {
            sumRay[i][j] = ray1[i][j] + ray2[i][j];
        }
    }
}

void partArraySUM(size_t r_sz, size_t c_sz, int ray1[r_sz][c_sz], int ray2[r_sz][c_sz], size_t r, size_t c, int sumRay[r][c])
{
    for(size_t i = 0; i < r; i++) {
        for(size_t j = 0; j < c; j++) {
            sumRay[i][j] = ray1[i][j] + ray2[i][j];
        }
    }
}

void printRay(size_t r, size_t c, int ray[r][c])
{
    for(size_t i = 0; i < r; i++) {
        for(size_t j = 0; j < c; j++) {
            printf("%8d",ray[i][j]);
        }
        putchar('\n');
    }
}

样本互动：

Enter the number of ROWS: 2
Enter the number of COLUMNS: 2

Input integers for Array 1.
Enter Number for [ROW:0 COL:0]: 1
Enter Number for [ROW:0 COL:1]: 2
Enter Number for [ROW:0 COL:2]: 3
Enter Number for [ROW:1 COL:0]: 4
Enter Number for [ROW:1 COL:1]: 5
Enter Number for [ROW:1 COL:2]: 6
Enter Number for [ROW:2 COL:0]: 7
Enter Number for [ROW:2 COL:1]: 8
Enter Number for [ROW:2 COL:2]: 9

       1       2       3
       4       5       6
       7       8       9

Input integers for Array 2.
Enter Number for [ROW:0 COL:0]: 1
Enter Number for [ROW:0 COL:1]: 1
Enter Number for [ROW:0 COL:2]: 1
Enter Number for [ROW:1 COL:0]: 1
Enter Number for [ROW:1 COL:1]: 1
Enter Number for [ROW:1 COL:2]: 1
Enter Number for [ROW:2 COL:0]: 1
Enter Number for [ROW:2 COL:1]: 1
Enter Number for [ROW:2 COL:2]: 1

       1       1       1
       1       1       1
       1       1       1

       2       3
       5       6

Answer 2

只需稍作更改即可。 在arraySUM（）中 ，在else部分中您已重新初始化

i = 0和j = 0。

但是在重新初始化之后，它们会增加。 因此它将变为1，因此在执行时它将读取ray [1]而不是ray [0] 。

只需将它们初始化为-1。

i = -1和j = -1