Angular2 组件的“this”在执行回调函数时未定义

IT技术 javascript arrays typescript angular
2021-01-21 13:18:02

我有一个组件，它调用服务从 RESTful 端点获取数据。该服务需要在获取所述数据后执行回调函数。

问题是当我尝试使用回调函数将数据附加到组件变量中的现有数据时，我得到一个EXCEPTION: TypeError: Cannot read property 'messages' of undefined. 为什么是this未定义的？

typescript版本：版本 1.8.10

控制器代码：

import {Component} from '@angular/core'
import {ApiService} from '...'

@Component({
    ...
})
export class MainComponent {

    private messages: Array<any>;

    constructor(private apiService: ApiService){}

    getMessages(){
        this.apiService.getMessages(gotMessages);
    }

    gotMessages(messagesFromApi){
        messagesFromApi.forEach((m) => {
            this.messages.push(m) // EXCEPTION: TypeError: Cannot read property 'messages' of undefined
        })
    }
}
5个回答

使用Function.prototype.bind函数：

getMessages() {
    this.apiService.getMessages(this.gotMessages.bind(this));
}

这里发生的情况是您将 传递gotMessages为回调，当它被执行时，范围是不同的，所以这this不是您所期望的。
该bind函数返回一个绑定到this您定义的新函数。

当然，您也可以在那里使用箭头函数：

getMessages() {
    this.apiService.getMessages(messages => this.gotMessages(messages));
}

我更喜欢bind语法，但这取决于你。

第三个选项，以便绑定方法开始：

export class MainComponent {
    getMessages = () => {
        ...
    }
}

或者

export class MainComponent {
    ...

    constructor(private apiService: ApiService) {
        this.getMessages = this.getMessages.bind(this);
    }

    getMessages(){
        this.apiService.getMessages(gotMessages);
    }
}
谢谢！OOP 风格泄漏 javascript 细节（绑定）的事实是可悲的。
2021-03-17 13:18:02
辛苦了，谢谢！感谢您解释为什么会发生这种情况。
2021-04-05 13:18:02
@skfd 完全同意 .bind( this ) 应该是默认的 TypeScript 行为，因为我们正在编程面向对象！:-) 但是谢谢，我会记得现在将它添加到类成员函数的所有回调中，在我的情况下总是如此，因为......
2021-04-06 13:18:02

或者你可以这样做

gotMessages(messagesFromApi){
    let that = this // somebody uses self 
    messagesFromApi.forEach((m) => {
        that.messages.push(m) // or self.messages.push(m) - if you used self
    })
}
你是我的英雄
2021-04-04 13:18:02

因为您只是在getMessages没有正确this上下文的情况下传递函数引用。

您可以使用 lambda 轻松解决该问题，该 lambda 表达式会自动绑定正确的this上下文以供在该匿名函数中使用：

getMessages(){
    this.apiService.getMessages((data) => this.gotMessages(data));
}
就是这样！谢谢
2021-03-22 13:18:02
完美的。最简单有效的解决方案。
2021-03-31 13:18:02

我有同样的问题，通过使用() => { } 代替function() 解决

这不会向现有答案添加任何信息
2021-03-21 13:18:02

请定义函数 

gotMessages = (messagesFromApi) => {
  messagesFromApi.forEach((m) => {
    this.messages.push(m)
  })
}
这不会向现有答案添加任何信息
2021-04-06 13:18:02
其它你可能感兴趣的问题
上一篇无法将属性“innerHTML”设置为 null 下一篇打印网页的特定部分