[Resolved] Help with foreach()

[Mike-2003]

I have the below code when ran it comes up with the following error:

Warning: Invalid argument supplied for foreach()

I thought the I have it right, but must be I have missed something. Any help would be appreciated.

Thanks,
Mike

<?

if (($result = dosql($sql)) && (mysql_num_rows($result))) {

//initialize subtotal variables
$cart_ct = 0;
$line_ct = 0;
$time_ct = 0;
$rec_ct = 0;
$error_ct =0;
$cs_error_ct=0;
$piece_ct=0;
$prev_emp_name = "Test";
$prev_zone = "ZZ";
$emp_cart_ct=0;
$emp_line_ct=0;
$emp_time_ct=0;
$emp_error_ct=0;
$emp_cs_error_ct=0;
$emp_piece_ct=0;


echo "<table width=100% bgcolor='" . $color . "' class='reportsborder' cellspacing=1 cellpadding=1>";
echo "<tr valign='top' align='left'>";
echo "<table width=100%><tr bgcolor='#ffffff'><td class='text9bold' width=2% align = 'left'>&nbsp</td>";
echo "<td class='text9bold' width=5% align='center'>Zone</td>";
echo "<td class='text9bold' width=10% align='center'>Employee Name</td>";
echo "<td class='text9bold' width=5% align='center'>Lines</td>";
echo "<td class='text9bold' width=5% align='center'>Pieces</td>";
echo "</tr></table>";
	while ($query_data = mysql_fetch_array($result)) {
		If ($prev_emp_name <> $query_data["EmployeeFull"] and $rec_ct <> 0) {


		 if(isset($zonesubtot[$query_data['Zone']])) {
   $zonelinessub[$query_data['zone']]+=(int)$query_data['zone'];
} else {
   $zonelinessub[$query_date['zone']]=(int)$query_data['zone'];
}


echo "<table width=100% bgcolor='#ffffff' cellspacing=1 cellpadding=1>";
echo "<tr valign='top' align='left' bgcolor='#ffffff'><td>&nbsp</td></tr></table>";

} else {
//no change
				}

echo "<table width=100% bgcolor='" . $color . "' class='reportsborder' cellspacing=1 cellpadding=1>";
echo "<tr valign='top' align='left'>";
?>
<table>
<thead>
<th>Zone</th>
<th>Sub-total</th>
</thead>
<?php
foreach($zonelinessub as $key => $value) {
?>

<tr>
<td><?php echo($key);?></td>
<td><?php echo($value);?></td>
</tr>
<?
    }
?>    

    </table>

<?
echo "<td class='text9' width=5% align='center'>".$query_data["Zone"]."</td>";
echo "<td class='text9' width=10% align='center'>".$query_data["LastName"].",".$query_data["FirstName"]."</td>";
echo "<td class='text9' width=5% align='center'>".$query_data["BatchLines"]."</td>";
echo "<td class='text9' width=5% align='center'>".$query_data["BatchPieces"]."</td>";
echo "</tr>";
echo "</table>";

if ($color == "#eeffee") $color = "#ffffff";
else $color = "#eeffee";
	}   
}

else {
echo "No Carts recorded in date range";
	}


?>

[buck2bcr]

Are you sure that the $zonelinessub variable is getting set?

do an if isset right before the foreach to see if its set.

That's were I would start.

[Mike-2003]

I'm not sure what you mean by isset right before the foreach(). Can you give me an example?

[laserlight]

I think buck2bcr presumed it was an incoming variable, and didnt actually look at the code.

Your $zonelinessub variable is declared at your loop:
while ($query_data = mysql_fetch_array($result))
right?

That could mean that the loop doesnt even run once, so the variable doesnt exist.

You might be able to circumvent that by declaring
$zonelinessub = array();
before the loop.

However, a surefire solution would be to check $zonelinessub before the foreach(), by using [man]is_array/man

[Mike-2003]

laserlight, Thanks! it works now.

[buck2bcr]

Laserlight,

I did look at the code and was wondering if the array was made at all for use later.

That's way I said to check it.

I said use the isset instead of is_array(), so thanks for straightening that out for me.

Buck