在静态javascript类方法中自我实例化是一种不好的做法？

Question

这是使用 OOP JS（这是 TS）的糟糕代码吗？

class KYC {
 public reference;
 public data = null;

 constructor(id: string) {
   this.reference = id? firestoreAdmin.collection('kyc').doc(id) :
           firestoreAdmin.collection('kyc').doc() :
 }

 async get() {
  const result = await this.reference.get();

  if(!result.exist) throw new Error('not found');

  this.data = result.data;

  return this;
 }

 static async getById(id: string) {
  return await new this(id: string).get();
 }
}

我之所以这样写是因为我发现在 express 中使用 new Kyc(id).get(); 有点不可读。

还有一个问题，这在某种程度上是一种不好的做法吗？反模式？

任何意见都会很棒！

Answer 1

在静态类方法中进行自实例化是一种不好的做法吗？

不，一点也不。将静态方法用作工厂完全没问题。

您的代码中奇怪的是您正在异步初始化data。这可能是故意的，但从我的角度来看，如果没有数据，这个类就毫无用处，所以我建议在静态方法中预先做这件事：

class KYC {
  constructor(
    public reference,
    public data,
  ) {}

  static async getFromReference(reference) {
    const result = await reference.get();
    if (!result.exist) throw new Error('not found');
    return result.data;
  }
  static async getById(id: string) {
    const reference = firestoreAdmin.collection('kyc').doc(id);
    return new this(reference, await this.getFromReference(reference));
  }

  … // further methods operating on this.data
}