вызов setinterval со ссылкой на объект - PullRequest
Новая
       21

вызов setinterval со ссылкой на объект

3 голосов
/ 18 декабря 2010

У меня есть ротатор баннеров, и я хотел использовать объекты вместо функций, чтобы сделать код более эффективным. В любом случае, я не могу заставить setInterval работать. Я думаю, что это как-то связано со ссылкой на объект. Кто-нибудь может объяснить это? Вот что у меня так далеко: 

window.addEvent('domready', function() {
function set_banner(divid, array)
{
    var banner = $(divid);
    banner.set('html', '<a href=""><img src="" alt=""/></a>');
    var banner_link = $(divid).getElement('a');
    var banner_image = $(divid).getElement('img');

    var delay = 0;  

    for (var keys in banner1array) {
        var callback = (function(key) { return function() {
            banner.setStyle('opacity', 0);
            var object = array[key];
            for (var property in object) {
                if (property == 'href') {
                    var href = object[property];
                }
                if (property == 'src') {
                    var src = object[property];
                }
            }
            if (!banner.getStyle('opacity')) {
                banner.set('tween', {duration:1000});                   
                banner_link.setProperty('href', href);
                banner_image.setProperty('src', src);
                banner.tween('opacity', 1);
            }
        }; })(keys);
        setTimeout(callback, delay);
        delay += 21000;
    }
}

var banner1 = set_banner('banner1', banner1array);
setInterval(function() {set_banner('banner1', banner1array);}, 84000);

var banner2 = set_banner('banner2', banner2array);
setInterval(function() {set_banner('banner2', banner2array);}, 84000);

});

Ответы [ 2 ]

2 голосов
/ 18 декабря 2010

Пара простых ошибок: 

 var banner1 = new set_banner('banner1');
               ^ ---------- creates a new object and uses set_banner as the constructor
                            your code already gets called here
                            and you get a new object back, which in this case has NO use

 ....
 setInterval(banner1(), 42000);
                     ^----------------- The parenthesis EXECUTE the function
                                        the RETURN VALUE is then passed to setInterval
                                        BUT... banner1() is NOT a function, so this fails

Что вы хотите сделать в случае, если вы хотите вызвать set_banner через 42 секунды И передать параметр, это использовать анонимную функцию, которая затем вызывает set_banner. 

setInterval(function() { // pass an anonymous function, this gets executed after 42 seconds...
    set_banner('banner1'); // ...and then calls set_banner from within itself
}, 42000);
1 голос
/ 18 декабря 2010

Что еще нужно учитывать: http://zetafleet.com/blog/why-i-consider-setinterval-harmful. (tl: dr Вместо setInterval используйте setTimeout.) Хотя я не уверен, что его аргументы применимы здесь, кажется, что полезно привыкнутьизбегать. 

function defer_banner(div, bannerArray, delay) {
  setTimeout(function() {
    setBanner(div, bannerArray);
    defer_banner(div, bannerArray, delay);
  }, delay); 
});
...