Как я могу реорганизовать этот код Ruby и Rails?

Question

Как я могу изменить этот код?

if env["rack.request.form_hash"] && env["rack.request.form_hash"]["authenticity_token"]
  env["rack.request.form_hash"]["authenticity_token"]=env["rack.request.form_hash"]["authenticity_token"].gsub("\r\n",'')
end

Answer 1

env["rack.request.form_hash"]["authenticity_token"] = env["rack.request.form_hash"]["authenticity_token"].gsub("\r\n",'') rescue nil

или с редактированием на месте

env["rack.request.form_hash"]["authenticity_token"].gsub!("\r\n",'') rescue nil

Answer 2

если у вас есть драгоценный камень и , вы можете пропустить проверку и сразу перейти к:

env["rack.request.form_hash"]["authenticity_token"].andand.gsub("\r\n",'')

Answer 3

Вместо того, чтобы использовать andand или try, я бы сделал:

if env.fetch("rack.request.form_hash", {})["authenticity_token"].to_s.gsub("\r\n",'')

или добавлю to_hash в список полезных NilClass методов (to_a, to_s,to_i и т. Д.):

class NilClass; def to_hash; {} end end

и выполните:

if env["rack.request.form_hash"].to_hash["authenticity_token"].to_s.gsub("\r\n",'')

Answer 4

Я бы порекомендовал:

if (rrf = env["rack.request.form_hash"]) && rrf_at = rrf["authenticity_token"] then rrf_at.gsub!("\r\n",'') end

или аналогичный, но более короткий:

rrf_at.gsub!("\r\n",'') if (rrf = env["rack.request.form_hash"]) && rrf_at = rrf["authenticity_token"]

Это СУХОЙ, лаконичный и не использует спасательные "хаки"; -D

Answer 5

Кажется, что хэш-индексы используются везде, возможно, вы можете начать там.

   key1 = "rack.request.form_hash"
   key2 = "authenticity_token"
   env[key1] && env[key1][key2]

Ничего умного, но значительно укорачивает линию.

Что-то вроде этого может работать:

    env[key1][key2].gsub!('\r\n','') if env.has_key?(key1) && env[key1].has_key?(key2)