Prawidłowy sposób asynchronicznie wysyłania wiadomości e-mail w ASP.NET ... (czy robię to dobrze?)

Od. NET 4.5 SmtpClient implementuje metodę asynchroniczną SendMailAsync. W rezultacie wysyłanie wiadomości e-mail asynchronicznie jest następujące:

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

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

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

Czy używasz. Net SmtpClient do wysyłania wiadomości e-mail? może już wysyłać wiadomości asynchroniczne .

Edit-jeśli {[0] } nie jest wrapperem nad SmtpClient, to chyba nie będzie to tak przydatne.

Threading nie jest tu złą opcją, ale jeśli sam nie poradzisz sobie z wyjątkiem, spowoduje to pęknięcie i awarię procesu. Nie ma znaczenia, na którym wątku to robisz.

Więc zamiast Mailera.SendTheMail () try this:

new Thread(() => { 
  try 
  {
    mailer.SendTheMail();
  }
  catch(Exception ex)
  {
    // Do something with the exception
  }
});

Jeszcze lepiej, Użyj asynchronicznych możliwości SmtpClient, jeśli możesz. Nadal będziesz musiał obsługiwać wyjątki.

Sugerowałbym nawet rzucenie okiem na nową bibliotekę Zadań Parallet. Net 4. Który ma dodatkową funkcjonalność co pozwala obsłużyć wyjątkowe przypadki i dobrze współpracuje z ASP.Net Pula wątków.

Jeśli używasz klas. Net SmtpClient i MailMessage, powinieneś zwrócić uwagę na kilka rzeczy. Po pierwsze, spodziewaj się błędów przy wysyłaniu, więc pułapka i obsłużyć je. Po drugie, w. Net 4 zaszły pewne zmiany w tych klasach, a obie implementują IDisposable(MailMessage od 3.5, SmtpClient nowy w 4.0). Z tego powodu tworzenie SmtpClient i MailMessage powinno być zawinięte w bloki lub jawnie usunięte. Jest to przełomowa zmiana, niektórzy ludzie nie są świadomi z.

Zobacz to pytanie, aby uzyskać więcej informacji na temat usuwania podczas korzystania z asynchronicznych wysyłek:

Jakie są najlepsze praktyki korzystania z SmtpClient, SendAsync i Dispose pod. NET 4.0

Dlaczego więc nie mieć osobnego Pollera/serwisu, który zajmuje się wyłącznie wysyłaniem maili? W ten sposób, pozwalając na rejestrację post-back, aby wykonać tylko w czasie potrzebnym do zapisu do bazy danych/kolejki wiadomości i opóźniając wysyłanie wiadomości e-mail do następnego interwału ankiety.

Zastanawiam się nad tym samym problemem właśnie teraz i myślę, że naprawdę nie chcę nawet inicjować wysyłania wiadomości e-mail w ramach żądania zwrotnego po serwerze. Proces obsługi stron WWW powinien być zainteresowany otrzymaniem odpowiedzi do Użytkownika jak najszybciej, im więcej pracy spróbujesz wykonać, tym wolniej będzie.

Spójrz na polecenie Query Segregation Principal ( http://martinfowler.com/bliki/CQRS.html ). Martin Fowler wyjaśnia, że w części poleceń operacji mogą być używane różne modele niż w części zapytań. W tym scenariuszu komendą byłoby "register user", zapytaniem byłoby e-mail aktywacyjny, używając luźnej analogii. Stosowny cytat byłby prawdopodobnie być:

Przez oddzielne modele rozumiemy najczęściej różne modele obiektowe, prawdopodobnie działające w różnych procesach logicznych

Warto też przeczytać artykuł Wikipedii o CQRS ( http://en.wikipedia.org/wiki/Command%E2%80%93query_separation ). ważnym punktem, który to podkreśla, jest:

Jest to ewidentnie zamierzona wskazówka programowa, a nie reguła dobrego kodowania

Znaczenie, użyj go tam, gdzie twój kod, program wykonanie i zrozumienie programisty byłoby korzystne. To dobry przykład.

To podejście ma dodatkową zaletę negowania wszystkich problemów związanych z gwintowaniem mufti i bólów głowy, które mogą przynieść.

Pracowałem nad tym samym problemem dla mojego projektu:

Pierwsza próba {[0] } tak jak ty:
- I loose context
- Problem z obsługą WYJĄTKÓW
- Powszechnie mówi się, Thread są złym pomysłem na IIS threadpool

Więc przełączam się i próbuję z asynchronously:
- 'asynchronicznie' jest fake w asp.net aplikacja internetowa. To po prostu umieścić wywołania kolejki i swicth kontekst

Więc robię usługę windows i pobieram wartości przez tabelę sql: happy end

Więc dla szybkiego rozwiązania: od ajax strony wykonaj wywołanie asynchroniczne powiedz użytkownikowi fake tak, ale kontynuuj Wysyłanie zadania w kontrolerze mvc

Użyj tej drogi-

private void email(object parameters)
    {
        Array arrayParameters = new object[2];
        arrayParameters = (Array)parameters;
        string Email = (string)arrayParameters.GetValue(0);
        string subjectEmail = (string)arrayParameters.GetValue(1);
        if (Email != "[email protected]")
        {
            OnlineSearch OnlineResult = new OnlineSearch();
            try
            {
                StringBuilder str = new StringBuilder();
                MailMessage mailMessage = new MailMessage();

                //here we set the address
                mailMessage.From = fromAddress;
                mailMessage.To.Add(Email);//here you can add multiple emailid
                mailMessage.Subject = "";
                //here we set add bcc address
                //mailMessage.Bcc.Add(new MailAddress("[email protected]"));
                str.Append("<html>");
                str.Append("<body>");
                str.Append("<table width=720 border=0 align=left cellpadding=0 cellspacing=5>");

                str.Append("</table>");
                str.Append("</body>");
                str.Append("</html>");
                //To determine email body is html or not
                mailMessage.IsBodyHtml = true;
                mailMessage.Body = str.ToString();
                //file attachment for this e-mail message.
                Attachment attach = new Attachment();
                mailMessage.Attachments.Add(attach);
                mailClient.Send(mailMessage);
            }

    }


  protected void btnEmail_Click(object sender, ImageClickEventArgs e)
    {
        try
        {
            string To = txtEmailTo.Text.Trim();
            string[] parameters = new string[2];
            parameters[0] = To;
            parameters[1] = PropCase(ViewState["StockStatusSub"].ToString());
            Thread SendingThreads = new Thread(email);
            SendingThreads.Start(parameters);
            lblEmail.Visible = true;
            lblEmail.Text = "Email Send Successfully ";
        }

Jeśli chcesz wykryć wycieki, musisz użyć profilera takiego jak ten:

Http://memprofiler.com/

Nie widzę nic złego z Twoim rozwiązaniem, ale mogę prawie zagwarantować, że to pytanie zostanie zamknięte jako subiektywne.

Inną opcją jest użycie jQuery do wywołania ajax do serwera i wywołania przepływu poczty e-mail. W ten sposób interfejs użytkownika nie jest zamknięty.

Powodzenia!

Matt