[RESOLVED] Explode function and recordset

[tnylsej]

How to display data from database using array of strings retrieved by explode function as parameter of recordset?

Part of the codes is shown below:

<tr>
       <?php 		 
	$exploded = explode(",",$row_Recordset1['Attributes']);
	$numExploded = count($exploded);
	for ($i=0; $i<=$numExploded; $i++){ 
		$strExploded = $exploded[$i];
		mysql_select_db($database_tsis, $tsis);
		$query_Recordset2 = "SELECT * FROM attribute WHERE Attribute='".$strExploded."'";
		$Recordset2 = mysql_query($query_Recordset2, $tsis) or die(mysql_error());
		$row_Recordset2 = mysql_fetch_assoc($Recordset2);
		$totalRows_Recordset2 = mysql_num_rows($Recordset2);				?>
         <th><?php echo $row_Recordset2['Name']; ?></th>
      <?php } ?>
</tr>

It only displays the result of $row_Recordset2[$exploded[0]].

What went wrong?

[Piranha]

I don't know exactly how your string looks like, but it is possible that the string (before it gets exploded) looks something like this: "Denmark, Sweden, Finland, Netherlands". If that is the case and you explode with , then the space will still be there in front of the other countries, and it will only show the first. A solution is to trim the result.

To see if it is really this you could echo $row_Recordset1['Attributes'] to see how it looks like. You could also echo $numExploded to see that there really is more than one result.

[tnylsej]

Thanks alot

It's because of the space..

Able to get everything works now =D

[Weedpacket]

You might want to review your database design; there aren't many reasons to justify having multiple values in a comma-delimited string in a database field. (And many reasons to justify not doing it: all that code plus the query to get the attributes field could be replaced by a single query and a loop over its results, for example.) Oh, and you only need to select the database once, not every time you query it.

[tnylsej]

haha.. those codes were generated by dreamweaver, dreamweaver always selects the database each time I create a new recordset.. =/

Anyway, I don't understand what you mean by:

all that code plus the query to get the attributes field could be replaced by a single query and a loop over its results

Would you kindly elaborate? (=

Thanks

[Weedpacket]

The phrase I recommend you google for is "database normalisation".