これを書いているもっと慣れ親しんだRubyの方法は何ですか？

Question

Rubyの人がこのように書くのを笑ってくれませんか？私が見たRubyのコードのように、特に簡潔には思われません。

Answer 1

それは私には完全に合理的です。あなたはif（parent_id = if params...）の前に課題を移動したり、3値を使うことができますが、結果が良く見えるとは思いません。

PARENT_IDは、あなたは、単に書くことができ、その行の前にゼロまたは未定義の場合：

parent_id = params[:parent_id] unless params[:parent_type] == "Order" 

Answer 2

それで本当に何も間違って-ですが、より簡潔に行うことができます。私はそれはそれは方法罰金だと思います

parent_id = if (params[:parent_type] == "Order") 
    nil 
else 
    params[:parent_id] 
end 

Answer 3

：また

parent_id = (params[:parent_type] == "Order") ? nil : params[:parent_id] 

。私はRubyの人です。そのように書くのはあなたを笑っていません。コードが何をしているのかは明らかですし、実際のコードの重複はないので、私はそれについて心配しません。

Answer 4

私が好き：

parent_id = (params[:parent_type] == "Order" ? nil : params[:parent_id])

Answer 5

もう一つのバリエーション：

parent_id = (params[:parent_type] == "Order") && params[:parent_id] 

Answer 6

我々はルビーを使用することができます三項演算子。そのような何か：あなたはそれをしなかった別の方法で簡単なルビーのif-elseを

parent_id = params[:parent_id] unless params[:parent_type] == "Order" 

または書き込み：parent_idがnil値が設定されている

parent_id = params[:parent_type] == "Order" ? nil : params[:parent_id] 

場合は、私たちがない限り構文を書くことができます

parent_id = if (params[:parent_type] == "Order") 
    nil 
else 
    params[:parent_id] 
end 

詳細については、Idiomatic Rubyのコードについては、awesome articleが見つかりました。