Если вы рассматриваете свои объекты как держатели данных, вы нарушите их инкапсуляцию, но этого не следует делать, как в реальной жизни, всегда следует запрашивать согласие.

Проблемы

  • Нарушение сокрытия информации
  • Нарушение инкапсуляции
  • Связь

Решения

  1. В сочетании с интерфейсами и поведением, никогда с данными.

Образец кода

Неправильный

Правильно

Обнаружение

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

Теги

  • Связь

Вывод

Если ваши классы загрязнены сеттерами, геттерами и общедоступными методами, у вас наверняка будут способы присоединиться к их случайной реализации.

Также известен как

  • Несоответствующая близость

связи



Code Smell 01 - Anemic Models
Ваши объекты - это набор общедоступных атрибутов без поведения. medium.com





Обоняние кода 28 - Сеттеры
Первое упражнение, которое делают юные программисты. IDE, обучающие программы и старшие разработчики продолжают учить их этому антипаттерну. medium.com



Больше информации



Объектная оргия
В компьютерном программировании объектная оргия - это термин, распространенный в сообществе программистов Perl, описывающий типичный сбой… en.wikipedia.org





Неуместная близость
Один класс использует внутренние поля и методы другого класса. refactoring.guru



Структура данных - это просто глупый язык программирования.

Билл Госпер



Великие цитаты о программной инженерии
Иногда короткая мысль может родить потрясающие идеи. medium.com



Эта статья является частью серии CodeSmell.



Как найти вонючие части вашего кода
Код плохо пахнет. Давайте посмотрим, как изменить ароматы. medium.com