小编典典

让构造函数返回 Promise 是不好的做法吗?

all

我正在尝试为博客平台创建一个构造函数,它内部正在进行许多异步操作。这些范围包括从目录中抓取帖子、解析它们、通过模板引擎发送它们等。

所以我的问题是,让我的构造函数返回一个承诺而不是他们调用的函数的对象是不明智的new

例如:

var engine = new Engine({path: '/path/to/posts'}).then(function (eng) {
   // allow user to interact with the newly created engine object inside 'then'
   engine.showPostsOnOnePage();
});

现在,用户也可能 提供补充 Promise 链链接:

var engine = new Engine({path: '/path/to/posts'});

// ERROR
// engine will not be available as an Engine object here

这可能会造成问题,因为用户可能会感到困惑,为什么 engine 在构建后不可用。

在构造函数中使用 Promise 的原因是有道理的。我希望整个博客在构建阶段之后能够正常运行。但是,在调用后几乎无法访问该对象似乎是一种气味new

我已经讨论过使用类似的东西engine.start().then()或者engine.init()会返回 Promise
的东西。但那些看起来也很臭。

编辑:这是在 Node.js 项目中。


阅读 55

收藏
2022-08-01

共1个答案

小编典典

是的,这是一种不好的做法。构造函数应该返回其类的实例,仅此而已。否则会弄乱new运算符和继承。

此外,构造函数应该只创建和初始化一个新实例。它应该设置数据结构和所有特定于实例的属性,但 不执行
任何任务。如果可能的话,它应该是一个没有副作用的纯函数,并具有所有的好处。

如果我想从我的构造函数中执行事情怎么办?

这应该在你的类的方法中。你想改变全局状态吗?然后显式调用该过程,而不是作为生成对象的副作用。此调用可以在实例化之后立即进行:

var engine = new Engine()
engine.displayPosts();

如果该任务是异步的,您现在可以轻松地从该方法返回其结果的承诺,以便轻松地等待它完成。
但是,当方法(异步)改变实例并且其他方法依赖于它时,我不会推荐这种模式,因为这会导致它们需要等待(即使它们实际上是同步的也变成异步的)并且你很快就会有一些内部队列管理正在进行中。不要将实例编码为存在但实际上不可用。

如果我想将数据异步加载到我的实例中怎么办?

问问自己: 你真的需要没有数据的实例吗? 你能以某种方式使用它吗?

如果答案是 否定 的,那么您不应该在拥有数据之前创建它。将数据 ifself 作为构造函数的参数,而不是告诉构造函数如何获取数据(或传递数据的承诺)。

然后,使用静态方法加载数据,从中返回一个 Promise。然后链接一个将数据包装在一个新实例中的调用:

Engine.load({path: '/path/to/posts'}).then(function(posts) {
    new Engine(posts).displayPosts();
});

这使得获取数据的方式具有更大的灵活性,并大大简化了构造函数。同样,您可以编写静态工厂函数来为Engine实例返回 Promise:

Engine.fromPosts = function(options) {
    return ajax(options.path).then(Engine.parsePosts).then(function(posts) {
        return new Engine(posts, options);
    });
};

...

Engine.fromPosts({path: '/path/to/posts'}).then(function(engine) {
    engine.registerWith(framework).then(function(framePage) {
        engine.showPostsOn(framePage);
    });
});
2022-08-01