如何产生Mobx.autorun以防止在类构造函数中触发?

时间:2016-05-04 09:03:36

标签: javascript typescript mobx

在这个类中,我使用初始化的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
  };

1 个答案:

答案 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传递给自动运行的函数。)