オートリンクを勝手に添削してみた

http://blog.mtg.ne.jp/comprules/others/20081202143200.html
↑にあったカード名オートリンク用のJSを勝手に添削してみた。

http://blog.mtg.ne.jp/comprules/gatherlink.js
このjsは素晴らしいんだけど、body全部のモードしかないのがいただけない。

var AutoCardComplete;
var AutoCardComplete.load = function(id){
window.onload = AutoCardComplete.complete(id)
}
var AutoCardComplete.complete = function(id){
if( id ) {
var target = document.getElementByID(id);
}
else {
var target = body;
}
target.innerHTML = target.innerHTML.replace(
…以下略
みたいにして、インポートの後にAutoCardComplete.load()を呼んでもらえば良いのに。

あ、あと細かい話だけど名前空間は汚さないようにした方が良いと思う。
関数は一つの名前空間に統一して、onload関数も切り分けて外に出すとonloadで別のことをしたいときに、gatherlink.jsの変更がいらなくなるから尚使いやすくなる。

ちなみに脳内でテストしていないので、スペルが間違っていたり、勘違いしていて上手く動かない可能性もあるので悪しからず。