IE6 + javascript

質問

すみません、先ほども質問したのですが、締め切ってしまったのでもう一度質問させてください。
下記コードが、Firefoxでは動作するのですが、IEではjavasciptの部分が動作していません。
もちろん、javascriptの設定は有効にしてあります。

var browser = navigator.appName;

if(browser.charAt(0) == "N") { window.addEventListener("load",change_page(page_id),false);
} else if (browser.charAt(0) == "Ｍ") { window.attachEvent("onLoad",change_page(page_id));
}
．．．．

function httpRequest() {
if (window.ActiveXObject) {
try {
httpObj = new ActiveXObject("Microsoft.XMLHTTP");
} catch(e) {
try {
httpObj = new ActiveXObject("Msxml2.XMLHTTP");
} catch(e) {
httpObj = false;
}
}
} else if (window.XMLHttpRequest){
try{
httpObj = new XMLHttpRequest();
} catch(e) {
httpObj = false;
}
}
return httpObj;
}

change_page()はfunctionです。
なぜIE6ではjavascriptが動かないのでしょう？
ソースのどこが悪いのかわからず、どなたかアドバイスあればお願いします！

回答No.6

BLUEPIXY · ベストアンサー

＃５のソースを眺めて見ましたけど、特に文字列の閉じ抜けがあるようには見えませんね。
動的に作成される部分で、「"」か「'」のデータが悪さをするのでしょうか・

＃５＞実は最初にロードしたときは関数は実行されているのです。
＃５＞でもリロードすると関数が呼ばれず、DBからデータを抽出して表示させているのですが、その表示がまったくなくなってしまします。
私もXMLHttpRequest を使った処理で同じような症状がでたことがあります。
それは（私の場合でいうと）、XMLHttpRequest のリクエストによる読込もキャッシュされて、全く同じリクエストを送信しても２回目は実際に読込が行われていませんでした。（ブラウザのリロードはソースのリロードであってXMLHttpRequest はリロードされない）
参考までに

補足

ご確認ありがとうございます。
XMLHttpRequestのところは大変参考になりました。
終了していない文字列の・・というエラーはいつも同じ行で起きています。
（jsファイル以外も確認してみたのですが、該当するようなところはありませんでした。）
ですが、ロードした時点ではjavascriptがきちんと動くようになりました。ので、’と”のデータが悪さをするのではないようです。
また違う問題なのですが、どうもディクリメント演算子が働いていません。インクリメントはうまくいっているのですが。
まったく同じ関数で、インクリメントとディクリメントとのみが違う関数があるのですが、ディクリメントのほうは動かないのです。ディクリメントをインクリメントに変えればOKなのですが・・・
もちろんFirefoxでは動いています。
もし何か経験談などアドバイスがあればお願いします。
なければない、とご返信ください。
お手数かけますが、よろしくお願いします。

回答No.7

BLUEPIXY

#6>まったく同じ関数で、インクリメントとディクリメントとのみが違う関数があるのですが、ディクリメントのほうは動かないのです
そう言われても実際の関数をみることなしに、この情報だけでコメントすることは、ムリです。（＾＾；）

お礼

すいません・・・
結局、下記の部分を取り除くことでIEもFirefoxと同じ動作になりました。

