firefox3.6でJavaScriptが動かない

質問

firefox2.xのころからだと思うのですが、
JavaScriptのOnChangeイベントが反応しません。

環境は、少し古いのですが、
ホームページビルダ１０です。

ホームページ上へコンボボックスを配置して０～４の値を選択するようにしています。

ソースは以下のようになっています。
＝＝＝＝＝＝＝
<script language="JavaScript">
<!--
function Pent(formobj,pent0) {
var v;
v = eval(pent0.value);
Pent.value = v;
}
//-->
</script>

<td valign="middle" height="28" width="13"><select size="1" style="font-size : 24px;" name="pent0" onchange="Pent(form,pent0)">
<option value="0" selected>0</option>
<option value="1">1</option>
<option value="2">2</option>
<option value="3">3</option>
<option value="4">4</option>
</select></td>
＝＝＝＝＝＝＝
このソースでIE6では問題なく選択できるのですが、
firefoxだとonchangeに反応していないようです。

対処方法をお教えください。

回答No.2

yambejp · ベストアンサー

結構、問題だらけのソースですね。
こんな風に書き変えないとまずいかもしれません。

<script>
function Pent(obj) {
var v=parseInt(obj.value);
obj.form.elements["pent1"].value = v;
}
</script>
<form>
<p>
<select size="1" style="font-size : 24px;" name="pent0" onchange="Pent(this)">
<option value="0" selected>0</option>
<option value="1">1</option>
<option value="2">2</option>
<option value="3">3</option>
<option value="4">4</option>
</select>
<input type="text" name="pent1" />
</p>
</form>

・関数名と要素名に同じものをつけない
・引数にオブジェクトをわたすときに、名前（name）でわたさない
（同じフォーム内の処理ならthisなどをつかう）
・evalなど不正処理がおこりやすいものはつかわない

お礼

ずいぶん昔に、ホームページビルダーが生成したコードと
どこかのホームページだったかを参考に見よう見まねで作りました。

少し、時間（気持ち？）に余裕ができたので、今まで気になっていた
「ＩＥでは動くけれど、ＦＦでは動かない」を考えてみようと思い、
質問させていただきました。
JavaScriptのバージョンの関係かと思ったりしましたが、
どうやらそうではないようですね。

勉強してみます。
ありがとうございました。

回答No.1

fujillin

FF3.5.9で確認しましたが…同様ですね。

反応しないのではなくて、エラーで止まっているだけでしょう。（要素の指定の方法がよくない）
引数のform、pent0ともに渡されていないので（というか意味が通じないので）、呼出すところで止まっているようです。
ＩＥでも、formはnullになっていますが、pent0はなんとか通じているみたい。

function Pent()が何をしたいものなのかよくわかりませんが、とりあえず動くようにしたいのなら、呼び出し側は
　onchange="Pent(this)"
とかにしておいて、function側は
　function Pent(pent0) {
にするとか…
（formはfunction内で使ってないみたいなので、省略）

お礼

ずいぶん昔に、ホームページビルダーが生成したコードと
どこかのホームページだったかを参考に見よう見まねで作りました。

少し、時間（気持ち？）に余裕ができたので、今まで気になっていた
「ＩＥでは動くけれど、ＦＦでは動かない」を考えてみようと思い、
質問させていただきました。
JavaScriptのバージョンの関係かと思ったりしましたが、
どうやらそうではないようですね。

勉強してみます。
ありがとうございました。