array.push（）は奇妙なエラーをスローする

Question

配列Xに要素をプッシュしようとしていますが、配列Yを反復しています。ある時点で、新しい要素を配列Xにプッシュしているときに、ブラウザコンソールに「予期しないトークン：」というエラーが表示されます。私はそれが失敗する前にいくつかの要素を適切に押すことができます（ほぼ毎回7番目の要素）。あなたがスクリプトから見ることができるように、私は8つの異なる位置付けでランダムに写真を表示しようとしている

function getPosition(img) { 
    var tmpRandPosition = Math.floor(Math.random() * (9)); 

    if($.inArray(galleryPositions[tmpRandPosition], populatedPositions) != -1) { 
     setTimeout("getPosition("+img+")",1); 
    } else { 
     populatedPositions.push(galleryPositions[tmpRandPosition]); 

     return true; 
    } 
} 

：

は、それが問題の原因かもしれない再帰関数、そしてある...ここでは、コードですHTMLの要素。

Answer 1

setTimeout機能に問題があるようです。連結の代わりに無名関数を使用してその関数に引数を渡すようにしてください：

setTimeout(function() { getPosition(img) }, 1); 

Answer 2

これは壊れます：

setTimeout("getPosition("+img+")",1); 

を、それが実際に書き込みますよう：

setTimeout("getPosition(img_path.jpg)",1); 

を、評価しようとしますそれは（evalを使用して）。

問題はJSがimg_path.jpgを変数とみなしていることです。

修正：

setTimeout("getPosition('"+img+"')",1); 

しかし、それは文字列を評価するために良いか高速ではないとして、このようにそれを行うことはありません。

のsetTimeoutに代わりに無名関数を送る：

REALFIX：

setTimeout(function() { 
    getPosition(img); 
}, 1); 

Answer 3

Don't call setTimeout with a string argument.常に機能を使用します。この場合の文字列を使用すると、img変数が脆弱な場合にこのコードが注入攻撃を受けます。