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 == cthena == 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 に答える