Java Magic Square - Sum列和Sum行错误

时间:2015-08-31 08:41:07

标签: java arrays methods compiler-errors magic-square

我被分配了一个学校作业来制作和检查用户在N * N矩阵中生成的二维数组中的“幻方”。

到目前为止,我已经获得了大部分代码(我已经单独测试了每个方法)。但是,我无法纠正在我的'sumColumn'和'sumRow'方法中不断出现的两个最终错误。这是我上述两种方法的代码:

public static int sumColumn(int[][] square, int columnNumber)
{
    int sum = 0 ;
    for (int j = 0; j < square.length ; j++)
    {
        for (int i = 0; i < square.length; i++)
        {
            sum = sum + square[i][j] ;
        }
    }                        
    return sum ;
}

public static int sumRow(int[][] square, int rowNumber)
{
    int sum = 0 ;
    for (int i = 0; i < square.length; i++)
    {
        for (int j = 0; j < square.length; j++)
        {
            sum = sum + square[i][j] ;
        }
    }
    return sum ;
}

这是输出以及从main方法调用时出现的错误:

Please enter a value for N:
1
Please enter 1 numbers: 
1
This is the square you input:
+-+
|1|
+-+
Exception in thread "main" java.lang.Error: Unresolved compilation problems: 
The method sumRow(int[][], int) in the type MagicSquares is not applicable for the arguments (int[][]) at squares.MagicSquares.validMagicSquare(MagicSquares.java:105)
The method sumColumn(int[][], int) in the type MagicSquares is not applicable for the arguments (int[][]) at squares.MagicSquares.main(MagicSquares.java:167)

一些摆弄'sumRow'和'sumColumn'会改为产生另一个错误:

Exception in thread "main" java.lang.Error: Unresolved compilation problems: 
rowNumber cannot be resolved to a variable at squares.MagicSquares.validMagicSquare(MagicSquares.java:105)
colNumber cannot be resolved to a variable at squares.MagicSquares.main(MagicSquares.java:167)

在解决此问题时,我们非常感谢您的帮助!谢谢!

PS:我上个月才开始编程,所以请善待:3

编辑:这是检查每个行,列以及主对角线和辅助对角线是否等于制作幻方的方法。

boolean status = true ;
    int sum = sumDiagonal(square) ;
    if (sumSecondaryDiagonal(square) != sum)
    {
        status = false ;
    }
    else
    {
        for (int row = 0; (row < square.length) && status; row ++)
        {
            if (sum != sumRow(square, square.length))
            {
                status = false ;
            }
        }
        for (int col = 0; (col < square.length) && status; col ++)
        {
            if (sum != sumColumn(square, square.length))
            {
                status = false ;
            }
        } 
    }
    return status;

2 个答案:

答案 0 :(得分:0)

您的sumColumn方法需要两个参数:

  1. int[][] square
  2. int columnNumber
  3. 在你的main方法中,你似乎只提供了一个参数int[][],而你忘了将columnNumber包含为第二个参数。

    同样适用于sumRow方法。

    我根据您发布的错误消息做出了上述调查结果:

    The method sumRow(int[][], int) in the type MagicSquares is not applicable for the arguments (int[][]) at squares.MagicSquares.validMagicSquare(MagicSquares.java:105)

    它表示,在您的MagicSquares.java文件中,行号为105,您调用sumRow方法。它进一步表明,您实施的方法(The method sumRow(int[][], int) in the type MagicSquares)无法使用/不适用于参数(int[][]),在第105行。

答案 1 :(得分:0)

您显示的代码没有语法错误。

但你的例外:

  

MagicSquares类型中的方法 sumRow(int [] [],int)不适用于squares.MagicSquares中的参数(int [] []) .validMagicSquare(MagicSquares.java:105)

调用时,您未对sumColumnsumRow进行正确的调用:

int[][] matrix = // your matrix;
sumRow(matrix);     // bad call
sumRow(matrix, 1);  // good call!!!!

但实际上,如果您检查方法,请不要使用columnNumberrowNumber,所以如果要对所有列/行求和,则重新声明您的方法,就像现在这样编写:

public static int sumAllColumns(int[][] square)
public static int sumAllRows(int[][] square)

确保第二个循环遍历列!!!

for (int j = 0; j < square[0].length; j++)
  

我不得改变任何方法的标题。

因此,只需使用已存在的方法,但请确保使用columnNumberrowNumber变量。如果方法描述是正确的,则只需要给定行/列的总和,因此不需要嵌套循环。

public static int sumColumn(int[][] square, int columnNumber) {
    int sum = 0 ;
    for (int i = 0; i < square.length; i++)
        sum = sum + square[columnNumber][i] ;
    return sum ; 
}

public static int sumRow(int[][] square, int rowNumber)
{
    int sum = 0 ;
    for (int i = 0; i < square[0].length; i++)
        sum = sum + square[i][rowNumber] ;
    return sum ;
}

<强> Check here a working DEMO

ADD ON的

安全检查:只要您提供任何columnNumberrowNumber方法,就可以抛出 IndexOutOfBounds 例外。 的
为避免这种情况,请检查数组内是否有给定的索引。

人性化:要对第一行或第一列求和,您必须给出0值。所以为了让用户以 human 方式引用行和列(如果是第一列,则为第1列) 的
为了避免这种情况,请在方法和检查中给出行/列-1:

public static int sumColumn(int[][] square, int columnNumber) {
    // humanize
    columnNumber = columnNumber - 1;
    // safe check
    if ((columnNumber) > square.length) 
        return 0;

    int sum = 0 ;
    for (int i = 0; i < square.length; i++)
        sum = sum + square[columnNumber][i] ;
    return sum ; 
}

public static int sumRow(int[][] square, int rowNumber)
{
    // humanize
    rowNumber = rowNumber - 1;
    // safe check
    if ((rowNumber) > square[0].length) 
        return 0;

    int sum = 0 ;
    for (int i = 0; i < square[0].length; i++)
        sum = sum + square[i][rowNumber] ;
    return sum ;
}