[ruby-dev:37217] Re: [ruby-cvs:27586] Ruby:r20368 (trunk): * ext/bigdecimal/bigdecimal.c (BigDecimal_div2): should return

まつもと ゆきひろです

In message "Re: [ruby-dev:37214] Re: [ruby-cvs:27586] Ruby:r20368 (trunk): * ext/bigdecimal/bigdecimal.c (BigDecimal_div2): should return"
    on Thu, 27 Nov 2008 23:59:44 +0900, Yukihiro Matsumoto <[e-mail]> writes:

||一方で、もしこの変更がありならば、floor、ceil、truncate、round あたりの
||仕様も見直せる可能性があると思います。
|
|ああ、そうですね。これらもBigDecimalではなく、整数を返した方
|がうれしいでしょうねえ。

と思ってソースを読んでみたのですが、これってオプショナルな引
数を受け付けて、小数点以下の桁も指定できるんですね。そうなる
と

  * このまま。引数によって戻り値の型が変わるのは気持ち悪い
  * 引数がない時はInteger。ある時はBigDecimal

という選択肢がありますね。どちらでも構わないと言えば構わない
のですが。個人的には前者かな。やや統一性に欠けますが。

||それと BigDecimal と Float 共通ですが、ひとつ目の要素がInteger だったり、
||Float だったりしますね。
||
||main@190-20081127> 1.0.divmod(1).collect{|x| [x,x.class]}
||#=> [[1, Fixnum], [0.0, Float]]
||main@190-20081127> 1.0.divmod(0).collect{|x| [x,x.class]}
||#=> [[NaN, Float], [NaN, Float]]
||
||main@190-20081127> BigDecimal('1').divmod(1).collect{|x| [x,x.class]}
||#=> [[#<BigDecimal:843b900,'0.1E1',4(12)>, BigDecimal], [#<BigDecimal:843b964,'0.0',4(12)>, BigDecimal]]
||main@190-20081127> BigDecimal('1').divmod(0).collect{|x| [x,x.class]}
||#=> [[#<BigDecimal:8413720,'NaN',4(4)>, BigDecimal], [#<BigDecimal:8413748,'NaN',4(4)>, BigDecimal]]
||
||divmod = div + modulo と考えると、他の言語で divmod やそれに相当するも
||の、modulo などを調べると (CL、Scheme、Smalltalk、Python など)、確認し
||たところではどれも零割でエラーになっているようですが、ruby でもエラーに
||していいのかもしれません。
|
|なるほど。

こちらは修正してみました。1.9.1には入れるべきではないと思い
ますが。

                                まつもと ゆきひろ /:|)