Ruby：if variable vs if variable.nil？

私はRubyを初めて使い、nilとfalseを除いてすべてのオブジェクトがtrueであることを知って驚いた。0でも真です。

言語のその特性についての良いところは、あなたが書くことができるということです：

if !variable
  # do stuff when variable is nil
end

経験豊富なRuby開発者である同僚は、.nilを使用する代わりにそれを選択すべきだと主張しています。そのようです：

if variable.nil?
  # do stuff when variable is nil
end

ただし、後者は次の2つの理由により優れたオプションであると考えています。2.私の主題の意見では、たとえそれがよりコンパクトでなくても、より読みやすいです。

ここで「初心者」の間違いを犯していますか？

あなたが意味することを書いてください。あなたが書いたものを意味します。

別のテストをしてみましょう。 .vowel?

if char == 'a'
   # do stuff
end

対

if char.vowel?
   # do stuff
end

さて、これらが同じことをしないことは明らかです。しかし、あなたがそうすることだけを期待しcharている場合、aまたはbそれが機能する場合。しかし、それは次の開発者を混乱させます-charがある条件のために2番目のブロックに入り[eiou]ます。ただし、aでは、正常に機能します。

はい、それはというケースであるnilとfalseRubyで唯一falsy値です。ただし、をテストすることでをテストしnilている場合nil || false、これはあなたが言うことではありません。

それは次の開発者（たまたまあなたの自宅の住所を知っている狂ったa殺人者）がコードを読み、なぜfalseそこにも入るべきなのか疑問に思うことを意味します。

正確にあなたが意味することを伝えるコードを書いてください。

以下を考慮してください。

response = responses.to_a.first

これはresponses、またはの最初の要素を返しnilます。そして、条件文を扱うためnilとしてfalse、我々は書くことができます。

response = responses.to_a.first
if response
  # do something ('happy path')
else
  # do something else
end

どちらがより読みやすいですか：

if response.nil?
  # do something else
else
  # do something ('happy path')
end

if (object)パターンは、Rubyのコードでは非常に一般的です。また、次のようにリファクタリングにうまくつながります：

if response = responses.to_a.first
  do_something_with(response)
else
  # response is equal to nil
end

また、Rubyメソッドはnilデフォルトで返されることも覚えておいてください。そう：

def initial_response
  if @condition
    @responses.to_a.first
  else
    nil
  end
end

次のように簡略化できます。

def initial_response
  if @condition
    @responses.to_a.first
  end
end

したがって、さらにリファクタリングできます。

if response = initial_response
  do_something_with(response)
else
  # response is equal to nil
end

帰国nilは必ずしも最良のアイデアとは限らない、と主張することができますnil。なぜなら、それはあらゆる場所をチェックすることを意味するからです。しかし、それは別の魚のやかんです。オブジェクトの「存在または不在」のテストが頻繁に要求されることを考えると、オブジェクトの真実性はRubyの有用な側面ですが、この言語の初心者にとっては驚くべきことです。

Rubyは緩やかに型付けされた言語であるためif variable.nil?、自然言語のように読まれるだけでなく、プログラマーが誤ってfalse値を割り当ててifステートメントに陥るのを防ぎます。

慣用的に、unless variableが望ましいif !variable。否定if文は、スキミング時に否定を見逃しやすいため、多くの場合、コードのにおいと見なされます。

ここでの全体像は、nilこのようなチェックを行うことも少しコード臭いことです。適切なオブジェクト指向のために、nullオブジェクトパターンを使用して、この種のチェックが不要になるようにします。http://devblog.avdi.org/2011/05/30/null-objects-and-falsiness/

オブジェクトに何をすべきかを伝え、何であるかを尋ねないでください。これは、「尋ねないでください」と呼ばれることもあります。