preg_match function

[Anon]

I am trying to write a script that generates a stylesheet for various platforms. Pretty basic I am sure.

The parser is hanging up on my preg_match function, however.

$agent = getenv("HTTP_USER_AGENT");

if (preg_match("/Win/i"),("$agent")) {
	$style = "win";	}

else if (preg_match("/Linux/i"),("$agent")) {
	$style = "linux";}

Any ideas why it spits out an error for the two lines that attempt to match the user agent variable?

Latest version of php and Apache on Win98 if that makes any difference.

Thanks
-phast

[Anon]

if (preg_match("/Win/i"),("$agent")) {
$style = "win"; }

should be :

if (eregi("win",$agent))
{
$style = "win";
}
if (eregi("linux",$agent))
{
$style = "linux";
}

rgds,

Alex

[Anon]

You have a syntax error in your preg_match command. All arguments to the command should be in a single list. What you've done is tried to supply two lists of one item each. Change to:

preg_match("/Linux/i", $agent)

If you aren't used to perl's regular expressions, you might want to fall back to the simpler regular expressions in the ereg and eregi commands.

-spp

[Anon]

Thanks for the help folks... eregi() fixed the problem.

And to think I copied the preg_match() sniipet right out of a fairly new PHP book verbatim. Go figure!

-j