Как я могу использовать DI с классами электронной почты и SMS
У меня есть этот интерфейс:
public interface IMessageSender
{
Task SendMessageAsync(parameters);
}
И это классы:
public class EmailSender : IMessageSender
{
}
public class SMSSender : IMessageSender
{
}
В моем контроллере я использовал их так:
public class MyController : Controller
{
private readonly IMessageSender _messageSender;
public MyController(IMessageSender messageSender)
{
_messageSender = messageSender;
}
}
И у меня есть два метода в этом контроллере:
public async Task<IActionResult> SendByEmail()
{
await _messageSender.SendMessageAsync(parameters);
}
public async Task<IActionResult> SendBySMS()
{
await _messageSender.SendMessageAsync(parameters);
}
В приведенных выше методах, как я могу сообщить ядру .net, что в методе SendByEmail мне нужен экземпляр класса EmailSender, а в методе SendBySMS мне нужен экземпляр класса SMSSender. Я уже добавил это в свой класс запуска, но он всегда создает экземпляр класса EmailSender:
services.AddTransient<IMessageSender, EmailSender>();
Причина, по которой я предлагаю этот подход, заключается в том, что многие люди испытывают искушение «эти два класса делают одно и то же, поэтому дайте им общий интерфейс». Но, как правило, с интерфейсами, которые думают об этом неправильно. Думая об интерфейсах, сосредоточьтесь в первую очередь на вызывающей стороне. Чего хочет абонент? Если звонящий хочет отправить электронное письмо, дайте ему интерфейс для этого. Если они хотят отправить SMS - дайте им для этого интерфейс. Их не волнуют ваши абстракции — их волнует отправка электронных писем или SMS.
Ответы 1
В настоящее время у вас есть:
public interface IMessageSender
{
Task SendMessageAsync(parameters);
}
и два класса, которые реализуют этот интерфейс. Это работает, но я бы предложил небольшую настройку этого подхода. Сохраните IMessageSender и добавьте два новых интерфейса:
public interface IEmailSender : IMessageSender
{
}
public interface ISMSSender : IMessageSender
{
}
Зачем это делать? Что ж, теперь он позволяет вам внедрить отправителя конкретный, который вас интересует (этот контроллер хочет отправлять электронные письма, этот контроллер хочет отправлять SMS).
Кроме того, поскольку они оба наследуются от IMessageSender, с некоторыми контейнерами IoC (например, Autofac — я не тестировал их все) вы также можете внедрить коллекцию IMessageSender (т. е. IEnumerable<IMessageSender>) — это не полезно для всегда, но может быть полезно, если вы хотите вызвать все отправителей сообщений (т. е. SMS это и электронная почта это). Если это бесполезно - не проблема, удалите IMessageSender вообще (и переместите методы, определяющие интерфейс, и в IEmailSender, и в ISMSSender).
Теперь измените свои классы, чтобы реализовать новые интерфейсы:
public class EmailSender : IEmailSender
{
}
public class SMSSender : ISMSSender
{
}
и зарегистрируйте свои классы для обоих интерфейсов с вашим контейнером. С Autofac это будет что-то вроде AsImplementedInterfaces.
Глядя на ваш существующий код, я бы предложил следующее решение:
services.AddTransient<IEmailSender, EmailSender>();
services.AddTransient<ISMSSender, SMSSender>();
services.AddTransient<IMessageSender, EmailSender>();
services.AddTransient<IMessageSender, SMSSender>();
Затем измените свой контроллер на:
public MyController(IEmailSender emailSender, ISMSSender smsSender)
Я думаю, что этот services.AddTransient<IMessageSender, EmailSender>(); неверен, потому что как он может отличить их позже, чтобы создать из них экземпляр? Фермер мне кажется более правильным. Не так ли?
Это полезно, если вы хотите внедрить IEnumerable<IMessageSender> (например, stackoverflow.com/a/39569277/34092). Вам это сейчас не нужно, я подозреваю. Прочитайте but it can be useful if в моем ответе.
Нет необходимости регистрировать IMessageSender, если он вам не нужен. Это только в том случае, если вам нужна абстракция типа "пришлите, а мне все равно как". Когда вы определяете IEmailSender, вы имеете в виду «отправить по электронной почте». За каждым интерфейсом, который вы определяете, должно стоять конкретное намерение. В какой-то момент IEmailSender может получить другие параметры и полностью отклониться от IMessageSender.