紅寶石：尼爾斯在IF語句

Question

我曾經在一個Rails應用程序下面的非常難看Ruby代碼我工作：

if params.present? 
    if params[:search].present? 
    if params[:search][:tags_name_in].present? 
     ... 
    end 
    end 
end 

所有我想要問的是是否PARAMS [：搜索] [ ：tags_name_in]已定義，但由於參數，可以而params [：搜索]，而params [：搜索] [：tags_name_in]可能全部是零，如果我用...

if params[:search][:tags_name_in].present? 

...我如果沒有參數或沒有搜索參數，會得到一個錯誤。

當然，必須有更好的方法來做到這一點......建議？

Answer 1

如果你只是想看看它的定義爲什麼不保持簡單並使用定義？功能？

if defined?(params[:search][:tags_name_in]) 

Answer 2

參數總是被定義的，所以你可以刪除它。

爲了減少代碼量，你可以做

if params[:search] && params[:search][:tags_name_in] 
    #code 
end 

如果沒有定義params[:search]，病情會短路並返回nil。

Answer 3

我通常最終會做這樣的事情：

if params.has_key?(:search) && params[:search].has_key?(:tags_name_in) 
... 

end 

但如果你在不介意對尼爾斯測試你的if語句也可以這樣做：

if params[:search] && params[:search][:tags_name_in] ... 

這不會拋出一個錯誤，因爲紅寶石short-circuits & &運算符。

Answer 4

您可以使用andand。它處理這個確切的情況：

if params[:search].andand[:tags_name_in].andand.present?

Answer 5

哈哈，如果你想成爲可怕的，猴補丁零：

class NilClass 
    def [](key) 
    nil 
    end 
end 

我會建議短路，如果像其他的答案表明，雖然。

Answer 6

你有很多的選擇，將返回params[:search][:tags_name_in]值或nil如果params[:search]是nil。

清除，但冗長：

params[:search] && params[:search][:tags_name_in] 

使用try（從active_support）：

params[:search].try(:[], :tags_name_in) 

使用救援：

params[:search][:tags_name_in] rescue nil 

使用fetch：

params.fetch(:search, {})[:tags_name_in] 

注意fetch可有時被用來避免if完全，特別是如果有什麼不指定參數是什麼時候做：

def deal_with_tags 
    MyModel.where :tags => params.fetch(:search){ return }[:tags_name_in] 
end