如果条件块很短,则使用三元组

Jay*_*266 1 ruby ruby-on-rails

我有一部分工作,但我被告知,如果条件块很短,我应该使用三元组.

module PayerContractsHelper

  def showPayerContractRate(val)
    json_val = JSON.parse(val)
    if(json_val['amount'].present?)
      "display:block"
    else
      "display:none"
    end
  end

end
Run Code Online (Sandbox Code Playgroud)

我怎么能用这段代码把它变成三元组呢?我假设代码更少,更精确.我怎样才能做到这一点?

Sim*_*tti 5

三元条件定义为:

condition ? if-true : if-false
Run Code Online (Sandbox Code Playgroud)

在您的情况下,以下代码

json_val = JSON.parse(val)
if json_val['amount'].present?
  "display:block"
else
  "display:none"
end
Run Code Online (Sandbox Code Playgroud)

会变成

JSON.parse(val)['amount'].present? ? "display:block" : "display:none"
Run Code Online (Sandbox Code Playgroud)

要么

json_val = JSON.parse(val)
json_val['amount'].present? ? "display:block" : "display:none"
Run Code Online (Sandbox Code Playgroud)

以下版本对我来说最具可读性

module PayerContractsHelper

  def showPayerContractRate(val)
    json_val = JSON.parse(val)
    "display:" + (json_val['amount'].present? ? "block" : "none")
  end

end
Run Code Online (Sandbox Code Playgroud)

另请注意,方法的Ruby命名约定是underscore_case:

module PayerContractsHelper

  def show_payer_contract_rate(val)
    json_val = JSON.parse(val)
    "display:" + (json_val['amount'].present? ? "block" : "none")
  end

end
Run Code Online (Sandbox Code Playgroud)