質問をすることでしか得られない、回答やアドバイスがある。

15分調べてもわからないことは、質問しよう!

新規登録して質問してみよう
ただいま回答率
85.50%
JavaScript

JavaScriptは、プログラミング言語のひとつです。ネットスケープコミュニケーションズで開発されました。 開発当初はLiveScriptと呼ばれていましたが、業務提携していたサン・マイクロシステムズが開発したJavaが脚光を浴びていたことから、JavaScriptと改名されました。 動きのあるWebページを作ることを目的に開発されたもので、主要なWebブラウザのほとんどに搭載されています。

jQuery

jQueryは、JavaScriptライブラリのひとつです。 簡単な記述で、JavaScriptコードを実行できるように設計されています。 2006年1月に、ジョン・レシグが発表しました。 jQueryは独特の記述法を用いており、機能のほとんどは「$関数」や「jQueryオブジェクト」のメソッドとして定義されています。

Q&A

解決済

3回答

2323閲覧

setInterval() が思うように動いてくれない

ippya

総合スコア31

JavaScript

JavaScriptは、プログラミング言語のひとつです。ネットスケープコミュニケーションズで開発されました。 開発当初はLiveScriptと呼ばれていましたが、業務提携していたサン・マイクロシステムズが開発したJavaが脚光を浴びていたことから、JavaScriptと改名されました。 動きのあるWebページを作ることを目的に開発されたもので、主要なWebブラウザのほとんどに搭載されています。

jQuery

jQueryは、JavaScriptライブラリのひとつです。 簡単な記述で、JavaScriptコードを実行できるように設計されています。 2006年1月に、ジョン・レシグが発表しました。 jQueryは独特の記述法を用いており、機能のほとんどは「$関数」や「jQueryオブジェクト」のメソッドとして定義されています。

0グッド

0クリップ

投稿2015/09/25 09:38

複数のタグに1秒間隔で時刻を表示したくて、下のように書いたのですが、最初に時刻が表示されるのみで、周期的に表示が更新されません。
どこに問題があるのか、皆さんのお知恵をお貸しください。

HTML

1<head> 2 <script src='http://ajax.googleapis.com/ajax/libs/jquery/2.1.4/jquery.min.js' type='text/javascript'></script> 3</head> 4<body> 5<div class="test"></div> 6<div class="test"></div> 7<div class="test"></div> 8<script> 9$(document).ready( function(e){ 10 $('.test').each( function(){ 11 setInterval( test_job( $(this) ), 1000); 12 }); 13}); 14 15function test_job(obj){ 16 var dd = new Date(); 17 obj.text( dd.toLocalString() ); 18} 19</script> 20</body>

気になる質問をクリップする

クリップした質問は、後からいつでもMYページで確認できます。

またクリップした質問に回答があった際、通知やメールを受け取ることができます。

バッドをするには、ログインかつ

こちらの条件を満たす必要があります。

guest

回答3

0

以下のように変更。

javascript

1$(document).ready( function(e){ 2 $('.test').each( function(){ 3 var self = this; 4 setInterval( function(){test_job( $(self) )}, 1000); 5 }); 6}); 7 8function test_job(obj){ 9 var dd = new Date(); 10 obj.text( dd.toLocaleString() ); 11}

変更した点は3つあります。

  • test_job( $(this) ) は undefined を返すので無名関数でラップした
  • グローバルスコープでは this は window を返すので、クロージャを使った
  • .toLocaleString() のtypo

参考になれば幸いです。

投稿2015/09/25 10:10

Lhankor_Mhy

総合スコア35860

バッドをするには、ログインかつ

こちらの条件を満たす必要があります。

ippya

2015/09/25 10:45

早速の回答ありがとうございます。 おかげさまで解決しました^^。
guest

0

ベストアンサー

javascript

1 2$(document).ready( function(e){ 3 $('.test').each( function(){ 4 var $self = $(this); 5 setInterval(function(){ test_job($self); }, 1000); 6 }); 7}); 8 9function test_job(obj){ 10 var dd = new Date(); 11 obj.text( dd.toLocaleString() ); 12}

で動きますでしょうか?

ポイントは setIntervalの第一引数に無名関数。
これは、呼び出したい関数に引数を与えるためです。

また、無名関数の中でthisを使うと、本来の$('.test')のものと異なるので、
一度別変数にthisを格納しておき、使用するところです。

なお、toLocalStringtoLocaleStringの誤植ですかね?

投稿2015/09/25 10:14

rk7fd3s

総合スコア61

バッドをするには、ログインかつ

こちらの条件を満たす必要があります。

rk7fd3s

2015/09/25 10:15

あぁ、4分差でかぶりましたねw
ippya

2015/09/25 10:45

回答ありがとうございます。 おっしゃる通り、toLocalString() は e が抜けてました^^;; おかげさまで解決しました^^。
guest

0

最初の例だとsetIntervalの第1引数が関数ではなく、test_job()の戻り値になっているのが問題です。
他の方の例のようにfunction(){}を1クッション挟むとうまくいくと思います。

投稿2015/09/25 10:17

gtakat

総合スコア206

バッドをするには、ログインかつ

こちらの条件を満たす必要があります。

ippya

2015/09/25 10:46

回答ありがとうございます。 setInterval に function(){} が必要だったんですね。 ありがとうございました^^。
guest

あなたの回答

tips

太字

斜体

打ち消し線

見出し

引用テキストの挿入

コードの挿入

リンクの挿入

リストの挿入

番号リストの挿入

表の挿入

水平線の挿入

プレビュー

15分調べてもわからないことは
teratailで質問しよう!

ただいまの回答率
85.50%

質問をまとめることで
思考を整理して素早く解決

テンプレート機能で
簡単に質問をまとめる

質問する

関連した質問