プロが教えるわが家の防犯対策術!

下記ソースのjavascriptにて外部javascript内容を読み込もうとしています

==呼び出すファイル(test.htm)==========================
<head>
<script type="text/javascript" src="common.js">
<!--
function init() {
setBrowser();
alert(getBrowser());
}
-->
</script>
</head>
<body onload="init();">
てすと
</body>
</html>
==================================================
==呼び出されるファイル(common.js)===================
<!--
var strBrowser;
function setBrowser() {
if(navigator.userAgent.indexOf("MSIE") != -1){
// IE
strBrowser = "IE";
} else if(navigator.userAgent.indexOf("Firefox") != -1){
// FireFox
strBrowser = "Firefox";
} else if(navigator.userAgent.indexOf("Safari") != -1){
// Chrome & Safari
strBrowser = "Safari";
} else{
strBrowser = "";
}
}
function getBrowser() {
return strBrowser;
}
//-->
=================================================

これで呼び出す事ができません。
何処が悪いのか、ご指摘頂ければ幸いです。

A 回答 (1件)

test.htmでは<script>タグが2回必要になります。

以下のように訂正してください。

<html>
<head>
<script type="text/javascript" src="common.js"></script>
<script type="text/javascript">
<!--
function init() {
setBrowser();
alert(getBrowser());
}
-->
</script>
</head>
<body onload="init();">
てすと
</body>
</html>

また、common.jsファイルはHTMLではないので、冒頭の<!--と、最後の//-->は不要です。以下のように削除してください。

var strBrowser;
function setBrowser() {
if(navigator.userAgent.indexOf("MSIE") != -1){
// IE
strBrowser = "IE";
} else if(navigator.userAgent.indexOf("Firefox") != -1){
// FireFox
strBrowser = "Firefox";
} else if(navigator.userAgent.indexOf("Safari") != -1){
// Chrome & Safari
strBrowser = "Safari";
} else{
strBrowser = "";
}
}
function getBrowser() {
return strBrowser;
}
    • good
    • 0
この回答へのお礼

できました!理想とする動きが!
本当にありがとうございます!

お礼日時:2010/11/20 13:42

お探しのQ&Aが見つからない時は、教えて!gooで質問しましょう!