这段代码怎样改能更简洁,更易维护
function Check()
{    
if(document.form1.hot.value ==0)
{
  alert("请输入酒店ID");
  document.form1.hot.focus();
  return false;
}
else if(document.form1.ctn.value == 0)
{
  alert("请输入内容介绍");
  document.form1.ctn.focus();
  return false;
}
else if(document.form1.start.value == 0)
{
alert("请输入开始时间");
document.form1.start.focus();
return false;
}
else if(document.form1.end.value == 0)
{
alert("请输入结束时间");
document.form1.end.focus();
return false;
}
else if(document.form1.s_ty.value == 0)
{
alert("请输入备注");
document.form1.s_ty.focus();
return false;
}
  return true;
}

解决方案 »

  1.   

    写2个方法  一个方法获得document.form1下面的所有组件  一个方法function Check(type)  在第一个方法中循环调用function Check(type)
      

  2.   

    如果只是要内容不能为空可以这样子var forminput=document.form1.elements;
    for(var i=0;i<forminput.length;i++)
    {
     if(forminput[i].value==0){
    alert("所有内容不能为空");
    return;
    break;
     }
    }
      

  3.   

    并不是所有的,其中一个INPUT可以为空
      

  4.   

    if (forminput[i].name !='**' && forminput[i].value==0)
    {}
    **就是你可为空的元素名
      

  5.   

    这是我改后的JS,
    我想获取所有文本框的名字或者ID(这里用ID名)
    之后在判断文本框不能为空
    function inCheck(){
    document.getElementById('text_name').focus();
    }
    function Check(){
    if(document.form1.value == 0){
    alert("请输入指定的信息");inCheck;
    return false;
    }
    return true;
    }
      

  6.   

    http://blog.csdn.net/xuzuning/archive/2010/09/03/5861501.aspx建议你使用这个
      

  7.   

    如果只是考虑判断非空的字段,那么楼上的兄弟确实正解。
    如果需要考虑到良好的用户体验,比如分别针对每个字段给出不同的提示,那么我建议楼主开辟一个数组
    数组里分别存放表单元素名(如:hot),以及对应的中文提示名(如:酒店ID)
    判断非空同样是用循环,遍历数组,比如判断表单元素hot的值为0,那么可以给出提示“请输入酒店ID“
    这样,用户非常容易找到自己哪个没填了。
      

  8.   

    可以试试jquery的验证插件啊...