2012-01-07 11 views
0

このコードは機能していないようです。私は10秒ごとに更新されるチャットルームの動的リストを作成しようとしています。私はまた、どの部屋が表示されるのかをフィルタリングするためにperamintersを設定できるようにしたい。 私はこのコードを使用していますが、何らかの理由で動作していないようです。setIntervalが機能していないようですか?

<script type="text/javascript"> 
function setRooms() 
{ 
    console.log('ok'); 
    if($('#only').is(':checked')) 
    { 
    var checked = 1; 
    } 
    else 
    { 
    var checked = 0;  
    } 
    if($('#open').is(':checked')) 
    { 
    var opened = 1; 
    } 
    else 
    { 
    var opened = 0; 
    } 
    $.post('backend/outputRooms', { 
     fLevel : $('#flevel').val(), 
     sLevel : $('#slevel').val(), 
     display : $('display').val(), 
     Checked : checked, 

     }, 
     function(data) 
     { 
      $('#text').html(data); 
     }); 
} 
$(document).ready(function(){ 
    setInterval(setRooms(), 10000); 
    }); 
</script> 

はconsole.log(「OK」)が同様に動作していないように見えるように私は、バックエンドのページに問題があることを除外しています。どんな助け?

+0

を、コンソールを言っています.logはヒットしていませんか?私はjsfiddleでそれを実行し、okが印刷されています。 – Travis

答えて

1

あなたの関数の前後に引用符を入れてみてください:

setInterval('setRooms()', 10000); 

代わりにあなたの方法を変更します。

var setRooms = function() 

とあなたがして使用することができます。

setInterval(setRooms, 10000); 
+1

どちらも必要ありません。 Vitalyの答えに示されているように関数の名前を渡すだけです。 – jfriend00

+0

私は今それを理解していますが、彼らはまだ仕事をしています。 – akiller

+1

はい、動作しますが、 'setRooms'は既に関数を表すシンボルであるため、最初のものは推奨されません(ベストプラクティスではありません)。 – jfriend00

12

setInterval(setRooms、10000);

関連する問題