那位大虾指点一下我的select的JS控制错在哪里？为什么它没起作用？因该如何改，谢谢

Question

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>

Answer 1

方法摘要 
 void clear() 
          从此映射中移除所有映射关系。 
 Object clone() 
          返回此 HashMap 实例的浅表复制：并不克隆键和值本身。 
 boolean containsKey(Object key) 
          如果此映射包含对于指定的键的映射关系，则返回 true。 
 boolean containsValue(Object value) 
          如果此映射将一个或多个键映射到指定值，则返回 true。 
 Set<Map.Entry<K,V>> entrySet() 
          返回此映射所包含的映射关系的 collection 视图。 
 V get(Object key) 
          返回指定键在此标识哈希映射中所映射的值，如果对于此键来说，映射不包含任何映射关系，则返回 null。 
 boolean isEmpty() 
          如果此映射不包含键-值映射关系，则返回 true。 
 Set<K> keySet() 
          返回此映射中所包含的键的 set 视图。 
 V put(K key, V value) 
          在此映射中关联指定值与指定键。 
 void putAll(Map<? extends K,? extends V> m) 
          将指定映射的所有映射关系复制到此映射中，这些映射关系将替换此映射目前针对指定映射的所有键的所有映射关系。 
 V remove(Object key) 
          如果此映射中存在该键的映射关系，则将其删除。 
 int size() 
          返回此映射中的键-值映射关系数。 
 Collection<V> values() 
          返回此映射所包含的值的 collection 视图。 
例:
import java.util.*;class hashmaptest
{
   public static void main(String[] args)
   {
      HashMap  register = new HashMap();     
      register.put("name","jingsp");
      register.put("date","1996-8");
      register.put("stuid","960515");
      register.put("status","overS");
      register.put("scrol","excellence");
      register.put("lasttime","1999-9");
      System.out.println(register);
      System.out.println("-------------------");
      Set s = register. entrySet();
      System.out.println(s);
      System.out.println("-------------------");
      Iterator it = s.iterator();
      while(it.hasNext())
      {
           Map.Entry map = ( Map.Entry)it.next();
           System.out.print(map+"\t");
           System.out.println("key:"+map.getKey()+"\tvalue:"+map.getValue());
      }
   }
}

Answer 2

var   element   = document.form1.moduleId;
试试

Answer 3

dragon_sea,非常感谢,很想知道为什么这样改一下就可以了呢?跟我那个有什么不同啊?

Answer 4

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