java fx パズルゲームでのシャッフル機能

Question

前提・実現したいこと

javafxを使ってパズルゲームを作成しています。
開始ボタンを押すと4x4の16枚のパズルがランダムな配置につくようプログラミングしたいのですが、いくつか同じx,y座標に位置するようになってしまいます。
resultX,resultYに与えられる座標の数値がいくつか同じものが出てしまうため同じ座標に位置してしまうのだと思われます。
一度使った座標（例えばxは20,yは270に一度配置されたらそこには配置されないような仕組みを作る）に配置しなくなるようにはどうしたらいいでしょうか？

発生している問題・エラーメッセージ

パズルをシャッフルする際に同じ座標に位置するものが出てしまう

該当のソースコード

Button buttons[] = { this.button1, this.button2, this.button3, this.button4,
this.button5, this.button6, this.button7, this.button8,
this.button9, this.button10, this.button11, this.button12,
this.button13, this.button14, this.button15, this.button16 };

	for (int i = 0; i < buttons.length; i++) {

		List<Integer> listX = Arrays.asList(20, 80, 140, 200);
		List<Integer> listY = Arrays.asList(270, 330, 390, 450);

		int indexX = new Random().nextInt(listX.size());
		Integer resultX = listX.get(indexX);

		int indexY = new Random().nextInt(listY.size());
		Integer resultY = listY.get(indexY);
	
	
		buttons[i].setLayoutX(resultX);
		buttons[i].setLayoutY(resultY);


		
	}

試したこと

ここに問題に対して試したことを記載してください。

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

ここにより詳細な情報を記載してください。

Answer 1

シャッフル機能。やりたいことそのもの名前のメソッドがあります。Collections#shuffle()

例は、Buttonの配列ではありませんが、やることは一緒です。

import java.security.SecureRandom;
import java.util.Arrays;
import java.util.Collections;
public class xxHoge {
  @Test
  public void bbb() {
    final String[] buttons = {"a", "b", "c", "d", "e", "f", "g", "h", "i", "j", "k", "l", "m", "n", "o", "p"};
    System.out.println(Arrays.toString(buttons));
    System.out.println("shuffle");
    Collections.shuffle(Arrays.asList(buttons), new SecureRandom());
    System.out.println(Arrays.toString(buttons));
  }
}
// [a, b, c, d, e, f, g, h, i, j, k, l, m, n, o, p]
// shuffle
// [o, e, d, p, m, b, k, g, l, j, c, f, n, i, a, h]

---

質問の意図を読み違えていました。追記します。

上記を使い、こんな感じではどうでしょうか。

import javafx.geometry.Point2D;
...
  @Test
  public void ccc() {
    final Button[] buttons = new Button[]{...};

    final List<Integer> listX = Arrays.asList(20, 80, 140, 200);
    final List<Integer> listY = Arrays.asList(270, 330, 390, 450);

    final List<Point2D> points =
        listX.stream().flatMap(x ->
            listY.stream().map(y -> new Point2D(x, y))
        ).collect(Collectors.toList());

    Collections.shuffle(points, new SecureRandom());

    for (int index = 0; index < buttons.length; index++) {
      buttons[index].setLayoutX(points.get(index).getX());
      buttons[index].setLayoutY(points.get(index).getY());
    }
  }

---

KosukeDayo_
2020/04/21 23:45
final List<Point2D> points = から Collections.shuffle(points, new SecureRandom());　までで行っている動作の説明をしていただいてもよろしいでしょうか・・・？初学者のため、申し訳ありません。

• java.util.stream.Stream
• #flatMap()
• #map()
• #collect()

    final List<Point2D> points = new ArrayList<>();
    for (int x : listX) {
      for (int y : listY) {
        points.add(new Point2D(x, y));
      }
    }

//    final List<Point2D> points =
//        listX.stream().flatMap(x ->
//            listY.stream().map(y -> new Point2D(x, y))
//        ).collect(Collectors.toList());

Comments

[KosukeDayo_]

回答ありがとうございます
このメソッドを使えばランダムな値を重複なく代入できるというのは分かったのですが、これをどう活用したらよいかが思いつきません………
良い案はありますでしょうか？

[shiketa]

x位置とy位置を組合せてから、shuffleする。追記しました。

[KosukeDayo_]

final List<Point2D> points = から Collections.shuffle(points, new SecureRandom());　までで行っている動作の説明をしていただいてもよろしいでしょうか・・・？初学者のため、申し訳ありません。

Answer 2

すいません、取り下げます。
これでは明らかにシャッフルとは言えませんね。。。

Answer 3

ボタンに映像が付いているのであれば、
ボタンのシャッフルをした後、その並びで座標を割り当てればいいのでは？

List<Button> buttons = Arrays.asList( this.button1, this.button2, this.button3, this.button4,
this.button5, this.button6, this.button7, this.button8,
this.button9, this.button10, this.button11, this.button12,
this.button13, this.button14, this.button15, this.button16 );
Collections.shuffle(buttons);

List<Integer> listX = Arrays.asList(20, 80, 140, 200);
List<Integer> listY = Arrays.asList(270, 330, 390, 450);

for (int i = 0; i < 4; i++) {
    int x = listX.get(i);
    for (int j = 0; j < 4; j++) {
        Button target = buttons.get(i * 4 + j);
        target.setLayoutX(resultX);
        target.setLayoutY(resultY);
    }

}