1

我为我的作业写了一个代码，上面写着输入是否创建一个魔方矩阵。在幻方矩阵中，所有行、列和对角线之和必须相等。我写了一些函数来计算行、列和对角线的总和。在代码的末尾，我需要比较它们以查看它们是否相等。我将函数的结果分配给不同的变量，并在代码末尾的 if 语句中对它们进行了比较。我想知道有没有更聪明的比较方法。我的意思是在我的 if 语句中有太多的变量和太多的相等性。我相信有一个更聪明的方法。

package lab03;

import java.util.Scanner;

public class E7_15 {

public static boolean checkNumbers(int[][] array){
    for(int i=0; i<4; i++){
        for(int j=0; j<4; j++){
            if (array[i][j] < 1 || array[i][j] > 16){
                System.out.println("You entered a wrong value");
                return false;
            }
        }
        }
    return true;
}

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

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

public static int diagonalSumRightToLeft(int[][] array){
    int sum = 0;
    for(int i=0; i<array.length; i++){
        sum += array[i][array.length-1-i];
    }
    return sum;
}

public static int diagonalSumLeftToRight(int[][] array) {
    int sum = 0;
    for(int i=0; i<array.length; i++){
        sum += array[i][i];
    }
    return sum;
}

public static void main (String [] args){

    int[][] intArray = new int [4][4];
    Scanner in = new Scanner(System.in);
    for (int i=0; i<4; i++) {
        for ( int j=0; j<4; j++) {
            System.out.println("!!!Please enter your numbers between 1-16!!!");
            System.out.println("Enter your number for row " + (i+1) + " and column " + (j+1) + ": ");
            intArray[i][j] = in.nextInt();
        }
    }
    boolean done = checkNumbers(intArray);
    int sumLRD = diagonalSumLeftToRight(intArray);
    int sumRLD = diagonalSumRightToLeft(intArray);
    int r1 = sumRow(intArray, 0);
    int r2 = sumRow(intArray, 1);
    int r3 = sumRow(intArray, 2);
    int r4 = sumRow(intArray, 3);
    int c1 = sumColumn(intArray, 0);
    int c2 = sumColumn(intArray, 1);
    int c3 = sumColumn(intArray, 2);
    int c4 = sumColumn(intArray, 3);

    if (done == true){
        if(sumLRD==sumRLD && sumLRD==r1 && sumLRD==r2 && sumLRD==r3 && sumLRD==r4 &&
        sumLRD==c1 && sumLRD==c2 && sumLRD==c3 && sumLRD==c4 && sumRLD==r1 && sumRLD==r2 &&
        sumRLD==r3 && sumRLD==r4 && sumRLD==c1 && sumRLD==c2 && sumRLD==c3 &&
        sumRLD==c4 && r1==r2 && r1==r3 && r1==r4 && r1==c1 && r1==c2 && r1==c3 && r1==c4 &&
        r2==r3 && r2==r4 && r2==c1 && r2==c2 && r2==c3 && r2==c4 && r3==r4 && r3==c1 &&
        r3==c2 && r3==c3 && r3==c4 && r4==c1 && r4==c2 && r4==c3 && r4==c4 && c1==c2 &&
        c1==c3 && c1==c4 && c2==c3 && c2==c4 && c3==c4){
            System.out.println("This is a magic square matrix");
        }
        else {
            System.out.println("This is NOT a magic square matrix");
        }
    }
    if (done == false){
        System.out.println("WRONG VALUE! START AGAIN!");
    }
    in.close();
}

}

4

2 回答 2

3

似乎您正在尝试查看所有这些数字是否相等。只需使用一个循环将列表中的每个变量与其后面的变量进行比较：

public boolean allEqual(int... values) {
    for (int i = 0; i < values.length-1; i++) {
        if (values[i] != values[i+1]) {
            return false;
        }
    }
    return true;
}

然后用以下内容替换您的巨型条件：

if (allEqual(sumLRD, sumRLD, r1, r2, ...)) { 
    // ... 
}

这是有效的，因为相等是可传递的——即 ifa == b和b == cthen a == c。（对于任何数量的变量都是一样的。）

另一种方法是将整个检查重构为：

boolean isMagicSquare(int[][] intArray) {
    // check diagonals
    int sum = diagonalSumLeftToRight(intArray);
    if (sum != diagonalSumRightToLeft(intArray)) {
        return false;
    }

    // check rows and columns
    for (int i = 0; i < 4; i++) {
        if (sum != sumRow(intArray, i) || sum != sumColumn(intArray, i)) {
            return false;
        }
    }

    return true;
}

// ...
if (isMagicSquare(intArray)) {
    // ...
}
于 2013-10-09T01:01:04.217 回答
0

您可以将所有总和放在一个数组中，并使用像这样的函数来检查所有值是否相等：

public static boolean allElementsTheSame(int[] array) {
    if (array.length == 0) {
        return true;
    } else {
        int first = array[0];
        for (int element : array) {
            if (element != first) {
                return false;
            }
        }
        return true;
    }
}
于 2013-10-09T01:01:31.597 回答