那位大虾指点一下我的select的JS控制错在哪里？为什么它没起作用？因该如何改，谢谢
function validate()
{
var element = getElementByName("moduleId");
if(element.value ==""){
alert("请选择发布的新闻所属Module!");
element.focus();
return false;
}
element = getElementByName("text");
if(element.value ==""){
alert("请选择发布的新闻标题!");
element.focus();
return false;
}
element = getElementByName("description"); 
if(element.value==""){
alert("请选择发布的新闻内容!");
element.focus();
return false;
}
return true;
}
</script>
<form name="form1" onsubmit="return validate()" method="post" action="${pageContext.request.contextPath}/manager/addLinks.do">
  <table width="91%" border="1" align="center" bordercolor="#CCCCFF" bgcolor="#ccddee" class="9P">
  <tr> 
  <td height="9" colspan="2" bgcolor="#Ccddee"> <div align="center" class="12p"> 
  <p class="Blue"><span class="pp"><strong>${admin.adminName }</strong></span><span class="9P"> 
  新闻发布</span><span class="11p">[</span><span class="green"><strong>正常  
  </strong>]</span><span class="pp"> 
  </span></p>
  </div>
  </td>
  </tr>
  <tr> 
  <td class="9P">图片链接新闻添加[link]添加所属模块：<font color="red">（此发布的新闻所属模块一定要选，否则会报错！）</font>
  <select name="moduleId" >
  <option value="" >请选择</option>
<c:forEach items="${modules}" var="module"> 
<option value="${module.moduleId}" >${module.moduleName }</option>
</c:forEach>
</select>
</td>
  </tr>
</table>

------解决方案--------------------
var element = document.form1.moduleId;
试试
------解决方案--------------------
用这个试一下
var element = document.getElementById("moduleId");
前面少了document是不行的