Типографские декораторы + Отражать метаданные

Question

Я использую декоратор свойства Field, который помещает свой ключ в свойство метаданных fields Reflect:

export function Field(): PropertyDecorator {
    return (target, key) => {
        const fields = Reflect.getMetadata('fields', target) || [];
        if (!fields.includes(key)) {
            fields.push(key)
        }
        Reflect.defineMetadata('fields', fields, target)
    }
}

Затем у меня есть абстрактный базовый класс Form, который обращается кметаданные в дополнительном модуле получения:

abstract class Form {
    get fields() {
        return Reflect.getMetadata('fields', this) || [];
    }
}

До сих пор я мог успешно использовать его, чтобы отличить поля формы от других свойств класса.Рассмотрим следующие классы:

abstract class UserForm extends Form {
    @Field()
    public firstName: string

    @Field()
    public lastName: string

    get fullName() {
        return this.firstName + ' ' + this.lastName;
    }
}

class AdminForm extends UserForm {
    @Field()
    roles: string[]
}

const form = new AdminForm()
console.log(form.fields)
// ['roles', 'firstName', 'lastName']

Проблема возникает, когда я определяю родственный класс для AdminForm - MemberForm.Когда существует несколько подклассов для Form, кажется, что fields getter возвращает все поля:

class MemberForm extends UserForm {
    @Field()
    memberSince: Date;
}

const form = new AdminForm()
console.log(form.fields)
// ['roles', 'firstName', 'lastName', 'memberSince'] <--!!!

Это не имеет смысла для меня.Почему поле memberSince появляется в экземпляре AdminForm?Как я могу определить разные поля на разных подклассах?

Answer 1

Проблема в том, что getMetadata идет вниз по цепочке прототипов и всегда будет возвращать то, что определено для базового типа (поскольку он назначается первым).Вам нужно использовать getOwnMetadata, чтобы получить поле массива текущего класса, только когда вы добавляете новое поле и когда получаете поля, вам нужно пройти вверх по цепочке свойств, чтобы получить все поля базового класса.

Этодолжно работать:

import 'reflect-metadata'
export function Field(): PropertyDecorator {
  return (target, key) => {
      const fields = Reflect.getOwnMetadata('fields', target) || [];
      if (!fields.includes(key)) {
          fields.push(key)
      }
      Reflect.defineMetadata('fields', fields, target)
  }
}

abstract class Form {
  get fields() {
      let fields = []
      let target = Object.getPrototypeOf(this);
      while(target != Object.prototype) {
        let childFields = Reflect.getOwnMetadata('fields', target) || [];
        fields.push(...childFields);
        target = Object.getPrototypeOf(target);
      }
      return fields;
  }
}

abstract class UserForm extends Form {
  @Field()
  public firstName!: string

  @Field()
  public lastName!: string

  get fullName() {
      return this.firstName + ' ' + this.lastName;
  }
}

class AdminForm extends UserForm {
  @Field()
  roles!: string[]
}

const form1 = new AdminForm()
console.log(form1.fields) // ['roles', 'firstName', 'lastName']

class MemberForm extends UserForm {
  @Field()
  memberSince!: Date;
}

const form2 = new MemberForm()
console.log(form2.fields) // ["memberSince", "firstName", "lastName"]