複数の数字が重複していないかを確認したい

Question

前提・実現したいこと

覆面算で数字が重複していないかを確認する関数を作っています
配列arrayを回して同じ値ならfalseで処理をやめ、全部回したときにそれぞれが一致しないときだけtrueを返そうと思っています。

該当のソースコード

if (checkNum(1, 2, 3, 4, 5, 6, 7, 8, 9)) {
        System.out.println("test is through");　　//通過しない
        }


  public static boolean checkNum(int s, int i, int x, int e, int v, int n, int t, int w, int y) {
        Integer[] array = { s, i, x, e, v, n, t, w, y };
        for (int j = 0; j < 9; j++) {
            for (int k = 1; k < 9; k++) {
                if (array[j].equals(array[k])) {
                    return false;
                }
            }
        }
        return true;
    }

試したこと

checkNum(1, 2, 3, 4, 5, 6, 7, 8, 9)を実行し中に入るか確かめた。
しかし、中には入らなかったため、checkNumが正しく動いてないことが分かった。

補足情報（FW/ツールのバージョンなど）

openjdk version "1.8.0_252"

Comments

[LouiS0616]

本筋から逸れますが、checkNumの引数には可変長引数を使うと簡潔に書けるかと思います。

[Jhon_McClane]

ご指摘ありがとうございます。
かなり簡潔に書くことができました。

Answer 1

ええと、array[N]の値をarray[N+1]～array[8]までと比較したいんですよね?

        for (int j = 0; j < 9 - 1; j++) {
            for (int k = j + 1; k < 9; k++) {
                if (array[j].equals(array[k])) {
                    return false;
                }
            }
        }

• 内側のforループの変数kの初期値はj + 1にする。自分自身と比較してfalseを返しているのが失敗の原因。また、0～j - 1までのチェックは、それ以前のループで終わっているので不要。
• 外側のforループの変数jの最終値は7でいい。array[7]をarray[8]と比較すれば、array[8]までの重複はすべてチェックしたことになるため。

なお、数値を入れたリストを作れば、HashSetにしたリストとのサイズ差で調べられます。

Comments

[Jhon_McClane]

原因が分かり納得しました。また、無駄なループ回数のご指定ありがとうございます。