Displaying mysql table with php

Question

I'm having trouble displaying my mysql table using php code. All it displays is the column names not the values associated with them. I know my username password and db are all correct but like I said the table is not displaying the values I added. Any help would be much appreciated This is my mysql code:

CREATE TABLE Guitars
(
  Brand  varchar(20)  NOT NULL,
  Model  varchar(20)  NOT NULL,
  PRIMARY KEY(Brand)
);

insert into Guitars values('Ibanez','RG');
insert into Guitars values('Ibanez','S');
insert into Guitars values('Gibson','Les Paul');
insert into Guitars values('Gibson','Explorer');

And this is my php code:

<?php
$db_host = '*****';
$db_user = '*****';
$db_pwd = '*****';

$database = '*****';
$table = 'Guitars';

if (!mysql_connect($db_host, $db_user, $db_pwd))
    die("Can't connect to database");

if (!mysql_select_db($database))
    die("Can't select database");

// sending query
$result = mysql_query("SELECT * FROM {$table}");
if (!$result) {
    die("Query to show fields from table failed");
} 

$fields_num = mysql_num_fields($result);

echo "<table border='1'><tr>";
// printing table headers
for($i=0; $i<$fields_num; $i++)
{
    $field = mysql_fetch_field($result);
    echo "<td>{$field->name}</td>";
}
echo "</tr>\n";
// printing table rows
while($row = mysql_fetch_row($result))
{
    echo "<tr>";

    // $row is array... foreach( .. ) puts every element
    // of $row to $cell variable
     foreach($row as $cell)
        echo "<td>$cell</td>";

    echo "</tr>\n";
}
mysql_free_result($result);
?>

Can you check your table from mysql/phpmyadmin ?

The Alpha
– The Alpha

2012-04-18 01:02:33 +00:00
Commented Apr 18, 2012 at 1:02 — The Alpha
– The Alpha, Commented Apr 18, 2012 at 1:02

GoSmash · Accepted Answer · 2012-04-18 00:47:01Z

4

Try this:

// printing table rows
while($row = mysql_fetch_row($result))
{
    echo "<tr>";

    echo "<td>$row[0]</td>";
    echo "<td>$row[1]</td>";

    echo "</tr>\n";
}

Update:

Note: You can't make brand as primary key since you gonna add same brand name for different models.

enter image description here

edited Apr 18, 2012 at 0:47

answered Apr 18, 2012 at 0:27

GoSmash

1,1081 gold badge12 silver badges38 bronze badges

Sign up to request clarification or add additional context in comments.

7 Comments

Ryan Sayles Over a year ago

I understand what this code is doing however it yields the same result as my previous code. It displays a table that only says Brand and Model and not the values

JT Smith Over a year ago

Have you ensured the table has values prior to calling this function?

GoSmash Over a year ago

i made a home work with this and i got result too. Please see update

Ryan Sayles Over a year ago

Ramesh is that screenshot the output of the code that I have except you replaced my while loop with yours? Or did you just add that while loop because I'm still getting an empty table

GoSmash Over a year ago

@rsay3 just used the code what you had i haven't changed anything. There is no problem with your code. Please check your table.

|

JT Smith · Accepted Answer · 2012-04-18 00:49:01Z

3

I don't see why you are using the fetch_field call. I'm assuming that you know ahead of time what the actual names of each field in your table is prior to calling it's data? I think for simplicity sake (less loops and nested loops) you should write the name of the fields manually, then loop through the data entering the values.

$feedback .= "<table border='1'><tr>";
$feedback .= "<th>Brand</th><th>Model</th></tr>";

while ($row = mysql_fetch_array($result)) {
   $feedback .= "<tr><td>" . $row['Brand'] . "</td>";
   $feedback .= "<td>" . $row['Model'] . "</td></tr>";
}

$feedback .= "</table>";

echo $feedback;

answered Apr 18, 2012 at 0:49

JT Smith

7414 silver badges12 bronze badges

Comments

Indrek · Accepted Answer · 2012-04-18 00:21:26Z

0

By the time you're done displaying the header, the query result's internal pointer will have reached the last row, so your mysql_fetch_row() calls fail because there are no more rows to fetch. Call mysql_data_seek(0); before printing the table rows, to move the internal pointer back to the first row.

answered Apr 18, 2012 at 0:21

Indrek

9068 silver badges29 bronze badges

1 Comment

Ryan Sayles Over a year ago

Do I have to call mysql_data_seek(0) before the while loop then?

The Alpha · Accepted Answer · 2012-04-18 00:43:56Z

0

You can also try for fetching data

while ($fielddata = mysql_fetch_array($result))
{
    echo '<tr>';
    for ($i = 0; $i<$fields_num; $i++) // $fields_num already exists in your code
    {
        $field = mysql_fetch_field($result, $i);
        echo '<td>' . $fielddata[$field->name] . '</td>';
    }
    echo '</tr>';
}

edited Apr 18, 2012 at 0:43

answered Apr 18, 2012 at 0:38

The Alpha

147k30 gold badges294 silver badges313 bronze badges

Collectives™ on Stack Overflow

Displaying mysql table with php

4 Answers 4

7 Comments

Comments

1 Comment

Comments

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

4 Answers 4

7 Comments

Comments

1 Comment

Comments

Your Answer

Sign up or log in

Post as a guest

Related