How to populate array data into a HTML table

Question

I am trying to populate an array into a HTML table, but I keep getting Notice: Undefined offset

What am I doing wrong?

<?php
class Invoices{
    public $database;
    public $exec;

    public function __construct(){
        $this->database = new Dbase();
    }

    public function show(){
        $query="SELECT * FROM invoices;";
        $this->exec=$this->database->fetch($this->database->run($query));
        echo '<table class="table table-striped table-bordered"><thead>
                <tr>
                <th>ID</th>
                <th>Customer ID</th>
                <th>Products</th>
                <th>Total</th>
                <th>Status</th>
                <th></th>
                </tr>
                </thead>';
        for ($i=0; $i<count($this->exec, 0); $i++){
            echo '<tr>';
            for ($j=0; $j<5; $j++){
                echo '<td>'.$this->exec[$i][$j].'</td>';
            }
            echo '</tr>';
        }
        echo '</table>';
    }
}
?>

Below are the results of var_dump() on the $exec array.

I think using a foreach loop instead would simplify things. — Don't Panic
– Don't Panic, Commented Jan 9, 2017 at 20:54

Don't Panic · Accepted Answer · 2017-01-09 21:05:33Z

2

If you use foreach loops, you won't need to worry about incorrect indexes.

In my opinion, a for loop is usually better for doing something a specific number of times,and if you're just iterating over every item in an array, foreach is usually more straightforward.

foreach ($this->exec as $invoice)
{
    echo '<tr>';
    foreach ($invoice as $column)
    {
        echo '<td>'.$column.'</td>';
    }
    echo '</tr>';
}

answered Jan 9, 2017 at 21:05

Don't Panic

41.9k11 gold badges70 silver badges87 bronze badges

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

Comments

I wrestled a bear once. · Accepted Answer · 2017-01-09 20:55:51Z

0

Hardcoding array lengths is a big no no

    foreach($this->exec[$i]as $j)
    {
        echo '<td>'.$this->exec[$i][$j].'</td>';
    }

But if you must harcode the count, make sure you get it right. according to your markup it looks like you;re expecting 6 elements ,not 5.

edited Jan 9, 2017 at 20:55

answered Jan 9, 2017 at 20:51

I wrestled a bear once.

23.5k20 gold badges74 silver badges122 bronze badges

Collectives™ on Stack Overflow

How to populate array data into a HTML table

2 Answers 2

Comments

Comments

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

2 Answers 2

Comments

Comments

Your Answer

Sign up or log in

Post as a guest

Related