Need some help for a newbie

arbjones

I am creating a test php program to access a mysql database and
am getting a few errors. Please let me know how I can correct them.
Thanks in advance

PHP 5.1.2
MYSQL 5.0
Windows 2000
IIS

Warning: mysqli_query() expects parameter 1 to be mysqli,
string given in C:\TESTIT\Ernie Test Connection.php on line 16

Warning: mysqli_fetch_array() expects parameter 1 to be mysqli_result,
null given in C:\Documents and Settings\Administrator\Desktop\TESTIT\Ernie Test
Connection.php on line 26

$user="root";
$host="localhost";
$password="";
$database = "memberstats";
$connect = mysqli_connect($host,$user,$password)
or die(mysql_error());
$db = mysqli_select_db($connect,$database)
or die(mysql_error());

$pettype = "dragon";
$query = "SELECT * FROM userinfo WHERE petType='$pettype'";
$result = mysqli_query($query,pettype)
or die(mysql_error());

/ Display results in table /

$pettype = ucfirst($pettype)."s";
echo "<h1>$pettype</h1>";
echo "<table cellspacing='15'>";
echo "<tr><td colspan='3'><hr></td></tr>";
while ($row = mysqli_fetch_array($result))
{

Houdini

This will run take note of the changes.

$user="root";
$host="localhost";
$password="";
$database = "memberstats";
$connect = mysqli_connect($host,$user,$password,$database)//with mysqli you use the database in the connection.
or die(mysql_error());
//$db = mysqli_select_db($connect,$database) Not need with mysqli it is done above
//or die(mysql_error());

$pettype = "dragon";
$query = "SELECT * FROM userinfo WHERE petType='$pettype'";
$result = mysqli_query($connect,$query)//use the link and the query (not field type)
or die(mysql_error());

/* Display results in table */
while ($row = mysqli_fetch_array($result)){
$pettype = ucfirst($pettype)."s";
echo "<h1>$pettype</h1>";
echo "<table cellspacing='15'>";
echo "<tr><td colspan='3'><hr></td></tr>";
//while ($row = mysqli_fetch_array($result))moved this up where it belongs and added the closing brace}
}

bradgrafelman

Couple of things...

Yes, the DB can be selected in the connect call... though either way is fine.
THe problem with your query was that you didn't pass the link resource, let alone in the right place. The manual for [man]mysqli_query/man can help you figure out which parameters are needed and in what order.
I believe Houdini wasn't reading your code correctly, as I believe you already have your while() statement in the correct place. Using his modification (moving it up and encompassing the HTML for the pet type and table tag) will definitely not work as expected, as you'll have a new table for each row returned (and an incorrectly formed table at that).