form validator in javascript

[Ikoster]

heey,
ik heb een for validator willen omvormen dat als er een veld niet/niet goed is ingevuld.
het betrefende veld een rode rand krijgt na submitten.

alles werkte goed tot ik de tweede stap wilde doen.
wat zit er fout?
[JS]
function validateForm()
{
var naam = $("#naam");
var email = $("#email");
var fnaam=document.forms["myForm"]["fnaam"].value;
var femail=document.forms["myForm"]["femail"].value;
var atpos=femail.indexOf("@");
var dotpos=femail.lastIndexOf(".");

if (fnaam==null || fnaam=="")
{
naam.parent().addClass("error");
return false;
}
if (atpos<1 || dotpos<atpos+2 || dotpos+2>=femail.length)
{
email.parent().addClass("error");
return false;
}
}
[/JS]

<style>
.error {
border-color:red;
}
</style>
<form name="myForm" action="demo_form.asp" onsubmit="return validateForm()" method="post">
First name: <input type="text" name="fnaam" id="naam" ><br />
Email: <input type="text" name="femail" id="email" >
<input type="submit" value="Submit">
</form>

bedankt voor het lezen.:thumb:

 

[phobia]

maak van je .error in je style eens
border: 1px solid red;

 

[Ikoster]

heey,
bedankt voor het reageren.
echter is dit niet het probleeem.
het is wel al een verbetering dank je wel.

maar de fout zit in dit regeltje:
var naam = $("#naam");
het object is leeg!!
wat moet ik hier aan veranderen.:thumb:

 

[phobia]

var naam=$("#naam").va();

 

[That Guy]

ik neem aan dat dit [js]var naam=$("#naam").val();[/js]moet zijn (met een 'L' erachter), typo? Niet dat ik ervaring met jQuery heb ofzo. Maargoed.

 

[phobia]

Klopt helemaal Tha Guy, het knippen en plakken ging niet helemaal naar wens. En ik heb niet de tijd genomen om het ff goed na te lezen!

My BooBoo

 

[Ikoster]

heey,
bedankt voor het reageren!
echter geeft dit nog steeds een error:
"de eigenschap of methode wordt niet door dit object ondersteund"
wat betekend dit en hoe moet het voledige script eruitzien?

[JS]function validateForm()
{
var naam = $("#naam").val();
var email = $("#email").val();
var fnaam=document.forms["myForm"]["fnaam"].value;
var femail=document.forms["myForm"]["femail"].value;
var atpos=femail.indexOf("@");
var dotpos=femail.lastIndexOf(".");
if (fnaam==null || fnaam=="")
{
naam.parent().addClass("error");
return false;
}
if (atpos<1 || dotpos<atpos+2 || dotpos+2>=femail.length)
{
email.parent().addClass("error");
return false;
}

}[/JS]

 

[Ikoster]

opgelost

heey,
bedankt voor het kijken maar ik heb intussen zelf een oplossing gemaakt.
[JS]
function validateForm()
{
var contactXemail = document.forms["contact"]["email"].value;
var atpos=contactXemail.indexOf("@");
var dotpos=contactXemail.lastIndexOf(".");
var statusXemail = "";
if (atpos<1 || dotpos<atpos+2 || dotpos+2>=contactXemail.length)
{
document.forms["contact"]["email"].style.borderColor='red';
document.getElementById('email').style.color='red';
var statusXemail = "false";
}
if (statusXemail == 'false'){return false;}
else{return;}
[/JS]

bekijk het resultaat op http://www.ikoster.nl