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

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

ただいまの
回答率

88.78%

イベントハンドラの記述の仕方による実行の違い

受付中

回答 2

投稿 編集

  • 評価
  • クリップ 1
  • VIEW 594

keip

score 33

JavaScriptの一部で、styleのアトリビュートで、displayがnoneとなっている時となっていない時でメッセージの表示を変えるという処理を書きました。

    function pic_status(){
        var ma = $("#message-area:first").find("span");
        if (pic_style === "none"){
            ma.text("the pic is hidden");
        }else{
            ma.text("the pic is shown");
        }
    }
    window.onload = pic_status();

この最後の行の

window.onload = pic_status();


というようにイベントハンドラを書いた場合にはページを読み込んだ際に判別して表示を変えてくれます。
ただし

window.onload = function(){
    pic_status();
}


window.addEventListener("load", function(){
    pic_status();
)}


などと記述した場合には、ページの読み込み時は表示を変えてくれず、どこかのボタンを押すなどした時にやっと反応します。

イベントハンドラの記述の仕方で挙動が異なるのはどういうことなのでしょうか?
よろしくお願いいたします。

<!DOCTYPE html>
<html>
    <head>
        <meta charset="UTF-8">
        <title>JS勉強</title>
        <link rel="stylesheet" type="text/css" href="../css/stylesheet.css">
        <link rel="stylesheet" type="text/css" href="./bootstrap/bootstrap-4.3.1-dist/css/bootstrap.min.css">
        <script src="../js-jq/jquery-3.4.1.min.js"></script>
        <script src="../js-jq/myjs.js"></script>
    </head>
    <body>
        <header>
            <h1>JavaScript</h1>
        </header>
        <div id="main" class="container">
        <div id="area51">
            <div id="changeable-area">
                <div id="text-area">
                    <span>something</span>
                </div>
            </div>
        </div>
        <div id="button-wrapper"><button id="trigger-button">TRIGGER</button></div>
        <div id="message-area">
            <span>some messages are written in this area.</span>
        </div>
        <p class="button-area"><button id="pic-button">PIC</button></p>
        <div class="pictures">
            <img src="../images/test1.jpg">
        </div>
        </div>
    </body>
    <footer>I'm learning JavaScript.</footer>
</html>
*{
    margin:0; padding:0;
}

#pics img{
    width: 300px;
}
body{
    text-align: center;
}
footer{
    text-align: center;
    height: 50px;
    line-height: 50px;
    color: whitesmoke;
    background-color: cornflowerblue;
    font-weight: bold;
    margin-top: 15px;
}
header{
    height: 60px;
    color: whitesmoke;
    background-color: #333333;
    line-height: 60px;
    margin-bottom: 15px;
}
header h1{
    margin-left: 10px;
}
#trigger-button{
    border-radius: 3px;
}
#area51{}

#changeable-area{
    width: 100%;
    height: 250px;
    border: solid 1px #333333;
    margin: 0 auto;
    font-size: 20px;
    font-weight: bold;
    border-radius: 4px;
    text-align: center;
}
#text-area{
    padding: 5px 15px 30px 15px;
}
#message-area{
    height: 50px;
    border: solid 1px #333333;
    border-radius: 4px;
    text-align: center;
    width: 60%;
    margin: 0 auto;
    overflow: auto;
}
#message-area span{
    font-weight: bold;
    font-size: 13px;
}

