higlighting problem: shouldn't highlight one letter at a time

[gilgalbiblewhee]

What's wrong with this:

PHP Code:

for($m=0; $m <  count($bigWords); $m++){
        $strText = preg_replace("/(".$keyword[$m].")/i", "<span class=\"\" id=\"\" style=\"color:".$COLORS[$m].";                             font-weight:bold;\">$1</span>", $strText);
        echo "<br />";
        echo $strText;
    } 


It's highlighting one letter at a time:

<span class="" id="" style="color:red; font-weight:bold;"></span>
t<span class="" id="" style="color:red; font-weight:bold;"></span>
h<span class="" id="" style="color:red; font-weight:bold;"></span>
e<span class="" id="" style="color:red; font-weight:bold;"></span>
 <span class="" id="" style="color:red; font-weight:bold;"></span>
w<span class="" id="" style="color:red; font-weight:bold;"></span>
h<span class="" id="" style="color:red; font-weight:bold;"></span>
o<span class="" id="" style="color:red; font-weight:bold;"></span>

 <span class="" id="" style="color:red; font-weight:bold;"></span>
a<span class="" id="" style="color:red; font-weight:bold;"></span>
n<span class="" id="" style="color:red; font-weight:bold;"></span>
d<span class="" id="" style="color:red; font-weight:bold;"></span>
 <span class="" id="" style="color:red; font-weight:bold;"></span>
b<span class="" id="" style="color:red; font-weight:bold;"></span>
u<span class="" id="" style="color:red; font-weight:bold;"></span>

[curtiss]

My first guess would be that $keyword is not really an array, but a string.

If you use a string in a scalar context (place a number in square brackets after the string name), it will use one letter from the string.

I do find it a bit strange that you're running the loop according to the count of a variable called "$bigWords", but you're utilizing a variable called $keyword inside.

Finally, unless your keyword will include a regexp at any point in time, you really should be using str_ireplace instead of preg_replace.

[gilgalbiblewhee]

Originally Posted by curtiss

My first guess would be that $keyword is not really an array, but a string.

If you use a string in a scalar context (place a number in square brackets after the string name), it will use one letter from the string.

I do find it a bit strange that you're running the loop according to the count of a variable called "$bigWords", but you're utilizing a variable called $keyword inside.

Finally, unless your keyword will include a regexp at any point in time, you really should be using str_ireplace instead of preg_replace.

I realized when I pasted that section the $keyword wasn't mentioned before. And echo $strtext should be placed outside the for loop.

[curtiss]

Does that mean that you solved your problem, or do you still need more help?

[gilgalbiblewhee]

Originally Posted by curtiss

Does that mean that you solved your problem, or do you still need more help?

I resolved it thanks.