Verificando se uma variável não é nula e não é zero em ruby

Estou usando o código a seguir para verificar se uma variável não é nula e não é zero

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

Existe uma maneira melhor de fazer isso?

a menos que discount.nil? || desconto == 0
  # ...
fim

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

Cuidado com as isenções de responsabilidade usuais ... grande poder / responsabilidade, remendos de macacos levando ao lado sombrio etc.

ok, depois de 5 anos se passaram ....

if discount.try :nonzero?
  ...
end

É importante observar que isso tryé definido na gema do ActiveSupport, portanto não está disponível em ruby ​​comum.

a menos que [zero, 0]. inclua? (desconto) 
  # ...
fim

A partir do Ruby 2.3.0 em diante, você pode combinar o operador de navegação segura ( &.) com Numeric#nonzero?. &.retorna nilse a instância foi nile nonzero?- se o número foi 0:

if discount&.nonzero?
  # ...
end

Ou postfix:

do_something if discount&.nonzero?

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

Você pode fazer isso:

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

A ordem é importante aqui, porque se discountfor nil, ela não terá um zero?método. A avaliação de curto-circuito de Ruby deve impedir que tente avaliar discount.zero?, no entanto, se discounté nil.

Você pode converter sua linha vazia em valor inteiro e marcar zero?

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

if discount and discount != 0
  ..
end

atualização, será falseparadiscount = false

Você pode tirar proveito do método NilClassfornecido #to_i, que retornará zero para nilvalores:

unless discount.to_i.zero?
  # Code here
end

Se discountpode ser um número fracionário, você pode usá-lo #to_fpara impedir que o número seja arredondado para zero.

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

Se passarmos "", ele retornará falso, enquanto em branco? retorna verdadeiro. É o mesmo caso quando data = false em branco? retorna verdadeiro para nulo, falso, vazio ou uma cadeia de espaços em branco. Então é melhor usar branco? método para evitar string vazia também.

Ao lidar com um registro de banco de dados , gosto de inicializar todos os valores vazios com 0, usando o assistente de migração:

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

Você pode inicializar o desconto para 0, desde que o seu código esteja garantido para não tentar usá-lo antes de ser inicializado. Isso removeria uma verificação, suponho, não consigo pensar em mais nada.

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

Eu prefiro usar uma abordagem mais limpa:

val.to_i.zero?

val.to_iretornará a 0se val for a nil,

Depois disso, tudo o que precisamos fazer é verificar se o valor final é zero .

Solução alternativa é usar refinamentos, da seguinte forma:

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

Eu acredito que o seguinte é bom o suficiente para o código ruby. Eu não acho que poderia escrever um teste de unidade que mostre alguma diferença entre isso e o original.

if discount != 0
end