組み合わせの処理内容で迷っています。

Question

ループ処理について質問です。

リストを1つ用意して、ランダムに値を2つ取得します。
同じ値同士だともう1回取得し、違う値だと処理を続けます。異なる組み合わせを"1回ずつ"行うにはどのように処理を行えばいいのか教えて欲しいです。(0,1)と(1,0)も1回ずつに含むので以下のリストだと合計で42回ループ処理を行うことになります。(0,1)はmovie1が0で、movie2が1という意味です。

// ランダムな値を取得
function rand() {
  var movieList = [0, 1, 2, 3, 4, 5, 6];
  var randNum1 = movieList[Math.floor(Math.random() * movieList.length)];
  var randNum2 = movieList[Math.floor(Math.random() * movieList.length)];
  return randNum1, randNum2;
}
var randNum1 = rand();
var randNum2 = rand();

// 取得した値を表示
function movie(randNum1, randNum2) {
  var movie1 = randNum1;
  var movie2 = randNum2;

  for(i = 0; i <= 41; i++) {
    if(movie1 === movie2) {
      continue;
    }
    else {

    }
    i++;
  }
  alert(movie1);
  alert(movie2);
}

forとelseの処理で迷っています。まだ全てコードは書けてませんが、このままだと42回のループの中に例えば、(1,0)が２回入っていてもループが継続され、(2,3)の組み合わせが1回も処理されずにループ処理が終了してしまいます。

どうか皆様のお力をお貸しください。。。

Answer 1

こういうときは問題を切り分けて考えましょう。一気に実装しようとすると混乱します。

1. 組み合わせを用意する

let movieList = [0, 1, 2, 3, 4, 5, 6];

// 組み合わせの用意
let movieComb = movieList.map( n => {
    return movieList
        .map( m => [n, m] )
        .filter( elm => elm[0] !== elm[1] );
}).flat();

2重ループで回し、値が違うものだけをリストに追加します。上記では map 関数等を使って多少記述を楽にしていますが、 for 文で済ませる場合でも同じように考えることができます。

let movieList = [0, 1, 2, 3, 4, 5, 6];

// 組み合わせの用意
let movieComb = [];
for (let i = 0; i < movieList.length; i++) {
    for (let j = 0; j < movieList.length; j++) {
        if (movieList[i] !== movieList[j]) {
            movieComb.push([movieList[i], movieList[j]]);
        }
    }
}

2. シャッフルする

配列をシャッフルしてしまえばランダムに取り出すのと同じです。

JavaScriptで配列をシャッフルする方法 - Qiita

このページの内容がわかりやすいです。

// シャッフルする
// https://qiita.com/komaji504/items/62a0f8ea43053e90555a
for (let i = movieComb.length - 1; i > 0; i--) {
    const r = Math.floor(Math.random() * (i + 1));
    const tmp = movieComb[i];
    movieComb[i] = movieComb[r];
    movieComb[r] = tmp;
}

3. 取り出す

最後に取り出します。特筆すべきことはないです。

ここで movie1 や movie2 に代入するといいでしょう。

let movie1, movie2;

// 取り出す
for (let i = 0; i < movieComb.length; i++) {
    // alert(movieComb[i]);
    document.write("<div>"+i+": "+movieComb[i]+"</div>");
    movie1 = movieComb[i][0];
    movie2 = movieComb[i][1];
    // 以降movie1,2を用いた処理
}

全体

let movieList = [0, 1, 2, 3, 4, 5, 6];
let movie1, movie2;

// 組み合わせの用意
let movieComb = movieList.map( n => {
    return movieList
        .map( m => [n, m] )
        .filter( elm => elm[0] !== elm[1] );
}).flat();

console.log(movieComb); // 42個生成されてることをデバッガーから確認できます。

// シャッフルする
// https://qiita.com/komaji504/items/62a0f8ea43053e90555a
for (let i = movieComb.length - 1; i > 0; i--) {
    const r = Math.floor(Math.random() * (i + 1));
    const tmp = movieComb[i];
    movieComb[i] = movieComb[r];
    movieComb[r] = tmp;
}

// 取り出す
for (let i = 0; i < movieComb.length; i++) {
    // alert(movieComb[i]);
    document.write("<div>"+i+": "+movieComb[i]+"</div>");
    movie1 = movieComb[i][0];
    movie2 = movieComb[i][1];
    // 以降movie1,2を用いた処理
}

---

冒頭に述べたとおり、問題を切り分けることで簡単に解決できます。