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

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

ただいまの
回答率

89.96%

以前のIEのバージョンでは出ていなかったpopup.jsのエラーがIE11にバージョンアップして出るようになりました。

解決済

回答 1

投稿 編集

  • 評価
  • クリップ 0
  • VIEW 1,153

higashi

score 5

前提・実現したいこと

Javascropt初心者です。
popup.jsを使用しています。以前のIEのバージョンでは出ていなかったエラーがIE11にバージョンアップしたところ、以下のようなエラーが出るようになりました。動作に問題はないのですが、閲覧者が不安になるかと思い、表示を出なくしたいのですが、ネットで調べてみても解決策が出てきませんでした。
解決策に心当たりがある方がいらっしゃいましたら、よろしくお願いいたします。

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

未定義または NULL 参照のプロパティ 'className' は取得できません

該当のソースコード

■HTML
jsの読み込み部
<script type="text/javascript" src="../js/popup.js"></script>


class popupの使用箇所
<li><a href="xxx" target="_blank"  class="popup" rel="500,800"><img src="xxxxx" /></a></li>

■popup.js
window.onload = function (){
var node_a = document.getElementsByTagName('a');
for (var i in node_a) {
if(node_a[i].className == 'popup'){
node_a[i].onclick = function() {
return winOpen(this.href, this.rel)
};
}
}
} ;

function winOpen(url, rel) {
var split = rel.split(',') ;
window.open(
url,'popup',
'width='+ split[0] +',height='+ split[1] +',toolbar=no,location=no,status=no,menubar=no,scrollbars=yes,resizable=yes');

return false;
};

試したこと

popup.jsの挿入位置を</body>の直前に移動したりしてみましたが結果は同じでした。

補足情報(言語/FW/ツール等のバージョンなど)

インターネットエクスプローラー11.0.31
windows8.1
2010年にサイトをリリースした際にはエラーは出ていなかったので、その当時だとIEのバージョンはおそらく8か9だと思います。エラーが出始めたのは、おそらくIE10になったタイミングだと思われます。

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

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

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

    クリップを取り消します

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

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

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

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

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

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

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

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

    質問の評価を下げる

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

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

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

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

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

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

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

    詳細な説明はこちら

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

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

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

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

  • yuji38kwmt

    2016/11/19 21:49

    IEのどのバージョンで、動いていましたか?

    キャンセル

  • higashi

    2016/11/20 10:25

    2010年にサイトをリリースした際にはエラーは出ていなかったので、その当時だとIEのバージョンはおそらく8か9だと思います。エラーが出始めたのは、おそらくIE10になったタイミングだと思われます。

    キャンセル

回答 1

checkベストアンサー

+1

 解決方法

ループの方法が悪いです。
下記コードのように、C言語風のループを使えば、エラーは出ないと思います。

for (var i=0; i<node_a.length; i++) { 
  if(node_a[i].className == 'popup'){ 
    //~~
  }
}

 エラーの原因

質問者様は、配列要素に対してfor-inループを使っています。
この場合注意が必要で、最初から設定されているプロパティ名(lengthやitemなど)も返してしまいます。
https://developer.mozilla.org/ja/docs/Web/JavaScript/Guide/Loops_and_iteration

  Array 要素に対して反復処理を行う方法として for...in 文を使用することができますが、これは数値のインデックスに加えてユーザ定義プロパティの名称も返します。そういうわけで、配列に対しての反復処理には、数値のインデックスを使い従来の for ループを使用するほうが良いです。というのも、カスタムプロパティやカスタムメソッドを追加するといった Array オブジェクトの変更を行った場合、for...in 文は配列要素に加えてユーザ定義プロパティに対しても反復処理するからです。

そのため、「node_a["length"]はNULLで、そこにclassNameプロパティなんかないよ」と、言われているのでしょう。

IE11にバージョンアップしたらエラーが出たというのは、IE11とその前でfor-inループの仕様が変わったためだと思います。

投稿

  • 回答の評価を上げる

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

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

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

  • 回答の評価を下げる

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

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

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

  • 2016/11/20 10:21

    早々のご連絡ありがとうございました。試したところ、エラーが解消されました。
    大変たすかりました。

    キャンセル

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

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

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

  • トップ
  • JSPに関する質問
  • 以前のIEのバージョンでは出ていなかったpopup.jsのエラーがIE11にバージョンアップして出るようになりました。