[ruby-dev:32193] Re: Date#+に大きな数字を与えるとおかしな日付に

山本です。

> date には速度の課題があって、一番の理由は、rational の速度なのですが、
> rational の速度を改善するか、なんらかの形で rational を排除するか考え
> ていました。僕としては、rational の速度の改善に期待していましたが、難
> しいようです。

なるほど、確かに手元の簡単なベンチマークでもrationalにしたものでは
3倍くらい遅くなりました。ずいぶんな違いが出るようですね。

でも個人的には最初に報告したエラーにもならず
おかしな日付になる現象はあまり良くないと思います。
僕の好みとしては遅くとも正しい結果が出るほうが好みです。
一意見としてお聞きください。

すごいいまさらですが、mathnなくても大丈夫でしたのでパッチのせておきます。

Index: date.rb
===================================================================
--- date.rb     (revision 13846)
+++ date.rb     (working copy)
@@ -385,10 +385,10 @@
        y -= 1
        m += 12
       end
-      a = (y / 100.0).floor
-      b = 2 - a + (a / 4.0).floor
-      jd = (365.25 * (y + 4716)).floor +
-       (30.6001 * (m + 1)).floor +
+      a = Rational(y, 100).to_i
+      b = 2 - a + Rational(a, 4).to_i
+      jd = (Rational(36525, 100) * (y + 4716)).to_i +
+       (Rational(306001, 10000) * (m + 1)).to_i +
        d + b - 1524
       if jd < sg
        jd -= b
@@ -406,14 +406,14 @@
       if jd < sg
        a = jd
       else
-       x = ((jd - 1867216.25) / 36524.25).floor
-       a = jd + 1 + x - (x / 4.0).floor
+       x = ((jd - Rational(186721625, 100)) / Rational(3652425, 100)).to_i
+       a = jd + 1 + x - Rational(x, 4).to_i
       end
       b = a + 1524
-      c = ((b - 122.1) / 365.25).floor
-      d = (365.25 * c).floor
-      e = ((b - d) / 30.6001).floor
-      dom = b - d - (30.6001 * e).floor
+      c = ((b - Rational(1221, 10)) / Rational(36525, 100)).to_i
+      d = (Rational(36525, 100) * c).to_i
+      e = ((b - d) / Rational(306001, 10000)).to_i
+      dom = b - d - (Rational(306001, 10000) * e).to_i
       if e <= 13
        m = e - 1
        y = c - 4716