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

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

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

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

Q&A

解決済

1回答

1975閲覧

動的に追加したオブジェクトに複数のonload属性を付与する

mimikku

総合スコア13

JavaScript

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

0グッド

0クリップ

投稿2017/01/09 03:00

###前提・実現したいこと
動的に追加したオブジェクトに複数のonload属性を付与する

###発生している問題・エラーメッセージ
以下のソースで実行したところ、ページがフリーズしてしまう

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

javascript

1var element = document.createElement('img'); 2 element.src = "img/stand.png"; 3 element.id = num; 4 element.style.position="absolute"; 5 element.width=60; 6 element.height=60; 7 element.style.left =120; 8 element.style.top =30; 9 num++; 10 element.onload = new Function("move(this);");//ここと直後の関数を付与したい 11 element.onload = new Function("sound(this);"); 12 element.onclick = new Function("remove(this)"); 13 14 var objBody = document.getElementsByTagName("body").item(0); 15 objBody.appendChild(element);

###試したこと
element.onload = new Function("move(this);"),new Function("sound(this);");
としたものの、フリーズはしなかったが動作が正常でなかった

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

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

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

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

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

guest

回答1

0

ベストアンサー

複数のリスナーを割り当てる場合はaddEventListenerを使用してください。

js

1element.addEventListener('load', move); 2element.addEventListener('load', sound);

あと、

js

1document.getElementsByTagName("body").item(0);

でbody要素を取得し、bodyに要素を追加していますが、document.bodyで取得することができます。

js

1document.body.appendChild(element);

投稿2017/01/09 03:08

turbgraphics200

総合スコア4267

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

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

think49

2017/01/09 03:47 編集

大筋は同意ですが、少しだけ捕捉します。 addEventListener の第三引数省略は DOM4 から規定された比較的新しい仕様なので、後方互換性の為に false を指定する事をお勧めします。 また、質問文のコードでは move, sound が第一引数で this を渡すように作られているようなので、event オブジェクトが渡されて動くように move, sound のコードを修正する必要があると思われます。 https://developer.mozilla.org/ja/docs/Web/API/EventTarget/addEventListener
mimikku

2017/01/09 05:55

ご回答ありがとうございます。 引数なんですが、 element.addEventListener('load', "move(this)", false); element.addEventListener('load', "sound(this)", false); としたところ正しい結果が得られませんでした。 正しい書きかたを教えていただければ幸いです
guest

あなたの回答

tips

太字

斜体

打ち消し線

見出し

引用テキストの挿入

コードの挿入

リンクの挿入

リストの挿入

番号リストの挿入

表の挿入

水平線の挿入

プレビュー

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

ただいまの回答率
85.48%

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

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

質問する

関連した質問