php error

PoorDoggie · 05-15-2006, 03:26 PM

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

PHP Code:

  $sub = $_SERVER['HTTP_HOST']; // get host

????: NamePros.com http://www.namepros.com/programming/197636-php-error.html
$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 · 05-15-2006, 03:35 PM

PHP Code:

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

that is your problem

presumably you meant to have:-

PHP Code:

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

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

PHP Code:

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

????: NamePros.com http://www.namepros.com/showthread.php?t=197636

Dan · 05-15-2006, 03:44 PM

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

All you need is a comma. ^^

BillyConnite · 05-15-2006, 04:30 PM

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 · 05-16-2006, 12:17 AM

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 · 05-16-2006, 11:21 AM

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

[QUOTE=BillyConniteSo line 6 would be: $sub = str_replace($domain,"",$sub);
[/QUOTE]

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 · 05-16-2006, 11:25 AM

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

Not case sensitive str_ireplace.

webmonkey: scroll down first?

Xyzer · 05-16-2006, 11:32 AM

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

05-15-2006, 03:26 PM	THREAD STARTER #1 (permalink)
PoorDoggie Soon to be RICHdoggie! Join Date: Jan 2005 Location: UK Posts: 2,408	php error It says unexpected t_variable or something - line 6 (highlighted) PHP Code: `$sub = $_SERVER['HTTP_HOST']; // get host ????: NamePros.com http://www.namepros.com/programming/197636-php-error.html $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 Last edited by SecondVersion; 05-15-2006 at 04:18 PM. Reason: [COLOR] doesn't work in [PHP] ;)

05-15-2006, 03:35 PM	#2 (permalink)
Peter NamePros Expert Join Date: Nov 2003 Location: Scotland Posts: 5,069	PHP Code: `[COLOR=Red]$sub = eregi_replace($domain, "" $sub);[/COLOR]` that is your problem presumably you meant to have:- PHP Code: `$sub = eregi_replace($domain, "" $sub);` but this is still wrong it should be (notice the comma after the quotes) :- PHP Code: `$sub = eregi_replace($domain, "", $sub); ????: NamePros.com http://www.namepros.com/showthread.php?t=197636`

05-15-2006, 04:30 PM	#4 (permalink)
BillyConnite Join Date: Jul 2005 Location: Coffs H, Australia Posts: 3,456	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. __________________ Free Forums / GoDaddy Coupon Codes (NEW DOMAIN!) / Free Arcade Script / <?='Your computer is '.(1?fine:broken).'.'?>

05-15-2006, 03:44 PM	#3 (permalink)
Dan Buy my domains. Join Date: Feb 2006 Posts: 2,796	He meant to have the [COLOR] stuff there so you could find it easier.. but that doesn't work in [PHP]. All you need is a comma. ^^

05-16-2006, 12:17 AM	#5 (permalink)
Xyzer Senior Member Join Date: Aug 2005 Location: United Kindom Posts: 1,502	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.

05-16-2006, 11:21 AM	THREAD STARTER #6 (permalink)
PoorDoggie Soon to be RICHdoggie! Join Date: Jan 2005 Location: UK Posts: 2,408	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 [QUOTE=BillyConniteSo line 6 would be: $sub = str_replace($domain,"",$sub); [/QUOTE] 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

05-16-2006, 11:25 AM	#7 (permalink)
Dan Buy my domains. Join Date: Feb 2006 Posts: 2,796	http://us3.php.net/manual/en/function.str-ireplace.php Not case sensitive str_ireplace. webmonkey: scroll down first?

05-16-2006, 11:32 AM	#8 (permalink)
Xyzer Senior Member Join Date: Aug 2005 Location: United Kindom Posts: 1,502	Good idea Dan, But i like to know that i know.

Currently Active Users Viewing This Thread: 1 (0 members and 1 guests)