致命错误：当它是对象时调用非对象上的成员函数

Question

我有一个类包含一组类。循环遍历数组并尝试运行 Class 函数时，出现错误：

Fatal error: Call to a member function getTDs() on a non-object on line 21

这是我的代码：

Cars.class

class Cars {
    private $cars = array();

    public function __construct(){
        $result = DB::getData("SELECT * FROM `cars` ORDER BY `name`");

        foreach($result as $row){
            $this->cars[] = new Car($row);
        }
    }

    public function printTable(){
        $html = '<table>';
        for($i=0, $l=count($this->cars); $i<$l; $i++){
            $html .= '<tr>';
            $html .= $this->cars[$i]->getTDs();
            $html .= '<td></td>';
            $i++;
            //print_r($this->cars[$i]);
            //print_r($this->cars[$i]->getTDS());
            $html .= $this->cars[$i]->getTDs(); //This is the supposed non-object
            $html .= '<td></td>';
            $i++;
            $html .= $this->cars[$i]->getTDs();
            $html .= '</tr>';
        }
        $html .= '</table>';
        echo($html);
    }
}

汽车类

class Car {
    public $data;

    public function __construct($data){
        $this->data = $data;
    }

    public function getTDs(){
        $html = '<td>'.$this->data['name'].'</td>';
        return $html;
    }
}

当在那个“非对象”（第 19 行）上使用 print_r 时，我得到了这个：

Car Object
(
    [data] => Array
    (
        [name] => 'Ferrari'
    )
)

当在“非对象”调用 getTDs()（第 20 行）上使用 print_r 时，我明白了：

<td>Ferrari</td>

那么，当我尝试将该结果添加到我的 $html 变量时，为什么在下一行它会中断？

Answer 1

你的 for 语句是：

for($i=0, $l=count($this->cars); $i<$l; $i++){

但在该循环中，您将 $i 增加了两倍。

$i++;
$i++;

所以在循环的最后一次迭代中，$i 指向 cars 的最后一个元素，但是当您再次增加 $i 时，您将到达数组的末尾。

所以在你到达太远之前停止循环。您的解决方法应该是：

for($i=0, $l=count($this->cars)-2; $i<$l; $i++){

编辑每次尝试访问索引时检查您是否位于 cars 数组的末尾会更明智。

Answer 2

你在循环中增加你的索引，你不需要这样做。这应该可以正常工作：

for($i=0, $l=count($this->cars); $i<$l; $i++){
        $html .= '<tr>';
        $html .= $this->cars[$i]->getTDs();
        $html .= '<td></td>';
        $html .= "</tr>";
}

另外，作为最佳实践，请尝试在循环外使用计数，它具有更好的性能。

$numCars = count($this->cars);
for($i=0; $i<$numCars; $i++)
{
  ...
}