Declared function error

baffled_in_UK

Hi,

I copied this function which highlights search results into my page:

function highlight( $needle, $haystack ) {
$parts = explode( strtolower($needle), strtolower($haystack) );
$pos = 0;
foreach( $parts as $key=>$part ){
  $parts[ $key ] = substr($haystack, $pos, strlen($part));
  $pos += strlen($part);
  $parts[ $key ] .= '<FONT COLOR="#009000"><B>'.substr($haystack, $pos, strlen($needle)).'</B></FONT>';
  $pos += strlen($needle);
}
return( join( '', $parts ) );
}

// highlight search results
if ($type == "area") { 
highlight($value, $area);
$area=$parts; }

When I run this code I get this error :

"Fatal error: Cannot redeclare highlight() (previously declared in /search_results.php3:49) in /search_results.php3 on line 49"

(line 49 is the "function" line)

The word "highlight" is only used in the two places as shown. I'm coming to the conclusion that I'm really thick 'cos I can't see what's wrong. Can you ?

Thanks

YappyDog

The error is trying to tell you that you've already declared your "highlight" function, and you are not allowed to do it again. Check to see if you include your function twice.

dalecosp

Only thing that caught my eye is that $parts shouldn't be available to your script outside the function, should it?

Don't you really want:

if ($type=="area") {
   $parts=highlight($value, $area);
   $area=$parts;
   }

I really am guessing here...HTH 🙂

baffled_in_UK

Yeah, I kind of figured that what was it meant 😉

But as the main thread says, I have not included it twice, that is why it is confusing me.

dalecosp

Originally posted by YappyDog
The error is trying to tell you that you've already declared your "highlight" function, and you are not allowed to do it again. Check to see if you include your function twice.

Yah, that's correct, of course. You might make sure you're not looping over that section of code somehow 🙂

baffled_in_UK

That was it, it was within the damn loop.

Thanks guys.