c - 如何将两个二维数组中的整数添加到用户指定长度的新二维数组中？

Question

我是新来的，今年是我第一次学习 c。我遇到了有关二维数组等的问题。问题是:编写一个程序，计算两个二维矩阵的和。

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

如果我的第一个二维数组和第二个数组有:

• {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

首先，您需要将函数原型(prototype)放在 main() 之前，或者至少将函数定义移到 main() 之前。我一点都不清楚为什么要为 sumRay[][] 使用 VLA，但为 ray1[][] 和 ray2 使用常量数组[][]。除非您有令人信服的理由，否则最好全面使用 VLA。

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

arraySum() 函数迭代两对重复的数组索引，原因尚不清楚。这里的逻辑错综复杂，过于复杂。您报告的虚假输出可以追溯到此功能；它难以阅读和理解，这应该是它需要重写的标志。如果您的意图是仅部分添加输入数组，那么名称并不反射(reflect)这种意图。我简化了这个功能，加强了逻辑并删除了重复项。有关执行部分数组添加的版本，请参阅下面的更新。

printSumRay() 函数似乎是多余的，因为 printRay() 函数可以做同样的工作，所以我删除了它，重写了 printRay() 使用 VLA 并收紧代码。原始代码在此处的循环控制表达式中使用魔数(Magic Number) 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，但不要定义全局常量 MAXROW 和 MAXCOL，而是定义 const size_t maxrow 和 const size_t maxcol 在 main() 的主体中。无论如何，您应该将这些数组维度传递给函数，而不是依赖于全局值。

添加了一个函数 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