Question

我使用的是Rails 3.2.11。我有一个动作（show），如果未设置模型中的某个属性，我想禁用它。现在我直接在行动中处理这个问题：

def show
    @model = Model.find(params[:id])
    if !@model.attribute
        raise ActionController::RoutingError.new('Something bad happened')
    end
end

这是可接受的还是有更好的方法来处理这种情况？我希望该行为与用户尝试访问不存在的记录时相同。

Answer 1

我更喜欢在 before_filter 中使用该逻辑，因此您的show动作将是干净的：

before_filter :check_attribute

...

def show
  # you can use straight @model here

end
...

private

def check_attribute
  @model = Model.find(params[:id])
  if !@model.attribute
    raise ActionController::RoutingError.new('Something bad happened')
  end
end

通过这种方式，您也可以将其用于其他操作。

Answer 2

是的，这是可以接受的。我个人会有条不紊地写一个单行。

raise ActionController::RoutingError.new('Something bad happened') unless @model.attribute?

处理:not_found响应的替代方法的一个很好的资源是this question。

仅在模型具有特定属性集时启用操作

2 个答案: