Discussion :

[Jilimay]

Bonjour à tous,

je suis peut-être un peu trop fatigué pour coder, mais le code suivant provoque le plantage du script :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
<SCRIPT language=javascript>
var i=0;
function Timer() {
 
    document.write(i+"<br/>");
    i++;
    setTimeout(Timer(),1000);
}
Timer();
 
</SCRIPT>

La temporisation n'est pas respectée et la fonction Timer est directement appelée, ce qui provoque l'erreur "too much recursion".

Donc : quel est le problème ?


Merci d'avance !

[sekaijin]

NE JAMAIS UTILISER
document.write

car ce dernier recharge la page si son chargement est arrivé à son terme

Code html :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
<SCRIPT id='ici' language=javascript>
var i=0;
var elementOuAjouterLesBR = document.getElementById('ici');
function Timer() {
 
    elementOuAjouterLesBR.appendChild(document.createTextNode(i));
    elementOuAjouterLesBR.appendChild(document.createElement('br'));
    i++;
    setTimeout(Timer(),1000);
}
setTimeout(Timer(),1000);
</SCRIPT>

[Jilimay]

Bonjour,

Merci pour ta réponse rapide !

Par contre, dois-je vraiment faire un appendChild sur le bloc javascript ? Peut-être faut-il plutôt créer une div quelconque avec l'id 'ici' ?

Ceci dit dans les deux cas, le problème reste le même. Je suis à peu près sûr que l'erreur est complètement débile, mais n'arrivant pas à y mettre le doigt dessus ...

[Bovino]

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

setTimeout(Timer(),1000);

C'est une référence à la fonction qu'il faut passer en paramètre, toi tu passes le résultat de l'exécution de la fonction.

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

setTimeout(Timer,1000);

[Jilimay]

Merveilleux ! Merci beaucoup !

[sekaijin]

tu fais l'append sur ce que tu veux
et bien vu
j'avais laissé passé le timer() à la place de timer

A+JYT