Improve company productivity with a Business Account.Sign Up

x
  • Status: Solved
  • Priority: Medium
  • Security: Public
  • Views: 193
  • Last Modified:

Check for "@" symbol as part of a alert in a domain prefix for web form.

Greetings Experts;

To resolve a problem I had with a form, I am using the example code from a previous post how to strip urls from form field before or on submit
I have modified the code just to check the prefix and not the suffix. I would also like to alert on the @ but have been baffled on how to add this.  Any suggestions? I have listed the modifed code.

Thanks

Dave

<script type="text/javascript">
function checkIt(o){
	if( /(<a +href|http:|www|@\.)/i.test(o.value)){
		alert('Please do not include the prefix "www, http, etc.. ');
		o.focus();
		return false;
	}
return true;
}
</script>

Open in new window

0
Davidjevans
Asked:
Davidjevans
  • 3
  • 2
1 Solution
 
Graceful_PenguinCommented:
You can do one of two things either do the test twice, once for the prefix and then again for the @. Or you could use the exec method of the regexp object. This method can check for multiple matchs as show in this example from the manual.
function matchDemo()
{
  var s;
  var re = new RegExp("d(b+)(d)","ig");
  var str = "cdbBdbsbdbdz";
  var arr = re.exec(str);
  s = "$1 contains: " + RegExp.$1 + "<BR>";
  s += "$2 contains: " + RegExp.$2 + "<BR>";
  s += "$3 contains: " + RegExp.$3;
  return(s);
}

Open in new window

0
 
DavidjevansAuthor Commented:
Thanks Graceful;
 
     I think my main issue is the fact that the "@" is not recongnized in the string. Also, to which manual are you referring to? I am a project manager turned programmer on this project so my only formal training goes back 25 years to assembler level language.

Regards

Dave
0
 
GwynforWebCommented:
I ran your code as below and tested with <a @. it ran as you wanted. ie I see no problem

<script type="text/javascript">

function checkIt(o){
        if( /(<a +href|http:|www|@\.)/i.test(o.value)){
                alert('Please do not include the prefix "www, http, etc.. ');
                o.focus();
                return false;
        }
return true;
}

</script>

<input type=text  onblur="alert(checkIt(this))">
0
What Kind of Coding Program is Right for You?

There are many ways to learn to code these days. From coding bootcamps like Flatiron School to online courses to totally free beginner resources. The best way to learn to code depends on many factors, but the most important one is you. See what course is best for you.

 
Graceful_PenguinCommented:
I modified the prev function to test for your specifications and no there is no problem with the @. Try each of your types of strings in this function and see if it will work for you.
function matchDemo()
{
  var s;
  var re = new RegExp("(^.*@.*$)|(^http:.*$)|^(www)");
  var str = "www.alfega.com";
  var arr = re.exec(str);
  s = "$1 contains: " + RegExp.$1 + "<BR>";
  s += "$2 contains: " + RegExp.$2 + "<BR>";
  s += "$3 contains: " + RegExp.$3;
  alert(s);
}

Open in new window

0
 
DavidjevansAuthor Commented:
Sorry for the delay in getting back to this. Priority projects jumped in.  

Gwynforweb. I think if you tried it without a period in front of the @ (I am guessing this is why you did not see the issue) you will find this does fail. For whatever reason, the group I am coding this for say's the get periodic "@" signs in the responses. As if they are entering an e-mail address.  

Graceful Penguin, thank you for the suggestion, however I am afraid it is a bit beyond me. I am a former training manager, turned project manager, turned programmer. (We do what we can to stay employed these days). Can you be a bit more descriptive for me?

Thanks

Dave
0
 
Graceful_PenguinCommented:
OK I modified the code to do exactly what you wanted. But let me explain what it does and why.
The attached code first compiles the regular expersion :
var re = new RegExp("(^.*@.*$)|(^http:.*$)|^(www)");
then it runs the regexp on o.value :
re.exec(o.value);
Then it tests each of the result strings to see if a match was found :
if ((RegExp.$1 != '') || (RegExp.$2 != '') || (RegExp.$3 != ''))

Let me explain what happens here, as you can see the regexp string "(^.*@.*$)|(^http:.*$)|^(www)" is divided into three separate tests for each of the conditions. The result of each of these test are stored in an array : RegExp.$ also each element can be access by RegExp.$N were N is the number of the test . (like I did in the code)

The reason I test each subset is to show you how you can supply more information to the user because you know exactly what part of the test failed. (I did not code that part of the function you can add that if you want to when you want to)

I hope you understand what I tried to explain. Feel free to ask if there is something you don't understand.
function matchDemo(o)
{
  var re = new RegExp("(^.*@.*$)|(^http:.*$)|^(www)");
  re.exec(o.value);
  if ((RegExp.$1 != '') || (RegExp.$2 != '') || (RegExp.$3 != ''))
  {
    alert('Please do not include the prefix "www, http, etc.. or an @');
    o.focus();
    return false;
  }
  else
    return true;
}

Open in new window

0
Question has a verified solution.

Are you are experiencing a similar issue? Get a personalized answer when you ask a related question.

Have a better answer? Share it in a comment.

Join & Write a Comment

Featured Post

What Kind of Coding Program is Right for You?

There are many ways to learn to code these days. From coding bootcamps like Flatiron School to online courses to totally free beginner resources. The best way to learn to code depends on many factors, but the most important one is you. See what course is best for you.

  • 3
  • 2
Tackle projects and never again get stuck behind a technical roadblock.
Join Now