170 похожих чатов

Предыстория: два сервиса каждый со своим интерфейсом. Макаронный код. Сервисы

достаточно проблемные в плане расширения, по этому попали под рефакторинг. Сервис2 работал со своей зоной ответственности, Сервис1 работал со своей и плюс частично с Сервис2, т.е. был миксом фасада и самого себя.
Я разделила полностью эти два сервиса, ниодин из них не вызывает больше другой, и вычленила из них общий интерфейс фасад. Для общих методов, инжектю этот новый фасад вместо конкретных сервисов. Но у сервисов остались еще другие методы, которые имеют смысл для одного сервиса, и не имеют смысла для другого. И для этих методов, сервисы инжектятся напрямую через interface1 и interface2 без фасада. Это норм или нужно фигачить новые сервисы?
По зоне ответственности, все методы внутри одного сервиса расположены релевантно, т.е. нет ничего такого, что можно было бы очевидно и однозначно выбросить

2 ответов

22 просмотра

Звучит как то, что нужны три интерфейса фасада. Иначе общая часть скрыта за фасадом, а два других в сумме включают то, что должно быть скрыто, и уже скрыто за фасадом. Неувязочка выходит.

Natali-Kolosova Автор вопроса
George Polevoy
Звучит как то, что нужны три интерфейса фасада. Ин...

то, что не влазит за фасад, это не приватные методы, и они не переиспользуются методами фасада. Они просто живут в сервисе, им не нужен фасад

Похожие вопросы

Обсуждают сегодня

30500 за редактор? )
Владимир
47
а через ESC-код ?
Alexey Kulakov
29
Чёт не понял, я ж правильной функцией воспользовался чтобы вывести отладочную информацию? но что-то она не ловится
notme
18
У меня есть функция где происходит это: write_bit(buffer, 1); write_bit(buffer, 0); write_bit(buffer, 1); write_bit(buffer, 1); write_bit(buffer, 1); w...
~
13
Недавно Google Project Zero нашёл багу в SQLite с помощью LLM, о чём достаточно было шумно в определённых интернетах, которые сопровождались рассказами, что скоро всех "ибешни...
Alex Sherbakov
5
Ребят в СИ можно реализовать ООП?
Николай
33
Как передать управляющий символ в открытую через CreateProcess консоль? Собсна, есть процедура: procedure TRedirectThread.WriteData(Data: OEMString); var Written: Cardinal;...
Serjone
6
в JclConsole объявлено так: function CtrlHandler(CtrlType: DWORD): BOOL; stdcall; - где ваше объявление с stdcall? у вас на картинке нет stdcall
Karagy
8
https://github.com/erlang/otp/blob/OTP-27.1/lib/kernel/src/logger_h_common.erl#L174 https://github.com/erlang/otp/blob/OTP-27.1/lib/kernel/src/logger_olp.erl#L76 15 лет назад...
Maksim Lapshin
20
program test; {$mode delphi} procedure proc(v: int32); overload; begin end; procedure proc(v: int64); overload; begin end; var x: uint64; begin proc(x); end. Уж не знаю...
notme
6
Карта сайта