obj.onreadystatuschange = function() {
if ((httpObj.readyState == 4) && (httpObj.status == 200)) {

不具合のある箇所を1文ずつ検証していくしかないのですね。
長々とお付き合いくださり、ありがとうございました。

回答No.5

BLUEPIXY

#3の回答のうち
"onLoad" は、"onload" にして下さい。

＃３＞終了していない文字列型の定数です
は、attachEvent というより、どこかの文字列定数がちゃんと閉じていないと思われます。
なんだったら関連する部分のソースを補足して下さい。

＃４＞change_page()関数でコールバックしているchange_girl関数だけ呼び出されているようなのです。ですが、終了していない文字列型の定数です。ライン11、というメッセージは変わりません。
多分、エラーが発生しているために途中まで実行されているのでしょう。エラーがでないように修正する必要があります。

補足

お忙しい中ありがとうございます。
実は最初にロードしたときは関数は実行されているのです。
でもリロードすると関数が呼ばれず、DBからデータを抽出して表示させているのですが、その表示がまったくなくなってしまします。
ちょっと長いのですが
var id = 0;
var page_id = 0;
var data_count;
var target_url = "db3.php";

window.onload = function() {change_page(page_id);};
function change_page(page_id) {
httpObj = httpRequest();
httpObj.open("GET",target_url, true);
httpObj.send("");
httpObj.onreadystatechange = function() {
if((httpObj.readyState==4)&&(httpObj.status==200)){
var girl_data=httpObj.responseText.split("<p>");
data_count = girl_data.length;
var list = 4;
if (page_id > data_count) {
page_id = data_count;
}
start_id = page_id;
end_id = start_id + list;
if (end_id > data_count) {
end_id = data_count;
}

last_page=Math.ceil(data_count/list)-1;
page = page_id / list;
disp_page_btn(last_page,page);

var num = 0;
for(page_idstart_id;page_id<end_id;page_id++){
data=girl_data[page_id].split(",");
div =document.createElement("div");
div.id = 'item' + num; htm='<divclass="head_bbs"><divclass="head_bbs_bg"><div class="head_bbs_bg_base" align="left">';
htm+=・・・
var content=document.createTextNode("");
div.appendChild(content); document.getElementById("list").appendChild(div);
document.getElementById("item" +num).innerHTML=htm;
num++;
}
change_girl();
}
}
}
function change_girl() {
change_btn();
httpObj = httpRequest();
httpObj.open("GET",target_url+'?id='+ id,true);
httpObj.send("");
httpObj.onreadystatechange = function() {
if ((httpObj.readyState == 4) && (httpObj.status == 200)) {
var girl_data = httpObj.responseText.split(",");
document.getElementById("girl_img").src = girl_data[4];
document.getElementById("name1").innerHTML = girl_data[1];
document.getElementById("name2").innerHTML = girl_data[1];
document.getElementById("last_access").innerHTML = girl_data[5];
}
}
}
function disp_page_btn(last_page,page) {
if ((last_page > page) && (page >= 0)){
go_next = '<a href="javascript:next_page()">次へ>></a>';
document.getElementById("page_next1").innerHTML = go_next;
document.getElementById("page_next2").innerHTML = go_next;
} else {
go_next = '次へ>>';
document.getElementById("page_next1").innerHTML = go_next;
document.getElementById("page_next2").innerHTML = go_next;
}
if ((0 < page) && (page <= last_page)) {
go_back = '<a href="javascript:pre_page()"><<前へ</a>   ';
document.getElementById("page_back1").innerHTML = go_back;
document.getElementById("page_back2").innerHTML = go_back;
} else {
go_back = '<<前へ   ';
document.getElementById("page_back1").innerHTML = go_back;
document.getElementById("page_back2").innerHTML = go_back;
}
}
関係あると思われる関数を記載してみました。

回答No.4

BLUEPIXY

ちなみに、イベントを１つしか割り当てないのだったら
window.onload=function(){
change_page(page_id);
};
でＩＥもFirefox も動作すると思います。

補足

window.onload=function(){change_page(page_id);};
このようにすると、一部うまくいくようになりました。
change_page()関数でコールバックしているchange_girl関数だけ呼び出されているようなのです。
ですが、終了していない文字列型の定数です。ライン11、というメッセージは変わりません。

回答No.3

BLUEPIXY

change_page()が関数を返す関数でないなら

window.attachEvent("onLoad",change_page(page_id))
は、
window.attachEvent("onLoad",change_page)
のように書きます。
その時に引数が必要なようなら
window.attachEvent("onLoad",function(){change_page(page_id);})
とでも書けばいいかと思います。

補足

ご回答ありがとうございます。
callback関数に引数は必要なので、
window.attachEvent("onLoad",function(){change_page(page_id);})
は試してみました。
しかし、↑で、終了していない文字列型の定数です。というスクリプトエラーは未だでます。
ソースに改行が入っているとエラーになることもある、という記事を読んだので改行なしのコードに書き換えてみましたが、状況は変わりません。

回答No.2

agharta

すいません、やはり外していましたね。

さて、デバッグの方法としては、
window.attachEvent("onLoad",change_page(page_id));
の返り値がどうなっているか。(trueまたはfalse)が返るはずです。

change_page関数は呼ばれているのかどうかのチェックも必要かと思います。

補足

ご回答ありがとうございます。
onLoad→onloadにするとエラーを表示したあと、IEの左下にエラー!という黄色い表示がなくなりました。
またいろいろに試しているうちに、引数のpage_idを間違ってpageとしたところ、エラーにpageは定義されていません、と出てきたのでchange_pageは呼ばれているよなのですが・・・動作はまったくしていません・・・
よくわからないまま、下記のように変更してみました。
} else if (browser.charAt(0) == "M") {
window.attachEvent('onload',function(e) {
change_girl();
if(e != window.event) {
alert(e);
}
for(var p in window.event) {
if(e[p] != window.event[p]) {
alert("property "+p+" is different");
}
}
});
} else {
alert("without support on your browser:" + browser);
}
すると必ずアラートで、[object]、property boundElements is different　と出てきます。
boundElemtntsもよくわからないのですが・・・

回答No.1

agharta

Ｍ ⇒ M
ってことはないですね(全角と半角の違い)

失礼しました。

補足

もちろんないです・・・というか私も試しました。
if(browser.charAt(0) == "N") {
window.addEventListener("load",change_page(page_id),false);
} else if (browser.charAt(0) == "M") {
window.attachEvent("onLoad",change_page(page_id));
} else {
alert("without support on your browser:" + browser);
}

コードを上記のように変えて、N、M以外だとアラートが出るようにしてみました。
半角Mではアラートが出ません。一致しているのです。
M→Sに変更するとちゃんとアラートが出て、IEのエラーはなくなります。
attachEventがおかしいはずなのですが、何がおかしいのか・・・
エラーメッセージでは、attachEventの行で文字列型が終了してません。
次の行で、種類が一致していません。というわけわからないエラーメッセージに苦しんでおります。