1

私は基本的に、Rails 3.0.7 用に最新化された Railscast #32 のソリューションを実装しようとしています。

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

私が何をしようとしても、2番目の救助に到達することはないようです. また、スキーマで日時として指定されている、deadline 属性にも奇妙な点があります。

最終的な解決策はこのようなものでしたが、おそらくいくつかのリファクタリングに耐えることができます. メソッド呼び出しでの設定が気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

リファクタリングは歓迎され、回答としてマークされます。

4

1 に答える 1

1

いくつかのこと:

$ Chronic.parse 'foo'

nil を生成し、例外ではありません。

$ Time.parse 'foo'

例外ではなく、Time オブジェクトを生成します。(編集: Ruby 1.8.7)

レスキューを使用する場合、レスキューする例外のタイプを指定する必要があります

rescue SyntaxError, NameError => boom
于 2011-05-25T04:57:50.450 に答える