好吧，我有一个这样的参数哈希：
params = { :q => { :id_eq => "" } }
测试是否id_eq为空白的最佳方法是什么？我目前正在使用if params[:q] && params[:q][:id_eq].blank?，但似乎有很多重复的代码。
有没有更好的办法？
问问题
289 次
3 回答
4
没有比你已经在做的更好的方法了。（至少在我看来。）依赖异常是一个坏主意，其他任何东西都明显不那么清晰。
于 2013-07-26T15:49:18.297 回答
0
if params.fetch(:q, {id_eq: true})[:id_eq].blank?
OP的原始代码与其他答案之间的比较。到目前为止，我的答案是唯一给出正确结果的答案。
params = { :q => { :id_eq => "" } }
params[:q] && params[:q][:id_eq].blank? # => true (OP)
params.fetch(:q, {id_eq: true})[:id_eq].blank? # => true (my code)
params[:q].try(:[], :id_eq).blank? # => true (Marek Lipka)
params.try(:values_at, :q).first[:id_eq].blank? # => true (mbratch old)
params[:q][:id_eq].blank? rescue false # => true (Doydle)
params.fetch(:q,{})[:id_eq].blank? # => true (Stefan)
params.try(:[], :q).try(:[], :id_eq).blank? # => true (mbratch new)
params = { :q => { :id_eq => "foo" } }
params[:q] && params[:q][:id_eq].blank? # => false
params.fetch(:q, {id_eq: true})[:id_eq].blank? # => false
params[:q].try(:[], :id_eq).blank? # => false
params.try(:values_at, :q).first[:id_eq].blank? # => false
params[:q][:id_eq].blank? rescue false # => false
params.fetch(:q,{})[:id_eq].blank? # => false
params.try(:[], :q).try(:[], :id_eq).blank? # => false
params = {}
params[:q] && params[:q][:id_eq].blank? # => nil (falesy value)
params.fetch(:q, {id_eq: true})[:id_eq].blank? # => false (falesy value)
params[:q].try(:[], :id_eq).blank? # => true # Wrong
params.try(:values_at, :q).first[:id_eq].blank? # => true # Wrong
params[:q][:id_eq].blank? rescue false # => true # Wrong
params.fetch(:q,{})[:id_eq].blank? # => true # Wrong
params.try(:[], :q).try(:[], :id_eq).blank? # => true # Wrong
nil出于条件的目的，和之间的区别false并不重要，但得到true而不是nil肯定是错误的。
于 2013-07-26T14:52:50.277 回答
-1
rescue始终是一种选择：
if (params[:q][:id_eq].blank? rescue false)
于 2013-07-26T15:11:45.170 回答