28

我有一个使用acts_as_nested_set分叉的模型，并且我在模型中添加了一种方法来保存模型并将节点移动到一个事务中的集合中。此方法调用验证方法以确保移动有效，该方法返回 true 或 false。如果验证失败，我希望我的保存方法引发ActiveRecord::Rollback以回滚事务，但也向调用者返回 false。

我的模型如下所示：

class Category < ActiveRecord::Base
  acts_as_nested_set :dependent => :destroy, :scope => :journal

  def save_with_place_in_set(parent_id)
    Category.transaction do
      return false if !save_without_place_in_set

      if !validate_move parent_id
        raise ActiveRecord::Rollback and return false
      else
        place_in_nested_set parent_id
        return true
      end
    end
  end

  alias_method_chain :save, :place_in_set

  def validate_move(parent_id)
    # return true or false if the move is valid
    # ...
  end

  def place_in_nested_set(parent_id)
    # place the node in the correct place in the set
    # ...
  end
end

但是，当我在失败的情况下调用 save 时，事务会回滚但函数会返回nil：

>> c = Category.new(:name => "test") 
=> #<Category id: nil, name: "test" parent_id: nil, lft: nil, rgt: nil>
>> c.save_with_place_in_set 47
=> nil
>> c.errors.full_messages
=> ["The specified parent is invalid"]
4

3 回答 3

34

您可以将希望从函数返回的值存储在变量中，并将其返回到事务块之外。例如

  def save_with_place_in_set(parent_id)
    return_value = false
    Category.transaction do
      if !save_without_place_in_set
        return_value = false
      elsif !validate_move parent_id
        return_value = false
        raise ActiveRecord::Rollback
      else
        place_in_nested_set parent_id
        return_value = true
      end
    end
    return return_value
  end

我最初将 return_value 设置为 false，因为您可以退出该事务块的唯一其他方法是如果ActiveRecord::Rollback我相信其他方法之一引发。

于 2009-06-29T15:21:52.837 回答
15

因为ActiveRecord::Rollback异常已被处理，但不是由 重新引发ActiveRecord::Transaction，所以我可以将我的 return 移出事务块，从而在事务回滚后返回一个值。

通过一点重构：

def save_with_place_in_set(parent_id = nil)
  Category.transaction do
    return false if !save_without_place_in_set
    raise ActiveRecord::Rollback if !validate_move parent_id

    place_in_nested_set parent_id
    return true
  end

  return false
end
于 2009-06-29T15:20:17.487 回答
3

我知道这可能有点晚了，但我遇到了同样的问题并且刚刚发现，在一个事务块中你可以简单地引发一个异常并拯救那个......Rails 隐式回滚整个事务。所以不需要 ActiveRecord::Rollback。

例如：

def create
  begin
    Model.transaction do
      # using create! will cause Exception on validation errors
      record = Model.create!({name: nil})
      check_something_afterwards(record)
      return true
    end
  rescue Exception => e
    puts e.message
    return false
  end
end

def check_something_afterwards(record)
  # just for demonstration purpose
  raise Exception, "name is missing" if record.name.nil?
end

我正在使用 Rails 3.2.15 和 Ruby 1.9.3。

于 2016-06-13T13:42:49.740 回答