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

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

ただいまの
回答率

90.52%

  • JavaScript

    16368questions

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

  • イベントハンドラ

    37questions

    マウスのクリックなどの特定の事象(イベント)が発生した時に実行される処理のことをイベントハンドラと呼びます。

javascript イベントハンドラ 実行タイミングについて

解決済

回答 2

投稿

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

gorira6

score 4

JavaScriptの課題について 

JavaScriptの研修を受けている初心者です。   
課題で不明な点がいくつかあり、ご質問させて頂きます。

課題は以下の通りです。

「背景黒」ボタンを押すと、ページ全体の背景色が黒で文字色が白、「背景白」ボタンを押すと、ページ全体の背景色が白で文字色が黒となるようプログラムを追記してください。

<!DOCTYPE html>
<html lang="ja">
<head>
    <meta charset="UTF-8">
    <title>実行タイミング</title>
    <script>
        /* 課題処理を追加 */
        function changeBack(color){
            var color = color;
            if (color === 1){
                var background = document.getElementById('background');
                background.style.backgroundColor = '#000000';
                var font = document.getElementById('font');
                font.style.color = '#ffffff';
            }
            else if (color === 2){
                var background = document.getElementById('background');
                background.style.backgroundColor = '#FFFFFF';
                var font = document.getElementById('font');
                font.style.color = '#000000';
            }
        }
        window.onload = function(){
            var black = document.getElementById('black');
            var white = document.getElementById('white');
            black.addEventListener('click',changeBack(1),false);
            white.addEventListener('click',changeBack(2),false);
        }
    </script>
</head>
<body id = "background">
    <p id = 'font'>ボタンクリックにより背景色と文字色を変化</p>
    <button id = "black">背景黒</button>
    <button id = "white">背景白</button>
</body>
</html>

発生している問題、疑問点

1, function(){}の中にある'click'が正常に作用していない
(例えば、背景⇒白 文字⇒黒にするためのコードだけ消すと、ボタンを押していないにも関わらず、初めから背景⇒黒 文字⇒白になってしまう。)

間違っているとしたら、どの部分を、どのように修正すれば良いか。 
(なぜ、間違っているかもご教示頂けるとありがたいです。)

大変お手数をおかけしますが、できれば小中学生でも分かるような説明で 
ご解説頂けると幸いです。

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

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

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

    クリップを取り消します

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

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

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

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

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

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

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

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

    質問の評価を下げる

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

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

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

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

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

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

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

    詳細な説明はこちら

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

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

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

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

  • 退会済みユーザー

    2017/04/12 02:11

    複数のユーザーから「やってほしいことだけを記載した丸投げの質問」という意見がありました
    「質問を編集する」ボタンから編集を行い、調査したこと・試したことを記入していただくと、回答が得られやすくなります。

回答 2

checkベストアンサー

+3

// black.addEventListener( 'click', changeBack(1), false );
// ↓
   black.addEventListener( 'click', function(){ changeBack(1) }, false );

投稿

  • 回答の評価を上げる

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

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

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

  • 回答の評価を下げる

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

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

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

  • 2017/04/12 02:18

    ご回答頂き有り難うございます!
    無事、問題が解決できました。

    また、度重なる質問で申し訳ないのですが、
    よろしければ、無名関数をchangeback(1)に使う前に
    clickが効かなかった原因、無名関数を使うことで
    問題が解決した理由を教えて頂いてもよろしいでしょうか?
    お手数かとは思いますが、よろしくお願い致します。

    キャンセル

  • 2017/04/12 02:24

    すみません。
    自己解決しました。
    失礼しました!

    キャンセル

+3

 addEvetnListener の第二引数に関数の実行結果を渡した場合

black.addEventListener('click',changeBack(1),false);
white.addEventListener('click',changeBack(2),false);

changeBack(1), changeBack(2) の返り値は undefined なので、上記コードは下記と等価です。

black.addEventListener('click',undefined,false);
white.addEventListener('click',undefined,false);

addEventListener の第二引数は関数または handleEvent を持つオブジェクトを指定して下さい。

 イベントハンドラ関数に引数を渡す

イベントハンドラ関数の引数は event オブジェクトが予約されているので、通常の方法では外部から引数を指定することは出来ず、特殊な方法を利用する事になります。

  • handleEvent を持つオブジェクトを指定して this 束縛
  • Function.prototype.bind
black.addEventListener('click', {handleEvent: changeBack, color: 'black'}, false);
white.addEventListener('click', {handleEvent: changeBack, color: 'white'}, false);

black.addEventListener('click', changeBack.bind(black, 'black'), false);
white.addEventListener('click' changeBack.bind(white, 'white'), false);

 event.target

イベントハンドラ関数内では event.target でイベントが発生した要素ノードを得られます。
対象の要素ノードは id="black" または id="white" になっているはずなので、element.id を見る事で条件分岐させることが可能です。
先述の「イベントハンドラ関数に引数を渡す」はやや特殊な実装なので、私はこの方法を推します。

function handleEvent (event) {
  switch (event.target.id) {
    case 'black':
      break;
    case  'white':
      break;
    default:
      throw new Error;
  }
}

Re: gorira6 さん

投稿

  • 回答の評価を上げる

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

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

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

  • 回答の評価を下げる

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

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

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

  • 2017/04/12 02:36

    詳しいご解説有り難うございます!

    キャンセル

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

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

関連した質問

  • 解決済

    onclickが子要素まで適用されて困っています。

    <div onclick="test()" id="fade" style="position: relative; top:0px; left:0px; width:100%;

  • 解決済

    三秒点灯させて三秒点滅 JS

    信号っぽいものを作ろうとしています。青をボタンを押してから三秒点灯させて、そこから自動的に三秒点滅させて、黄色というようにしたいのですが、三秒点灯させてから点滅させるやり方がよくわ

  • 解決済

    Jquery部分作動せず 信号制作

    信号制作で クリック、青三秒点灯、青三秒点滅、からの黄色二秒点灯、最後に赤五秒点灯させたいのですがJQueryで書いた部分、青点滅終了と黄色点灯開始が反応しません。なぜでしょうか?

  • 解決済

    javascript ストップウォッチを作る

    前提・実現したいこと 現在ドットインストールを見ながらストップウォッチを作成中です。しかし、今までうまく動いていたのに、急にボタンを押してもストップウォッチのスタートボタンを押し

  • 解決済

    Google chromeでjsの関数書き換えができない

    前提・実現したいこと ホームページ上で試験対策用の記憶補助のために「赤シート」暗記方法の様なページを作っています。 テキストのカラーと背景色を同じにすることで目隠しして、クリッ

  • 解決済

    JS 基本質問

    JS初心者です。有識者からすると非常に簡単かと思いますが、 出来るだけきれいなコードを書きたいので教えていただければと思います。 <input type = 'button

  • 受付中

    cake phpでソートさせるボタンを押した際にそのボタンのcssのスタイルを変更したい

    cake phpでソートさせるボタンを押した際にそのボタンのcssのスタイルを変更したい 初心者なものでうまく言葉にできなくて申し訳無いのですが、 cake phpで構築されたウェ

  • 解決済

    自分のWebページに指定したURLのページを表示する

    前提・実現したいこと Webページを自分で作って、2行目以降に指定したURLのページを表示するWebページを作ることはできないでしょうか。 発生している問題・エラーメッセージ

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

  • JavaScript

    16368questions

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

  • イベントハンドラ

    37questions

    マウスのクリックなどの特定の事象(イベント)が発生した時に実行される処理のことをイベントハンドラと呼びます。