多次元配列と再帰呼び出しについて

Question

多次元配列で[0,1], [2,3], [4,5]みたいなものを作りたいのですが、main関数の結果は下記の写真のように[x,x],[x,x]となってしまいます。

そこで3つ質問です。
① [[0,1], [2,3], [4,5]]みたいに配列の中に配列を作るときは下記のmain関数の処理をどのように変更すれば実現できますか？

② rand()をstimu変数に入れているのでmain(stimu)の値が何回やっても同じになるのは分かるんですが、どうすればランダムに取り出してmain()のendList[]に追加できますか？

③ rand()の取り出す処理でdocument.write("<div>"+combo+"</div>");を追加すると結果は下記の写真みたいに42個の組み合わせ全てを取得できるんですが、return combo;すると最初の1組だけ取得されてしまいます。なぜでしょうか？

//main関数
function main(stimu) {
  var endList = [];

  endList.push([stimu[0], stimu[1]]);
  console.log(endList[0]);
  endList.push([stimu[0], stimu[1]]);
  console.log(endList[1]);
}


//ランダムな値を取得
function rand() {
  var movieList = [0, 1, 2, 3, 4, 5, 6];
  var movie_1, movie_2;

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

  // シャッフルする
  for (var i = movieComb.length - 1; i > 0; i--) {
      var r = Math.floor(Math.random() * (i + 1));
      var tmp = movieComb[i];
      movieComb[i] = movieComb[r];
      movieComb[r] = tmp;
  }
  // 取り出す
  for (var i = 0; i < movieComb.length; i++) {
      movie_1 = movieComb[i][0];
      movie_2 = movieComb[i][1];
      var combo = movieComb[I];
      document.write("<div>"+combo+"</div>");
      return combo;
  }
}
var stimu = rand();

どうか、解決方法をよろしくお願い致します。

Comments

[tanishi_a]

ちゃんと読んでないですが、下記の i が大文字なのは怪しいですね。

> movieComb[I];

[gyoruo]

ありがとうございます。
そこを直してもできませんでした、、、

[tanishi_a]

そうですね。その他のところ含めて回答欄に書いたので確認してみてください。

Answer 1

こんな感じで

const a=Array(7).fill(null).map((x,y)=>y);
var b=[];
a.forEach(x=>a.forEach(y=>b.push([x,y])));
b=b.filter(x=>x[0]!==x[1]).map(x=>[x,Math.random()]).sort((x,y)=>x[1]-y[1]).map(x=>x[0]);
console.log(b);

Answer 2

③ は、 combo という１要素しか返してないからですね。
たぶん返したいのは 全部の組み合わせなので、こうだと思います。

// 取り出す
var combos = [];
for (...) {
   ...
   var combo = movieComb[i]; // ※ 大文字 => 小文字にした
   combos.push(combo);
}
return combos;

①②は読んだら追記します。

＃＃ 以下、追記

③については、
rand 関数内の // 取り出す 以下は不要で、return movieComb; したらいいと思います

①②については、

やろうとしてることは、こういうことだと思いますが、

function main(stimu) {
  var endList = [];
  endList.push(stimu[Math.floor(Math.random() * stimu.length)]);
  endList.push(stimu[Math.floor(Math.random() * stimu.length)]);
  console.log(endList);
}

これだと重複してしまうのと、既にシャッフルしてあるようなので、以下のように順番に取り出せば良いと思います。

function main(stimu) {
  var endList = [];
  endList.push(stimu[0]);
  endList.push(stimu[1]);
  console.log(endList);
}

・・・

※ それから 本質とは関係ない補足ですが、

関数名ですが rand() は乱数を返す関数に見えてしまうので、別の名前にしたほうが良いと思いました。 なんでもいいですが、 getRandomCombinations() とか。 stimu も何だかわからないですが。

それと、タイトルの「再帰呼び出し」は関係ないように思います。

・・・

＃＃＃ さらに追記

もとのコードを見るに、書きたかったのは、こういうことだったかも？

//main関数
function main() {
  var endList = [];

  endList.push(rand());
  endList.push(rand());
  console.log(endList);
}


//ランダムな値を取得
function rand() {
  var movieList = [0, 1, 2, 3, 4, 5, 6];

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

  // シャッフルする
  for (var i = movieComb.length - 1; i > 0; i--) {
      var r = Math.floor(Math.random() * (i + 1));
      var tmp = movieComb[i];
      movieComb[i] = movieComb[r];
      movieComb[r] = tmp;
  }

  // 取り出す
  return movieComb[Math.floor(Math.random() * movieComb.length)];
}

main();