[ruby-dev:37221] Re: RSS::Maker.create(version)

At Thu, 27 Nov 2008 19:11:01 +0900,
Kouhei Sutou wrote:
> In <[e-mail]>
>   "[ruby-dev:37210] RSS::Maker.create(version)" on Thu, 27 Nov 2008 15:22:05 +0900,
>   "Akinori MUSHA" <[e-mail]> wrote:
>
> > 　RSS::Maker で、 "2.0" 等の文字列でフィードのフォーマットを渡す
> > インターフェースは事実上 RSS::Maker.make(version) {|maker| ... }
> > しかありません。(maker = RSS::Maker::XXXXX.new 相当ができない)
> >
> > 　しかし、RSSの生成が処理の主眼でない場合や、複数のRSSフィードを
> > 同時に作ったりする場合、ブロックの導入が必須のインターフェースは
> > 使いづらいです。
>
> わかりました。
>
> > 　そこで、 RSS::Maker.create(version) のようなメソッドを追加する
> > のはどうでしょうか。つまり、
>
> RSS::Maker.createではなく、RSS::Maker.newではどうでしょうか？
> 字面的にはcreateよりもnewの方が自然な気がします。モジュール
> にnewをするのや、newで返ってくるオブジェクトがレシーバのクラ
> スではないのは気持ち悪いですが。。。

　私にとっては、特に後者の気持ち悪さが致命的なレベルでした。

　create は Factory Method パターンでも new Klass() をラップする
インスタンス生成メソッド名によく使われる語なので、違和感はないと
思いました。

　
　もし気にされるのであれば、クラスを返すメソッド(以前の仕様では
RSS::Maker.maker がそれでした)をたとえば class_for() のような
名前で用意した方が柔軟かもしれません。

    rssmaker = RSS::Maker.class_for(version).new

という具合です。

--
Akinori MUSHA / http://akinori.org/