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

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

ただいまの
回答率

89.20%

Javascriptの基礎的な質問です。

解決済

回答 2

投稿 編集

  • 評価
  • クリップ 0
  • VIEW 919

fama

score 100

現在、ボタンをクリックするとページを下に3秒毎に繰り返しスクロールする処理を実装しようとしています。

function startScroll()
{
   timer = setInterval(scrollDown(),3000);
}

function scrollDown()
{  
    var webview = document.getElementById('mainWebview');
    webview.executeJavaScript('window.scrollTo(0, 100000000)');
}

このように記述した場合、スクロールが1回のみ行われ処理が終わります。
なぜ1回のみしか処理が行われないのかがわからず試行錯誤して timer = setInterval(scrollDown(),3000); をfunctionの外に出してみました。

timer = setInterval(scrollDown(),3000);

function scrollDown()
{  
    var webview = document.getElementById('mainWebview');
    webview.executeJavaScript('window.scrollTo(0, 100000000)');
}


すると、求めている処理(ボタンを押すとスクロールループ)とは違いますが、ページが開いてから繰り返しスクロールをループしてくれました。

このように、 timer = setInterval(scrollDown(),3000); をfunctionの中に入れてonclickイベントで呼び出した場合はなぜ1回のみの実行になり、functionの外に出した場合は正常にループしてくれるのか仕組みがわからず困っています。

わかりやすく説明して頂ける方はいらっしゃいませんでしょうか?

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

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

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

    クリップを取り消します

  • 良い質問の評価を上げる

    以下のような質問は評価を上げましょう

    • 質問内容が明確
    • 自分も答えを知りたい
    • 質問者以外のユーザにも役立つ

    評価が高い質問は、TOPページの「注目」タブのフィードに表示されやすくなります。

    質問の評価を上げたことを取り消します

  • 評価を下げられる数の上限に達しました

    評価を下げることができません

    • 1日5回まで評価を下げられます
    • 1日に1ユーザに対して2回まで評価を下げられます

    質問の評価を下げる

    teratailでは下記のような質問を「具体的に困っていることがない質問」、「サイトポリシーに違反する質問」と定義し、推奨していません。

    • プログラミングに関係のない質問
    • やってほしいことだけを記載した丸投げの質問
    • 問題・課題が含まれていない質問
    • 意図的に内容が抹消された質問
    • 過去に投稿した質問と同じ内容の質問
    • 広告と受け取られるような投稿

    評価が下がると、TOPページの「アクティブ」「注目」タブのフィードに表示されにくくなります。

    質問の評価を下げたことを取り消します

    この機能は開放されていません

    評価を下げる条件を満たしてません

    評価を下げる理由を選択してください

    詳細な説明はこちら

    上記に当てはまらず、質問内容が明確になっていない質問には「情報の追加・修正依頼」機能からコメントをしてください。

    質問の評価を下げる機能の利用条件

    この機能を利用するためには、以下の事項を行う必要があります。

質問への追記・修正、ベストアンサー選択の依頼

  • think49

    2016/02/20 12:28

    重複質問です。https://teratail.com/questions/27751

    キャンセル

  • fama

    2016/02/20 15:42

    ご指摘ありがとうございます!修正致します!

    キャンセル

回答 2

checkベストアンサー

0

質問の内容を変更する場合は質問の編集機能を使用してください。重複質問ととられてしまうのが普通です。

setInterval 関数が第一引数に取るのは Function オブジェクト、すなわち関数です。
下の (1) と (2) の記述の意味の違いに注意してください。
scrollDown 関数は値を返さないので setInterval には undefined が渡されます。

// (1) scrollDown 関数が渡される(3 秒間隔の実行が予約される)
setInterval(scrollDown, 3000);
// (2) scrollDown 関数の実行結果が渡される(副作用的にこの時点で一度だけ実行される)
setInterval(scrollDown(), 3000);

次に scrollDown の中身ですが、これは原点(画面の左上)を基準にして、(x: 0, y: 100000000) の位置にスクロールするものです。何秒経ってもこの位置は同じですがこれは意図通りですか?
たとえば 3 秒ごとに少しずつスクロールする場合は window.scrollBy を使用してください。

投稿

  • 回答の評価を上げる

    以下のような回答は評価を上げましょう

    • 正しい回答
    • わかりやすい回答
    • ためになる回答

    評価が高い回答ほどページの上位に表示されます。

  • 回答の評価を下げる

    下記のような回答は推奨されていません。

    • 間違っている回答
    • 質問の回答になっていない投稿
    • スパムや攻撃的な表現を用いた投稿

    評価を下げる際はその理由を明確に伝え、適切な回答に修正してもらいましょう。

  • 2016/02/20 17:48

    書き足した結果どのような処理になったか不透明なので質問欄かどこかに現在のコードを書いてもらえますか?(コメント欄は見にくいのでw)
    onclick に追加する処理も書いてもらえれば原因が分かるかもしれません。

    キャンセル

  • 2016/02/20 18:45

    なるほどです!teratailの使い方も勉強になります!
    後、今回問題が解決致しました!
    私の初歩的なミスでして、正常にコードを書くところまではできていたのですが、呼び出す処理をhtml側で間違えていました 涙

    キャンセル

  • 2016/02/20 18:46

    解決してよかったですね。

    キャンセル

0

setInterval(scrollDown(),3000);


より、

setInterval(
    function(){
        //処理
    }, 3000
);


の方が私は好きです。
ほかのメソッドからも呼び出すようなメソッドでなければ、この方が見やすいと思います。
変数のスコープとかもわかりやすいし。

投稿

  • 回答の評価を上げる

    以下のような回答は評価を上げましょう

    • 正しい回答
    • わかりやすい回答
    • ためになる回答

    評価が高い回答ほどページの上位に表示されます。

  • 回答の評価を下げる

    下記のような回答は推奨されていません。

    • 間違っている回答
    • 質問の回答になっていない投稿
    • スパムや攻撃的な表現を用いた投稿

    評価を下げる際はその理由を明確に伝え、適切な回答に修正してもらいましょう。

  • 2016/02/20 15:31

    otftroughさんアドバイスありがとうございます!
    たしかにこれは見やすいですね!ちょっと記述してみます★

    キャンセル

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

  • ただいまの回答率 89.20%
  • 質問をまとめることで、思考を整理して素早く解決
  • テンプレート機能で、簡単に質問をまとめられる