#text-area{
    height: 250px;
    word-break: break-all;
    margin: 0 auto;
    overflow: auto;
}
#trigger-button{
    margin: 0 10px;
    width: 80px;
    line-height: 25px;
    border-radius: 3px;
}
#button-wrapper{
    text-align: center;
    margin: 10px 0;
}
.pictures{
    margin: 0 auto;
    width: 640px;
}
#pic-button{
    width: 50px;
    border-radius: 4px;
}
.button-area{
    text-align: center;
    margin-top: 10px;
}
$(document).ready(function(){
    function pic_status(){
        var ma = $("#message-area:first").find("span");
        if (pic_style === "none"){
            ma.text("the pic is hidden");
        }else{
            ma.text("the pic is shown");
        }
    }
    window.onload = pic_status();

    $("#trigger-button").click(function(){
        $("#text-area").prepend("<span>a text was created. </span>");
    })


    var pics = $(".pictures");
    var pb = $("#pic-button");
    var pic_style;
    pb.click(function(){
        pics.toggle();
        pic_style = pics.css("display");
        pic_status();
    })


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

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

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

    クリップを取り消します

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

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

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

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

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

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

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

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

    質問の評価を下げる

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

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

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

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

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

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

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

    詳細な説明はこちら

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

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

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

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

  • m.ts10806

    2019/05/21 00:01

    再現可能なようにHTMLとjsの記述もあわせてご提示いただけますか?

    キャンセル

  • keip

    2019/05/21 00:13

    ありがとうございます。
    ソースコードを貼りました。
    よろしくお願いします。

    キャンセル

回答 2

+4

window.onload

エラーの原因はSyntaxErrorかと思いますが、関数呼び出しと関数オブジェクトの違いを意識する事から始めると良いように思います。

window.onload = pic_status();

関数名に () を付けるという事は関数pic_statusをその場で実行するということです。
関数pic_statusの返り値は undefined ですので、このコードは下記コードと等価です。

pic_status();
window.onload = undefined; // 関数pic_statusの返り値 undefined を代入する

window.onload には undefined が代入されますので、onloadのタイミングでは何も発生しません。
質問者さんな無名関数式を使って、この問題を解いたようですが、もっと簡潔な方法があります。

window.onload = pic_status; // 関数オブジェクト pic_status を代入する

window.onload  には関数オブジェクトを代入しなければなりませんので、関数pic_statusをそのまま代入すれば良いのです。

window.addEventListener

addEventListenerもonloadと同様の書き方が可能です。

window.addEventListener('load', pic_status, false); // 関数オブジェクト pic_status を第二引数に指定する

addEventListenerは第三引数を省略可能ですが、後方互換性の為に明示的に指定する事をお勧めします。
https://developer.mozilla.org/ja/docs/Web/API/EventTarget/addEventListener

$(document).ready()

$(document).ready(function(){
    // 中略
});

このコードは DOMContentLoaded とほぼ同じタイミングで発動します(等価ではありません。
DOM構築完了が保証されているので、そもそも論でいえば、loadイベントで待つ必要はありません
https://api.jquery.com/ready/

Re: keip さん

投稿

編集

  • 回答の評価を上げる

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

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

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

  • 回答の評価を下げる

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

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

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

  • 2019/05/21 11:03

    think49さん
    ありがとうございます。
    なるほど、window.onload = pic_status();はそういうことになっているのですね。

    なるほど、$(document).ready()はloadでイベントを待つ必要はないのですね。勉強になりました。
    ありがとうございました。

    キャンセル

  • 2019/05/21 12:18

    addEventListener の第三引数の話は Firefox 5 のことでしょうか?
    jQuery 2.2 から内部的に第三引数を指定しなくなっていますし、あまり気にしても仕方ないかなと思いました。

    キャンセル

+3

こちらのコードが動作しないのは
書き方が間違っているからです。

window.addEventListener("load", function(){
  pic_status();
)}

正しくは)}ではなく})です。

window.addEventListener("load", function(){
  pic_status();
})

もう一方のこちらは書き方は間違っておらず
私の方で動作が確認できたため
動作しないというのは勘違いです。

window.onload = function(){
  pic_status();
}

余談ですがfooterタグがbodyタグの外側にあるため
正しく内側に記述しましょう。

投稿

  • 回答の評価を上げる

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

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

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

  • 回答の評価を下げる

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

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

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

  • 2019/05/21 11:01 編集

    yasutomiさん
    ありがとうございます。
    addEvetListenerの方は書き方が間違っていたんですね。

    window.onload = function(){
    pic_status();
    }

    の方をもう一度試してみましたが、やはりページ読み込み時に実行してくれませんでした。
    ローカルで開発しているのが原因とかあるのでしょうか。

    footerタグはbodyの中に書くのですね。そこは完全に外に書くものであると勘違いしていました。
    訂正していただき感謝いたします。

    キャンセル

  • 2019/05/21 11:25 編集

    どちらにしろwindow.onloadという書き方は
    現在はほとんど使用されないのでthink49さんも説明されていますが
    window.addEventListener('load', pic_status);を使用したほうが良いでしょう。

    後方互換性の為にfalseが必要と書かれてありましたが、
    現在はIE11で省略しても動作するため、省略して問題ないです。

    キャンセル

  • 2019/05/21 11:37

    そうなんですね。window.onloadというイベントハンドラの書き方よりもaddEventListenerの方がよいのですね。
    追加情報ありがとうございます。

    キャンセル

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

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

関連した質問

同じタグがついた質問を見る