Rails在虚拟属性上验证,设置表单错误。 Railscast#32

时间:2011-05-25 03:56:34

标签: ruby-on-rails validation ruby-1.9.2 railscasts virtual-attribute

我基本上是尝试从Railscast#32实现解决方案,为Rails 3.0.7进行了现代化改进

http://railscasts.com/episodes/32-time-in-text-field

class Task < ActiveRecord::Base

attr_accessible :description, :complete, :deadline

validate :deadline_string_no_errors


def deadline_string
  self.deadline.to_s
end

def deadline_string=(deadline_str)
  unless deadline_str.blank?
    begin
      self.deadline = Chronic.parse(deadline_str)
    rescue
      self.deadline = Time.parse(deadline_str)
    rescue
      @deadline_invalid = true
    end
  end
end

def deadline_string_no_errors
  errors.add(:deadline_string, "Is Invalid") if @deadline_invalid
end

我希望通过控制台或(预期)表单条目将字符串(例如'foobar')放入#deadline_string =时设置验证错误。

我特别关注的是更严格的时间格式,Chronic.parse('2011-05-25 08:19:00 UTC')会引发TypeError。在这种情况下,我想回退到Time.parse,它可以理解这种格式。

此备用表单也不适用于deadline_string="foobar"作为初始调用。

# snip
def deadline_string=(deadline_str)
  unless deadline_str.blank?
    self.deadline = ( parse_time(deadline_str, Chronic) || parse_time(deadline_str, Time) )
  end
end

private

def parse_time(string, parser)
  begin
    parser.parse(string)
  rescue
    @deadline_invalid = true
  end
end

def deadline_string_no_errors
  #snip
end

无论我尝试什么,似乎都没有达到第二次救援。截止时间属性也有一些奇怪之处,它在模式中指定为日期时间。

最终的解决方案是这样的,但它可能会受到一些重构。我担心在方法调用中设置self.deadline,但不确定为什么我应该这样做。

Class Task < ActiveRecord::Base
  attr_accessible: description, :complete, :deadline

  validate :deadline_string_no_errors

  def deadline_string
    self.deadline.to_s
  end

  def deadline_string=(deadline_str)
    unless deadline_str.blank?
      self.deadline = ( parse_time(deadline_str, Chronic) || parse_time(deadline_str, Time) )
      @deadline_invalid = true if self.deadline.nil?
    end
  end

  private

  def parse_time(string, parser)
    begin
      parser.parse(string)
    rescue
    end
  end

  def deadline_string_no_errors
    errors.add(:deadline_string, "Is Invalid") if @deadline_invalid
  end

欢迎重构,并将其标记为答案。

1 个答案:

答案 0 :(得分:1)

一些事情:

$ Chronic.parse 'foo'

产生零,而不是例外。

$ Time.parse 'foo'

产生一个Time对象,而不是异常。 (编辑:在Ruby 1.8.7中)

当您使用救援时,您需要指定您正在救援的例外类型

rescue SyntaxError, NameError => boom