tDiary

ただただしです。

2009/9/1 Kazuhiko <kazuhiko@...>:
> (*) キモいevalの例 navi_user.rb:
> eval( <<MODIFY_CLASS, TOPLEVEL_BINDING )
> module TDiary
>  class TDiaryMonthWithoutFilter < TDiaryMonth
>    def referer_filter(referer); end
>  end
> end
> MODIFY_CLASS
>
> たぶん近いうちにRuby 1.9側で修正してくれる気がしますが、それはそれとし
> て、ここで、ただのmodule定義ではなく、キモいevalをしている理由って何でし
> たっけ？

(例によっていい加減な)記憶の糸をたどると、かつてのRubyではこうしないと
トップレベルのモジュールを深いところで定義できなかったんだと思う。うそ
かも知れないけど。いまならeval抜きでこうできますね:

module ::TDiary
   〜
end

試しにmakerss.rbの実装を上記のように変更してみました。ruby 1.8.7-p72以降
と1.9.2では問題なく動作してるようです。

> また、何らかの理由があるとしても、オフィシャルプラグインで必要とされてい
> る機能で、特に害をあたえなさそうなものは、本体（tdiary.rb）側にとりこん
> でしまいませんか？

少なくともuser_navi.rbはその機能全体を取り込むのは規定路線なのでなんとか
したいです。その他のプラグインに関しては吟味が必要かも。update時にしか
動かないものは入れなくてもいいだろうし(makerss.rbはこっち)。

> というわけで、以下、キモいevalのあるプラグイン一覧。
> counter.rb
> disp_referrer.rb
> makelirs.rb
> makerss.rb
> navi_user.rb
> pb-show.rb
> recent_list.rb
> tb-show.rb
> title_tag.rb

「自分はこれのメンテナだ」という自覚のある人、ちょっと確認してみてもら
えますか?

-- 
ただただし - ツッコミは、短く鋭く愛を込めて♪
http://tdtds.jp/
@tdtds

TADA Tadashi wrote:
>> (*) キモいevalの例 navi_user.rb:
>> eval( <<MODIFY_CLASS, TOPLEVEL_BINDING )
>> module TDiary
>>  class TDiaryMonthWithoutFilter < TDiaryMonth
>>    def referer_filter(referer); end
>>  end
>> end
>> MODIFY_CLASS
>>
>> たぶん近いうちにRuby 1.9側で修正してくれる気がしますが、それはそれとし
>> て、ここで、ただのmodule定義ではなく、キモいevalをしている理由って何でし
>> たっけ？
> 
> (例によっていい加減な)記憶の糸をたどると、かつてのRubyではこうしないと
> トップレベルのモジュールを深いところで定義できなかったんだと思う。うそ
> かも知れないけど。いまならeval抜きでこうできますね:
> 
> module ::TDiary
>    〜
> end

nadoka方面で聞いてみたところ、「1.7 featureです」とのことでしたので、
Ruby 1.8.2 以上対応の今となっては、過去の遺物のようですね。
http://www.ruby-lang.org/ja/man/html/ruby_1_8_feature.html#a2003-02-20

>> また、何らかの理由があるとしても、オフィシャルプラグインで必要とされてい
>> る機能で、特に害をあたえなさそうなものは、本体（tdiary.rb）側にとりこん
>> でしまいませんか？
> 
> 少なくともuser_navi.rbはその機能全体を取り込むのは規定路線なのでなんとか
> したいです。その他のプラグインに関しては吟味が必要かも。update時にしか
> 動かないものは入れなくてもいいだろうし(makerss.rbはこっち)。

例えば、トラックバック関係は、tdiary.rbの中にトラックバック用のコードが
既にあるので、そういう機能のプラグインについては、クラスをいじるような
コードはtdiary.rb側に移動していいんじゃないかなぁと。

# 本当はプラグインを毎回evalしなくてもいいようにしたいところだけど、どう
やったら実現できるのかはまだ考えていません。

>> というわけで、以下、キモいevalのあるプラグイン一覧。
>> counter.rb
>> disp_referrer.rb
>> makelirs.rb
>> makerss.rb
>> navi_user.rb
>> pb-show.rb
>> recent_list.rb
>> tb-show.rb
>> title_tag.rb
> 
> 「自分はこれのメンテナだ」という自覚のある人、ちょっと確認してみてもら
> えますか?

とりあえずは、キモいevalをほどくところからやってみます。

かずひこ

反応が究極に遅いzundaです。むーん。

--- TADA Tadashi <t@...> wrote:

> 2009/9/1 Kazuhiko <kazuhiko@...>:
> > (*) キモいevalの例 navi_user.rb:
> > eval( <<MODIFY_CLASS, TOPLEVEL_BINDING )
> > module TDiary
> >  class TDiaryMonthWithoutFilter < TDiaryMonth
> >    def referer_filter(referer); end
> >  end
> > end
> > MODIFY_CLASS

> 「自分はこれのメンテナだ」という自覚のある人、ちょっと確認してみてもら
> えますか?

> > disp_referrer.rb

は僕だと思うのですが、キモいevalは見つけられませんでした T_T
(disp_referer.rbではリンク元置換リストにRubyコードを書けるように
していて、それを解釈するために必要なevalはあります。それは見逃し
てもらえるとうれしい。)

ここですよん!とご存じの方は教えてもらえるとうれしいです。



--------------------------------------
Yahoo! JAPAN - Internet Security for teenagers and parents.
http://pr.mail.yahoo.co.jp/security/

ただただしです。

2009/10/1 zunda <zunda616e@...>:
>> > disp_referrer.rb
>
> は僕だと思うのですが、キモいevalは見つけられませんでした T_T
> (disp_referer.rbではリンク元置換リストにRubyコードを書けるように
> していて、それを解釈するために必要なevalはあります。それは見逃し
> てもらえるとうれしい。)

というか、このリストは単にgrep -l *.rbしただけのもののような
気がする(笑)。

-- 
ただただし - ツッコミは、短く鋭く愛を込めて♪
http://tdtds.jp/
@tdtds