[JS] setTimeout Opcje

[MaestroS]

Napisałem taką funkcję:

[HTML] pobierz, plaintext 
function Renew(user, button)
{
advAJAX.post({
          url : "plik.php",
          parameters:{
               user: user
 
  },
  onSuccess : function(obj) {
        if (obj.responseText == "renewal_ok")
        {
                alert('Odnowa przebiegła pomyślnie.\nMożesz kontynuować swoją przygodę.');
                button.disabled = true;
        }
        else
        {
 
                document.getElementById('status').innerHTML=obj.responseText;
                setTimeout(Renew(user, button), 10000);
                button.disabled = true;
        }
  },
});
}
[HTML] pobierz, plaintext 

Tak ją wywołuję:

[HTML] pobierz, plaintext 
<div id="status"></div>
<BR><input type="button" value="Odpocznij" id="rest" onClick="Renew('' . $GlobalUserAccount['character_name'] . '', document.getElementById('rest'))">
[HTML] pobierz, plaintext 

Moje pytanie:
Dlaczego setTimeout nie działa ? Klikając na 'Odpocznij' - oczywiscie, statystyki sie odnawiaja, ale co 0,01 sec., a nie jak setTimeout nakazuje: co 10 sekund... poprostu staty odnawiaja sie za szybko. Gdzie popelnilem blad ?

Ten post edytował MaestroS 28.06.2007, 18:23:55

[sticker]

czas jest w mikrosekundach wiec pewnie sie pomyliłes z zerem :]

[abc667]

no bo ty wywołujesz funkcje tam a nie przekazujesz jej do timeouta

można to zrobić np tak

[HTML] pobierz, plaintext 
setTimeout(function() { Renew(user, button); }, 10000);
[HTML] pobierz, plaintext 

[atomix1040]

tudziez podac paretr jako zmienna znakowa

setTimeout ("funckja()", 10000);
a nie
setTimeout (funkcja(), 10000);

[abc667]

ale tak nie przekaże się łatwo np referencji do obiektu a tak jak podałem jest optymalnie

[atomix1040]

jesli potrzeba przekazac parametry do funkcji to jasne ze nie ale jest to jest prosta metoda update naprzyklad z Prototype Updater Ajax to wystarczy, mniej kodu to zawsze lepiej.