for loop in a function?

[7724]

Hi,

I have a for loop, that i've tested and it works perfectly well, but when I put it in a function I get the message:

[] operator not supported for strings in these lines:

$headline[] = $row['headline'];
$story_id[] = $row['story_id'];

Any ideas how I can solve this?

$SQL = "SELECT headline, story_id from site ORDER BY story_id desc LIMIT 2";      
$result = mysql_query($SQL) OR die(mysql_error()); 

$row = mysql_fetch_array($result);

$n = mysql_num_rows($result);

$active = ($active=='active' ? 'inactive' : 'active');
$inactive = ($inactive=='inactive' ? 'active' : 'inactive');


for ($i = 1; $i <= $n; $i++ ) {

$headline[] = $row['headline']; 
$story_id[] = $row['story_id']; 
     
    echo '
      <div id="leaderPanel'.$n=i.'" class="leadContainer" style="display:inline">
      <a class="leader-'.$inactive.'"> '.$headline[0].' <br /> '.$story_id[0].'</a>
      <a class="leader-'.$active.'">'.$headline[1].' <br /> '.$story_id[1].'</a>
      </div>
     '; 

[Kailash Badu]

The error message implies that the variable $headline and $story_id were used to hold string values somewhere before the loop. Find that out.

You can apply a quick fix by declaring both $headline and $story_id as array just before the loop.

$headline = array();
$story_id  = array();

[spikeZ]

Further to Kailash's answer, the problem is that the arrays are not declared inside the function so are out of its scope when you try and fill them.

Put the 2 lines in Kailash's andser inside your function.

Off Topic:


Nice to see you again

[7724]

thanks guys