Page 1 of 1

[solved] email form validation script

Posted: Fri Jan 19, 2007 3:04 pm
by jyhm
Can anyone tell the difference between these two scripts? They are simple validation scripts for an email form. It is called via the form action

Code: Select all

<form name="myform" action="testjscript.htm" method="post" onSubmit="return CheckData(this)">
They look completely identical to me, but only the first one works. This is driving me nuts! :evil:

Code: Select all

<script type="text/javascript" language="JavaScript">
				<!--
				/* This one works. */
				re = /[\<\>@#\$%\^\&\+=\{\}\[\]\|\\/]/;
				
			function CheckData(myform)	{
					
					if (myform.name.value == "")	{
						alert("Please Enter Your Name.");
						myform.name.focus();
						myform.name.select();
						var problem = true;
					}
					
					if (myform.email.value == "")	{
						alert("Please Enter Your Email.");
						myform.email.focus();
						myform.email.select();
						var problem = true;
					}
					
					if (myform.message.value == "")	{
						alert("Please Enter a Message.");
						myform.message.focus();
						myform.message.select();
						var problem = true;
					}

					if	(re.test(myform.message.value))	{
					 	alert("These symbols are not allowed: @ # $ % ^ & + = { } [ ] | \ / < >");
					 	myform.message.focus();
					 	myform.message.select();
					 	var problem = true;
					 	
					}
					
					if (problem==true)	{
						return false;
					} else	{
						return true;
					}
					
				}
		//-->
		</script>

Code: Select all

<script type="text/javascript" language="JavaScript">
				<!--
				/* This one does not work. */
				re = /[\<\>@#\$%\^\&\+=\{\}\[\]\|\\/]/;
				
			function CheckData(myform)	{
					
					
					if (myform.name.value == "")	{
						alert("Please Enter Your Name.");
						myform.name.focus();
						myform.name.select();
						var problem = true;
					}
					
					if (myform.email.value == "")	{
						alert("Please Enter Your Email.");
						myform.email.focus();
						myform.email.select();
						var problem = true;
						
					
					
					if (myform.message.value == "")	{
						alert("Please Enter a Message.");
						myform.message.focus();
						myform.message.select();
						var problem = true;
					}

					if	(re.test(myform.message.value))	{
					 	alert("These symbols are not allowed: @ # $ % ^ & + = { } [ ] | \ / < >");
					 	myform.message.focus();
					 	myform.message.select();
					 	var problem = true;
					 	
					}
					
					if (problem==true)	{
						return false;
					} else	{
						return true;
					}
					
				}
		//-->
		</script>

Posted: Fri Jan 19, 2007 3:10 pm
by jyhm
Alas! I forgot a:

Code: Select all

'}'
at

Code: Select all

if (myform.email.value == "")   {
 alert("Please Enter Your Email.");
 myform.email.focus();
myform.email.select();
var problem = true;
It must look different when I post it because I could not find it on my editor.
Thanks for all your help guys! :D

Posted: Fri Jan 19, 2007 3:11 pm
by Luke
glad I could help

Posted: Fri Jan 19, 2007 3:33 pm
by Kieran Huggins
You go NSG!