Проверка, если переменная не ноль и не ноль в ruby

Я использую следующий код, чтобы проверить, если переменная не ноль и не ноль

if(discount != nil && discount != 0) 
  ...
end

Есть лучший способ сделать это?

разве что discount.nil? || скидка == 0
  # ...
конец

class Object
  def nil_zero?
    self.nil? || self == 0
  end
end

# which lets you do
nil.nil_zero? # returns true
0.nil_zero?   # returns true
1.nil_zero?   # returns false
"a".nil_zero? # returns false

unless discount.nil_zero?
  # do stuff...
end

Остерегайтесь обычных заявлений об отказе от ответственности ... великой власти / ответственности, исправлений обезьян, ведущих на темную сторону и т. Д.

хорошо, через 5 лет прошло ....

if discount.try :nonzero?
  ...
end

Важно отметить, что tryон определен в геме ActiveSupport, поэтому он недоступен в обычном ruby.

если только [nil, 0] .include? (скидка) 
  # ...
конец

Начиная с версии 2.3.0, вы можете комбинировать оператор безопасной навигации ( &.) с Numeric#nonzero?. &.возвращает, nilесли экземпляр был nilи nonzero?- если число было 0:

if discount&.nonzero?
  # ...
end

Или постфикс:

do_something if discount&.nonzero?

if (discount||0) != 0
  #...
end

Вы могли бы сделать это:

if (!discount.nil? && !discount.zero?)

Здесь важен порядок, потому что если discountесть nil, то у него не будет zero?метода. Оценка Руби по короткому замыканию должна препятствовать тому, чтобы он пытался оценить discount.zero?, однако, если discountесть nil.

Вы можете преобразовать вашу пустую строку в целочисленное значение и проверить ноль ?.

"".to_i.zero? => true
nil.to_i.zero? => true

if discount and discount != 0
  ..
end

обновить, это будет falseдляdiscount = false

Вы можете воспользоваться NilClassпредоставленным #to_iметодом, который будет возвращать ноль для nilзначений:

unless discount.to_i.zero?
  # Code here
end

Если это discountмогут быть дробные числа, вы можете использовать #to_fвместо них, чтобы предотвратить округление числа до нуля.

def is_nil_and_zero(data)
     data.blank? || data == 0 
end  

Если мы передадим "", он вернет false, тогда как пусто? возвращает истину. То же самое в случае, когда data = false blank? возвращает true для nil, false, empty или строки пробела. Так что лучше использовать бланк? метод, чтобы избежать пустой строки.

При работе с записями в базе данных мне нравится инициализировать все пустые значения с 0, используя помощник по миграции:

add_column :products, :price, :integer, default: 0

Вы можете инициализировать скидку до 0, если ваш код гарантированно не будет пытаться использовать ее до инициализации. Полагаю, это сняло бы одну проверку, я больше ни о чем не думаю.

if discount.nil? || discount == 0
  [do something]
end

Я предпочитаю использовать более чистый подход:

val.to_i.zero?

val.to_iвернет, 0если val является nil,

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

Альтернативное решение - использовать уточнения, например:

module Nothingness
  refine Numeric do
    alias_method :nothing?, :zero?
  end

  refine NilClass do
    alias_method :nothing?, :nil?
  end
end

using Nothingness

if discount.nothing?
  # do something
end

Я считаю, что следующее достаточно для рубинового кода. Я не думаю, что смогу написать модульный тест, который показал бы разницу между этим и оригиналом.

if discount != 0
end