Каковы лучшие практики использования SmtpClient, SendAsync и Dispose в .NET 4.0

Я немного озадачен тем, как управлять SmtpClient теперь, когда он одноразовый, особенно если я звоню с помощью SendAsync. По-видимому, мне не следует вызывать Dispose до завершения SendAsync. Но должен ли я когда-нибудь называть это (например, используя "использование"). Сценарий представляет собой службу WCF, которая периодически рассылает электронную почту при совершении звонков. Большая часть вычислений выполняется быстро, но отправка электронной почты может занять секунду или около того, поэтому предпочтительнее использовать Async.

Следует ли мне создавать новый SmtpClient каждый раз при отправке почты? Должен ли я создать его для всего WCF? Помогите!

Обновление. Если это имеет значение, каждое электронное письмо всегда настраивается под пользователя. WCF размещен в Azure, а Gmail используется в качестве почтовой программы.

Примечание. .NET 4.5 SmtpClient реализует async awaitableметод SendMailAsync. Для более ранних версий используйте, SendAsyncкак описано ниже.

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

var message = new MailMessage("from", "to", "subject", "body"))
var client = new SmtpClient("host");
client.SendCompleted += (s, e) => {
                           client.Dispose();
                           message.Dispose();
                        };
client.SendAsync(message, null);

Это немного раздражает, потому что SendAsyncне принимает обратный вызов.

Исходный вопрос был задан для .NET 4, но если он помогает, начиная с .NET 4.5, SmtpClient реализует метод async awaitable SendMailAsync .

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

public async Task SendEmail(string toEmailAddress, string emailSubject, string emailMessage)
{
    using (var message = new MailMessage())
    {
        message.To.Add(toEmailAddress);

        message.Subject = emailSubject;
        message.Body = emailMessage;

        using (var smtpClient = new SmtpClient())
        {
            await smtpClient.SendMailAsync(message);
        }
    }
}

Лучше избегать использования метода SendAsync.

В общем, объекты IDisposable следует удалять как можно скорее; реализация IDisposable на объекте предназначена для сообщения того факта, что рассматриваемый класс содержит дорогие ресурсы, которые должны быть детерминированно освобождены. Однако, если создание этих ресурсов является дорогостоящим и вам нужно создать множество этих объектов, может быть лучше (с точки зрения производительности) сохранить один экземпляр в памяти и повторно использовать его. Есть только один способ узнать, имеет ли это значение: профилировать это!

Re: утилизация и Async: usingочевидно, вы не можете использовать . Вместо этого вы обычно удаляете объект в событии SendCompleted:

var smtpClient = new SmtpClient();
smtpClient.SendCompleted += (s, e) => smtpClient.Dispose();
smtpClient.SendAsync(...);

Хорошо, старый вопрос я знаю. Но я сам наткнулся на это, когда мне нужно было реализовать что-то подобное. Я просто хотел поделиться кодом.

Я перебираю несколько SmtpClients, чтобы отправлять несколько писем асинхронно. Мое решение похоже на TheCodeKing, но вместо этого я удаляю объект обратного вызова. Я также передаю MailMessage как userToken, чтобы получить его в событии SendCompleted, чтобы я мог также вызвать dispose для этого. Как это:

foreach (Customer customer in Customers)
{
    SmtpClient smtpClient = new SmtpClient(); //SmtpClient configuration out of this scope
    MailMessage message = new MailMessage(); //MailMessage configuration out of this scope

    smtpClient.SendCompleted += (s, e) =>
    {
        SmtpClient callbackClient = s as SmtpClient;
        MailMessage callbackMailMessage = e.UserState as MailMessage;
        callbackClient.Dispose();
        callbackMailMessage.Dispose();
    };

    smtpClient.SendAsync(message, message);
}

Вы можете понять, почему особенно важно избавиться от SmtpClient, из следующего комментария:

public class SmtpClient : IDisposable
   // Summary:
    //     Sends a QUIT message to the SMTP server, gracefully ends the TCP connection,
    //     and releases all resources used by the current instance of the System.Net.Mail.SmtpClient
    //     class.
    public void Dispose();

В моем сценарии отправки нескольких писем с помощью Gmail без удаления клиента я обычно получал:

Сообщение: служба недоступна, закрывается канал передачи. Ответ сервера был: 4.7.0 Временная системная проблема. Повторите попытку позже (WS). oo3sm17830090pdb.64 - gsmtp