6

次のようなコールバックで関数を呼び出しています。

$(function() {
    //get all the items
    search.init('.result tbody tr');
    search.parseresults(function(announcementID){
        //query every single page
        var myCompany = new company(announcementID);
        myCompany.requestPage(function(){
            //on response parse the data.
            myCompany.parsedata()
            var myPerson = new person(myCompany )
            myPerson.getPhone(function(){
                console.log('test')
            });
        })
    });
});

問題があるのは、console.log('test') を使用した最後のコールバックです。

これは getPhone 関数です。

person.prototype.getPhone = function(callback){
    this.attempt++
    if( this.attempt === 1){
        var who = this.lastname;
        var where = this.adress+' '+this.postal;
    }else if(this.attempt === 2){
        var who = this.firstname+' '+this.lastname;
        var where = this.adress+' '+this.postal;
    }else{
        var who = this.firstname+' '+this.lastname;
        var where = this.adress+' '+this.postal;
        var url = 'http://personer.eniro.se/resultat/'+who+'/'+where;
        console.debug('')
        //console.debug('fail')
        console.debug(url)
        console.debug(this)
        return
    }
    var self = this;

    var url = 'http://personer.eniro.se/resultat/'+who+'/'+where;
    GM_xmlhttpRequest({
        method: "GET",
        url: url,
        onload: function(data) {
            data = $.parseHTML(data.response);
            var vCard = $(data).find('.vcard')
            if (vCard.length === 1){
                var phone = vCard.find('.tel.row a').map(function(){
                    return this.text
                }).get()

                self.officePhone = phone[0];
                if(phone.length > 1){
                    self.mobilePhone = phone[1];
                }else{
                    self.mobilePhone = '';
                }
                callback();

            } else if(vCard.length > 1){
                self.getPhone()
            }
        }
    })
}

コールバックは、想定されているときにトリガーされます。しかし、コールバックが存在する場合、エラーが発生します:

undefined は関数ではありません

4

2 に答える 2

9 
else if(vCard.length > 1){
    self.getPhone()
}

次の試行を行っているとき、callback- then はその呼び出しで未定義です。コールバックを呼び出す前に、コールバックが関数であるかどうかを常にテストする必要があります。

if (vCard.length === 1){
    var phone = vCard.find('.tel.row a').map(function(){
        return this.text
    }).get()

    self.officePhone = phone[0];
    if(phone.length > 1){
        self.mobilePhone = phone[1];
    }else{
        self.mobilePhone = '';
    }
    // also pass some reasonable result:
    if (typeof callback=="function") callback(phone);

} else if(vCard.length > 1) {
    self.getPhone(callback)
}
于 2013-08-09T12:43:04.913 に答える