javascript - 在TS中为长类定义动态装饰器而无需重新定义所有方法的方法

Question

我有一个类，里面有很多方法。我们在这里不关注这个类可能会被重构并重写为几个类，因为它最终会被重构，但不知道。我想要一个装饰器，它只装饰这个类的一种方法。我们可以通过多种方式做到这一点。

类接口(interface):

interface IFoo {
    method1 (): number;
    method2 (a: string): number;
    method3 (b: IFoo): number;
    method4 (c: string | (() => string)): number;
    method5 (d: number, e: string): number;
}

1. 经典的 OOP 解决方案:定义重新定义特定方法的装饰器，并为所有其他方法调用 super 实现。像这样的东西。

   class FooDecorator implements IFoo {
       constructor (
           private readonly provider: IFoo
       ) {}
   
       public method1 (): number {
           return this.provider.method1() + 1;
       }
   
       public method2 (a: string): number {
           return this.provider.method2.apply(this.provider, arguments);
       }
   
       public method3 (b: IFoo): number {
           return this.provider.method3.apply(this.provider, arguments);
       }
   
       public method4 (c: string | (() => string)): number {
           return this.provider.method4.apply(this.provider, arguments);
       }
   
       public method5 (d: number, e: string): number {
           return this.provider.method5.apply(this.provider, arguments);
       }
   }
   

   如您所见，写得很长并且代码重复。

2. 尝试利用一些 JS 功能。

   interface IFooDecorator {
       method1: IFoo["method1"];
   }
   
   class FooDecorator implements IFooDecorator {
       constructor (
           private readonly provider: IFoo
       ) {
           Object.setPrototypeOf(this, provider);
       }
   
       public method1 (): number {
           return this.provider.method1() + 1;
       }
   }
   

   明显的缺点是错误的类型化和 setPrototypeOf 的使用

我也尝试过使用代理，但对使用代理的打字支持也很差。还有其他解决方案吗？第一种方法很好，以防我们可以通过 .apply 调用自动重新定义非修饰方法。

Answer 1

将装饰器实现为工厂函数，使用 Object.create 而不是 class 语法，这样您就不需要使用 Object.setPrototypeOf:

function fooDecorator<T>(provider: IFoo): T implements IFoo {
    return Object.create(provider, {
        method1: {
            value(): number {
                return provider.method1() + 1;
            },
            writable: true,
            configurable: true
        }
    });
}