3

Есть троллерская паста на которую я случайно наткнулся. Не стоит воспринимать код всерьёз. Посмотрел, а там объекты создаются в методах класса. Сам никогда так не делал, объекты передаю в конструктор или сеттер, создавая их заранее.

У меня появился вопрос. Даже два.

  • На сколько правильно создавать объекты в методах и какие принципы это нарушает?
  • Считается ли такое поведение "фабрикой"?
3
  • 1
    Хорошие у вас вопросы возникают. Рекомендую ко всему прочему познакомиться с парадигмой инверсии управления.
    – vp_arth
    2 мая 2018 в 16:13
  • @vp_arth то чувство, когда одна ссылка даёт понять многие моменты, и пропадает смысл отвечать... :₽ 2 мая 2018 в 16:30
  • 1
    На самом деле ссылки недостаточно для новичка. Вы интерпретируете содержимое с высоты своего опыта, которого у новичков нет. Поэтому отвечать смысл есть.
    – vp_arth
    2 мая 2018 в 16:34

1 ответ 1

2

Ответ на первый вопрос.

Это не правильно! Дело в том что, создания объектов внутри других классов, полностью рушит масштабируемость и в целом делает ваш код менее безопасным и более запутанным,а также более зависимым от внешней стреды, так например если класс из которого мы пытаемся получить объект не существует в нашей области видимости, нам потребуется переписать архитектуру и сложить области видимости, или напрямую вызывать класс из области видимости, В итоге, так делать не правильно, это путает разработчиков, и делает код менее понятным, это абсолютное нарушения одного из самых популярных и стандартных паттернов под названием "внедрение зависимостей" - DI, прочтите про него, вот.

Нет, такое поведение считается плохим не масштабируемым и зависимым, небезопасным и не очень профессиональным кодом.

UPD: также уточню, если вы и правда создавали объекты заранее, и передавил их в конструктор например, вы вероятно как раз следовали DI, соответственно ваш код на порядок лучше в этом плане, чем тот который вы представили на осмотр.

19
  • Хорошо. Я думаю так же. Ведь получается что класс использующий такой подход, становится зависим от класса, объект которого он создает и, например, при изменении имени класса создаваемого объекта, придется менять имя и в методе, который его создает. Но на счёт вашего второго высказывания есть сомнения. В порождающих шаблонах проектирования сказано, что все фабрики создают объекты внутри себя. Это имеет смысл, потому что фабрике делегируют функции создания объектов, это её прямое назначение.
    – user283622
    2 мая 2018 в 13:11
  • @Петр честно говоря, за счет того что я с телефона, я не рассматривал весь код который вы представили, но в том отрывке который я увидел, я не увидел реализации фабрики как таковой. 2 мая 2018 в 13:13
  • Класс HelloWorldStrategyFactory. Вообще у меня возник небольшой диссонанс. Когда вы описываете класс, который должен создать объект внутри себя, как вы это делаете? Я запрашиваю через параметры конструктора объект фабрики/DI и создаю объект через метод этого посредника
    – user283622
    2 мая 2018 в 13:24
  • 1
    Добавлю свои 5 копеек - DI конкретного класса - тоже зависимость. Везде, где возможно, нужно инжектить интерфейсы, а не конкретные классы.
    – vp_arth
    2 мая 2018 в 16:10
  • 1
    @Петр, ничего плохого в зависимости UserManager от User нет. Оба эти класса считаются client-space классами. По большому счёту - это его назначение - создавать вам юзеров. Всё нужно применять с умом, без фанатизма. Где-то Вам всё равно придётся написать new, просто желательно это делать в неком ServiceLocator/Injector/IoCContainer/ClassLoader, кто как называет. Его инстанс вы передаёте в конструктор класса приложения в точке входа. Ни в коем случае не делайте его глобально доступным(Singleton), я неоднократно сталкивался с необходимостью иметь одновременно несколько контейнеров.
    – vp_arth
    2 мая 2018 в 18:40

Ваш ответ

By clicking “Отправить ответ”, you agree to our terms of service and acknowledge you have read our privacy policy.