如何讓這個Ruby方法更漂亮？

Question

我相信條件可以寫得更好：

def update
  if allow_update?
    if @company.update_attributes(params[:company])
      flash.now[:success] = "success message"
    else
      flash.now[:error] = "error message"
    end
  else
    flash.now[:error] = "error message"
  end

  render :show
end

我怎么能改寫這個？

Answer 1

使用&&運算符，可以將兩個if合並為一個：

def update
  if allow_update? && @company.update_attributes(params[:company])
    flash.now[:success] = "success message"
  else
    flash.now[:error] = "error message"
  end

  render :show
end

Answer 2

您可以使用一些字符串插值和三元運算符來縮短代碼。

def update
  allow_update? && @company.update_attributes(params[:company]) ? result = "success" : 
  result = "error"
  flash.now[result.to_sym] = "#{result} messege"
  render :show
end