confirm passwords

[onressy]

this must be a better way with regular expressions, any suggestions?

if (objFrm.frmPassword.value == ""){
objFrm.frmPassword.focus();
alert("Please enter a password.");
return false;
}

if (objFrm.frmPasswordConfirm.value == ""){
objFrm.frmPasswordConfirm.focus();
alert("Please confirm your password.");
return false;
}else if(!objFrm.frmPassword.value==objFrm.frmPasswordConfirm.value)
alert("passwords do not match")
}

 

[onressy]

sry typeO , this line should be:
}else if(objFrm.frmPassword.value!=objFrm.frmPasswordConfirm.value)

not: (!objFrm.frmPassword.value==objFrm.frmPasswordConfirm.value

 

[BabyJeffy]

There are other ways... but I wouldn't necessarily call them better ways. You can use regular expressions... but they will not be as easy for you (or the next person maintaining the code) to understand.

If you are unable to create your own regular expression for this simple task - then I suggest you stick with what you have... it is easy to understand for everyone.

Cheers,
Jeff

[tt]Jeff's Page @ Code Couch

http://www.codecouch.com/jeff/

http://www.codecouch.com/jeff/blog/

[/tt]

What is Javascript? FAQ216-6094

 

[cLFlaVA]

this isn't really a case for regular expressions. you could clean up your code a little, but i wouldn't change much, functionally...



*cLFlaVA
----------------------------
[tt]( <P> <B>)¹³ * (<P> <.</B>)[/tt]

[URL unfurl="true"]http://www.coryarthus.com/[/url]

 

[BillyRayPreachersSon]

Why do you want to use regexp here? If all you want to do is see if a field is empty, then what you have is the best way to go.

If you want something more, then you'll need to tell us - we're not psychic ;-)

Hope this helps,
Dan

[tt]Dan's Page [blue]@[/blue] Code Couch

http://www.codecouch.com/dan/

[/tt]