From: Yusuke ENDOH Date: 2010-01-29T19:54:19+09:00 Subject: [ruby-dev:40238] Re: [Bug:trunk] three bugs of Matrix::Scalar 遠藤です。 2010年1月29日19:24 石塚圭樹 : >>見てみたのですが、現在の実装では、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