は一度

Question

タイトルとしてで複数のボタンにリスナーをクリックして追加し、私は一度に複数のボタンをクリックしてリスナーを追加しようとしています。 私は、以下のソースをしようとするけど、リスナーが正常に追加されますが、それはすべてのリンクを開きません。私は、関数が各要素にリスナーを追加するとき、socialLinks配列が未定義であることを意味します。誰かが私を助けることができる？

$('.IndexBody').arrive('#shortUrl', function(){ 
     var shortUrl = $('#shortUrl').text(); 
     var socialTags = ['.entypo-twitter', '.entypo-facebook', '.entypo-gplus']; 
     var socialLinks = ["http://twitter.com/share?text=http://"+shortUrl+" — Link made by ", 
          "https://www.facebook.com/sharer/sharer.php?u="+shortUrl+"&src=sdkpreparse", 
          "https://plus.google.com/share?url="+shortUrl]; 
     console.log(socialLinks); 
     for(var i=0; i<3; i++){ 
      $(socialTags[i]).click(function(e) { 
      var width = 575, 
       height = 400, 
       left = ($(window).width() - width)/2, 
       top = ($(window).height() - height)/2, 
       opts = 'status=1' + 
         ',width=' + width + 
         ',height=' + height + 
         ',top=' + top + 
         ',left=' + left; 
      window.open(socialLinks[i], 'facebook', opts); 
      console.log(socialLinks[i]); 
      return false; 
      }) 
     } 
     }); 

Answer 1

問題は、あなたがclosure内部iをキャプチャしているということですので、あなたは、関数を作成したときに、ボタンがクリックされたとき、それは値なのは、現在の値が、値ではなくなります。

このような何か試してみてください：

for(var i=0; i<3; i++){ 
    $(socialTags[i]).click(createHandler(socialLinks, i)); 
} 

function createHandler (socialLinks, i) { 
    return function (e) { 
     var width = 575, 
      height = 400, 
      left = ($(window).width() - width)/2, 
      top = ($(window).height() - height)/2, 
      opts = 'status=1' + 
        ',width=' + width + 
        ',height=' + height + 
        ',top=' + top + 
        ',left=' + left; 
     window.open(socialLinks[i], 'facebook', opts); 
     console.log(socialLinks[i]); 
     return false; 
    }; 
}