42個の組み合わせを作りたいのに、43組みになってしまう問題について

Question

42個の組み合わせ((1,1)などの同じ値の組み合わせは無視する)を表示させたいのですが、なぜか43組みになってしまいます。
解決策をお願い致します。

function main(shuffleList) {
  var quotient, remainder;

  for (i = 0; i < shuffleList.length; i++) {
    var x = shuffleList[i];
    quotient = Math.floor(x / 7);
    remainder = x % 7;
    if (quotient !== remainder) {
      console.log(quotient);
      console.log(remainder);
      console.log("##############");
    }
    else { continue; }
  }
}

function shuffle() {
  var list = [1,2,3,4,5,6,7,8,9,10,11,12,13,14,15,16,17,18,19,20,21,22,23,24,25,26,27,28,29,30,31,32,33,34,35,36,37,38,39,40,41,42,43,44,45,46,47,48,49];

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

Comments

[think49]

main(shuffleList); が抜けているのでは？
このままでは動きません。

Answer 1

[0,0] と [7,7] のペアが成立しないから。

49 - 6 === 43

※個人的には、[0,1,2,3,4,5,6,7] の中から「42通りのペア」を抽出する意義が謎です。
[1,2,3,4,5,6,7] から「42通りのペア」を抽出するなら分かりますが。

Re: gyoruo さん

Comments

[gyoruo]

0〜7はこちらの都合でそうさせていただきました。
詳しく記載してなく、申し訳ございません。

[think49]

0-7がお望みなら、jun68yktさんの回答は0-6なので、期待通りではありません。
次の質問の「1から始まる」も0を含まない為、期待通りではありません。
https://teratail.com/questions/217068

Answer 2

こんにちは

解決策をお願い致します。

とのことなので、ご質問のタイトルにある「42個の組み合わせを作りたい」との要件を満たすために、取り急ぎの

解決策

を挙げますと、function shuffle() の冒頭で

修正前：

var list = [1,2,3,4,5,6,7,8,9,10,11,12,13,14,15,16,17,18,19,20,21,22,23,24,25,26,27,28,29,30,31,32,33,34,35,36,37,38,39,40,41,42,43,44,45,46,47,48,49];

としている部分を、 0始まりの49個の配列、すなわち、0以上48以下の整数を含む配列

修正前：

var list = [0,1,2,3,4,5,6,7,8,9,10,11,12,13,14,15,16,17,18,19,20,21,22,23,24,25,26,27,28,29,30,31,32,33,34,35,36,37,38,39,40,41,42,43,44,45,46,47,48];

に修正することかなと思われます。
上記を動作確認するコードを、以下のCodePenに作成しました。

• https://codepen.io/jun68ykt/pen/GRRoXrN?editors=0011

これは、ご質問に挙げられているコードの var list を上記のように修正し、さらに、 main の中で、組み合わせの数をカウントして出力するようにしたもので、 main の終了時に "組み合わせの数： 42" と表示されます。

補足：
本題とは関係ありませんが、上記の０以上48以下の整数を昇順に含む配列 list は、49個の数を並べる他に、以下でも作れます。

var list = [...Array(49)].map((_,i) => i);

以上、参考になれば幸いです。

Comments

[gyoruo]

ご丁寧な対応、ありがとうございます。
最後の補足でもできるんですね！
教えていただき、ありがとうございます。

[jun68ykt]

どういたしまして。次のご質問にも回答しましたので、参考になれば幸いです。