Откуда ошибка в контроллере?

Question

[Дониёр Маматкулов]

Здравствуйте!
Yii2 basic. Есть контроллер и в нём метод:

<?php
namespace app\controllers;

class UserController extends AppController {
	
	public $menu;
	
	public function getMenuHtml($tree) {
        if(!empty($tree)) {
            foreach($tree as $category) {
                $this->menu .= "<li data-jstree='{\"icon\":\"fa fa-user-circle\"}' data-id=\"".$category['id']."\">".$category['name'];
                if(isset($category['childs'])) {
                    $this->menu .= "<ul>";
                    self::getMenuHtml($category['childs']);
                    $this->menu .= "</ul>";
                }
                $this->menu .= "</li>";
            }
            return $this->menu;
        }
    }
}
?>

Сейчас вылазит ошибка Using $this when not in object context в строке "$this->menu"
Так стало после переключения на php семёрку.
Как исправить эту ошибку?

Comments

[Павел Корнилов]

self::menu() ?

Answer 1

[Антон Шаманов]

статический вызов динамического метода, должно быть $this->getMenuHtml

Comments

[Дониёр Маматкулов]

Нет, ошибка возникает в строке "$this->menu" См. скриншот:

[Антон Шаманов]

Дониёр Маматкулов, нет? ... ответ

[Антон Шаманов]

aleksejjjj, Дониёр Маматкулов, дело в том что в php не запрещен вызов статических методов как динамических и наоборот, но в контексте статического метода $this не существует, поэтому ошибка возникает именно в этой строке.

[Антон Шаманов]

aleksejjjj, и что мы получим в итоге? очередной СИ? в большой свободе большая ответственность

[Антон Шаманов]

aleksejjjj, "ты нуб и не хочешь говнокода? иди учи яву/cи" - слоган php и это ты еще ruby не видел)

[Антон Шаманов]

aleksejjjj,

если метод потом поменяет мой коллега? Это недопустимое поведение и это должно быть ограничено на уровне языка.

прости, но это показывает то что ты не понимаешь докрин php, метод не должен доверять данным поступившим из вне.

[Антон Шаманов]

aleksejjjj,

когда эта "фича" полезна

это не фича, это ошибка и если ты думаешь что оно свойственно новичкам, то разочарую - вордпресс, битрикс, вебасист и т.д. зачастую этим грешат.

акие еще данные поступившие извне?

те, что не передаются приватным методам класса другими его методами.

Добавить в метод проверку контекста? Это поняли даже разрабы PHP, а вы всё сопротивляетесь.

self и $this имеют немного отличий.
ладно, хорош флеймить, я пойду прогуляюсь - оставляю последнее слово за тобой.

[Дониёр Маматкулов]

Сделал так:

$t = new UserController();
$t->getMenuHtml($category['childs']);

Ошибка сохраняется. Меня мучает вопрос, почему ошибка возникает в строке $this->menu???

[Антон Шаманов]

Дониёр Маматкулов, потому что в статичных методах волшебная переменная $this недоступна. Значит как и сказал Алексей где-то в другом месте идет статический вызов данного метода.

Answer 2

[morricone85]

во-первых, почему контроллер генерирует html код? Это не правильно.
во-вторых, какое отношение имеет меню к контроллеру User?;
ну, а по теме, вам уже указали, где ошибка - self::getMenuHtml($category['childs'])