это будет полезно в случае 4-х описанных бекендов

1-2 не всчет

3-4 бекенда это кейс 5%

1 2 бэкенда это по факту где то четыре условия

в деве не должно отправляться

потом, в разныэ энвироментах тоже разные бэкенды могут юзаться.

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

и оно все абсолютно бесшовно будет работать

к тому же зачем городить switch case если есть готовый гем?

ты же не опишешь все бекенды

а ты опишешь докер-образ моего приложения?

просто конфигурируй в DSL и все.

?

нету докер-образа моего аппа — не буду использовать!

нету докер-образа моего аппа — не буду использовать!

обычно образ подходит под какой-то тип приложений

обычно смс бэкенд подходит под публичные сервисы

а теперь представь что поверх моей абстракции я могу active mailer like интерфейс зарядить

с смс мейлерами

очень много работы

ну делай я не против

Госпаде, в чем работа то?

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

то это полезно

Это 90% кейсов.

а если у меня есть смс гейт и мне надо его настроить быстро то хз

почему нет то?

зато потом ты сможешь переключиться по быстрому. Тебе не надо будет думать над case when, за тебя уже типовые примеры сделаны.

Подключил и юзаешь, круто же.

И вся конфигурация однотипная, в одном месте.

ну мб

Такой же инструмент как ActiveRecord например, поверх него кучу гемов наделали.

В конце концов тебе никто этот гем не навязывает, а попав в проект где он юзается ты будешь не против (я уверен)

В свою очередь я могу сделать обертки над гемами компаний и настроить автоматическое слежение за изменениями. Breaking change? выпускаю фикс за день.

Разве что они юзают ENV'ы например

мне кажется там очень много работы по поддержке

или другой гавнокод.

Мммм нет.

Если настроить автоматическое оповещение об изменениях то проблем не должно возникнуть.

В конце концов мои косты - не твои косты. Ты просто берешь и юзаешь.

ты видно не ленивый

В худшем случае все что тебе надо будет сделать это форкнуть гем и поправить какие то вещи, если тебе реально критично.

ты видно не ленивый

Там не так много работы как тебе кажется. Написать обертку это просто. Проблемы начинаются когда гем компании заюзает антипаттерн

например будет брать токен из ENV

@pimpmyyoba у тебя гитхаб заполнен чем нибудь полезным?

например будет брать токен из ENV

это плохо? сам не юзаю, но видел

Это антипаттерн

например это сложно тестировать

ну мб

видимо, аналогия с докером донесла мысль)

@pimpmyyoba у тебя гитхаб заполнен чем нибудь полезным?

https://github.com/ifokeev?tab=repositories

например это сложно тестировать

ENV не меняется как Hash?

меняется

меняется

в чём сложность тестов?

а почему этот код зависит от глобального состояния?

а если я запущу тесты в паралели?

что у тебя из YAML Hash, что из ENV — разница-то?

сразу ~4 теста запустятся в паралели и все будут тестировать этот гем

а почему этот код зависит от глобального состояния?

а если ты меняешь в коде руби ENV — он что, в шелл уходит?

так вот, начнутся рандомные падения (могут)

а

Хм, вообще д

параллель в самом процессе? а нах, если GIL?

ENV наверное отдельный на свой поток

аргумент снимается

ENV наверное отдельный на свой поток

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

в приложениях ENV это отличная вещь

собсна шелл сам тоже прога, он просто даёт этот енв всем прогам, запускаемым из него

а вот в конкретном коде библиотеки это вредная штука

мне перед каждым вызовом придется его проставлять и потом возвращать

ERROR: S client not available

а если у меня паралельный код как в пуме?

а ты не можешь сделать общий файл с конфигурацией?

могу

точно также я могу и глобальные $переменные выставлять

только все это антипаттерны.

Класс доллжен принимать в конструкторе все что ему нужно, в том числе опции

либо иметь возможность их задать, в рубях через configure например

Таким образом этот класс легко тестировать и легко таскать с места на место, плюс все что ему нужно явно видно где находится.

Таким образом этот класс легко тестировать и легко таскать с места на место, плюс все что ему нужно явно видно где находится.

так, а откуда взять конфиг, который прокинуть классу? YAML, ENV, anything else?

так, а откуда взять конфиг, который прокинуть классу? YAML, ENV, anything else?

если у тебя класс требует тонны конфигов то принцип единой ответственности извращенно нарушен

если у тебя класс требует тонны конфигов то принцип единой ответственности извращенно нарушен

Я про способ хранения знаний, которые ему нужно прокринуть. Как лучше их хранить?

как угодно, разве это имеет значение?

если ты хранишь секреты то это одно

если другие данные то просто в коде их прописать

у себя я секреты через ENV пробрасываю в secrets.yml

дальше уже их на местах потребляю.

типа Rails.application.config.secrets[:db][:host]

таким образом в secrets.yml у меня видно какие вообще секреты есть в приложении

YAML + ENV? ? Зачем? Почему не что-то одно?

Чтобы в одном месте было видно.

Либо у тебя ENV по всему коду будет размазан

и ты можешь легко что то забыть а увидишь только по эксепшену

Кстати, по сабжу: как я уже сказал, мне не нравится функциональность (классовые методы), в том числе в ActiveSMS. Вот, помимо вкуса, ещё один аргумент: класс — это константа на весь код, как ENV. Ты у него меняешь переменную (configure), как внутри Hash. Следовательно, на один код не настроешь по-разному (для параллельных задач) без наследования. Почему не через инициализацию?

Это паттерн синглтон

я сделал так чтобы было легко юзать

Чтобы в одном месте было видно.

Зачем это? И зачем ENV вообще, если есть YAML? Почему только один? В одном месте — это можно в одной директории разбить на файлы

Ээээ

а как ты в secrets.yml засунешь пароли без ENV то?

Это паттерн синглтон

Да ну? Синглтон — это же когда у объекта добавляешь метод, нет?

а как ты в secrets.yml засунешь пароли без ENV то?

Ну тип запишу.

Синглтон это когда класс глобальный

ты не можешь получить уникальный инстанс класса, он один на все приложение