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

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

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;
     }
     }
 }
}