Primeramente, quita el primer mysql_close(); solo con que lo cierres una vez y al final tienes mas que suficiente.
Otra cosa, si has hecho una funcion para conectar, tb puedes hacer una para las querys ( lo digo pq no existe ningun mensaje que verifique si tu consulta es ok o no ). De esta manera podras generar un fichero que contenga todas la funciones relacionadas con mysql, incluirla en este fichero (el actual donde llamarias a las funciones) con un require o un include -> ej. require('mysql.php').
Finalmente, puedes ahorrarte varias lineas de codigo en:
Código:
// Existe coincidencia de usuario
if(mysql_num_rows($result)>0)
{
?>
<script>document.location = 'index.htm';</script>
<?php
}
// No existe coincidencia
else
{
?>
<script>document.location = 'Login.php';</script>
<?php
}
y poner en su lugar una cosa tal que así:
Código:
<script>document.location =<?php (mysql_num_rows($result)>0)? ?>'index.htm';<?php : ?>'Login.php';</script>
Pruebalo y ya diras
