サンプルコードのエラー? 使い方の問題?

[naoshi]

本の109ページのサンプルコードの8行目でエラーが発生しました。
http://firefox3hacks.org/source/ch3/20-109.txt

コード: 全て選択

    init : function() {
        var target = this.target;
        this.types.forEach(function(aType) {
            target.events.addListener(aType, target); // ←ココ
        });
    },

エラーの内容。

//エラー: uncaught exception: [Exception... "Could not convert JavaScript argument arg 1 [extIEvents.removeListener]" nsresult: "0x80570009 (NS_ERROR_XPC_BAD_CONVERT_JS)" location: "JS frame :: chrome://helloworld/content/helloworld.js :: anonymous :: line 14" data: no]

forEachの前に var tmp=this として、addLister(aType, tmp) とすることで
期待した動作になっているように見えますが、あっているでしょうか?
よろしくお願いいたします。

# まだFirefoxとJavaScriptをはじめたばかりなので、
# 実行方法などの問題でしたら申し訳ありません

[あ]

forEachの前に var tmp=this として、addLister(aType, tmp) とすることで
期待した動作になっているように見えますが、あっているでしょうか?

それであっていると思います。自分なら

コード: 全て選択

for (var i in this.types) {
  target.events.addListener(this.types[i], this);
}

とするかな。

[naoshi]

functionの中にthisを渡すよりすっきりですね。
ありがとうございました。

[Piro]

あわわわ。すみません。リファクタリングの時かなんかにやらかしてしまったようです。以下を「正式版」とさせてください。（増刷分からはこちらに差し替えてもらいます）

コード: 全て選択

function FUELEventListener(aTypes, aTarget, aHandler) {
    this.types = aTypes.split(/,\s*|\s+/);
    this.target = aTarget;
    this.handleEvent = aHandler;
}
FUELEventListener.prototype = {
    init : function() {
        this.types.forEach(function(aType) {
            this.target.events.addListener(aType, this);
        }, this);
    },
    destroy : function() {
        this.types.forEach(function(aType) {
            this.target.events.removeListener(aType, this);
        }, this);
        this.types = this.target = this.handleEvent = null;
    }
};
var pref = Application.prefs.get('browser.sessionstore.resume_session_once');
var listener = new FUELEventListener('change', pref,
      function(aEventItem) {
        alert('pref is changed!');
        this.destroy();
      }
    );
listener.init();