Php error

[PoorDoggie]

It says unexpected t_variable or something - line 6 (highlighted)

$sub = $_SERVER['HTTP_HOST']; // get host
$domain = "6yd.net";
$sub = eregi_replace($domain, "" $sub); //<-- line 6?!
$sub = eregi_replace("www\.", "", $sub);
$sub = strtolower($sub);
if(!empty($sub)){
  include("shorturl/do_short_url.php");
}

anyone know what's wrong?

Thanks
Tom

 

[Peter]

[COLOR=Red]$sub = eregi_replace($domain, "" $sub);[/COLOR]

that is your problem

presumably you meant to have:-

$sub = eregi_replace($domain, "" $sub);

but this is still wrong it should be (notice the comma after the quotes) :-

$sub = eregi_replace($domain, "", $sub);

 

[Dan]

He meant to have the [COLOR] stuff there so you could find it easier.. but that doesn't work in

.

All you need is a comma. ^^

 

[yeppers]

As Peter showed, the leaving out of the comma was the problem.

But just a note, you should always try to optimize what you can in PHP to make things nice and quick, every little bit counts.

So for the first line I'd suggest using str_replace rather than eregi_replace, as its quicker to process.

So line 6 would be: $sub = str_replace($domain,"",$sub);

All the best!
Rhett.

 

[Xyzer]

I hate it when i come to programming threads, i work it out and find the problem, scroll down the thread and its finished lol.

 

[PoorDoggie]

Thanks a lot everyone, I can never figure it out, post it here, and then feel so stupid because it is sooooo obvious. Thanks for all your help though

is str_replace() case dependant though? eregi_replace isn't which is why I use it. I suppose in this case I could use "strtolower" or something.

Thanks
Tom

 

[Dan]

http://us3.php.net/manual/en/function.str-ireplace.php

Not case sensitive str_ireplace.

webmonkey: scroll down first?

 

[Xyzer]

Good idea Dan, But i like to know that i know.