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

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]

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