Я работаю с Ruby on Rails и хочу проверить две разные модели:
if (model1.valid? && model2.valid?)
...
end
Однако оператор «&&» использует оценку короткого замыкания (т.е. он оценивает «model2.valid?», Только если «model1.valid?» Истинно), что предотвращает выполнение model2.valids, если model1 недействителен.
Есть ли эквивалент «&&», который не использовал бы оценку короткого замыкания? Мне нужно оценить два выражения.
Попробуй это:
([model1, model2].map(&:valid?)).all?
Он вернет true, если оба действительны, и создаст ошибки в обоих экземплярах.
map(&:valid?) важен: [model1, model2].all?(&:valid?) - это операция короткого замыкания (он не будет собирать ошибки на model2, если есть ошибки на model1.)
- person Elliot Nelson; 19.03.2014
all?, так как он замыкается на первом ложном, поэтому вы не получаете сообщения об ошибках на каждой модели.
- person Kris; 02.07.2015
& работает нормально.
irb(main):007:0> def a
irb(main):008:1> puts "a"
irb(main):009:1> false
irb(main):010:1> end
=> nil
irb(main):011:0> def b
irb(main):012:1> puts "b"
irb(main):013:1> true
irb(main):014:1> end
=> nil
irb(main):015:0> a && b
a
=> false
irb(main):016:0> a & b
a
b
=> false
irb(main):017:0> a and b
a
=> false
Как насчет:
if [model1.valid?,model2.valid?].all?
...
end
Работает для меня.
Оцените их отдельно и сохраните результат в переменной. Затем используйте простой && между этими логическими значениями :)
Одной из ключевых концепций, которые позволяют сделать фрагмент кода более удобным для сопровождения для будущего разработчика, является его выразительность .
Рассмотрим следующие примеры:
([model1, model2].map(&:valid?)).all?
or
[model1.valid?,model2.valid?].all?
Оба они хорошо справляются со своей работой, но когда будущий разработчик столкнется с любым из них, не увидев никаких пояснительных комментариев, документов о ваших намерениях или не связавшись с вами напрямую, этот разработчик изменит их, не подозревая, что целью было избежать оценка короткого замыкания.
Еще хуже становится, когда у тебя нет тестов.
Вот почему я предлагаю ввести небольшой метод-оболочку, который сразу все прояснит.
def without_short_circuit_evaluation(*conditions)
conditions.all?
end
А потом где-нибудь в вашей кодовой базе.
if without_short_circuit_evaluation(model1.valid?, model2.valid?)
# do something
end
Вместо создания дополнительного массива с картой вы можете передать блок в all?.
[model_instance_1, model_instance_2].all? {|i| i.valid? }
