编写此 if then else 子句的更好方法是什么？

Question

我有以下代码。我仍然是 Ruby on Rails 的新手。正如你所看到的，我重复了四次。

我尝试了这样的事情：

if @property.nil? || @property.status_id == 144 || (@property.status_id <= 16 && current_user.nil?) || (@property.status_id <= 16 && current_user.id != @property.user_id)

但如果 @property 为零，它会给我很多错误。因为@property.status_id不能被调用，因为@property为零。

无论如何，我认为经验丰富的 Ruby on Rails 编码人员会明白这一点。

  def show
    @property = Property.find(params[:id]) rescue nil
    if @property.nil?
      flash[:error]=t("The_property_was_not_found")
      redirect_to root_path
      return
    end
    if @property.status_id == 144
      flash[:error]=t("The_property_was_not_found")
      redirect_to root_path
      return
    end
    if @property.status_id <= 16 && current_user.nil?
      flash[:error]=t("The_property_was_not_found")
      redirect_to root_path
      return
    end
    if @property.status_id <= 16 && current_user.id != @property.user_id
      flash[:error]=t("The_property_was_not_found")
      redirect_to root_path
      return
    end
    @images = Image.find(:all, :conditions =>{:property_id => params[:id]})
  end

根

Answer 1

def show
    @property = Property.find(params[:id]) rescue nil
    if @property.nil? || @property.status_id == 144 || (@property.status_id <= 16 && (current_user.nil? || current_user.id != @property.user_id))
      flash[:error]=t("The_property_was_not_found")
      redirect_to root_path
    else
      @images = Image.find(:all, :conditions =>{:property_id => params[:id]})
    end
  end

我不熟悉 Ruby 语法，所以这可能无法真正编译，但你明白了。

Answer 2

这真的是确切的代码吗？ || 短路 和 nil 值应该不是问题。

@property=nil
if @property.nil? || @property.status_id == 144
   puts @property.class.to_s
end

输出
NilClass

Answer 3

我认为您应该通过将“可以显示”逻辑定义为一个简单的辅助方法来解决这个问题，您可以调用它来做出决定，而不是用最终使相同操作发生的各种分支来混乱您的显示方法。

def can_show_property?(property)
  return false unless (property)

  return false if (property.status_id == 144 or property.status_id > 16)

  return false unless (current_user && current_user.id == property.user_id)

  true
end

def show
  @property = Property.find(params[:id]) rescue nil

  unless (can_show_property?(@property))
    flash[:error]=t("The_property_was_not_found")
    redirect_to root_path
    return
  end

  @images = Image.find(:all, :conditions =>{ :property_id => params[:id] })
end

在代码中包含像 144 这样的“神奇”数字确实会导致人们询问为什么没有为它们分配常量。当明确标记 MyApp::PROPERTY_LOCKED 时，通常更容易理解。

Answer 4

你可以试试这个：

def show
begin 
  @property = Property.find(params[:id])
  if [144,16,15,14,13,12,11,10,9,8,7,6,5,4,3,2,1,0].include?(@property.status_id)
    flash[:error]=t("The_property_was_not_found")
    if current_user && (current_user.id != @property.user_id)
      redirect_to myimmonatie_path 
    else
      redirect_to root_path 
    end
rescue
  flash[:error]=t("The_property_was_not_found")
  redirect_to root_path
end
@images = Image.find(:all, :conditions =>{:property_id => params[:id]})

结束