php error

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

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

PHP Code:

  $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 · 05-15-2006, 02: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);

Dan · 05-15-2006, 02: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, 03: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-15-2006, 11:17 PM

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, 10: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, 10: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, 10:32 AM

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

05-15-2006, 02:26 PM	#1 (permalink)
PoorDoggie Soon to be RICHdoggie! Join Date: Jan 2005 Location: UK Posts: 2,390 316.50 NP$ (Donate)	php error It says unexpected t_variable or something - line 6 (highlighted) PHP Code: `$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 __________________ █ Exchange your long URL for a much smaller one! █ Free Anonymizer/Proxy █ Taz Martin's blog - Learn all about internet marketing, promoting your website and much more! Last edited by SecondVersion; 05-15-2006 at 03:18 PM. Reason: [COLOR] doesn't work in [PHP] ;)

05-15-2006, 02:35 PM	#2 (permalink)
Peter Senior Member Join Date: Nov 2003 Location: Scotland Posts: 4,900 0.60 NP$ (Donate)	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);`

05-15-2006, 03:30 PM	#4 (permalink)
BillyConnite Join Date: Jul 2005 Location: Coffs H, Australia Posts: 3,107 47.00 NP$ (Donate)	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. __________________ <?php if(1===1){ $computer="fine."; }else{ $computer="broken."; } echo "Your computer is ".$computer; ?>

05-16-2006, 10:21 AM	#6 (permalink)
PoorDoggie Soon to be RICHdoggie! Join Date: Jan 2005 Location: UK Posts: 2,390 316.50 NP$ (Donate)	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 __________________ █ Exchange your long URL for a much smaller one! █ Free Anonymizer/Proxy █ Taz Martin's blog - Learn all about internet marketing, promoting your website and much more!

05-15-2006, 02:44 PM	#3 (permalink)
Dan Buy my domains. Join Date: Feb 2006 Posts: 2,801 56.00 NP$ (Donate)	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-15-2006, 11:17 PM	#5 (permalink)
Xyzer Senior Member Join Date: Aug 2005 Location: United Kindom Posts: 1,506 90.70 NP$ (Donate)	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, 10:25 AM	#7 (permalink)
Dan Buy my domains. Join Date: Feb 2006 Posts: 2,801 56.00 NP$ (Donate)	http://us3.php.net/manual/en/function.str-ireplace.php Not case sensitive str_ireplace. webmonkey: scroll down first?

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

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