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

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

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

HTML5 (Hyper Text Markup Language、バージョン 5)は、マークアップ言語であるHTMLの第5版です。

JavaScript

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

Q&A

解決済

3回答

4704閲覧

【JavaScript】即時関数のおまじないを追加するとエラーが

himejiy3

総合スコア77

HTML5

HTML5 (Hyper Text Markup Language、バージョン 5)は、マークアップ言語であるHTMLの第5版です。

JavaScript

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

0グッド

1クリップ

投稿2017/07/02 16:33

###前提・実現したいこと
JavaScript言語を参考書とネットで学習しながら進めている中で、

・コード全体を即時関数でくくると、変数の意図しない競合を防ぐことができる
・strictモード推奨

とのことで「ほうほう、そんな技が」と
普通に動いていた練習中のプログラムを『(function(){~}());』で囲ってみたところ、
以下のinputでどうしても『'disp' is undefined』のエラーが出て、
その理由を突き止められずにいます。

###発生している問題・エラーメッセージ

即時関数で囲わなければ、取りあえずは思った通りに動いてくれていました。(多くの妥協点はありますが。) Strictモードを有効にしても問題無かったです。 『(function~』で囲ったときだけ、dispが見えなくなるっぽいのですが その原因が分かりません。 回避方法はいくらでもあるでしょうけど、それよりも、このエラーの理由を知りたいです。 そもそも論として、何か大きな勘違いをしているのだろうとは思います・・

###該当のソースコード

HTML

1<!DOCTYPE html> 2<html lang="ja"> 3<head> 4 <meta charset="UTF-8"> 5 <title>お試し</title> 6</head> 7<body> 8<label for="guess">数値をどうぞ</label> 9<input type="number" id="guess" size="2" value="0" maxlength="2" /> 10<input type="button" id="kettei" value="決定" onClick="disp()" /> 11 12<div id="hyouji">表示</div> 13 14<script type="text/javascript" src="js/main.js"></script> 15</body> 16</html> 17

JavaScript

1(function() { 2 'use strict'; 3 4function disp() { 5 let hyouji = document.getElementById("hyouji"); 6 let guess = document.getElementById("guess").value; 7 hyouji.textContent = "はい、" + guess + "ですね!"; 8} 9 10}()); 11

###試したこと
元の完成形は単純な数当てゲームで、エラーが出ない大丈夫だったところを削っていくと、上記の部分が残りました。
jsファイルの1行目『(function(){』と、最終行『}());』をコメントアウトすると問題無く動きます。最終行は『})();』の形も試しました。(特に変わりませんが)

ネットで無名関数や即時関数、Strictモードについても検索して読みましたが、欲しい情報に到達することが出来ませんでした。
input文とonClickも調べたつもりです。
しかしまだまだあまりにもド素人すぎて、自分の知識不足・理解不足を痛感するばかりでした。

###補足情報(言語/FW/ツール等のバージョンなど)
即時関数で囲わなければ動くのだから、いったん保留して基礎学習を進めるしかないか・・と思っていたところ、こちらのサイトを見つけました。よろしくお願いします。

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

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

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

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

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

guest

回答3

0

HTML側の onClick をはずして、即時関数内で下記のようにするか addEventListener を使用すればよいと思います。

JavaScript

1document.getElementById("kettei").onclick = disp; /* 20170708: 修正onClick ⇒ onclick */

【EventTarget.addEventListener - Web API インターフェイス | MDN】
https://developer.mozilla.org/ja/docs/Web/API/EventTarget/addEventListener

【addEventListener()-JavaScript入門】
http://www.pori2.net/js/event/2.html

投稿2017/07/02 16:50

編集2017/07/08 02:05
kei344

総合スコア69364

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

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

himejiy3

2017/07/02 18:16

HTML側のonClickはちょっと気になっていました。 HTMLはページ構成でCSSはレイアウトの原則に反するような感じがして、何となくモヤモヤするものがあって。 onClickは外す方向で考えるのと、addEventListenerを学習しておきます。
think49

2017/07/07 23:43

