在这个类中,我使用初始化的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
};
答案 0 :(得分:3)
我认为这是解决问题的一种很好的方法,但初始化的字段也应该是可观察的。否则更改_initialized
将不会导致自动运行重新运行。
但在这种情况下,我不确定初始化变量在你的确实达到了什么,因为你在autorun之后的第一个语句是将initialized设置为true?
所以我不完全确定你要实现的目标:将自动运行/ 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将参数firstRun
传递给自动运行的函数。)