RegEXP Help 1

[Bastien]

Hi All

Regexp go me confused. I need to filter some characters out of a text box with regexp. I have the follwoing code

function checkString(s,oElement)
{
  var thisKey = s.value;
  myRegExp = /'&<>"/i;
  
  if (myRegExp.test( thisKey ))
  {
    alert("You have entered an invalid character in the field.\n\nYou are not allowed to use the following characters in the name.\n\n\tless than symbol( < )  \n\tgreater than symbol ( > )  \n\t ampersand ( & ) \n\tsingle quote( ' ) \n\tdouble quote ( \" )\n\nPlease adjust your entry.");
    document.getElementById(oElement).focus();
    clearField(oElement);
  }  
}



called by:


<input type="textbox" name="base_value_1" id="base_value_1" value="something" onkeyup="checkString(this,'base_value_1');">

I can't seem to get the regexp to test properly. Where I am going wrong?

TIA


Bastien

I wish my computer would do what I want it to do,
instead of what I tell it to do...

 

[kaht]

It's not triggering because your regex is looking for a run of characters in this order: '&<>"

So, unless you have all those characters, in that exact order, the regex will never return true.

There's 2 ways to fix this. First, put an or character between each character:

/'|&|<|>|"/

Second, this would be the preferred method - put brackets around the characters so the regex looks for any of the characters in the set:

/['&<>"]/

-kaht

Looking for a puppy?

http://www.silkypups.com

[small](Silky Terriers are hypoallergenic dogs that make great indoor pets due to their lack of shedding and small size)[/small]

 

[monksnake]

I like RegEx questions, they are like a puzzle.
Here is what I got, and it does work (tested):

  myRegExp = new RegExp("\'|&|<|>|\"");

 

[Bastien]

thanks, guys

Bastien

I wish my computer would do what I want it to do,
instead of what I tell it to do...