JavaScriptで全くの初心者なのですが、

質問

JavaScriptで全くの初心者なのですが、
フィッシャーイェーツのシャッフルをaのリスト内でしてから、
そこから一つ値をとり、
別の関数内で、その関数内での新たな変数に代入するようにしたいです。（function(){var b = this.result;}のような）
プログラミングはやらないのに急に使うことになり困っているのでだれか助けてください。

a = [1,1,1,5,5,5,9,9,9];
for (var a=[],i=0;i<9;++i) a[i]=i;
this.result = a[shuffle(a.length)] ;
function shuffle(array) {
var tmp, current, top = array.length;
if(top) while(--top) {
current = Math.floor(Math.random() * (top + 1));
tmp = array[current];
array[current] = array[top];
array[top] = tmp;
}
return array;
}

回答No.1

AsarKingChang · ベストアンサー

いまいち何がしたいのか、よくわかりません
a = [1,1,1,5,5,5,9,9,9];←まず代入
for (var a=[],i=0;i<9;++i) a[i]=i;←ここで、再度aを初期化[]して、a[i]=i;
とすれば、先の行は無意味です。

this.result = a[shuffle(a.length)] ;
shuffle(a.length)←aの長さつまり0~8個なので９の「数値」をshuffleは受け取りますが、function shuffle(array) は、arrayとかいてあり、arrayではなく、数値になってます。
またこの関数は受け取り値そのものがarrayにしているので、
this.result = shuffle(a)[0]でいいようにも思える反面
arrayという書き方は「型」なので「変数名」にするべきですよ。
function shuffle(hoge) { ←とか、別にarrayじゃなければなんでもいい

最後に、
a[shuffle(a.length)] ;
ですが、変数メモリの「最後の位置」は、常にUndefinedです。
”123456789”
と入れているつもりでしょうが、0オリジンなので、
これの順番は、
”123456789”
-012345678＝合計9個。しかし配列番号に９はない。
したがってメモリの外を参照している。
また、この関数がシャッフルが目的なら、
0～８のどれを書いても、それ自体に意味はなく、結果としても
たいした意味を持たない。

"this"が行方不明
this.result

という事で、
なんだろうな～
やりたいことをそのまま書いてみませんか？
恐らくは、流れそのものに問題があるように感じたもので。
（酒飲んでるので、意味不明回答になってたらごめんなさい）

で、プログラムするとき何でもかんでも表示してみると
上達は早いですよ。
console.log("A=" + a);
とか、でデバッグコンソールに表示させて
表示した瞬間の値をどんどん表示させると、動きがわかって
勉強になるのと、覚えも早くなりますよ～
では！がんば～！

お礼

ご報告おくれましたが、本当にありがとうございます！

回答No.2

kt_yuka

やりたいのはこれかな？
https://qiita.com/artistan/items/9eb9a0fb14f4ec3a8764

お礼

ありがとうございます。一応そのurlは見ていました。