ランダムバナーのリンク先をポップアップウィンドウで開きたい。

質問

ランダムバナーのリンク先をポップアップウィンドウで開きたい。

当方javascript初心者なのですが、色々なサイトを参考にして上記の方法を模索しています。
以下に記載する内容で画像をランダムに表示するところまで出来たのですが、
画像をクリックすると通常通り、リンク先に飛ぶだけでポップアップウィンドウが開きません。

この記述の間違い点、問題点をご指摘、助言をいただきたく思います。
よろしくお願いたします。

以下js記述↓

<SCRIPT language="JavaScript">
<!--
jmp = new Array();
img = new Array();
// ジャンプ先のアドレス(数字は画像と対応)
jmp[0] = "http://www.yahoo.co.jp/";
jmp[1] = "http://google.com/";
jmp[2] = "http://www.microsoft.com/";

img[0] = "banner1.jpg";
img[1] = "banner2.jpg";
img[2] = "banner3.jpg";

n = Math.floor(Math.random()*jmp.length);
document.write("<a href='"+jmp[n]+"' onClick='javascript:window.open('"+jmp[n]+"','','width=600'); return false;'>");
document.write("<img src='"+img[n]+"' border='0' width='100' height='100' />");
document.write("</a>");
//-->
</SCRIPT>

回答No.1

fujillin · ベストアンサー

ダブルクォートの中のシングルクォートまでは良いけれど、さらにその中でシングルクォートを使っているのが問題なのでは？
確認していませんが、エスケープすればOKではないでしょうか？

お礼

さっそくの回答ありがとうございました。

教えていただきました通り、シングルクオートをエスケープしたところ↓

document.write("<a href='"+jmp[n]+"' onClick=\"javascript:window.open('"+jmp[n]+"','','width=600'); return false;\">");

ポップアップウインドウが開きました！

自分の勉強不足が恥ずかしいと思いながら、Javascriptをもっと勉強したくなりました。
本当に、本当にありがとうございました。