onClickではなく、onclickですね。 JavaScriptは大文字/小文字を区別するので、このコードは期待通りに動作しないと思います…。 参考までに、<input onClick="..."> が通用するのは、HTMLの属性名が大文字/小文字を区別しない仕様だからであって、XHTMLでは大文字/小文字が区別されて通用しなくなります。 https://html.spec.whatwg.org/multipage/webappapis.html#event-handlers-on-elements,-document-objects,-and-window-objects
himejiy3

2017/07/08 06:35

そんな細かい注意点が。と思ったら、手持ちの参考書に載っていました。 まだ隅々まで熟読していないので、こういう部分も気を付けます。
guest

0

ベストアンサー

即時関数を入れるという事はスコープを作るという事です。ですので今まで見えていた disp 関数が即時関数を入れる事で見えなくなってしまいます。
解決方法としては2つあります。

一つ目は即時関数を作らないこと

二つ目はイベントの登録も即時関数内でやってしまうこと

後者をやるには、HTML が全てロードされた後でないとできません。一番簡単な方法は window.onload を使う事です。

javascript

1window.onload = function() { 2 function disp() { 3 ...... 4 } 5 document.getElementById('kettei').onclick = disp; 6}

投稿2017/07/02 16:51

mattn

総合スコア5030

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

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

himejiy3

2017/07/02 18:18

おおぉ・・動きました・・ありがとうございます。 取りあえず「『見えなくなる』状態は正しい」と分かって、スッキリしました。単純なスペルミスだったらどうしようと思っていたので。 onloadイベント、学習しておきます。
guest

0

解決済としましたが、自分が当初思い描いていた(こうなってほしかった)形にかなり近付けた感じがするので書いておくことにします。

html

1<!DOCTYPE html> 2<html lang="ja"> 3<head> 4 <meta charset="UTF-8"> 5 <title>お試し</title> 6</head> 7<body> 8 <label for="suuchi">数値入力をどうぞ (1-10)?</label> 9 <input type="number" id="suuchi" size="2" value="0" maxlength="2" /> 10 11 <input type="button" id="kettei" value="決定" /> 12 13 <div id="hyouji">表示</div> 14 15 <script type="text/javascript" src="js/main.js"></script> 16</body> 17</html> 18

JavaScript

1(function() { 2 'use strict'; 3 4let suuchi = document.getElementById("suuchi"); 5let kettei = document.getElementById("kettei"); 6let hyouji = document.getElementById("hyouji"); 7 8window.onload = function(){ 9 suuchi.value = 0; 10} 11 12document.addEventListener('DOMContentLoaded',function(){ 13 kettei.addEventListener('click',function(){ 14 disp(); 15 },false); 16},false); 17 18function disp() 19 { 20 hyouji.textContent = "はい、" + suuchi.value + "ですね!"; 21 } 22 23})(); 24

現時点の知識ではコレが精一杯でした。
早く初心者から抜け出せるよう、今後も精進します。
回答がとても参考になりました。ありがとうございました。

投稿2017/07/07 15:57

himejiy3

総合スコア77

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

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

mattn

2017/07/07 16:03

window.onload もういらないすね。
himejiy3

2017/07/08 08:02

やはりそうだったんですね。 何故こんなコードを追加していたかというと、プログラムを最初から走らせるべくブラウザ更新した時、 Edgeでは数値の初期値に0が入ってくれず、苦肉の策でした。 でも今考えてみると、追加するのであれば DOMContentLoadedのfunctionの中に suuchi.value = 0と1行加えれば済むことだったかも。 それに学習を続けているうち、そもそもHTMLのinputの中身が怪しいのが気になっています。 手を加え始めると切りが無さそう。 自分のブラウザ使用頻度はChrome7割・Edge3割ぐらいで Edgeの挙動は無視しても良いのですが、何となく試行錯誤してしまいます。
mattn

2017/07/08 08:04

はい。window.onload は他のスクリプトと混ぜられないので DOMContentLoaded を使った方が良いですね。
guest

あなたの回答

tips

太字

斜体

打ち消し線

見出し

引用テキストの挿入

コードの挿入

リンクの挿入

リストの挿入

番号リストの挿入

表の挿入

水平線の挿入

プレビュー

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

ただいまの回答率
85.50%

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

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

質問する

関連した質問