setTimeout перестает работать

Question

У меня есть страница, где я хочу сохранить значение из файла, обновленного все время.

У меня есть сценарий, сохраняющий некоторые данные для просмотра в Интернете в txt-файле, а затем я хочу показать эти данные в Интернете. Данные в текстовом файле обновляются каждые 20 секунд.

Он работает хорошо, как и раньше, через 3 минуты или около того, после чего страница останавливается. Есть идеи, почему это происходит?

function updatepot(elementid) { 
     var xmlhttp = new XMLHttpRequest(); 
     xmlhttp.onreadystatechange = function() { 
      if (this.readyState == 4 && this.status == 200) { 
       document.getElementById(elementid).innerHTML = this.responseText; 
      } 
     }; 
     xmlhttp.open("GET", "../readData.php?q=" + elementid, true); 
     xmlhttp.send(); 

} 

function updatePots() 
{ 
     updatepot("pot0"); 
} 
function keeprunning() 
{ 
    setTimeout(updatePots, 1000); 
    keeprunning(); 
} 


<?php  
// get the q parameter from URL 
$file = $_REQUEST["q"] . ".txt"; 
$myfile = fopen("potData/".$file, "r") or die("Unable to open file!"); 
$myData; 
while(!feof($myfile)) { 
    $myData = $myData . fgets($myfile); 
} 
fclose($myfile); 
echo $myData; 
?> 

Answer 1

Если вы хотите функцию запускать периодически, используйте setInterval вместо setTimeout - таким образом, вы не должны иметь дело с регулировкой интервала:

function updatepots() 
{ 
    updatepot("pot0"); 
} 

window.setInterval(updatePots, 1000); 

Answer 2

Ваш метод keepRunning вызывается немедленно, как только вызывается функция setTimeout. Это означает, что вместо того, чтобы вызывать каждую секунду, как вы, вероятно, намереваетесь, она называется постоянно (тысячи раз в секунду) - вы быстро столкнетесь с проблемами памяти, и все перестанет работать.

Чтобы это исправить, вызовите keepRunning в конце вашей функции updatePots:

function updatePots() 
{ 
     updatepot("pot0"); 
     keeprunning(); 
} 
function keeprunning() 
{ 
    setTimeout(updatePots, 1000); 
} 

Answer 3

вызов его таким образом:

function updatePots() 
{ 
    updatepot("pot0"); 
    keeprunning(); 
    setTimeout(updatePots, 1000); 
}