Можно ли обращаться к фасаду из класса, который этот фасад вызывает?

Question

[evomed]

Есть фасад Payments. Он связан с реализацией PaymentManage который вызывает различные классы платежей. Например CreatePayment и CheckStatus. Эти классы связаны с абстрактным PaymentService и у них внутренние связи налажены через parent. Но есть еще PriceConvert, который также вызывается в PaymentManage. И вдруг в CheckStatus мне понадобился PriceConvert. Я могу вызывать его через фасад Payments::priceConvert? Или мне нужно создать объект PriceConvert и вызвать нужный метод? Просто с одной стороны фасад Payments лежит ниже чем эти классы - это просто точка доступа для контроллеров итд. И я не могу обращаться с высокого уровня на низкий. С другой стороны создавать объекты, когда есть фасад тоже не хочется. По эстетическим соображениям и пока не понимаю что может произойти т.к. все равно юзаю фасад и если что подменяться будет как раз реализация, а не фасад. Как будет правильно?

Answer 1

[Василий Банников]

Весь вопрос звучит как какой-то оверинжиниринг.
0. Без диаграмм вообще поток мыслей сложно понять.
1. Фасад - это не способ легализовать божественный класс.
2. Циклические зависимости - это не всегда смертельно плохо, но всегда очень подозрительно.
3. Не нужно накидывать все паттерны, какие ты знаешь, если только что прочитал какую-то умную книжку или статью. Иногда лучше всё-таки сделать по принципу KISS и YAGNI.

Или мне нужно создать объект PriceConvert и вызвать нужный метод?

Зачем на каждую мелкую операцию делать отдельный класс и создавать постоянно новые объекты?

Просто с одной стороны фасад Payments лежит ниже чем эти классы - это просто точка доступа для контроллеров итд.

Фасад же типа по определению должен быть выше всех, как наиболее высокий уровень абстракции. С чего это вдруг он оказался ниже всех? И что у тебя подразумевается под контроллерами? Обычно контроллерами в вебе называют обработчики http-запросов. Тогда контроллеры должны зависеть от фасада, а не наоборот.

и у них внутренние связи налажены через parent

.
Что это вообще значит?

Payments::priceConvert

Почему вообще платежи отвечают за конвертацию курсов валют? Или что делает priceConvert?

Comments

[evomed]

Пример набросал во время вопроса, чтобы было понятно. Фасад служит только доступом к реализации через контейнер https://laravel.com/docs/9.x/container логика лежит выше. Вопрос можно ли вызывать фасад из логики (которую он вызывает/запускает) или нет. Я так уже делаю, но есть ощущение, что это не правильно.

[Василий Банников]

evomed, Всё ещё сужу исключительно по словам (и тому как я себе в голове картину нарисовал), тк не увидел ни диаграммы, ни кода.

Фасад служит только доступом к реализации через контейнер

Звучит ещё хуже, если честно)
Чем не устроил нормальный dependency injection?
Какую тогда вообще пользу даёт фасад? Почему мы потребителю напрямую к этому контейнеру не обратиться?

Вопрос можно ли вызывать фасад из логики (которую он вызывает/запускает) или нет.

Нет, тк в текущей реализации сразу несколько проблем
1. Циклическая зависимость (которая потенциально может к рекурсии привести и когда-нибудь приведёт)
2. Гвоздями прибиваешь свой домен к инфраструктуре

Чем не устроила "тупая" архитектура?
Контроллер (обработчик http-запросов) -> сервис (доменный, который отвечает за бизнес-правила) -> базовая инфраструктура (запросы к сторонним сервисам, очереди, базы данных)
И связать их все при помощи dependency injection.

[evomed]

Василий Банников, как бы не звучало, но фасады Ларавел именно так и работают) Здесь об использовании и преимуществах https://laravel.com/docs/11.x/facades. По идее да, правильней замкнуть связи внутри сервисов. Завтра посмотрю, как у Ларавел это решается.

[Василий Банников]

evomed, наверное стоило в теги ларавел добавить, раз речь не о фасадах как патерне, а о фасадах из ларавела).
Впрочем даже по документации, которую вы скинули - не выглядит так, что они под ваш кейс подходят.

Может всё-таки будет конкретный пример?

[evomed]

Василий Банников, почему не подходят? Фасады созданы для работы с сервисами. Пример есть в вопросе.

[evomed]

В общем, посмотрел, как у Ларавел и выпилил фасады из сервисов, которые этими фасадами вызываются, что в принципе и так логично)

[Василий Банников]

evomed, не вижу пример в вопросе

[evomed]

Василий Банников, с этим не могу помочь

Answer 2

[evomed]

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