Mysql Fetch Array Foreach

Question

I am trying to use a foreach loop to set value to populate a table, for some reason my array is showing null inside values. But when I var dump the $result_list[] it shows the array of products, Am i doing this wrong? Thanks

$result = mysql_query("SELECT id, product, price FROM products");


$result_list = array();
while($row = mysql_fetch_array($result)) {
   $result_list[] = $row;
}

foreach($result_list as $row) {

    $productitems[] = array(
        'id' => $row->id,
        'product'  => $row->product,
        'price'  => $row->price
    );              
}

var_dump($productitems);


array(2) { [0]=> array(3) { ["id"]=> NULL ["product"]=> NULL ["price"]=> NULL } [1]=> array(3) { ["id"]=> NULL ["product"]=> NULL ["price"]=> NULL } }

The mysql extension is deprecated and will be removed in the future: use mysqli or PDO instead. Anyway. you need foreach($result_list[] as $row. you forgot the []. — Kees Sonnema
– Kees Sonnema, Commented Jun 6, 2013 at 13:46

som · Accepted Answer · 2013-06-06 13:46:05Z

6

foreach($result_list as $row) {

    $productitems[] = array(
        'id' => $row['id'],
        'product'  => $row['product'],
        'price'  => $row['price']
    );              
}

Try this.

answered Jun 6, 2013 at 13:46

som

4,6542 gold badges23 silver badges36 bronze badges

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

Comments

GGio · Accepted Answer · 2013-06-06 13:46:13Z

1

to answer your question you are getting back row as an array and not object but yet you still try to access it as an object $row->id. Instead use $row['id'], $row['product'], $row['price'].

Do not use mysql_* functions instead use PDO or MySQLi, for example look how simple it is with PDO:

$productitems = $pdo->fetchAll(\PDO::FETCH_ASSOC); and then use foreach

answered Jun 6, 2013 at 13:46

GGio

7,67311 gold badges48 silver badges85 bronze badges

Comments

Zoe - Save the data dump · Accepted Answer · 2019-05-25 14:23:08Z

1

Change it to,

foreach($result_list as $row) {
    $productitems[] = array(
       'id' => $row['id'],
       'product'  => $row['product'],
       'price'  => $row['price']
    );
}

Note: Please, don't use mysql_* functions in new code. They are no longer maintained and are officially deprecated. See the red box? Learn about prepared statements instead, and use PDO, or MySQLi - this article will help you decide which. If you choose PDO, here is a good tutorial.

edited May 25, 2019 at 14:23

Zoe - Save the data dump

28.4k22 gold badges130 silver badges163 bronze badges

answered Jun 6, 2013 at 13:45

Rikesh

26.5k14 gold badges82 silver badges90 bronze badges

Collectives™ on Stack Overflow

Mysql Fetch Array Foreach

3 Answers 3

Comments

Comments

Comments

Your Answer

Linked

Hot Network Questions

Collectives™ on Stack Overflow

3 Answers 3

Comments

Comments

Comments

Your Answer

Sign up or log in

Post as a guest

Linked

Related