PHP Что делать ошибка function foreach() on null?

Question

[wedytd]

Есть код ниже но он не работает ошибка Fatal error: Uncaught Error: Call to a member function foreach() on null на 77 строке

Diary.php:
class Diary
{
    public array $days = []; // Список дней

    /**
     * Конструктор объекта
     * В качестве параметра нужно передавать информацию, получаемую от метода REST API "rest/diary"
     * 
     * @param array $info - массив информации о журнале
     */
    public function __construct (array $info)
    {
        $this->days = array_map (fn ($day) => new Day ($day), $info['days'] ?? []);
    }

    /**
     * Проход по дням недели
     * 
     * @param callable $callable - коллбэк для вызова
     * 
     * @return Diary
     */
    public function foreach (callable $callable): Diary
    {
        foreach ($this->days as $day)
            $callable ($day);

        return $this;
    }
}
index.php:
    $diary->foreach (function ($day) //77 строка
{
    echo 'Дата: '. $day->date . PHP_EOL;
    echo 'Уроки:'. PHP_EOL . PHP_EOL;

    $i = 0;

    $day->foreach (function ($lesson) use (&$i)
    {
        echo ++$i .') '. $lesson->discipline . PHP_EOL;

        $space = str_repeat (' ', strlen ($i) + 2);

        echo $space .'Время: '. $lesson->begin_time .' - '. $lesson->end_time . PHP_EOL;
        echo $space .'Кабинет: '. $lesson->room . PHP_EOL;
        echo $space .'Учитель: '. $lesson->teacher . PHP_EOL;
        echo $space .'Тема: '. $lesson->subject . PHP_EOL;

        echo PHP_EOL;
    });

    echo PHP_EOL;
});

Answer 1

[ThunderCat]

Очевидно что в переменной $diary ничего нет(NULL), и вы пытаетесь вызвать метод от пустой переменной. Предполагается что для начала нужно создать в ней объект класса Diary.

Comments

[wedytd]

Объект есть и он подключен в отдельном файле а в index сделано require

[ThunderCat]

wedytd, Это так не работает.
https://www.php.net/manual/en/language.types.object.php

Кроме того, то что у вас "типа объект", по сути ничего с объектно-ориентированным кодом общего не имеет, так как процедурный код завернутый в тег объекта, не перестает быть процедурным. Если вы заворачиваете кизяк в упаковку от шоколада, слаще он не станет.

[Ипатьев]

Я много говнокода повидал на своем веку, но такого ещё не встречал.
Это ж надо было такого наворотить
За один только метод с названием foreach в любом коллективе за такое долго и смачно били бы ногами.

[wedytd]

Ипатьев, Код не мой https://github.com/krypt0nn/BarsAPI

[ThunderCat]

wedytd,

Код не мой

то что кто-то выложил какаху на гитхаб, не делает этот код качественным, или даже рабочим.

[Ипатьев]

А я и не говорю что код ваш.
Но лучше конечно самому написать, чем вот этим пользоваться.

[tukreb]

wedytd, использовать студенческий код особенно опасно.
Мне было бы вообще стыдно выкладывать то, что я писал в студенческие годы, но видимо krypt0nn считает иначе :D

[toxa82]

wedytd, объекта нет и об этом тебе говорит ошибка. То что ты думаешь что он есть и подключается как ты описал это самообман. Ищи почему не создаётся или не передаётся нужный объект.