FOREACH beginner

[faaal]

Hi there.

I have a MySQL query that checks a "Compatible" column for a number. For instance "1".

In this table there are 4 rows: Type, Compatible, StockLevel and Cost.

I need to display the Type field of every row in Compatible which matches the number 1.

EG:

TYPE COMPATIBLE
[Honda] - [1] - 0 - £1
[Ford] - [1] - 0 - £1


Currently I have

$cartridge_query = mysql_query("SELECT * FROM cartridges WHERE Compatible LIKE '1'");
$cartridge_result = mysql_fetch_assoc($cartridge_query);
$cartridge_count = mysql_num_rows($cartridge_query);

echo "<select>";

foreach ($cartridge_result["Compatible"] as '1') {
echo "<option>".$cartridge_result["Type"]."</option>";

}

echo "</select>";

I have a feeling I'm close but it's not working. Any ideas?

[hyperionXS]

You are wrong using foreach. It should be

foreach($cartridge_result as $cr){
   if($cr['Compatible']==1){
       echo "<option>".$cr["Type"]."</option>";
   }
} 

[faaal]

You are wrong using foreach. It should be

foreach($cartridge_result as $cr){
   if($cr['Compatible']==1){
       echo "<option>".$cr["Type"]."</option>";
   }
} 

OOH I get it!

I had to get my mind wrapped around a while script and it clearly influenced what I was trying with foreach. That makes sense though!


OK I have tried this and I am using:

$cartridge_query = mysql_query("SELECT * FROM cartridges");
$cartridge_result = mysql_fetch_assoc($cartridge_query);
$cartridge_count = mysql_num_rows($cartridge_query); 

echo "<select>";

    foreach($cartridge_result as $cr){ 
         if($cr['Compatible']==$_POST["printer"]){ 
         echo "<option>".$cr["Type"]."</option>"; 
         } 
        } 

echo "</select>"; 

Where am I going wrong? My brain hasn't switched on this morning :/

[faaal]

BTW $_POST["printer"] is 1/2/3/4/5 etc... just one value.

In the option field I am not seeing any names, just the printer ID.

[hyperionXS]

$cartridge_query = mysql_query("SELECT * FROM cartridges");
$cartridge_result = array();`
while ($row = mysql_fetch_assoc($cartridge_query)) {
	$cartridge_result[] = $row;
}

echo "<select>"; 

foreach($cartridge_result as $cr){ 
    if($cr['Compatible']==$_POST["printer"]){ 
        echo "<option>".$cr["Type"]."</option>"; 
    } 
}

echo "</select>"; 

[faaal]

Thats great, thank you. It works!

I forgot one important element though which is that I was using the explode function, because in the table some cartridges are compatible with several printers.

Take these for insance:

HP xx BLACK
HP xx CYAN
HP xx YELLOW

In the compatible field may be "1, 9" (Printer ID #1 and Printer ID #9). The array you wrote makes me think this isn't possible on top?

[hyperionXS]

I guess you can use explode and in_array