onClick

Hej

Är nybörjare på JavaScript, och vet inte (har ingen aning) om vad som är fel:

<SCRIPT LANGUAGE="JavaScript">
<!--

/* This JavaScript checks the variables and alert if they're wrong! */

function controllValues(){
if(email == ""){
alert("Please fill in all inputs marked with a *!")
}
if(password == ""){
alert("Please fill in all inputs marked with a *!")
}
if(login == ""){
alert("Please fill in all inputs marked with a *!")
}
}

-->
</SCRIPT>

	<FORM METHOD="post" ACTION="index.php?inc=signup&cat=members" onClick="controllValues()"><BR>
Realname:<BR><INPUT TYPE="text" NAME="name"><BR>
Login name*:<BR><INPUT TYPE="text" NAME="login"><BR>
Password*:<BR><INPUT TYPE="password" NAME="password"><BR>
Email:<BR><INPUT TYPE="text" NAME="email"><BR>
<INPUT TYPE="submit" VALUE="Submit!"><BR>
* = You've got to fill in!
</FORM>

Nån som känner för att ge en hjälpande hand? :)

Vad jag vill är alltså att kontrollera om någon av email, name eller password är tomma fält! :)

------------------
Sprit och hårdrock?

Börja med att ändra på onClick="controllValues()" till onSubmit="return controllValues(this)"

Sedan scriptet;

function controllValues(formobj){
if(formobj.email.value == ""){
  alert("Please fill in all inputs marked with a *!");
  return false;
} else if(formobj.password.value == ""){
  alert("Please fill in all inputs marked with a *!")
  return false;
} else if(formobj.login.value == ""){
  alert("Please fill in all inputs marked with a *!")
  return false;
} else {
  return true;
}

------------------
/ J

OK...
Jag håller precis på och lära mig om JavaScript..

<SCRIPT LANGUAGE="JavaScript">
<!--
/* This JavaScript checks the variables and alert if they're wrong! */

Fungerar /* lika väl som // ??

Lika bra att lära sig alla alternativ med en gång :e

------------------
*Om man aldrig frågar lär man inte få veta*

// Detta är en kommentar på en rad

/* Detta är
en kommentar
på flera
rader och
måste då
avslutas med */

------------------
/ J

Tack! :)

------------------
Sprit och hårdrock?

Tack J !!! :e

...måste berätta det för min lärare! :p

Förvisso kanske han redan vet :OO
(fortsätter jag så här kan jag ju lära mig mycke! ;) )

------------------
*Om man aldrig frågar lär man inte få veta*