AP
in_array($definition[1], get_class_methods($definition[0]) ?? [], true) - А вот эта часть для какого use case-а?
Size: a a a
2020 August 16
AM
Arman Poghosyan
is_array($definition) && array_keys($definition) === [0, 1] - это по сути любой callable в виде массива, то есть первая проверка is_callable то же самое
в PHP 8 нет
AM
Это всё для него.
AP
Я что-то видать пропустил по PHP 8, сейчас пересмотрю.
AM
Там нельзя стало вызывать нестатичный метод как статичный.
А
но конкретно тут кейс другой, скорее всего. Это для отложенного создания сущности и вызова нестатичного метода
AP
https://wiki.php.net/rfc/consistent_callables - нашел, да, тада может сделать как-то явно?
AP
например: \function_exists('is_callable_type') && \is_callable_type($definition) чтоб потом все было понятней
AP
но конкретно тут кейс другой, скорее всего. Это для отложенного создания сущности и вызова нестатичного метода
Не совсем представил
А
Arman Poghosyan
Не совсем представил
ты задаёшь
$def=[SomeService::class, 'someMethod'] и когда придёт очередь это вызвать, будет примерно такое: container->get($def[0])->{$def[1]}() только через InjectorА
т.е. это не callable с вызовом статики. Это получение объекта и вызов обычного метода
AP
точно, вижу. Я полюбому перепишу, чтоб было явно видно для чего это, чтоб когда будет какая-то X версия с PHP минимум 8, сразу было явно, что надо убрать
AP
А то сижу, голову ломаю, что за дополнительная проверка))
А
лучше напиши комментарий
А
а то комменты никто не любит писать и порой сидишь и тупишь "что это и зачем?"
AP
🆗
AP
А может убрать этот формат вообще? И сделать что-то типо ClassName::action или Classname@action как в многих других роутерах? Тогда typehint array можно будет вообще убрать в PHP 8
AP
В PHP 8 будут же union types и просто можно будет type hint string|callable и не делать опять проверку на callable
AP
Если я конечно правильно представляю все это
А
оставь это на потом