[SOLVED] preg_match() error

[soycharliente]

Warning: preg_match(): No ending delimiter '^' found in events.php on line 3

 

I have no idea what that means.

 

<?php
$checkEventMonth = preg_match("^[0-9]+$", $_GET["m"]);
?>

 

Does that check that $_GET["m"] starts with a number and every character after the first is also a number?

[soycharliente]

If someone is going to post a tutorial about how to do something, then IT NEEDS TO BE RIGHT!!!

LOL. Geez. (not anyone here. something i found on-line.)

 

So I _think_ I fixed the error. But now all of my statements are coming up 0 when they are clearly 1.

What am I doing wrong?

<?php

$checkEventMonth = preg_match("/^[0-9]+$/", $_GET["m"]);
$checkEventDay = preg_match("/^[0-9]+$/", $_GET["d"]);
$checkEventYear = preg_match("/^[0-9]+$/", $_GET["y"]);
if(($eventMonth && $eventDay && $eventYear) == 1) {
$eventMonth = $_GET["m"];
$eventDay = $_GET["d"];
$eventYear = $_GET["y"];
$eventDate = date("Y-m-d", mktime(0, 0, 0, $eventMonth, $eventDay, $eventYear));
/*dbconnect();
$query = "SELECT * FROM events WHERE theDate='$eventDate' ORDER BY title ASC";
$result = mysql_query($query) OR DIE("Unable to run event query.");
dbclose();*/
$eventData = "Events for $eventMonth.$eventDay.$eventYear:\n";
} else {
$eventData = "Not a valid date.\n";
}

?>
<?php echo $eventData; ?>

[TEENFRONT]

erm

 

im not a regex guy but im not sure your doing it right.

 

preg_match("/^[0-9]+$/", $_GET["m"]);

 

to ME that looks like your only trying to see if "m" has a number between 0 and 9.

 

AND you dont do anything with "$checkEventMonth". you just do the preg_match then leave the vars alone.

 

Im guessing you may mean to do this ?

 

if(($checkEventMonth && $checkEventDay && $checkEventYear) == 1) {

[btherl]

to ME that looks like your only trying to see if "m" has a number between 0 and 9.

 

Not quite, it checks that EVERY character is between 0 and 9.  The ^ and $ anchor the start and end of the string, so the expression only matches if every character from the start to the end is 0-9.  It also checks that there is at least one character (that's done by using + instead of * )

[ToonMariner]

It checks that the characters of $_GET['m'] are numeric which could be done like so

 

<?php
preg_match("/\d+/", $_GET["m"]);
?>

[soycharliente]

How would you add on that it can't be empty? Like strlen() != 0.

Or set how long it can be?

[ToonMariner]

that is what the + is for - it means one or more times...

[btherl]

Some examples:

 

'|^[0-9]+|' => Match 1 or more digits (Doesn't match empty string)

'|^[0-9]*|' => Match 0 or more digits (DOES match empty string)

'|^[0-9]{5}|' => Match exactly 5 digits

'|^[0-9]{3,5}|' => Match 3, 4 or 5 digits.

'|^[0-9]?|' => Match 0 or 1 digits.  Usually used to say "This character can be there or it can be missing, I don't care".  Equivalent to {0,1}