Simple Regex problem :(

[Snetty]

I'm trying to write a regular expression that validates a form entry. It's designed to only allow uppercase alphabetic characters and numbers, it must also be between 2 and 4 characters in length (inclusive).

I came up with the following.. but it's allowing pretty much everything through, can anyone see where I'm going wrong?

PHP Code:

if(preg_match('/^([A-Z0-9]{2,4})$/', $_POST['prefix']) === FALSE){ fail } 


Any help much appreciated.

[abduraooft]

See http://in.php.net/preg_match

Return Values

preg_match() returns the number of times pattern matches. That will be either 0 times (no match) or 1 time because preg_match() will stop searching after the first match. preg_match_all() on the contrary will continue until it reaches the end of subject . preg_match() returns FALSE if an error occurred.

[Iszak]

PHP Code:

<?php

if (!preg_match('/^[A-Z0-9]{2,4}$/', $_POST['prefix'))
{
  echo "Invalid";
}

?>

Regards

[Snetty]

excellent, thanks for the help guys. I used to use eregi and I guess I just didn't think to check how preg_match deals with not finding a match.