新手,一个有关问题,多谢!

新手,请教各位一个问题,谢谢!!!!
<script language="javascript" type="">


function LoginSubmit(){

var user=document.getElementById("loginName").value;
var pass=document.getElementById("password").value;
if(user==null||user==""){
alert("请填写用户名:");
}
else if(pass==null||pass==""){
alert("请填写密码");
}
else document.getElementById("Login").submit();
}
</script>
  </head>
  
  <body>
    <form method="Post"  name="Login" id="Login"  action="LoginServlet">
    用户名:<input type="text" name="loginName" id="LoginName"><br>
    密码:<input type="password" name="password" id="password"><br>
    <input type="button" value="提交"  onclick="LoginSubmit()"><br>
    <input type="reset" value="重置" ><br>
    </form>
  </body>
</html>
这段代码有什么不对的吗?
这是写的一个登陆界面
------解决思路----------------------

<!DOCTYPE html>
<html lang="en">
<head>
<meta charset="utf-8" />

<title>open source</title>

  </head>
  
  <body>
    <form method="Post"  name="Login" id="Login"  action="LoginServlet">
    用户名:<input type="text" name="loginName" id="loginName"><br>
    密码:<input type="password" name="password" id="password"><br>
    <input type="button" value="提交"  onclick="LoginSubmit()"><br>
    <input type="reset" value="重置" ><br>
    </form>
  </body>
<script type="text/javascript">
function LoginSubmit(){
        var user=document.getElementById("loginName").value;
        var pass=document.getElementById("password").value;
        if(user == "" ){
                alert("请填写用户名:");
        }
        if(pass==""){
                alert("请填写密码");
        }
        if(user.length>3 && pass.length>6){
                document.getElementById("Login").submit();
        }
}
</script>
</html>