跳转到考试计划批改的页面

如果有什么不对的请大家指出来，我尽量改

function toupdate()
{?
?? if(document.form2.checkbox == null) return;
??? //获得checkbox输出的个数
? ?var numRow = document.form2.checkbox.length;
? ?//定义index只是判断是否对一条信息进行修改,无任何实际意义
? ?var index = 0;
? ?//考试计划的ID
? ?var planid;
? ?//考试计划状态
? ?var planState;
?if(numRow != null )
?{
??for (var i = 0; i < numRow; i++)
??{
???if (eval("document.form2.checkbox[" + i + "].checked") == true)
???{
????? ?index ++;
????? ?planid = document.form2.checkbox[i].value;
????? ?planState = document.form2.planState1[i].value;
?? ?}
??}
??//如果它是一条数据就执行,并把考试计划的ID和考试计划状态传入后台
??? if(index == 1)
??? {
????????? document.form2.action="<%=path%>/DbExamPlanAction.do?domethod=findByPlanId&planId="+planid+"&planState1="+planState;
?????? document.form2.submit();
??????
??????? } else {
???????? if(index == 0)
???????? {
??????? alert("对不起,您需要选择一条数据进行修改!");
?????????? return false;
???????? }else{
??????? alert("对不起,您只可以选择一条数据进行修改!");
??????? return false;
???? }
???? }
?}
}