Question

我为我的作业编写了一个代码，表示输入会创建一个魔方矩阵，或者不会。在魔术方阵中，所有的行，列和对角线总和必须相等。我写了一些函数来计算行，列和对角线的总和。在代码的最后我需要比较它们，看它们是否相等。我将函数的结果分配给不同的变量，并在代码末尾的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();
}

}

Answer 1

看起来你正试图看看所有这些数字是否相等。只需使用一个循环，将列表中的每个变量与其后面的变量进行比较：

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

然后用：

替换你的megacondition

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

这是有效的，因为等式传递 - 即a == b和b == c然后是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)) {
    // ...
}

Answer 2

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

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;
    }
}

比较if语句中的太多变量

2 个答案: