ты хочешь персистить объекты между запросами?

не думаю, что это хорошая затея

ты хочешь персистить объекты между запросами?

да, а что нет, если они чистые

в этом же прикол

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

ну... я даже не знаю, типа сервис отсылки смс, в который dependency передается апишка, а в call - юзер?

ага

ну а если апишек отсылки несколько

и какая нужна - определяется в реквест-тайм

и в тестах прокидываешь замоканую апизу

так я и в моем случае могу мокать

Foo.new(deps_and_params).call

просто я с трудом представляю объект, который будет одинаковым для всех запросов

так и не надо один объект делать для всего

ну так как же не надо, когда ты хочешь создать и закешировать его

на моем текущем проекте почти все сервисы на вход принимают исключительно request-specific ифну

Антон правильно говорит

нет понятия "правильно"

есть один подход, и есть другой

phone = Phone.new('79287239292') phone.send_sms(a,b) phone.send_sms(c,d)

нет понятия "правильно"

есть

правильно - это не через жопу

phone = Phone.new('79287239292') phone.send_sms(a,b) phone.send_sms(c,d)

если такое действительно надо - то тогда да

но это не то, что говорил Антон

SendSmsService.new('7282837', a,b).call - это через жопу

с чего бы это череж жопу?

SendSmsService.new('7282837', a,b).call SendSmsService.new('7282837', с,d).call вот так да, глупо

не спорю

ну а к чему вопрос

вопрос к тому, что если у меня sendsmsservice вызывается с конкретным номером один раз за реквест, то мне нет смысла делать new(123).send('asd')

Привет! Посоветуйте плиз норм сервис для пуш уведомлений

думаю юзать какой-нибудь onesignal

вопрос к тому, что если у меня sendsmsservice вызывается с конкретным номером один раз за реквест, то мне нет смысла делать new(123).send('asd')

ну у тебя тогда кот нереюзабельный и негибкий

завтра тебе надо дважды отправить и все тут

думаю юзать какой-нибудь onesignal

мы юзаем onesignal

а послезавтра окажется, что для этих вызовов есть еще что-то общее

например?

завтра тебе надо дважды отправить и все тут

например PhoneService.new(phone, mode).send_message(asd)

и ты все равно пойдешь править

это не тоже самое что и отправлять соазу же

мы юзаем onesignal

и как впечатления? все ок? были какие-то косяки?

это не тоже самое что и отправлять соазу же

я ж не говорю отправлять сразу же

new.call это не сразу же

я ж не говорю отправлять сразу же

ну я про SendSmsService

new.call это не сразу же

почему

то есть нет такого Foo.new(dependencies).call(params)

а если тебе нужен будет сам обхект без результата?

теряется переиспользуемость

ну не делай call

ну и зачем такие неявные вещи тогда нужны

какие неявные?

это конструктор + вызов

ты можешь не писать в 1 строку все

ну что можно на класс тут вызывать а тут можно на обхект вызывать

ну не делай call

и какая разница? я тем же обьектом не смогу кудато в другое место отправить смс

вот у меня например x = Foo.new и как тепер на этот x вызвать call

и какая разница? я тем же обьектом не смогу кудато в другое место отправить смс

ну я и говорю, что обычно мне такое не надо, но если надо, то я меняю сервис. ничего такого в этом нет

вот у меня например x = Foo.new и как тепер на этот x вызвать call

x.call

но у тебя call это .self метод

он и self и не self

ну я и говорю, что обычно мне такое не надо, но если надо, то я меняю сервис. ничего такого в этом нет

почему не сделать сразу нормально?

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

почему не сделать сразу нормально?

потому что непонятно, что будет нужно в конструктор, а что в вызов

делать наперед не имея требований - тоже неправильно

и как впечатления? все ок? были какие-то косяки?

он порой падает) два раза за 3 месяца было такое. а так в целом ок. только там еще гемор со сборками у мобильщиков. они должны знать где тест а где прод окружение. иначе onesignal будет выдавать mismatched environment и пуши приходить не будут. там либо руками править окружение плееров, либо в АПИ у себя выносить чтоб условный админ мог править. Апишка там суперлегкая. Есть гем для работы с этим. там еще минус что он к телефону привязывает юзера у себя в базе. то есть ты на дев и прод окружении будешь иметь 2 юзера с одним и тем же телефоном. мне вот менеджер задачу кинул что у нее дублируются сообщения) она зарегана и там и там.

YAGNI

потому что непонятно, что будет нужно в конструктор, а что в вызов

не вижу причин почему

ну так ты и делаешь наперёд

ERROR: S client not available

типа можно вполне обойтись без селф метода и жить себе

неа, я делаю интерфейс

не вижу причин почему

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

и как впечатления? все ок? были какие-то косяки?

это мелочи на самом деле. просто это неявно у них

ну опять же смотри уже у двух разрабов возникли вопросы "почему" "зачем" значит какой то не интуитивно понятнй интерфейс выходит

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

самый прямой вариант это передать в конструктор всю инфу

у тебя нет требования "надо будет отправлять несколько смс на один и тот же номер дважды"

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

но теряется тогда переиспользуемость

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

ты все равно никогда с первого раза не напрограммиш на все изменения бизнеса

а какой тогда смысл отдельном классе

поэтому зачем это делать?

отдельный класс для инкапсуляции логики и отделения

яж не говорю всё наперёд

проблема в том что у тебя класс слабый

не вижу как это касается текущего обсуждения

ну так а ещё отдельные классы для переиспользоания

это где такое написано?

это мелочи на самом деле. просто это неявно у них

сенкс за фидбек)

ну и я могу переиспользовать конечно

для разных телефонов и месседжеей

нужно писать код под текущую задачу, а не под ту, которая может возникнуть

возникнет - переделаешь

ну понятно, что надо во всем без фанатизма

если это что-то очевидное, то да, можно и сразу

я короче костноязычный

но таки считаю что делать self.call не стоит

сами придумайте почему

типа def self.call self.new.call end

ну у каждого есть свое мнение =)

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