@proRuby
Damir25.08.2018
16:30:15
16:30:15
def holy_war
как вы называете свои сервисы/интеракторы/команды
например классический "создание аккаунта". это CreateAccountService? или Account::CreateService? но тогда скорее всего класс Account уже есть в моделях, и получится неймспейс внутри класса - что нехорошо
end
Почему бы не в модуле accounts?
Vasiliy25.08.2018
16:30:31
16:30:31
Сначала конечно смущало, но вроде не стреляло
Tim25.08.2018
16:36:17
16:36:17
из-за top level constant lookup
GoogleTim25.08.2018
16:36:35
16:36:35
хорошо я это заметил когда тесты писал
Crypt25.08.2018
16:36:40
16:36:40
Реально создаёте сервис под одну команду?)
Tim25.08.2018
16:37:09
16:37:09
def holy_war
как вы называете свои сервисы/интеракторы/команды
например классический "создание аккаунта". это CreateAccountService? или Account::CreateService? но тогда скорее всего класс Account уже есть в моделях, и получится неймспейс внутри класса - что нехорошо
end
можно модуль accounts как тут уже сказали, класс Create с методом performнахуя в названии писать service
Vasiliy25.08.2018
16:37:34
16:37:34
С топ левел лукап было, но решалось множественным числом
Tim25.08.2018
16:37:42
16:37:42
это никакой информационной нагрузки не несёт
С топ левел лукап было, но решалось множественным числом
модуль с названием во множественном числе? да, так же делалVasiliy25.08.2018
16:38:13
16:38:13
Да да
Если обсер шел
Сервис бывает пишу чтобы разграничить что модуль какие то действия делает вне контекста моделей
Типа работа с апи или расчеты какие
GoogleVasiliy25.08.2018
16:40:07
16:40:07
Ну или там логика более 2-3 моделей взаимодействует
А хотя не, посмотрел сейчас, прост в папке services все это лежит
Gleb25.08.2018
16:52:21
16:52:21
::Account::Create и особо проблем не было никогда
Vasiliy25.08.2018
16:55:21
16:55:21
Такое стреляет иногда
Roman25.08.2018
16:55:26
16:55:26
ну короче да Accounts::Create
Vasiliy25.08.2018
16:55:34
16:55:34
Если вложенные сервисы делаешь
Crypt25.08.2018
17:21:26
17:21:26
ну короче да Accounts::Create
Мне вот любопытно почему не Services::Accounts идальше уже .create, .update, ...?Roman25.08.2018
17:22:20
17:22:20
у сервисов интерфейс с 1 публичным методом
Crypt25.08.2018
17:22:26
17:22:26
ну так и будет одноцелевой класс - управление акаунтами
Roman25.08.2018
17:22:29
17:22:29
ну так и будет одноцелевой класс - управление акаунтами
так это не работает. давай я сделаю одноцелевой класс - управление приложениеми все буду класть в 1 класс
Crypt25.08.2018
17:23:08
17:23:08
ну блин, это не одно и то же
Ilya25.08.2018
17:23:19
17:23:19
Roman25.08.2018
17:23:26
17:23:26
так я ее читал =)
не все правда применяю
сколько там у нее, 5 строк в методе
GoogleRoman25.08.2018
17:24:12
17:24:12
потому что можно делать "класс который отвечает за взаимодействие с базой" и там будет код всех моделей
Crypt25.08.2018
17:24:31
17:24:31
теперь понятно откуда это пошло, чтобы приложение разобрать, нужно 100500 файлов по 1 методу открыть
и всю эту лапшу втянуть
Roman25.08.2018
17:24:44
17:24:44
надо баланс
там не будет по 1 методу
там будет по одному публичному методу
например для создания чего-то, полюбому надо что-то там верифицировать или обработать входные параметры
это выносится в приватные
Darth25.08.2018
17:26:07
17:26:07
это как в ханами где по 1 классу на каждый экшон контроллера и все такие "сразу нахуй"
Roman25.08.2018
17:28:16
17:28:16
это как в ханами где по 1 классу на каждый экшон контроллера и все такие "сразу нахуй"
потому что рельсами вскормлены все =)Darth25.08.2018
17:28:30
17:28:30
Ни джависты, ни шарписты, ни_кто
Знаешь почему не делают?
Roman25.08.2018
17:29:04
17:29:04
а чо нам смотреть на другие языки чот не понятно
Darth25.08.2018
17:29:10
17:29:10
Потому что неудобно )
Roman25.08.2018
17:29:32
17:29:32
не ну смотри, никто не заставляет так делать
GoogleRoman25.08.2018
17:29:37
17:29:37
вообще никто никого не заставляет никак делать
если неудобно - не делай
какой самый нормальный способ сделать из 123 45 тугриков 12345? .tr(' ', '')[/\d+/] вообще нормально выглядит?
Black25.08.2018
18:25:02
18:25:02
Я программист уровня мидл. Не подскажете, как соединить два строковых переменных?
?25.08.2018
18:40:12
18:40:12
Если первое строка, то что-нибудь типа "123 45".split(" ").join("") ?
Ilya25.08.2018
18:40:40
18:40:40
мульон способов) странно слышать такие вопросы от мидла на самом деле)
?25.08.2018
18:41:01
18:41:01
Наверняка для этого есть либа
AdminERROR: S client not available
Ilya25.08.2018
18:41:19
18:41:19
)
?25.08.2018
18:41:19
18:41:19
Можно рекурсивно парсить посимвольно
Anton25.08.2018
18:42:41
18:42:41
Anton25.08.2018
18:45:06
18:45:06
он самый
Ilya25.08.2018
18:45:31
18:45:31
оо нееет
GoogleIlya25.08.2018
18:46:12
18:46:12
и тут я пойму что я в аду и мои дни повторяются)
Black25.08.2018
18:51:11
18:51:11
Но это не мастер лир
Vasya25.08.2018
18:51:37
18:51:37
Чем докажешь?
Vasiliy25.08.2018
18:55:23
18:55:23
Мне вот любопытно почему не Services::Accounts идальше уже .create, .update, ...?
В твоём случае если сервисы в апп лежат, надо путь АПП/сервисес/сервисес/аккаунтсЕсли сервисы не в апп, то надо прописывать путь автозагрузки
Damir25.08.2018
20:44:33
20:44:33
нахуя в названии писать service
Вероятно, потому что мы не хотим путать сервисы с другими классами?
А вдруг у тебя форма будет с аналогичным именем?
А вдруг какой-нибудь еще query-object с аналогичным названием?Vasya25.08.2018
21:05:44
21:05:44
Vasiliy25.08.2018
21:10:08
21:10:08
Ну как б тут немного ебаненькая архитектура рельсы в этом плане
Антон
Открыть в Telegram