Телеграмм чат группы javarush страница 4702

Ну вынести этот сервис в несколько сервисов

19:54пожаловаться #1

C

Первый вариант - это вынести логику в сам enum (как у дядь):

enum ЕнумТип {
abstract void делайТоЧтоНужно();
ТИП1 {
какойтоБизнесСервис.делайТо();
},
ТИП2 {
какойтоБизнесСервис.делайСё();
}
}

И вызывать в МоёмТупойКлассе уже без switch:

МойТупойКласс {
void метод(ЕнумТип енумТип) {
енумТип.делайТоЧтоНужно();
}
}

Классно, только откуда в ЕнумТип'е взять КакойтоБизнесСервис? В енумы без костылей за вайрится.

19:58пожаловаться #2

VS

Первый вариант - это вынести логику в сам enum (как у дядь):

enum ЕнумТип {
abstract void делайТоЧтоНужно();
ТИП1 {
какойтоБизнесСервис.делайТо();
},
ТИП2 {
какойтоБизнесСервис.делайСё();
}
}

И вызывать в МоёмТупойКлассе уже без switch:

МойТупойКласс {
void метод(ЕнумТип енумТип) {
енумТип.делайТоЧтоНужно();
}
}

Классно, только откуда в ЕнумТип'е взять КакойтоБизнесСервис? В енумы без костылей за вайрится.

Так себе варик) енам это про перечисления, а не про логику

C

Так себе варик) енам это про перечисления, а не про логику

О том и речь, что я бы вынес логику в туда-куда-обычно-выносят-логику, только как это заинджектить в енум?

20:01пожаловаться #4

VS

Да заинжектить не проблема, вопрос то не в этом. Это просто совсем уебищный вариант с точки зрения размазывания логики

20:02пожаловаться #5

C

Ну и другой вариант, как я писал выше - создать сервис под каждый ЕнумТип и в МойТупойКласс инджектить мапу<ЕнумТип, ЭтотМойНовыйСервис> - и вот это мне не понавилось тем, что я просто создаю ЭтотМойНовыйСервис под каждый ЕнумТип.
Будет ЭтотМойНовыйСервисЕнумТип1, ЭтотМойНовыйСервисЕнумТип2 и т.д.

20:02пожаловаться #6

VS

Ну и другой вариант, как я писал выше - создать сервис под каждый ЕнумТип и в МойТупойКласс инджектить мапу<ЕнумТип, ЭтотМойНовыйСервис> - и вот это мне не понавилось тем, что я просто создаю ЭтотМойНовыйСервис под каждый ЕнумТип.
Будет ЭтотМойНовыйСервисЕнумТип1, ЭтотМойНовыйСервисЕнумТип2 и т.д.

Ну а что такого в таких названиях сервисов?

20:02пожаловаться #7

VS

Это один хрен лучше чем свич)

20:03пожаловаться #8

C

По сути дублирование логики.

20:04пожаловаться #9

C

То есть где всё про Тип1? В ЕнумТипе и ешё в ЭтотМойНовыйСервисЕнумТип1.
А как нам переименовать Тип1 в ТипZ? Переименуй элемент в ЕнумТипе и ещё переименуй ЭтотМойНовыйСервисЕнумТип1. Тоже чё-т не нравится.

Кода дофига, логика размазалась по двум классам вместо одного.

20:05пожаловаться #10

C

Соотношение типов в енуме к имплементаций сервиса один к одному, так нафига при соотношении один к одному две отдельные сущности?
А как объединить в одну, непонятно.

20:06пожаловаться #11

VS

Ну запихни логику в енум, посмотрим что на это скажут твои тимлиды))

20:08пожаловаться #12

C

сам себе тимлид

20:08пожаловаться #13

РН

Роман Нагаев in Java & Co

То есть где всё про Тип1? В ЕнумТипе и ешё в ЭтотМойНовыйСервисЕнумТип1.
А как нам переименовать Тип1 в ТипZ? Переименуй элемент в ЕнумТипе и ещё переименуй ЭтотМойНовыйСервисЕнумТип1. Тоже чё-т не нравится.

Кода дофига, логика размазалась по двум классам вместо одного.

выглядит как повод заюзать стратегию

20:08пожаловаться #14

C

Роман Нагаев

выглядит как повод заюзать стратегию

Так это оно и есть, только мне не нравится размазывание / дублирование, как я написал выше.

20:10пожаловаться #15

РН

Роман Нагаев in Java & Co

Так это оно и есть, только мне не нравится размазывание / дублирование, как я написал выше.

switch case нужно вынести из бизнес сервиса, но не в енам, а что-то типо strategyProvider, оно будет принимать значение енама и возвращать нужную стратегию

20:15пожаловаться #16

РН

Роман Нагаев in Java & Co

бизнес сервис - действие без способа реализации
стратегия - способ реализации
провайдер стратегии - выбор способа реализации для действия (не уверен что эта штука так называется)

20:18пожаловаться #17

C

Вот хз, ребят, вроде сделал, как у лучших собаководов, только
было:
в одном классе свитч на 19 строчек из 3 элементов

стало:
код вызова сократился до 6 строк, без свитча;
добавился код по составлении мапы<енум, реализация> ~ 5 строк;
добавился 1 новый интерфейс и 3 новых класса-реализаций общим количеством 120+ строк.

Стало ли это всё проще? Мне кажется, что нет.

20:38пожаловаться #18

B

Bogdan in Java & Co

Roman K

А на новом месте работы я ковыряюсь в коде, который писал большой любитель акки и рефлекшена. И его где-то 19 мегабайт.

Кода?😳😳😳😳

20:45пожаловаться #19

RK

Roman K in Java & Co

Да