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

вроде многим такая конструкция тоже не всем нравится

https://github.com/airbnb/ruby#nested-conditionals

ну здесь видишь ретёрн всё равно как бы после действия

я именно о тех случаях говорил что "вроде не всем нравистя" когда ретёрн перед действием

ну не тпиа if ? code else false end

!!condition && code

давот это тоже вроде ответ вполне

хотя

если кондишн фолс

а код тру

то мне то надо чтоб тру вернулось

а это вроде фолс вернёт тогде не?

Так тебе нужно после выполнения кода вернуть true, а не результат выполнения кода?

результат

то мне то надо чтоб тру вернулось

?

ну если код тру

типа то что сделал код вернуло тру

что тебе вообще сделать надо?

не вернуть, а именно сделать

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

!!condition && code

какой-то js-ный прикол, так в Руби делают вообще часто?

у меня ща так if STATUS_AND_FORM_KEY_TO_DISABLE.key?(status) STATUS_AND_FORM_KEY_TO_DISABLE[status].include?(form_key) else false end но я думаю что можно лучше

я часто делаю, только без !!

STATUS_AND_FORM_KEY_TO_DISABLE.key?(status) && TATUS_AND_FORM_KEY_TO_DISABLE[status].include?(form_key)

STATUS_AND_FORM_KEY_TO_DISABLE.key?(status) && TATUS_AND_FORM_KEY_TO_DISABLE[status].include?(form_key)

?

ну или совсем тупо if STATUS_AND_FORM_KEY_TO_DISABLE.key?(status) STATUS_AND_FORM_KEY_TO_DISABLE[status].include?(form_key) end

а

ну или совсем тупо if STATUS_AND_FORM_KEY_TO_DISABLE.key?(status) STATUS_AND_FORM_KEY_TO_DISABLE[status].include?(form_key) end

разве вернётся тут фолс если ключа нет в константе?

не смотри на варинат выше

STATUS_AND_FORM_KEY_TO_DISABLE.key?(status) && TATUS_AND_FORM_KEY_TO_DISABLE[status].include?(form_key)

вот это юзай

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

спасибо

STATUS_AND_FORM_KEY_TO_DISABLE.key?(status) && TATUS_AND_FORM_KEY_TO_DISABLE[status].include?(form_key)

или даже пизже вариант есть

что в статусе возвращаться должно?

массив?

массив?

да

nil

nil

+

да

STATUS_AND_FORM_KEY_TO_DISABLE.fetch(status, []).include?(form_key)

ооооо

охуенно

спасибо

обращайся

nil

в смысле? там же условный оператор, с чего там nil-то?

ой ребят ещё вопрос

аа, тьфу, это ж условие, сорян) я про следующий код и забыл вовсе)

аа, тьфу, это ж условие, сорян) я про следующий код и забыл вовсе)

кек

вот мы когда пишем о том что мол такой то метод экземпяра в таком то классе, то используем такую нотацию Class#method

бывает такое, сам часто ебусь

а есть какая нотация чтоб показать что типа метод приватный?

у меня ща так if STATUS_AND_FORM_KEY_TO_DISABLE.key?(status) STATUS_AND_FORM_KEY_TO_DISABLE[status].include?(form_key) else false end но я думаю что можно лучше

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

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

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

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

в котором и будет твоя логика

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

вот это важнее чем кодсмелл

ну или совсем тупо if STATUS_AND_FORM_KEY_TO_DISABLE.key?(status) STATUS_AND_FORM_KEY_TO_DISABLE[status].include?(form_key) end

STATUS_AND_FORM_KEY_TO_DISABLE&.[](status)&.include?(form_key)

ааааааа

STATUS_AND_FORM_KEY_TO_DISABLE&.[](status)&.include?(form_key)

не читаемо же

вполне норм

ну можно .dig(status)

не читаемо из-за названия константы

ну смотри, вместо вызова 2ух методов, ты вызываешь 4

ну смотри, вместо вызова 2ух методов, ты вызываешь 4

почему это?

2 метода

или ты имеешь в виду безопасный вызов?

STATUSES&.[](status)&.include?(form_key) STATUSES.fetch(status, []).include?(form_key)

или ты имеешь в виду безопасный вызов?

fuf

ага

ну твой вариант тоже ок

ну типо вот это совсем боль &.[](status)

я пишу через безопасный вызов мне ок

можно было сделать даже проще, вот так STATUSES[status]&.include?(form_key)

всё боль

если в названии переменной больше чем существительное(+прилагательное), то в ней слишком много логики

у меня ща так if STATUS_AND_FORM_KEY_TO_DISABLE.key?(status) STATUS_AND_FORM_KEY_TO_DISABLE[status].include?(form_key) else false end но я думаю что можно лучше

скажи че такое статус и что такое форм ки

и что их связывает и какой смысл и как используется

если в названии переменной больше чем существительное(+прилагательное), то в ней слишком много логики

Ты смотрел недавние скринкасты от DHH? он там в одном из первых переменную вообще почти целым предложением называет

Ты смотрел недавние скринкасты от DHH? он там в одном из первых переменную вообще почти целым предложением называет

DHH протух

class Status def initialize(form_keys) @form_keys = form_keys end def form_key?(form_key) @form_keys.include?(form_key) end end class Statuses def initialize(status_and_form_key_hash) @statuses = status_and_form_key_hash.map { |k, v| [k, Status.new(v)] }.to_h end # naming of this method is debatable def include?(status_name, form_key) @statuses[status_name]&.include?(form_key) end end to_delete = Statuses.new(STATUS_AND_FORM_KEY_TO_DELETE) to_delete.include?(status, form_key)

разрешаю обоссать данный код

(если сможете)

Ты смотрел недавние скринкасты от DHH? он там в одном из первых переменную вообще почти целым предложением называет

я рад за DHH

разрешаю обоссать данный код

код без задачи

@statuses[status_name]&.include?(form_key) а вот это вообще работает?

@statuses[status_name]&.include?(form_key) а вот это вообще работает?

конечно

аперсант зачем?

чому бы нет?

аперсант зачем?

ты ща серьезно? )))

что бы не писать @statuses[status_name] && @statuses[status_name].include?(form_key)

ты ща серьезно? )))

о карманный хейтре, не на долго же тебя хватило без меня.

a&.b(c) тождественно a && a.b(c)

a&.b(c) тождественно a && a.b(c)

я не знал.

я не знал.

подпишись на ruby weekly