-1

このコードを書くためのより良い方法はありますか?私はそれが悪いと思います.

if !params[:project].blank?
  redirect_to project_path(params[:project])
else
  redirect_to root_url
end
4

4 に答える 4

6

IMO、もう少しよく読んでください。

if params[:project].blank?
  redirect_to root_path
else
  redirect_to project_path(params[:project])
end
于 2013-02-07T01:34:00.353 に答える
1

三項演算子を使用できます。

redirect_to params[:project].blank? ? root_url : project_path(params[:project])

于 2013-02-07T01:30:47.417 に答える
0
redirect_to(
  params[:project]
  .tap{|p| break p.blank? ? root_url : project_path(p)}
)
于 2013-02-07T01:49:22.283 に答える
0

非常に読みやすい 2 ライナー IMO。

redirect_to project_path(params[:project]) unless params[:project].blank?
redirect_to root_url if params[:project].blank?
于 2013-02-07T02:18:31.193 に答える