Как типизировать переменную под хранение классов?

Question

[johnymkp]

У меня есть интерфейс вида
interface IStorage<T, K>
и класс, содержащий базовую реализацию
class Storage<T, K> implements IStorage<T, K>
а еще несколько классов, которые закрывают типы в этом дженерике, например
class MatchStorage extends Storage<Match, number>
У меня есть конфиг, согласно которому создаются хранилища в indexeddb. В том же конфиге я хочу сделать поле, в котором должен храниться класс, соответствующий этому хранилищу. Выглядит примерно так:

const matchStorageConfig: IStorageConfig = {
  storageName: StorageName.Matches,
  oftype: MatchStorage,  // MatchStorage - это класс из примера выше

Когда я создаю хранилища, то следом создаю экземпляры класса для работы с этими хранилищами. Т.е. это по задумке должно выглядеть примерно так: берем из поля oftype класс и создаем экземпляр:
const instance = new oftype(тут аргументы);

Вопрос в том, как мне правильно типизировать oftype, чтобы туда:
* Можно было положить только классы, реализующие IStorage.
* Типы в IStorage могли быть любые, например, IStorage<User, number>, IStorage<Player, string> и т.д.

interface IStorageConfig {
  oftype: ??????? как описать тип под такую задачу

Comments

[Василий Банников]

Покажи как интерфейс IStorage выглядит.

[johnymkp]

Василий Банников, примерно так:

export interface IStorage<T, K> {
  save(data: T | T[]): Promise<DbOperationResult<T>[]>;

там несколько методов, все выглядят так же как save по сути.

[Василий Банников]

johnymkp, покажи метод или поле, в котором фигурирует K.
И как потом будет использоваться поле oftype из IStorageConfig

[johnymkp]

Василий Банников, oftype используется примерно так:

private createStorageInstances(connection: IDBDatabase) {
  this.storages = new Map();
  this.config.storages.forEach(s => {
    if (s.oftype) {
      this.storages.set(s.storageName, new s.oftype(connection, s.storageName));  // ТУТъ
    }
  });
}

public getStorage(storageName: StorageName) {
  return this.storages.get(storageName);
}

А "К" нужен просто чтобы тип ключа задать, больше ни для чего не используется. Интерфейс IStorage имеет методы записи, чтения, удаления. Допустим, метод чтения принимает ключ типа K, а возвращает массив типа T (массив лежит внутри DbOperationResult, там массив успешно считанных данных и массив ошибок, если что-то не считалось):

public read(key?: K | K[]): Promise<DbOperationResult<T>> {

Вот так К и используется. Вся логика работы с хранилищами реализована внутри класса Storage. А "конкретные" классы вроде MatchStorage нужны просто чтобы закрыть типы в Storage:

export class MatchStorage extends Storage<Match, number> {
  public constructor(database: IDBDatabase, storageName: string) {
    super(database, storageName);
  }
}

И вот эти "конкретные" классы я хочу разместить в конфиге, в поле oftype.

export const matchStorageConfig: IStorageConfig = {
  storageName: StorageName.Matches,
  oftype: MatchStorage,  // !!!!!!!! ВОТъ
  options: {
    keyPath: "id"
  },
  indexes: [
    {
      storageIndexName: "lobby_type_index",
      keyPath: "lobbyType",
    },
    {
      storageIndexName: "start_date_index",
      keyPath: "startDateTime",
    }
  ]
};

....

export const defaultDbConfig: IDbConfig = {
  version: 1,
  dbname: 'test123',
  storages: [
    matchStorageConfig,
  ]
}

Answer 1

[Alexandroppolus]

interface IStorageConfig<T, K> {
    oftype: new () => IStorage<T, K>;
}

const matchStorageConfig: IStorageConfig<Match, number> = {
    oftype: MatchStorage
}

const s = new matchStorageConfig.oftype();

здесь предполагается, что у всех твоих стораджей есть конструктор без параметров.
Если хочешь какие-то параметры, то их надо либо делать одинаковыми для всех стораджей (и тогда указать в круглых скобках сразу после new в интерфейсе IStorageConfig), либо в этом нет смысла

Comments

[johnymkp]

Написать
IStorageConfig<T, K>
не могу, потому что у меня этот интерфейс используется в конфиге всей БД, вот так:

export interface IDbConfig {
  dbname: string;
  version: number;
  storages: IStorageConfig[];  // ЗДЕСЬ
}

Соответственно, закрыть T и K негде.
Параметры у конструкторов всех хранилищ одинаковые. Я написал вот так:

export interface IStorageConfig {
  storageName: StorageName;
  oftype: new (database: IDBDatabase, storageName: string) => IStorage<any, any>;

Нормально? В плане того, что any в данном случае это не моветон? Как раз как будто бы логично выглядит, что тип данных и ключ хранилища может быть любой.

[Alexandroppolus]

В плане того, что any в данном случае это не моветон? Как раз как будто бы логично выглядит, что тип данных и ключ хранилища может быть любой.

так себе.

если допускается любой тип данных и любой ключ, то нафига параметры в IStorage? Сделай его обычным типом, не генериком:

export interface IStorage {
  save(data: unknown): Promise<DbOperationResult<unknown>[]>;
}

[johnymkp]

P.S. Про "негде закрыть" я похоже ошибся. Могу же написать вот так:

export interface IDbConfig {
  dbname: string;
  version: number;
  storages: IStorageConfig<any, any>[];  // Закрываю типы тут
}

export interface IStorageConfig<T, K> {
  storageName: StorageName;
  oftype: new (database: IDBDatabase, storageName: StorageName) => IStorage<T, K>;

Тогда если сам объект конфига напишу вот так:

export const matchStorageConfig: IStorageConfig<Match, number> = {
  storageName: StorageName.Matches,
  oftype: MatchStorage,

то компилятор не позволит положить в oftype конструктор, который реализует, например Storage<string, number>, а только тот, что реализует Storage<Match, number>

Вопрос про any остается - в данном случае any это нормально?

[johnymkp]

Alexandroppolus, не могу сходу сказать, насколько что-то изменится, если написать unknown. Но я попробую переделать места, которых это все касается. Если окажется, что дженерики и правда не нужны, тогда отмечу решением. А если какие-нибудь проблемы по ходу возникнут из-за unknown, то еще напишу сюда.