Как провести рефракторинг различающихся сигнатур методов в потомках (php7)? php7 бросает warning (раньше был "незаметный" strict) сигнатура метода наследника не совпадает с родителем.PHP Warning: Declaration of ns\Captcha::destroy($memberId) should be compatible with captcha\Obj::destroy()
Код наследникаpublic function destroy($memberId){
parent::destroy();
db::query("update member set ncaptcha = 0 where id = :memberId", $memberId);
}
Ситуация не единичная, таких методов полная кошелка. Посему ломаю голову, как правильней проводить рефракторинг подобного кода. Понимаю, что каждый случай уникальный, но все же. Кто какие варианты рефракторинга подскажет?В голове вертятся варианты:
1. В потомке новый метод destroyByMember($memberId) , но тогда мы получаем в наследство "неправильный метод" destroy(). Получается его нужно переводить e родителя в protected и в потомках публиковать свои методы destroyBy... c нужной сигнатурой.
2. Прилепить в потомке отдельный сеттер setMember($memberId) и через него указывать memberId для переопреленного destroy()
Как провести рефракторинг различающихся сигнатур методов в потомках (php7)? php7 бросает warning (раньше был "незаметный" strict) сигнатура метода наследника не совпадает с родителем.PHP Warning: Declaration of ns\Captcha::destroy($memberId) should be compatible with captcha\Obj::destroy()
Код наследникаpublic function destroy($memberId){
parent::destroy();
db::query("update member set ncaptcha = 0 where id = :memberId", $memberId);
}
Ситуация не единичная, таких методов полная кошелка. Посему ломаю голову, как правильней проводить рефракторинг подобного кода. Понимаю, что каждый случай уникальный, но все же. Кто какие варианты рефракторинга подскажет?В голове вертятся варианты:
1. В потомке новый метод destroyByMember($memberId) , но тогда мы получаем в наследство "неправильный метод" destroy(). Получается его нужно переводить e родителя в protected и в потомках публиковать свои методы destroyBy... c нужной сигнатурой.
2. Прилепить в потомке отдельный сеттер setMember($memberId) и через него указывать memberId для переопреленного destroy()
Код наследникаpublic function destroy($memberId){
parent::destroy();
db::query("update member set ncaptcha = 0 where id = :memberId", $memberId);
}
Ситуация не единичная, таких методов полная кошелка. Посему ломаю голову, как правильней проводить рефракторинг подобного кода. Понимаю, что каждый случай уникальный, но все же. Кто какие варианты рефракторинга подскажет?В голове вертятся варианты:
1. В потомке новый метод destroyByMember($memberId) , но тогда мы получаем в наследство "неправильный метод" destroy(). Получается его нужно переводить e родителя в protected и в потомках публиковать свои методы destroyBy... c нужной сигнатурой.
2. Прилепить в потомке отдельный сеттер setMember($memberId) и через него указывать memberId для переопреленного destroy()
Не можешь разобраться в этой теме?
Обратись за помощью к экспертам
Гарантированные бесплатные доработки
Быстрое выполнение от 2 часов
Проверка работы на плагиат
Интересные статьи из справочника
Поможем написать учебную работу
Отвечай на вопросы, зарабатывай баллы и трать их на призы.
Подробнее
Подробнее
Использовать интерфейсы для определения сигнатур методов и обеспечения их соответствия в потомках. Создать интерфейс с необходимыми методами и заставить потомков его реализовать, обеспечивая тем самым одинаковую сигнатуру методов.
Использовать типизацию аргументов и возвращаемых значений в PHP 7. С помощью объявления типов можно убедиться, что сигнатура метода в потомке совпадает с родителем.
Переименовать метод destroy в родителе на destroyByDefault и оставить его как protected. В потомке создать новый public метод destroy с нужной сигнатурой.
Если изменение сигнатуры метода в потомке неизбежно, то необходимо пересмотреть архитектуру кода и возможно разделить функционал на более мелкие и изолированные методы, чтобы сохранить соответствие сигнатур.
Выбор оптимального варианта будет зависеть от конкретного контекста и требований проекта.