Как сделать тип универсальным?

Question

[Александр Иванов]

метод sendConfirmMail это отправка почты при регистрации, просто пример реализации, она принимает юзера с интерфейсом IEmailUser который должен передаваться в главный интерфейс IMail и расширять его

export interface IEmailUser {
  email: string
  name: string
  phone: number
}

async sendConfirmMail(user: IEmailUser) {
    const mailContent:IMail<IEmailUser> = {
      to: user.email,
      subject: 'Confirm registration', //Example
      template: 'confirmReg',
      context: {
        email: user.email,
        name: user.name,
        phone: user.phone
      }
    }
    this.mailService.emailSend(mailContent)
  }

после чего уже в сервисе должен в методе emailSend переменная mailData должна иметь интерфейс уже расширеный IMail

export interface IMail<T> {
  to: string
  subject: string,
  template: string,
  context: T
}

async emailSend(mailData:IMail<T>) {
    const urlConfirmAddress = this.configService.get<string>(
      'URL_CONFIRM_ADDRESS',
    );

    return this.mailerService.sendMail({
      to: mailData.to,
      subject: mailData.subject,
      template: join(__dirname, '/../templates', mailData.template),
      context: mailData.context
    }).catch((e) => {
      throw new HttpException(
        `Ошибка работы почты: ${JSON.stringify(e)}`,
        HttpStatus.UNPROCESSABLE_ENTITY,
      );
    });
  }

как сделать в IMail context универсальным? потому что интерфейс T изменяемый, как это сделать?

у меня выходит ошибка

Не удается найти имя "T".ts(2304)
Параметр "mailData" общего метода из экспортированного класса имеет или использует закрытое имя "T".ts(4073)

Comments

[Alexandroppolus]

какой тип у this.mailerService.sendMail ? Надо на него ориентироваться, ты просто передаешь туда context как есть

[Александр Иванов]

Alexandroppolus,

import { HttpException, HttpStatus, Injectable } from '@nestjs/common';
import { MailerService } from '@nestjs-modules/mailer';
import { ConfigService } from '@nestjs/config';
import { join } from 'path';
import { IMail } from './interfaces/mail.interface';

@Injectable()
export class MailService {
  constructor(
    private readonly mailerService: MailerService,
    private configService: ConfigService
  ) {}

  async emailSend(mailData:IMail<T>) {
    const urlConfirmAddress = this.configService.get<string>(
      'URL_CONFIRM_ADDRESS',
    );

    return this.mailerService.sendMail({
      to: mailData.to,
      subject: mailData.subject,
      template: join(__dirname, '/../templates', mailData.template),
      context: mailData.context
    }).catch((e) => {
      throw new HttpException(
        `Ошибка работы почты: ${JSON.stringify(e)}`,
        HttpStatus.UNPROCESSABLE_ENTITY,
      );
    });
  }
}

[Alexandroppolus]

Александр Иванов, https://github1s.com/nest-modules/mailer/blob/main...

context?: {
    [name: string]: any;
  };

итого ты можешь просто написать

export interface IMail {
  to: string
  subject: string,
  template: string,
  context: Record<string, unknown>;
}

и не заморачиваться с генериком

[Александр Иванов]

Alexandroppolus, наверно мы не поняли друг друга, контекст должен принимать в себя любой другой интерфейс, потому то переменные в нем будут меняться по необходимости, я заведомо не могу знать каким он будет

[WbICHA]

Александр Иванов,

я заведомо не могу знать каким он будет

Если ты сам не знаешь, то откуда тс это может знать?

[Alexandroppolus]

Александр Иванов, ты просто закидываешь в этот контекст какие-то значения, и потом передаешь его в mailerService.sendMail. Больше никаких действий с ним не в твоем коде не делается.

mailerService.sendMail готов принять Record< string, unknown >, тебя такой тип тоже устроит, потому незачем что-то придумывать.

[Александр Иванов]

WbICHA, потому что тс это дед мороз который умеет читать мысли и знает все. я описал выше что пытаюсь сделать, если можно так сказать, универсальную типизацию, т.е. есть главный интерфейс IMail в котором есть элемент context и этот контекст изменяемый, некий T и этот интерфейс T может быть как IEmailUser так и другим передаваемым интерфейсом. чтоб я в конечном итоге мог принимать этот уже "полный" интерфейс, т.е. в данном примере это получается IMail в функции async emailSend(mailData:IMail) и вот как это сделать!

[Александр Иванов]

Alexandroppolus, меня такой тип не устраивает потому что в context должны передаваться свои поля которые я так же хочу типизировать

[Alexandroppolus]

Александр Иванов, ок, согласен, в sendConfirmMail и тому подобных конкретных функциях можно оставить IMail< IEmailUser > для борьбы с опечатками.
Ну а в универсальной функции emailSend всё равно будет достаточно IMail< Record< string, unknown > >

async emailSend(mailData: IMail<Record<string, unknown>>) {

как альтернатива, вместо Record< string, unknown > здесь можно перечислить через | все возможные варианты контекста:

async emailSend(mailData: IMail<IEmailUser | IEmailOther1 | IEmailOther2 | etc>) {

но, по моему, это менее гибко.

----
делать emailSend генериком, как предложено в ответе, нет надобности - от этого никаких плюсов. Да и опять же генерик надо будет ограничить типом Record< string, unknown >, чтобы данные можно было передать в mailerService.sendMail

async emailSend<T extends Record<string, unknown>>(mailData: IMail<T>)

Answer 1

[Александр]

async emailSend<T>(mailData: IMail<T>)

Comments

[Александр Иванов]

я так делал, ошибка пропадает но тогда в context: mailData.context нет отображаются в подсказках переменные из того самого изменяемого интерфейса

[Александр]

Александр Иванов, так внутри метода и не известно что пришло. Какую задачу пытаетесь решить?

[Александр Иванов]

Александр, если можно так сказать, универсальную типизацию, т.е. есть главный интерфейс IMail в котором есть элемент context и этот контекст изменяемый, некий T и этот интерфейс T может быть как IEmailUser так и другим передаваемым интерфейсом. чтоб я в конечном итоге мог принимать этот уже "полный" интерфейс, т.е. в данном примере это получается IMail в функции async emailSend(mailData:IMail) и вот как это сделать!

[WbICHA]

Александр Иванов, у тебя это уже сделано... Ты не понимаешь чего ты хочешь.