Почему вызов функции Deprecated?

Здравствуйте!
рассматриваю код, с такими моментами, на которые выводится deprecated

Deprecated: Non-static method app\core\DbModel::findUser() should not be called statically
Это ошибка в коде или такой способ записи реально устаревший?

0

Сообщение от Fatmarmelad такой способ записи реально устаревший?

Да. Такой способ в принципе нелогичен (вдруг findUser использует $this, которой нет при статичном вызове), не знаю, зачем когда-то допускали его использование.

0

Сообщение от Fatmarmelad рассматриваю код

у вас в коде ошибки. приведи код в рабочее состояние.

Сообщение от Fatmarmelad PHP 1 2 3 4 5     ...     public function login(){         $user = User::findUser(); //  предупреждение выбивает в этой строке.     }     ...

Сообщение от Fatmarmelad способ записи реально устаревший?

Сообщение от Para bellum Да

вплоть до версии 8.1.5 - работает и не является устаревшим.

0

Сообщение от Fart83 вплоть до версии 8.1.5 - работает и не является устаревшим.

https://onlinephp.io/c/b177b

0

Сообщение от Jewbacabra https://onlinephp.io/c/b177b

https://onlinephp.io/c/bfd84

0

Сообщение от Fart83 https://onlinephp.io/c/bfd84

Вызов метода родителя (не обязательно прямого) и вызов нестатического метода как статического - разные вещи, несмотря на похожий синтаксис.

0

Сообщение от Jewbacabra Вызов метода родителя (не обязательно прямого) и вызов нестатического метода как статического - разные вещи, несмотря но похожий синтаксис.

вы пример ТС смотрели? я думаю, что нет

Добавлено через 1 минуту

Сообщение от Jewbacabra разные вещи

это когда вы показываете другой пример... и пытаетесь изменить тематику ТС

0

Сообщение от Fart83 вплоть до версии 8.1.5 - работает и не является устаревшим.

То что работает не отменяет того, что такое поведение не объявлено depricated. Рано или поздно это будет вызывать ошибку.
Собственно это абсолютно правильно. Метод НЕ объявлен статическим в абстрактном классе. Абстрактный класс по сути задает "соглашение" и правила.

Тут правильнее говорить не "устаревший"/"не устаревший". А о том, что "плохой код"/"правильный код". В данном конкретном случае надо понимать, что допустимое (согласно соглашения накладываемого классом DbModel) изменение класса User может вызвать "поломку" кода в совершенно другом месте/местах.


Если это не устраивает необходимо создавать новый абстрактный класс (или интерфейс) вместо DbModel. который будет описывать соглашение о существовании статического метода

0

Сообщение от Fart83 вы пример ТС смотрели? я думаю, что нет

Как раз смотрел. И там именно что попытка вызвать нестатический метод как статический.

0

Сообщение от voral Тут правильнее говорить не "устаревший"/"не устаревший".

Сообщение от Fatmarmelad Deprecated

вас не смущает это слово и приведенный пример?

0

Сообщение от Fart83 вас не смущает это слово?

"Осуждаемый" ? нет
Оно как раз и говорит о том, что я пояснил

0

Сообщение от Jewbacabra Как раз смотрел.

и ошибки в коде... тоже видели? не смутило вас?

0

Сообщение от Fart83 тоже видели? не смутило вас?

А что конкретно и почему меня должно смутить? Код явно не полный, поэтому отсутствие реализации абстрактных методов пропускаем. Лишнее ключевое слово class однозначно исправляется. Иерархия классов понятна.

1

Сообщение от Jewbacabra А что конкретно и почему меня должно смутить? Код явно не полный, поэтому отсутствие реализации абстрактных методов пропускаем. Лишнее ключевое слово class однозначно исправляется. Иерархия классов понятна.

Добавлю, что как конкретно реализован класс User тут вообще не важно

0

Сообщение от Jewbacabra Код явно не полный

как и ваш пример

Сообщение от Jewbacabra Лишнее ключевое слово class однозначно исправляется

Parse error: syntax error, unexpected token "class" ...

Сообщение от Jewbacabra Иерархия классов понятна.

приму это как за аргумент

0

Сообщение от Fart83 как и ваш пример

Мой код полный, и выполнимый (кроме 8.1, где такой вызов падает с ошибкой)

Сообщение от Fart83 Parse error: syntax error, unexpected token "class"

Так я и пишу, что однозначно (удалением class) исправляется. Но так как код не полный, все равно не будет работать, но чтобы понять иерархию классов это и не нужно.

0

Сообщение от Jewbacabra https://onlinephp.io/c/b177b

Result for 7.0.33:
Deprecated: Non-static method A::a1() should not be called statically in /home/user/scripts/code.php on line 6

0

Fatmarmelad, выводится этот варнинг или нет не особо важно

Такое написание объективно можно было считать "depricated" с той самой версии php когда там появились классы, статические и не статические методы.

И, на конец то, уже сам язык начинает приучать писать код правильно

Добавлено через 2 минуты
Ради интереса глянул... статические методы появились в php 5.0 ( 2004 год)

1

Какой выход в подобных ситуациях? дописывать слово static в определении метода?

0

Тут зависит от проекта.

Но в общем случае, (если иметь в виду хорошие практики) То нужно создавать отдельную версию класса DbModel в которой этот метод будет статическим. Если в этом классе есть еще какие то методы, которые не изменятся, то их выделить в более "старший" класс. И оба эти класса (DBModel и новый) унаследовать от него.

Т.е. у вас будет новое соглашение для нового метода и нового класса. При этом не будет необходимости искать по всему коду, как оно использовалось, и доводить эту инфу до остальных разработчиков на этом проекте (или использующих данный класс)

Если DBModel это класс сторонних разработчиков, а loginUser - уже ваш класс. Тут только через создание объекта
Если это проект сугубо ваш, можно принять решение о рефакторинге. Ищите все вхождения, покрываете тестами. И рефакторите, добавив static в DbModel

Но при рефакторинге тут вообще можно копнуть и глубже, судя по названиям. У вас User это и отдельная сущность, и в тоже время осуществляет поиск пользователей. Т.е. есть подозрение на смешение "уровней" архитектуры. Но это тема уже отдельная

1