Блог

В качестве заголовка я пытаюсь добавить прослушиватель щелчков к нескольким кнопкам одновременно. но когда я пробую исходный код ниже, слушатель успешно добавлен, но он не открывает никаких ссылок. Я имею в виду, что массив 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 "amp;amp;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;
          })
        }
      });
  

Проблема в том, что вы записываете данные 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;
    };
}