火狐的兼容问题 js

Question

下面的这段代码如和才能兼容火狐？function Checker() {
    if (!form1.item.length){
        if (form1.items.checked)　　　form1.items.value = form1.item.value;
    }else{
        for (i = 0; i<form1.item.length; i++){
            if (form1.item(i).checked){　　　　
                form1.items.value = form1.item(i).value;
                for (j = i + 1; j<form1.item.length; j++){
                    if (form1.item(j).checked){　　　　　　
                        form1.items.value += ","; //用空格做分割符
                        form1.items.value += form1.item(j).value;
                    }
                }
                break;
            }
        }
    }
}

Answer 1

<script type="text/javascript">
<!--
function Checker() {
var form1 = document.form1;
var item = form1.getElementsByTagName("input");
var v=""; for (i = 0; i<item.length; i++){
if(item[i].name=="item"){
if (item[i].checked){　　　　
v += item[i].value + ",";
}
}
if(item[i].name=="items") var items = item[i];
}
items.value = v.substr(0,v.length-1);
} //-->
</script>
<form method="post" name="form1" action="">
<input type="checkbox" name="item" value="1">1
<input type="checkbox" name="item" value="2">2
<input type="checkbox" name="item" value="3">3
<input type="text" name="items">
<input type="button" value="Click" onclick="Checker()"/>
</form>

Answer 2

form1：建议使用document.getElementById("form1")的形式form1.item(j)：数组用[]而不是()asp/.net默认提交的name值一样的checkbox是以,分割的数据

Answer 3

我再补充一点，循环变量定义都不加var!!!!!。想把i啊，j啊当全局变量使用啊