<b>Warning</b>: Invalid argument supplied for foreach()

[powah]

php foreachtest.php

Content-type: text/html
X-Powered-By: PHP/4.3.3

<br />
<b>Warning</b>: Invalid argument supplied for foreach() in <b>/opt/lampp/htdocs/unittest/foreachtest.php</b> on line <b>20</b><br />

This is foreachtest.php:
<?php
$portArray;

function test() {
for ($unit = 0; $unit < 3; $unit++) {
$GLOBALS['portArray'][$unit] = $unit;
}

}

if ($SERVER["REQUEST_METHOD"] == "GET")
{
test();
} else if ($SERVER["REQUEST_METHOD"] == "POST")
{
//...
test();
}

foreach ($portArray as $i) {
echo "i $i";
}

?>

The web browser display the data:
i 0i 1i 2
Why the warning?
How to fix this problem?

[halojoy]

This is how I should do it

function test(){
for($i=0;$i<3;$i++)$arr[$i]=$i;
return $arr;}
$portArray=test();
// do your foreach

You still get warning using this?

[ysu]

well, the qoted function should and does work, I just did a test here (php5.0.4) - maybe it's a problem with an earlier version?
if you want to be sure, use

$portArray = array();

to initialize the variable.

anyway here's how I would do it - which results a re-usable function:

<?php 
$portArray = array();

function test( &$arr, $n ) { 
	for ($unit = 0; $unit < $n; $unit++) { 
		$arr[$unit] = $unit; 
	} 
} 

test($portArray,3); 

foreach ($portArray as $i) {
	echo "i $i<br />"; 
} 
?>

[Drakla]

Originally posted by halojoy
at php.net says something like
'superglobals can not be used within functions'

I don't think people need to be told rubbish like that without proving it first. You can use $GLOBALS['foo'][] = 'something'; in a function and it'll work just as you expect it to.

[Installer]

I guess in as sense "superglobals can not be used in functions" is something like "superglobals can be used in functions". Only one word difference.

What the manual says exactly like is "...they are automatically global--i.e., automatically available in every scope."

[halojoy]

Okay. Thanks Drakla, for info. Most kind of you. I remove that.
Will make your quote look a bit odd :p Sorry.