在 C 中检查单位矩阵

Question

我正在编写一个 C 程序来检查矩阵是否为单位。我没有得到任何输出。谁能找出我的代码有什么问题？

#include <stdio.h>

#include <stdlib.h>
int main()
{
    int **matrix;
    int i,j,n,flag=0;
    matrix=(int**)malloc(n*sizeof(int*));
    scanf("%d",&n);
    for(i=0;i<n;i++)
    {
        matrix[i]=(int*)malloc(n*sizeof(int));
    }
    for(i=0;i<n;i++)
    {
        for(j=0;j<n;j++)
        {
            scanf("%d",&matrix[i][j]);
        }
    }
    for(i=0;i<n;i++)
    {
        for(j=0;j<n;j++)
        {
            if(matrix[i][j]!=1 && matrix[j][i]!=0)
                flag=0;
            break;
        }
    }
    if(flag==1)
        printf("\nIt is identity");
    else
        printf("\nIt isn't");
    for(i=0;i<n;i++)
    {
        free(matrix[i]);
    }
    free(matrix);
    return 0;
}

Answer 1

有几个错误。例如，您在初始化之前使用变量 n

matrix=(int**)malloc(n*sizeof(int*));
scanf("%d",&n);

应该有

scanf("%d",&n);
matrix=(int**)malloc(n*sizeof(int*));

您永远不会将变量标志设置为 1。因此它的值始终等于 0，与矩阵值无关。

这个循环中的break语句

    for(j=0;j<n;j++)
    {
        if(matrix[i][j]!=1 && matrix[j][i]!=0)
            flag=0;
        break;
    }

必须在 if 语句的复合语句中。否则没有意义。

例如，循环可以如下所示

flag = 1;
for ( i = 0; flag && i < n; i++ )
{
    for ( j = 0; flag && j < n; j++ )
    {
        if ( matrix[i][j]!=1 && matrix[j][i]!=0 ) flag = 0;
    }
}

考虑到 C 结构之间的空行是代码中非常重要的元素。它们使代码更加清晰易读。