Проверка Rails на виртуальном атрибуте, установка ошибки формы. Railscast # 32 - PullRequest
Новая
       18

Проверка Rails на виртуальном атрибуте, установка ошибки формы. Railscast # 32

1 голос
/ 25 мая 2011

Я в основном пытаюсь реализовать решение из 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')Ошибка типа.В этом случае я хочу вернуться к 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 Ответ

1 голос
/ 25 мая 2011

Несколько вещей: 

$ Chronic.parse 'foo'

возвращает ноль и не исключение. 

$ Time.parse 'foo'

дает объект Time, а не исключение.(Изменить: в Ruby 1.8.7)

Когда вы используете спасение, вам нужно указать тип исключения, которое вы спасаете 

rescue SyntaxError, NameError => boom
...