配列のJava ArrayListには予期しない結果が返される

Question

コンピュータとユーザーの間でTic-Tac-Toeのゲームのコードを記述しています。移動を行うために、ボード上で占有されていないスポットのリストがパラメータとして提供され、ユーザにとっては、入力がリスト内にあるかどうかを比較するのに本当に有用です。そうであれば、それは合法的な動きになります。

ここに私のコードです。それは新しい動きがリストに含まれていないと私はなぜ知りませんと言い続ける。私はここでデータベースを検索して同様の問題を発見したが、いくつかの関連性があるが決定的なものは見つからなかった。

import java.util.ArrayList; 
import java.util.Arrays; 
import java.util.Scanner; 

public class UserTTTPlayer implements TicTacToePlayer{ 

@Override 
public int[] makeMove(ArrayList<int []> unusedMoves) { 

    Scanner in = new Scanner (System.in); 
    System.out.println("Your move, user?"); 
    String input = in.nextLine(); 
    int [] move = checkInput(input, unusedMoves); 
    while (move == null){ 
     input = in.nextLine(); 
     move = checkInput(input, unusedMoves); 
    } 

    return move; 
} 

private int [] checkInput(String input, ArrayList<int []> unusedMoves){ 

    System.out.println("Unused moves: "); 
    for (int [] move: unusedMoves) 
     System.out.println(Arrays.toString(move)); 

    //error checking for the length of the input 
    if (input.length() < 1 || input.length() > 2){ 
     System.out.println("Invalid input. Please try again."); 
     return null; 
    } 
    else{ 

     //convert the input from string to int 
     int col = input.charAt(0) - 'a'; 
     int row = input.charAt(1) - '0'; 
     int [] move = {row, col}; 
     System.out.println("Intended move: " + Arrays.toString(move)); 
     System.out.println(unusedMoves.contains(move)); 

     //error checking for the bounds of the board 
     if (col > 3 || col < 0 || row > 3 || col < 0){ 
      System.out.println("Invalid input."); 
      return null; 
     } 

     //error checking for if the space is available 
     else if (!unusedMoves.contains(move)){ 
      System.out.println("That space is already occupied."); 
      return null; 
     } 
     return move; 
    } 
} 
} 

これはその出力です。ボードや他の印刷は別のクラスからのものですが、問題とは関係ないと思います。私はそれが新しい動きを持っていると言うリストを出力しますが、まだfalseを返します。

You go first. You will be X's. 
    a b c 
0 - - - 
1 - - - 
2 - - - 

Your move, user? 
a0 
Unused moves: 
[0, 0] 
[0, 1] 
[0, 2] 
[1, 0] 
[1, 1] 
[1, 2] 
[2, 0] 
[2, 1] 
[2, 2] 
Intended move: [0, 0] 
false 
That space is already occupied. 

助けてください。

Answer 1

コメントが説明されているように、unusedMoves.contains(move)は期待通りに機能しません。 .containsメソッドは、リスト内のオブジェクトの.equalsメソッドを使用して、指定された値が要素の1つと等しいかどうかを確認します。しかし、配列の.equalsメソッドは要素を比較しません。==を使用して配列の同一性のみを比較します。例えば

：、あなたは、contains(unusedMoves, move)とunusedMoves.contains(move)を置き換えることができます

private boolean contains(List<int[]> list, int[] arr) { 
    return list.stream().anyMatch(x -> Arrays.equals(x, arr)); 
} 

このヘルパーメソッドではと期待どおりに動作するはずです：

int[] arr = {1, 2}; 

// prints false, as the array in the list is not *identical* 
System.out.println(Arrays.asList(new int[]{1, 2}).contains(arr)); 

// prints true, as the array in the list *is* identical 
System.out.println(Arrays.asList(arr).contains(arr)); 

は、このヘルパーメソッドを考えてみましょう。