ตรวจดูให้ด้วยครับ ว่าผมผิดพลาดตรงไหน JS มันไม่ยอมทำงานครับ ลองมาครึ่งวันแล้วไม่รู้ผิดตรงไหน

function notEmpty()
 {
	if(document.frm-user.txt-user_id.value.length!=5)
	{
		alert('รหัสต้องมี 5 หลัก');
		document.frm-user.txt-user_id.focus();
		return false;
	}
	else if(document.frm-user.txt-username.value=="")
	{
		alert('กรุณากรอก username');
		document.frm-user.txt-username.focus();
		return false;
	}
	else if(document.frm-user.txt-password.value=="")
	{
		alert('กรุณากรอก password');
		document.frm-user.txt-password.focus();
		return false;
	}
	else if(document.frm-user.txt-name.value=="")
	{
		alert('กรุณากรอกชื่อ');
		document.frm-user.txt-name.focus();
		return false;
	}
    else if(document.frm-user.txt-lname.value=="")
	{
		alert('กรุณากรอกนามสกุล');
		document.frm-user.txt-lname.focus();
		return false;
	}
	else if(document.frm-user.select_depart.value=="ABC")
	{
		alert('กรุณาเลือกหน่วยงาน');
		document.frm-user.select_depart.focus();
		return false;
	}
	else if(document.frm-user.select_role.value=="ABC")
	{
		alert('กรุณาเลือกสิทธิ์การใช้งาน');
		document.frm-user.select_role.focus();
		return false;
	}
	 return true;
}

<form action="#" name="frm-user" method="post"  id="frm-user_ID" onsubmit="javascript:notEmpty()">
<table  border="1" class="reference" width="95%">
  <tr>
    <th style="border:1px solid #7F9DB9; font-family : 'tahoma'; font-size : 13px;">USER ID</th>
    <th style="border:1px solid #7F9DB9; font-family : 'tahoma'; font-size : 13px;">Username</th>
    <th style="border:1px solid #7F9DB9; font-family : 'tahoma'; font-size : 13px;">Password</th>
    <th style="border:1px solid #7F9DB9; font-family : 'tahoma'; font-size : 13px;">Name</th>
    <th style="border:1px solid #7F9DB9; font-family : 'tahoma'; font-size : 13px;">Lastname</th>
    <th style="border:1px solid #7F9DB9; font-family : 'tahoma'; font-size : 13px;">Department</th>
    <th style="border:1px solid #7F9DB9; font-family : 'tahoma'; font-size : 13px;">Level</th>
  </tr>
  <tr>
 <td>
    <input type="text" name="txt-user_id" id="user_id" MAXLENGTH=5 style="border:1px solid #7F9DB9; font-family : 'tahoma'; font-size : 11px; color : #333333;  height :20px; width: 100px;">
 </td>
  <td>
  <input type="text" name="txt-username" id="username" style="border:1px solid #7F9DB9; font-family : 'tahoma'; font-size : 11px; color : #333333;  height :20px; width: 130px;">
  </td>
  <td>
  <input type="text" name="txt-password"  id="password" style="border:1px solid #7F9DB9; font-family : 'tahoma'; font-size : 11px; color : #333333;  height :20px; width: 130px;">
  </td>
  <td>
  <input type="text" name="txt-name"  id="name" style="border:1px solid #7F9DB9; font-family : 'tahoma'; font-size : 11px; color : #333333;  height :20px; width: 200px;">
  </td>
  <td><input type="text" name="txt-lname" id="name" style="border:1px solid #7F9DB9; font-family : 'tahoma'; font-size : 11px; color : #333333;  height :20px; width: 200px;"></td>
  <td>
        <select  id="select_depart" name="depart" style="border:1px solid #7F9DB9; font-family : 'tahoma'; font-size : 11px; color : #333333;  height :22px; width: 230px;" >
        <option selected="selected" value="ABC">----------------หน่วยงานทั้งหมด----------------</option>
        <option value=""></option>
         </select>
    </td>
    <td>
    <select  id="select_role" name="role" style="border:1px solid #7F9DB9; font-family : 'tahoma'; font-size : 11px; color : #333333;  height :22px; width: 180px;" >
        <option selected="selected" value="ABC">----------------Level----------------</option>
        <option value="2">USER</option>  
        <option value="1">ADMINISTRATOR</option>         
         </select>
    </td>
  </tr>
  </table>
  <br />
  <input type="submit" name="submit" value="submit" style="border:1px solid #7F9DB9; font-family : 'tahoma'; font-size : 15px; color : #333333;  height :30px; width: 100px;">
  &nbsp;&nbsp;
  <input type="reset" name="reset" value="reset" style="border:1px solid #7F9DB9; font-family : 'tahoma'; font-size : 15px; color : #333333;  height :30px; width: 100px;" onClick="this.frm-user.reset()">
  </form>