[ruby-dev:40238] Re: [Bug:trunk] three bugs of Matrix::Scalar

遠藤です。

2010年1月29日19:24 石塚圭樹 <[e-mail]>:
>>見てみたのですが、現在の実装では、compare_by_row_vectors は比較で
>>使われていない (というか、どこでも使われていない) 気がします。
>>なので、比較では compare_by_row_vectors のテストにはならなそうです。
>
> ん?
>
>  def test_compare_by_row_vectors
>    assert(@m1.compare_by_row_vectors(Matrix[[1, 2, 3], [4, 5, 6]]))
>  end
>
> と言う意味で言ったんですが?

ああーなるほど。compare_by_row_vectors は Matrix#== の比較の内部実装として
存在する、という意味だと理解してました。そうしておきます。

いずれにしても、vectors という名前にはあわないですね。



>>あと、このチケットと関係ないですが、[ruby-core:27901] を見てもらえると
>>助かります > いしつかさん
>
> これねぇ...
>
>  Integer#[], Integer#size
>
> の名前を変えた方がよいんじゃない? って言いたいところですが...

Integer#[] はたまに便利ですが #size については完全に同意します。
無くてもいい、というか無いほうがいいとすら思います。


> どう見ても他のクラスの[], sizeと振る舞いが違いますしねぇ, duck type 的
> にもイマイチな気がします.
>
> それはともかく, NArrayのNVector などのVectorの別実装が実際にありますし,
> Vector, Arrayに限定するのは, やはり duck type 的 にイマイチでしょう.

別に「duck typing 的にそういうもの」といって reject してもらっても構わない
と思います。すでに rubyspec に載ってるので、見解を示してもらえると修正する
根拠になるので助かります。

-- 
Yusuke ENDOH <[e-mail]>