AD
спасибо! <3
Size: a a a
2020 April 09
w
ненавижу людей, которые пишут return unless ... || ...
w
хуже если после этого еще какая-то логика
return unless ... || ... && ... || ...
return unless ... || ... && ... || ...
w
и совсем хочется убивать, если в качестве кусочков логики используются не булевые выражения, а какие-нибудь мутации
return unless model.save! || ...
return unless model.save! || ...
АШ
ну так model.save! бросит эксепшн если чо
АШ
так что return всегда красивый будет, стабильный :)
DE
ну так model.save! бросит эксепшн если чо
попался!
АШ
не, я так не пишу )
АШ
Ваня подтвердит
w
ну не знаю, мне твой код не попадался
TL
А я больше всего люблю многострочные присваивания с инлайновым if на конце
w
на одном проекте у нас практиковалось писать длинные строчки, что в экран не влезают, а в конце rescue nil
w
очень интересно было их дебажить
TL
Ага, а потом это начало заменяться на var&.some&.thing&.another
VK
А в рубокопе правило настроить на длину строк? И тогда не будет такого что в экран не влезают 😅
w
Все хороши ретроспективно
TL
Именно так и появляются многострочники с if на конце)
TL
Спасает только коп на длину метода=)
2020 April 10
SK
в unless если больше одного выражения уже тяжело читать, надо менять на if
NB
в идеале впринципе всегда по возможности использовать if, например вместо unless something.present? писать if something.blank? . И да, не более одного условия для unless.