Size: a a a

Ruby, Rails, Hanami | dry-rb

2019 October 15

АД

Антон Дьячук in Ruby, Rails, Hanami | dry-rb
Tim
ну пусть первый кому помешает откроет тикет и скажет че за говно давайте выпилим. вот в этом коммите добавили. и заменшенит автора
нет
код проходит ревью, и мелочи решаются до принятия комита
источник

RP

Roman Patkin in Ruby, Rails, Hanami | dry-rb
Igor Morozov
а зачем валидашки в отдельном месте? они переиспользуются как-то?

я так и не нашел причин этого делать за 3.5 года
я их могу мокать и тестировать отдельно
источник

T

Tim in Ruby, Rails, Hanami | dry-rb
Антон Дьячук
нет
код проходит ревью, и мелочи решаются до принятия комита
ок, тогда в ревью надо сказать выпиливай как запилил
источник

АД

Антон Дьячук in Ruby, Rails, Hanami | dry-rb
Tim
повторяю, это мелочи
на эти мелочи ревьюверы тратят время, а время стоит деньги
источник

D

Dima in Ruby, Rails, Hanami | dry-rb
источник

АД

Антон Дьячук in Ruby, Rails, Hanami | dry-rb
Tim
ок, тогда в ревью надо сказать выпиливай как запилил
да, но было бы приятно не тратить на такие мелочи ресурсы
источник

IM

Igor Morozov in Ruby, Rails, Hanami | dry-rb
Roman Patkin
а где их еще держать? сразу в сервисе?
зависит от того, как используешь. я вот использую их для валидаций того, что пришло извне — там норм место в экшоне / каком-нибудь высокоуровневом интеракторе

когда юзаю в доменном сервисе (внешних данных нет, просто код должен соблюсти контракт) — храню в сервисе
источник

IM

Igor Morozov in Ruby, Rails, Hanami | dry-rb
Roman Patkin
я их могу мокать и тестировать отдельно
а как отдельная папка помогает?
источник

АД

Антон Дьячук in Ruby, Rails, Hanami | dry-rb
ну не про руби же разговаривать?
источник

АД

Антон Дьячук in Ruby, Rails, Hanami | dry-rb
с руби все к счастью понятно
источник

T

Tim in Ruby, Rails, Hanami | dry-rb
Антон Дьячук
на эти мелочи ревьюверы тратят время, а время стоит деньги
а на говнокод потом все тратят время, в том числе на переписывание. и куда большее чем на ревью мелочей. написать "выпиливай" занимает ровно две секунды
источник

Т

Тенпеннай in Ruby, Rails, Hanami | dry-rb
Igor Morozov
а как отдельная папка помогает?
знаешь, что вот тут у тебя валидаторы
источник

RP

Roman Patkin in Ruby, Rails, Hanami | dry-rb
Igor Morozov
зависит от того, как используешь. я вот использую их для валидаций того, что пришло извне — там норм место в экшоне / каком-нибудь высокоуровневом интеракторе

когда юзаю в доменном сервисе (внешних данных нет, просто код должен соблюсти контракт) — храню в сервисе
я валидашке инжектю в сервис, а сами правила в отдельном классе
источник

IM

Igor Morozov in Ruby, Rails, Hanami | dry-rb
Roman Patkin
я валидашке инжектю в сервис, а сами правила в отдельном классе
я тоже, в общем-то 🤔
источник

АД

Антон Дьячук in Ruby, Rails, Hanami | dry-rb
Tim
а на говнокод потом все тратят время, в том числе на переписывание. и куда большее чем на ревью мелочей. написать "выпиливай" занимает ровно две секунды
и то и то требует времени, как одно от другого отделить?
источник

IM

Igor Morozov in Ruby, Rails, Hanami | dry-rb
Тенпеннай
знаешь, что вот тут у тебя валидаторы
у меня валидаторы во всех местах, где есть граница с внешним миром 🤔
источник

T

Tim in Ruby, Rails, Hanami | dry-rb
Антон Дьячук
и то и то требует времени, как одно от другого отделить?
очевидно по количеству
источник

RP

Roman Patkin in Ruby, Rails, Hanami | dry-rb
Igor Morozov
а как отдельная папка помогает?
да просто сервисы почище
источник

k

kolas in Ruby, Rails, Hanami | dry-rb
что за срач без димы и алекса? 🤨
источник

IM

Igor Morozov in Ruby, Rails, Hanami | dry-rb
Igor Morozov
я тоже, в общем-то 🤔
только я не через контейнер это делаю, а через конструктор. это может быть важно
источник