testo temporizzato in un div

[alladr2]

dev'essere una sciocchezza, ma non riesco a trovare l'errore.
l'effetto di questo script dovrebbe essere quello di cambiare ogni minuto il testo contenuto in in div (id='titolo'), ma non funzia:

function cambiatitolo(){
var minuto = new Date();
var txt = document.getElementById("titolo");
if((minuto.getMinutes() % 2) != 0){
txt.innerHTML = "testo1";
}
else{
txt.innerHTML = "testo2";
clearInterval(timerID);
}
}

timerID = setInterval('cambiatitolo()', 10000);

consigli?

[homezappa]

Dunque, ricapitoliamo:
Il tuo codice fa esattamete quello che gli hai detto di fare, e cioè

• fa partire un timer che ad ogni 10 secondi chiama la cambiatitolo()
• allo scattare del timer, se il minuto corrente modulo 2 è diverso da zero scrive 'testo1' nel div
• se invece il minuto corrente modulo 2 è uguale a zero, scrive 'testo2' nel div e ... FERMA IL TIMER! ... perché?

Io l'avrei fatta così:

function cambiatitolo(){
    var minuto = new Date();
    var txt = document.getElementById("titolo");
    if((minuto.getMinutes() % 2) != 0){
        txt.innerHTML = "testo1";
    }
    else {
        txt.innerHTML = "testo2";
    }
}

timerID = setInterval(cambiatitolo, 10000); 

HTH
Zappa

[alladr2]

ferma tutto perché ho il terrore di sovraccaricare il processore (eh, lo so, è un'idiozia).
però il mio problema è che in realtà questo script non funziona e non capisco perché.
hai qualche consiglio da darmi?

[alladr2]

cazz... erano le virgolette, credo. funziona.
grazie!

[PaoloIT]

E' sempre preferibile comunque utilizzare un setTimeout in una funzione che richiama se stessa.

[alladr2]

magari ci metto un contatore, e quando raggiunge le 10 ripetizioni lo faccio bloccare.

[PaoloIT]

esatto.
Io in genere utilizzo dei contatori come proprietà di un oggetto in modo da avere una persistenza specifica dell'evento.

[alladr2]

cioè, ehm, come fai? (io faccio solo prove! sono un dannato lamer!)

[PaoloIT]

Se è un div, ad esempio a scatenare l'evento identifico il div con il suo "id" appositamente attribuitogli e successivamente lo identifico con:

var ilMioDiv = document.getElementById("idDelMioDiv");

Andrò poi ad assegnargli una proprietà "custom", che so, "times":

ilMioDiv.times=0;

Nella funzione di temporizzazione andrò a controllare ed implementare la proprietà:

if (ilMioDiv.times<10){
ilMioDiv.times+=1;
/* etcoetera etcoetera */
}
else{
/* etcoetera etcoetera */
}

Ho abbandonato da tempo innerHTML e document.write() in favore di funzioni DOM standard ECMA.

[alladr2]

fichissimo. ho provato ad implemantare questa soluzione (con a sola differenza che faccio clearinterval se txt.volte >9 perché anche se non riesco a spiegarlo mi pare più economico). eppure non funziona.
potrebbe essere dalla funzione isdefined usata a naso?

function cambiatitolo(){
var txt = document.getElementById("titolo");
var minuto = new Date();
if(isDefined(txt.volte)){txt.volte+=1;} else{txt.volte = 0;}
if((minuto.getMinutes() % 2) != 0){
txt.innerHTML = "testo1 "+ txt.volte;
}
else{
txt.innerHTML = "testo2 "+ txt.volte;
}
if(txt.volte>9){clearInterval(timerID);}
}

timerID = setInterval(cambiatitolo, 10000);