在这个类中,我使用一个初始化的bool状态来产生mobx.autorun执行。otherwize“this”未完全分配并导致错误。还有其他更清洁的方法吗?
class GameMaster{
private _initialized:boolean = false;
private _store:IDomainStore;
private _moveDisposer:Lambda;
/**
*
* @param store - client or server store
*/
constructor(store:IDomainStore){
this._store = store;
console.log(this._store);
//todo abstract services to decouple client device from GameMaster because it is also used on the server.
this._moveDisposer = autorun(()=>{
// prevent firing in the constructor
if(this._initialized) {
this.present(
<IIntent>{
fromId: 'GeoLocation.service',
toIds: [Meteor.userId()],
wish: actions.playerActions.types.CHANGE_PLAYER_GEO_COORDINATES,
data: [System.GeolocationService.coordinates.lng, System.GeolocationService.coordinates.lat]
});
}
});
this._initialized = true;
}
public present(intent:IIntent):boolean{
...
}
...
}
这是我在另一个文件中看到的:
@observable coordinates = {
lng:0,
lat:0
};
最佳答案
我认为这是解决问题的一个很好的方法,但是初始化字段也应该是可观察的。否则,更改_initialized
不会导致自动运行重新运行。
但在这种情况下,我不确定初始化变量在您的中到底实现了什么,因为自动运行之后的第一条语句是将initialized设置为true?
所以我不完全确定您要实现什么:将autorun/present
调用推迟到构造函数的末尾,或者跳过第一个present
调用?
更新的答案
如果你想防止副作用(在这种情况下发送present
),有一个简单的模式。提示是确保你计算出副作用所需的任何值,但不要自己触发副作用。所以在你的例子中
constructor(store:IDomainStore){
let firstRun = true;
this._moveDisposer = autorun(()=>{
// make sure all information is tracked
const presenceInfo = <IIntent>{
fromId: 'GeoLocation.service',
toIds: [Meteor.userId()],
wish: actions.playerActions.types.CHANGE_PLAYER_GEO_COORDINATES,
data: [System.GeolocationService.coordinates.lng, System.GeolocationService.coordinates.lat]
}
// but prevent the side effect in the first run
if(!firstRun) {
this.present(presenceInfo);
} else {
firstRun = false;
}
});
}
(请注意,将来可能不再需要该标志,因为有一个现有的proposal要将param
firstRun
传递给autorunned函数)。关于javascript - 如何产生Mobx.autorun以防止在类构造函数中触发?,我们在Stack Overflow上找到一个类似的问题:https://stackoverflow.com/questions/37023109/