かずひこです。
やっとコードを眺めることができました。
At Fri, 3 Feb 2006 01:36:56 +0900,
TADA Tadashi wrote:
> * default_io専用です。他のIOには影響がないように実装したつもりですが、
> 予想外の問題が出るかもしれません。default_io以外を使っている場合は
> 様子見した方がいいでしょう
(snip)
> * プラグインファイル名は05referer.rbですが、今後変更になる可能性があり
> ます。今後のアップデート時には注意してください
core/plugin/05referer.rb に DefaultIO 依存のコード入ると、他の IO での運
用がちょっと面倒になるのですが、プラグイン集の方に移動することはできませ
んか?
その他に気になったことをいくつか挙げます。
○ add_referer の IO 行き
Dbi_IO では、「この日のリファラ群を一気に記録」ではなく「このリファラを
追加」という書き込みの方が断然有利(update 一行で書ける)なのですが、
add_referer(と add_comment)を IO に持っていくことはできませんか?
たぶん、↑こうならない限り、DbiIO は新しい DefaultIO にパフォーマンスで
勝てない気がします。
○ volatile の記録
所詮 volatile なんだし、PStore の方が速くないでしょうか?
○ skel/referer.rhtml ってどこで使ってる?
このファイルって使われていますか?
○ リファラ関連の正規表現の作成
今回の変更と関係はないのですが、
* tdiary/filter/default.rb
@conf.no_referer.each do |noref|
return false if /#{noref}/i =~ ref
end
とか、
* plugin/05referer.rb
@conf.only_volatile.each do |volatile|
if /#{volatile}/i =~ ref then
only_volatile = true
break
end
end
とかの処理は、設定する正規表現の数が増えるとコストが高そうなのですが、設
定の書き込み時に Regexp.union(*no_refer_array).inspect で保存して、それ
と一発でマッチさせるのはどうでしょうか?
あー、でも設定画面で現在の設定を textarea に展開するための元ネタがいるの
か。まとめた正規表現と元ネタと両方保存しておけばいい?
# ていうか、リファラは DBI をやめて 05referer.rb をそのまま使うのが楽だ
# なぁと思いはじめていたり。。。
--
かずひこ <http://wiki.fdiary.net/kazuhiko/>
「恋とハックはアジャイルが命!」
|