Когда родительские и дочерние методы сталкиваются!  TL;DR: Избегайте использования закрытых методов в родительских классах с именами, которые могут использовать дочерние классы.  Проблемы  Нарушение принципа наименьшего удивления  Неожиданное поведение и  дефекты  Скрытые зависимости  Ограниченная расширяемость  Неоднозначность кода  Нарушение принципа открытости/закрытости   Вводящий в заблуждение дизайн  Решения  Избегайте  иерархий  Переименовать частные методы  Поддерживайте четкое наименование  Избегайте дублирования имен.  Избегайте  защищенных методов  Подклассифицировать для   , не  основных отношений использовать код повторно  Контекст  Когда вы используете одно и то же имя метода в родительских и дочерних классах, вы создаете путаницу.  Закрытый метод в родительском классе не может быть переопределен, даже если в дочернем классе существует открытый метод с таким же именем.  Это проблема большинства статических языков, которые есть в их дизайне. Это разъединение приводит к ошибкам и затрудняет поддержку кода.  Образец кода  Неправильный   <? class ParentClass { private function greet() { // This method is private return "Hello from ParentClass"; } public function callGreet() { return $this->greet(); } } class ChildClass extends ParentClass { public function greet() { // Overriding a concrete method is a code smell // Compilers SHOULD warn you return "Hello from ChildClass"; } } $child = new ChildClass(); echo $child->callGreet(); // When callGreet() is invoked on the $child object, // it executes the following: // It calls $this->greet(), // which refers to the greet() method of ParentClass // because the original method is private // and cannot be overridden or accessed from ChildClass. // The unexpected output is 'Hello from ParentClass'  Верно   <? class ParentClass { protected function greet() { // notice the 'protected qualifier' return "Hello from ParentClass"; } public function callGreet() { return $this->greet(); } } class ChildClass extends ParentClass { public function greet() { return "Hello from ChildClass"; } } $child = new ChildClass(); echo $child->callGreet(); // The output is "Hello from ChildClass" // This is the standard (and wrong) solution // Also fixed by most AIs   <? abstract class ParentClass { // Declare greet() as an abstract method // Following the template-method design pattern abstract protected function greet(); public function callGreet() { return $this->greet(); } } class ChildClass extends ParentClass { protected function greet() { return "Hello from ChildClass"; } } class OtherChild extends ParentClass { protected function greet() { return "Hello from OtherChild"; } } $child = new ChildClass(); echo $child->callGreet(); // Output: Hello from ChildClass $otherChild = new OtherChild(); echo $otherChild->callGreet(); // Output: Hello from OtherChild  Обнаружение    Полуавтоматический [x]  Вы можете обнаружить этот запах, посмотрев на частные методы в родительских классах и проверив, определяют ли дочерние классы методы с тем же именем. Вы также должны проверить родительские методы, вызывающие частные методы.  Теги  Иерархия  Уровень    Средний [x]  Почему важна биекция  Понятный и предсказуемый код должен отражать   иерархию, которую он моделирует. реальную  При использовании закрытых методов с перекрывающимися именами создается разрыв   между моделью и реализацией. биекции  Этот пробел сбивает с толку разработчиков, увеличивает количество дефектов и нарушает принципы чистого кода.  Генерация ИИ  Генераторы искусственного интеллекта часто создают этот запах, когда генерируют шаблонные родительско-детские отношения.  Они могут не проверять уровни доступа или не учитывать последствия наследования.  Обнаружение ИИ  Инструменты ИИ могут устранить этот запах с помощью четких инструкций.  Вы можете попросить ИИ проверить наличие перекрывающихся имен методов и рефакторинг иерархий.   Попробуйте!  Помните: помощники на основе искусственного интеллекта совершают много ошибок  Без надлежащих инструкций  С конкретными инструкциями   ЧатGPT   ЧатGPT   Клод   Клод   Недоумение   Недоумение   Второй пилот   Второй пилот   Близнецы   Близнецы  Заключение  При проектировании родительских и дочерних классов следует использовать методы, которые четко определяют наследование и доступность.  Избегайте частных методов, которые перекрываются с дочерними методами. Это сохраняет ваш код читаемым, расширяемым и соответствующим принципам чистого кода.  Такие языки, как   позволяют переопределять родительские методы независимо от их имен, в то время как   строго соблюдает уровни доступа. Python, Java    ведет себя аналогично   . Эти различия означают, что вам необходимо понимать конкретные правила языка, с которым вы работаете, чтобы избежать неожиданного поведения. C# Java  Отношения   https://hackernoon.com/how-to-find-the-stinky-parts-of-your-code-part-xxviii   https://hackernoon.com/how-to-find-the-stinky-parts-of-your-code-part-xii   https://hackernoon.com/how-to-find-the-stinky-parts-of-your-code-part-iii-t7h3zkv   https://hackernoon.com/how-to-find-the-stinky-parts-of-your-code-part-xxv   https://hackernoon.com/how-to-find-the-stinky-parts-of-your-code-part-viii-8mn3352   Отказ от ответственности: Code Smells — это мое   . мнение   Авторы: Фото   на  Мэтта Артца Unsplash  Наследование — это хорошо, но никогда не следует забывать, что оно вносит тесную связь.   Роберт С. Мартин   https://hackernoon.com/400- Thought-provoking-software-engineering-quotes?embedable=true   Эта статья является частью серии CodeSmell на HackerNoon.   https://hackernoon.com/how-to-find-the-stinky-parts-of-your-code-part-i-xqz3evd?embedable=true

Walkthroughs, tutorials, guides, and tips. This story will teach you how to do something new or how to do something better.

Winner of Noonies 2020 for the award Hacker Noon Contributor of the Year - REFACTORING.

2020 - Hacker Noon Contributor of the Year in REFACTORING

2022 - HackerNoon Contributor of the Year - Clean Code

2022 - HackerNoon Contributor of the Year - Debugging

2022 - HackerNoon Contributor of the Year - Github

2022 - HackerNoon Contributor of the Year - Machine Learning

2022 - HackerNoon Contributor of the Year - Quality Assurance

2022 - HackerNoon Contributor of the Year - Refactoring

2022 - HackerNoon Contributor of the Year - Software Architecture

2022 - the Greatest Storyteller

Buy My Book

Share your ideas with me!

Read My Stories

Buy Me a Coffee

Software Engineer and Book Author

Этот звук создан на языке оригинала истории!

Слишком долго; Читать

Make resilience your competitive advantage

Код запаха 286 - Метод родитель-ребенок перекрывается

About Author

КОММЕНТАРИИ

БИРКИ

ЭТА СТАТЬЯ БЫЛА ПРЕДСТАВЛЕНА В

Related Stories

Невидимые слои: почему интервью с пользователями являются незаменимым активом

Как улучшить ваш рабочий процесс в 10 раз: 17 основных приложений

Краткое введение в теорию мозга Больцмана

Telegram: мост Крипто-острова на материк

Невидимые слои: почему интервью с пользователями являются незаменимым активом

Как улучшить ваш рабочий процесс в 10 раз: 17 основных приложений

Краткое введение в теорию мозга Больцмана

Telegram: мост Крипто-острова на материк

Light-Mode

Classic

Newspaper

Dark-Mode

Neon Noir

Minty

HN StartUps