ajax problem z wielokrotnie wykonywaną funkcją Opcje

[Xardian]

posiadam takie 2 pliczki

[HTML] pobierz, plaintext 
var http_request = false;
function makePOSTRequest(url, parameters, diva) {
http_request = false;
 
if (window.XMLHttpRequest) {
http_request = new XMLHttpRequest();
if (http_request.overrideMimeType) {
http_request.overrideMimeType('text/html');
}
} else if (window.ActiveXObject) {
try {
http_request = new ActiveXObject("Msxml2.XMLHTTP");
} catch (e) {
try {
http_request = new ActiveXObject("Microsoft.XMLHTTP");
} catch (e) {}
}
}
 
http_request.onreadystatechange = function(){
if (http_request.readyState == 1) {
document.getElementById(diva).innerHTML = "Ładowanie...";
 
}
if (http_request.readyState == 4) {
if (http_request.status == 200) {
result = http_request.responseText;
document.getElementById(diva).innerHTML = result;
czas();
 
}
else {
document.getElementById(diva).innerHTML = "Wystąpił błąd!";
}
}
};
 
http_request.open('POST', url, true);
http_request.setRequestHeader('Content-Type', 'application/x-www-form-urlencoded');
http_request.send(parameters);
 
}
[HTML] pobierz, plaintext 

jak łatwo można zauważyc dodałem wywołanie funkcji czas() i tu jest własnie problem, ponieważ po kliknięciu w jakikolwiek link z jaxem uruchomione funkcje czas() mnożą się powodując efekt zbyt szybkiego wykonywania :/

[HTML] pobierz, plaintext 
function czas()
{
liczba = document.getElementById('czasq').value;
 
sec = liczba % 60;
liczba = liczba - sec;
min =( liczba % 3600)/60 ;
liczba = liczba - min * 60;
god = liczba /3600;
 
document.getElementById('godz').innerHTML=god;
document.getElementById('min').innerHTML=min;
document.getElementById('sec').innerHTML=sec;
 
odliczaj();
 
 
}
function odlicz(){
cc= 1;
if(god == 0 && min == 0 && sec == 0)
{ 
document.getElementById('czas').innerHTML='zakończono';
}
else
{
setTimeout(function() { odliczaj(); } ,1000);
}};
 
function odliczaj()
{
if(sec > 0)
{
sec=sec - 1 ;
}
else
{
 
sec = 59;
 
if(min > 0)
{
min=min - 1 ;
}
else
{
if(god > 0){
god = god -1;
min = 59;
};};};
document.getElementById('godz').innerHTML=god;
document.getElementById('min').innerHTML=min;
document.getElementById('sec').innerHTML=sec;
setTimeout("odlicz()" ,0);
}
[HTML] pobierz, plaintext 

ma ktoś pomysł jak to zablokować żeby tylko 1 raz czas() działał w danej chwili lub gdzie go umieścić ;/
z niewiadomych przyczyn ten skrypt niedziala pod IE a pod mozillii śmig ale się wielokrotnie ładuje -.-

Ten post edytował Xardian 10.12.2007, 13:53:20

[Anatejms]

dzieje sie tak zapewne dlatego ze wywolania sa co 1s i sie zapentalaja. uzyj identyfikatora do funkcji setTimeout i pozniej zeruj funkcje przy uzyciu clearTimeout('identyfikator'). a poza tym nie wiem po co uzywasz setTimeout("odlicz()" ,0); to toz ci niezle zamula skrypt napisz po prostu odlicz();

nie testowalem ale tak powinno działać

[HTML] pobierz, plaintext 
function czas()
{
liczba = document.getElementById('czasq').value;
 
sec = liczba % 60;
liczba = liczba - sec;
min =( liczba % 3600)/60 ;
liczba = liczba - min * 60;
god = liczba /3600;
 
document.getElementById('godz').innerHTML=god;
document.getElementById('min').innerHTML=min;
document.getElementById('sec').innerHTML=sec;
 
odliczaj();
 
 
}
function odlicz(){
cc= 1;
if(god == 0 && min == 0 && sec == 0)
{ 
document.getElementById('czas').innerHTML='zakończono';
}
else
{
weit= setTimeout('odliczaj()' ,1000);
}};
 
function odliczaj()
{
clearTimeout('weit');
if(sec > 0)
{
sec=sec - 1 ;
}
else
{
 
sec = 59;
 
if(min > 0)
{
min=min - 1 ;
}
else
{
if(god > 0){
god = god -1;
min = 59;
};};};
document.getElementById('godz').innerHTML=god;
document.getElementById('min').innerHTML=min;
document.getElementById('sec').innerHTML=sec;
odlicz();
}
[HTML] pobierz, plaintext 

a i dla przejzystosci kodu uzywaj wcieć (taka rada ) latwiej sie patrzy gdzie sie koncza if'y i wogole