Jakie są najważniejsze wskazówki dotyczące używania SmtpClient, SendAsync i Dispose w .NET 4.0

Jestem trochę zakłopotany tym, jak zarządzać SmtpClient teraz, gdy jest on jednorazowy, zwłaszcza jeśli wykonuję połączenia za pomocą SendAsync. Przypuszczalnie nie powinienem wywoływać Dispose, dopóki SendAsync nie zakończy się. Ale czy powinienem to kiedykolwiek nazwać (np. Używając „używając”). Scenariusz to usługa WCF, która okresowo wysyła wiadomości e-mail, gdy wykonywane są połączenia. Większość obliczeń jest szybka, ale wysłanie wiadomości e-mail może zająć około sekundy, więc preferowany byłby Async.

Czy powinienem tworzyć nowego SmtpClient za każdym razem, gdy wysyłam pocztę? Czy powinienem utworzyć jeden dla całego programu WCF? Wsparcie!

Aktualizacja Jeśli ma to znaczenie, każdy e-mail jest zawsze dostosowany do użytkownika. Usługa WCF jest hostowana na platformie Azure, a Gmail jest używany jako poczta.

Uwaga: .NET 4.5 SmtpClient implementuje async awaitablemetodę SendMailAsync. W przypadku niższych wersji użyj SendAsynczgodnie z opisem poniżej.

Zawsze powinieneś IDisposablejak najszybciej pozbywać się instancji. W przypadku wywołań asynchronicznych jest to wywołanie zwrotne po wysłaniu wiadomości.

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);

To trochę denerwujące, że SendAsyncnie przyjmuje oddzwonienia.

Pierwotne pytanie dotyczyło platformy .NET 4, ale jeśli to pomaga, od wersji .NET 4.5 SmtpClient implementuje metodę async awaitable SendMailAsync.

W rezultacie asynchroniczne wysyłanie wiadomości e-mail wygląda następująco:

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);
        }
    }
}

Lepiej unikać używania metody SendAsync.

Ogólnie rzecz biorąc, obiekty IDisposable powinny zostać usunięte tak szybko, jak to możliwe; implementacja IDisposable na obiekcie ma na celu zakomunikowanie faktu, że dana klasa zawiera kosztowne zasoby, które powinny być deterministycznie uwalniane. Jeśli jednak tworzenie tych zasobów jest kosztowne i musisz skonstruować wiele z tych obiektów, może być lepiej (z punktu widzenia wydajności) zachować jedną instancję w pamięci i użyć jej ponownie. Jest tylko jeden sposób, aby dowiedzieć się, czy to robi jakąkolwiek różnicę: profiluj to!

Re: disposing and Async: usingoczywiście nie możesz użyć . Zamiast tego zazwyczaj usuwasz obiekt w zdarzeniu SendCompleted:

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

Ok, stare pytanie, które znam. Ale sam się na to natknąłem, gdy potrzebowałem zaimplementować coś podobnego. Chciałem tylko udostępnić jakiś kod.

Iteruję przez kilka SmtpClients, aby wysłać kilka wiadomości e-mail asynchronicznie. Moje rozwiązanie jest podobne do TheCodeKing, ale zamiast tego usuwam obiekt wywołania zwrotnego. Przekazuję również MailMessage jako userToken, aby uzyskać go w zdarzeniu SendCompleted, więc mogę również wywołać dispose. Lubię to:

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);
}

Możesz zobaczyć, dlaczego pozbycie się SmtpClient jest szczególnie ważne, po następującym komentarzu:

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();

W moim scenariuszu wysyłania wielu wiadomości za pomocą Gmaila bez usuwania klienta otrzymywałem:

Komunikat: Usługa niedostępna, zamykanie kanału transmisji. Odpowiedź serwera brzmiała: 4.7.0 Tymczasowy problem z systemem. Spróbuj ponownie później (WS). oo3sm17830090pdb.64 - gsmtp