[RESOLVED] mysql_fetch_row problem

[deltaforce229]

I'm having problem using mysql_fetch_row in the following code.

<?php
mysql_connect("localhost", "****", "****") or die ("Cant connect to db");
mysql_select_db("****") or die ("Cant select db");
// Groups
$cus_groups = mysql_query("SELECT * FROM `cms_bbuser_group` WHERE `group_id` = 2");
while ($cus_displaygroups = mysql_fetch_row($cus_groups)){
  // Members
  $cus_members = mysql_query("SELECT * FROM `cms_users` WHERE `group_id` = '.$cus_displaygroups[1].'");
  while ($cus_displaymembers = mysql_fetch_row($cus_members)){
    echo $cus_displaymembers[2];
  }
}
?>

It returns the following error

Warning: mysql_fetch_row(): supplied argument is not a valid MySQL result resource in /home/bloodzom/public_html/test.php on line 9

[laserlight]

The problem is that you did not properly end your string. It should be:

  $cus_members = mysql_query("SELECT * FROM `cms_users` WHERE `group_id` = '".$cus_displaygroups[1]."'");

That said, I would recommend that you select precisely the columns you need instead of doing a SELECT *, and also use mysql_fetch_assoc() instead of mysql_fetch_row(). Chances are you can actually just use one query (e.g., with a join), but given your current SELECT * and mysql_fetch_row() usage I cannot even begin such an attempt.

[zamshed_farhan]

hello dear.
you use variables in wrong way in the query,so it returns a wrong query
and that returns no results.

[deltaforce229]

@ thanks that works much better and cleaner.

@zamshed.farhan you don't think i know that? Learn to be helpful instead of pointing out the obvious, especially on your first post.