Ruby: Nils en una instrucción IF

Question

Tengo el siguiente código de rubí muy feo en una aplicación de rieles que estoy trabajando:

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

Todo lo que estoy tratando de hacer es si params [:] [búsqueda : tags_name_in] se ha definido, sino porque params, y params [:] de búsqueda, y params [:] [búsqueda: tags_name_in] podría ser nulo todo, si uso ...

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

... I obtener un error si no hay params o no hay parámetros de búsqueda.

Seguramente debe haber una mejor manera de hacer esto ... sugerencias?

Answer 1

si solo está tratando de ver si está definido ¿por qué no mantenerlo simple y usar el definido? ¿función?

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

Answer 2

Los parametros siempre se definirán, por lo que puede eliminarlos.

Para reducir la cantidad de código que puede hacer

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

Si params[:search] no está definido, la condición de cortocircuito y volver nil.

Answer 3

por lo general termino haciendo algo como esto:

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

end 

Aunque si no te importa las pruebas contra Nils en su sentencia if también se puede hacer esto:

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

esto no será arrojar un error porque ruby ​​short-circuits el operador & &.

Answer 4

Puede usar andand para esto. Se ocupa de esta situación exacta:

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

Answer 5

Jaja, si quieres ser terrible y monkeypatch nula:

class NilClass 
    def [](key) 
    nil 
    end 
end 

Yo recomendaría una cortocircuitado si como las otras respuestas sugieren, sin embargo.

Answer 6

usted tiene muchas opciones que devolverá el valor de params[:search][:tags_name_in] o nilparams[:search] si es nil.

Claro, pero muy largo:

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

Usando try (de active_support):

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

Uso de rescate:

params[:search][:tags_name_in] rescue nil 

Usando fetch:

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

Tenga en cuenta que fetch en algún momento se puede utilizar para evitar la if por completo, especialmente si no hay nada que hacer cuando no se especifica el parámetro:

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