Question

我收到了以下代码段

Class Artist < ActiveRecord
   attr_accessible :solo #boolean

  def change_solo!
    if self.solo == false
     self.solo = true
    else
     self.solo = false
    end
   self.save
  end

end

有更好的方式来写吗？提前致谢欢呼tabaluga

Answer 1

不要写这个方法。请改用ActiveRecord::Base#toggle!：

artist_object.toggle!(:solo)

如果尚未升级，则此方法也存在于Rails 2中。

Answer 2

啊哈！我可以进一步缩小它，呵呵：

def change_solo!
  update_attribute :solo, !solo
end

这会自动保存。

但没有测试就不会完整：

def test_change_solo_true_to_false
  Artist.create :solo => true
  assert Artist.change_solo!
  assert !Artist.solo?
end

def test_change_solo_false_to_true
  Artist.create :solo => false
  assert Artist.change_solo!
  assert Artist.solo?
end

顺便说一下，上面我使用的惯例是ActiveRecord中的任何布尔属性都可以在末尾有一个问号，使其更加不言自明。

Answer 3

这样的事情：

Class Artist < ActiveRecord 
attr_accessible :solo #boolean 
  def change_solo! 
   self.solo = !self.solo
   self.save 
  end 
end

Answer 4

仅供参考，ActiveRecord :: Base＃update_attribute不运行验证，因此您很少想使用此方法。

我怎么能写这个“ruby-ish”？

4 个答案: