demo 11 里面的 this.isMounted 的作用是什么?

[yuxino]

阮老师你好。我想请教下这一段里面的this.isMounted的作用。

react-demos/demo11/index.html

Lines 20 to 30 in 15a5be3

	componentDidMount: function() {
	$.get(this.props.source, function(result) {
	var lastGist = result[0];
	if (this.isMounted()) {
	this.setState({
	username: lastGist.owner.login,
	lastGistUrl: lastGist.html_url
	});
	}
	}.bind(this));
	},

看README的描述

did methods which are called right after something happens.

componentDidMount这个钩子调用的时候 this.isMounted应该总是为true吧。那么这个判断是不是多余的。

[ruanyf]

谢谢指出，这个判断确实是多余的。我把它删掉。

[vipcxj]

并不一定吧~ isMounted是在回调函数中，并且涉及网络请求，没人能保证它执行的时候仍旧是Mounted的

[yuxino]

@vipcxj componentDidMount是在isMounted状态为true的时候执行的回调函数,所以是可以保证的。我刚才仔细看了一下README的实例是正确的,有误的只是demo11文件夹里面的那个实例代码。而且这个东西不好

[vipcxj]

@Nbsaw 没记错的话，应该是执行get网络请求后才调用的回调函数，如果网比较差，这就会有一个时间差了啊，如果期间被unmount了，也是可能的

[ruanyf]

理论上，获取到数据的时候，组件是可能被卸载的。但是，这个例子里，确实是多余的判断。

另外，isMounted这个 API 也已经被废掉了，所以还是去掉比较好。facebook/react#5465

[vipcxj]

@ruanyf 我把整个issure都看了下，基本就是react的开发者嫌isMounted不够优雅，想废除它，但又没在react内部给一个完美的代替方案，所以另一个react的贡献者就不同意了。然后就是提出各种workaround，然而所有workaround都不是react自带的，全部仅仅是workaround～ 没看到这个issure的岂不是要被坑？

[ruanyf]

isMounted 确实被废掉了。https://github.com/facebook/react/blob/3c977dea6b96f6a9bb39f09886848da870748441/packages/react/src/ReactBaseClasses.js#L89