Invalid argument supplied for foreach() really simple syntax. easy question!

numerical25

Trying to figure out whats wrong with this syntax

<?php 

$ImusSays = array('says' =>'those are some','says'=>'nappy headed','says'=>'hoes.'); 

foreach($ImusSays['hair'] as $var) 
    { 
        echo $var." ";
    } 

echo "</br> therefore Imus is FIRED!";

?>

I get this error

Warning: Invalid argument supplied for foreach() in C:\www\apache\htdocs\test.php on line 10

whats the problem. What exactly is the invalid argument suppied.

NogDog

The first argument to foreach must be an array, not a single element of an array (unless it's a multi-dimensional array and that specified array element is, in fact, an array). Also, you define an array with one name on the first line, but then you're using a different name for the array in the foreach() statement, which I suspect is not what you intended?)

numerical25

$ImusSays = array('says' =>'those are some','says'=>'nappy headed','says'=>'hoes'); 

foreach($ImusSays as $saythis) 
    { 
        echo $saythis." ";
    } 

echo "Therefore imus is Fired!";

?>

Is this how its suppose to look ??

bradgrafelman

numerical25 wrote:
Is this how its suppose to look ??

Not really, couple of things to note:

You can't have multiple array items with the same key. In fact, the first line of your code is equivalent to doing something like this:
```
$ImusSays = array('says' => 'those are some');
$ImusSays['says'] = 'nappy headed';
$ImusSays['says'] = 'hoes';
```
In other words, you're simply overwriting the value of the 'says' key each time, NOT creating new elements.
Now, if you had properly formed your array, you could get rid of your entire for loop and replace it with this:
```
echo implode(' ', $ImusSays) . ' ';
```

NogDog

Everything looks good except that you use the same array index ('says') for each array element in the array() command, so you end up with a single array element with the last value assigned. You either need to make each index different (e.g.: 'says1', 'says2', 'says3') or use an enumerated array and let the indexes be assigned automatically:

$myArray = ('value 1', 'value 2', 'value 3');

numerical25

Yea I understand. that. I was just trying to figure out what I was doing wrong all together. you and nog solved thanks alot. Im just trying to figure out what im doing wrong in a nother problem im having. which I posted here almost close to the bottom of the page there is the real problem im trying to solve but I think i figure it out. I was just a little mislead. thanks

numerical25

Im trying to create a form that displays a list of products. When the customer clicks on a checkbox under delete, the form gets submitted in a $_POST. im trying to take the list of values and find the id numbers in a SQL and delete them. here is the syntax. is this correct ??

$list = implode(",", $_POST); // Will create a comma delimited string 
			$sql = "DELETE FROM products WHERE ProductID IS {$list}";

NogDog

// "IN" instead of "IS", and parens instead of curly braces:
$sql = "DELETE FROM products WHERE ProductID IN ($list)";

Looks like that should work if the values in $list are numeric. If they are strings, then:

$list = "'" . implode("','", $_POST) ."'";

numerical25

how do i make the values numeric ??? cause i think they are strings

numerical25

$sql = "DELETE FROM products WHERE ProductID IN ($list)";

Can I use this for a SELECT statement. Like...

SELECT productName FROM products WHERE productID IN ($list)

NogDog

Yes.