SmtpClient.Send को कॉल करते समय हमें समय-समय पर "एक असीमित कॉल पहले से प्रगति पर क्यों मिलता है"?

Question

हमारे पास कुछ (सिंक्रोनस) ईमेल कोड है, जो एक वर्ग बनाता है जो एक SmtpClient बनाता है, और उसके बाद एक ईमेल भेजता है। SmtpClient का पुन: उपयोग नहीं किया जाता है; लेकिन हम हर अब और फिर निम्न अपवाद प्राप्त करें:

System.Web.HttpUnhandledException (0x80004005): Exception of type 'System.Web.HttpUnhandledException' was thrown. ---> System.InvalidOperationException: An asynchronous call is already in progress. It must be completed or canceled before you can call this method. 
    at System.Net.Mail.SmtpClient.Send(MailMessage message) 
    at EmailSender.SendMail(MailAddress fromMailAddress, string to, String subject, String body, Boolean highPriority) in ...\EmailSender.cs:line 143 

कोड इस तरह दिखता है:

// ... 
var emailSender = new EmailSender(); 
emailSender.SendMail(toEmail, subject, body, true); 
// emailSender not used past this point 
// ... 

public class EmailSender : IEmailSender 
{ 
    private readonly SmtpClient smtp; 

    public EmailSender() 
    { 
     smtp = new SmtpClient(); 
    } 

    public void SendMail(MailAddress fromMailAddress, string to, string subject, string body, bool highPriority) 
    { 
     if (fromMailAddress == null) 
      throw new Exception(); 
     if (to == null) 
      throw new ArgumentException("No valid recipients were supplied.", "to"); 

     // Mail initialization 
     var mailMsg = new MailMessage 
     { 
      From = fromMailAddress, 
      Subject = subject, 
      Body = body, 
      IsBodyHtml = true, 
      Priority = (highPriority) ? MailPriority.High : MailPriority.Normal 
     }; 

     mailMsg.To.Add(to); 


     smtp.Send(mailMsg); 
    } 
} 

Answer 1

मेरा अनुमान है कि SmtpClient से अधिक संदेशों समवर्ती भेजने के लिए तैयार नहीं किया गया है।

मैं इस बजाय तरह वर्ग बदल जाएगा: वहाँ वास्तव में निर्माता में ग्राहक बनाने के लिए कोई कारण नहीं हैं

public class EmailSender : IEmailSender 
{ 
    Queue<MailMessage> _messages = new Queue<MailMessage>(); 
    SmtpClient _client = new SmtpClient(); 

    public EmailSender() 
    { 
    } 

    public void SendMail(MailAddress fromMailAddress, string to, string subject, string body, bool highPriority) 
    { 
     if (fromMailAddress == null) 
      throw new ArgumentNullException("fromMailAddress"); 
     if (to == null) 
      throw new ArgumentException("No valid recipients were supplied.", "to"); 

     // Mail initialization 
     var mailMsg = new MailMessage 
     { 
      From = fromMailAddress, 
      Subject = subject, 
      Body = body, 
      IsBodyHtml = true, 
      Priority = (highPriority) ? MailPriority.High : MailPriority.Normal 
     }; 

     mailMsg.To.Add(to); 

     lock (_messages) 
     { 
      _messages.Enqueue(mailMsg); 
      if (_messages.Count == 1) 
      { 
       ThreadPool.QueueUserWorkItem(SendEmailInternal); 
      } 
     } 
    } 

    protected virtual void SendEmailInternal(object state) 
    { 
     while (true) 
     { 
      MailMessage msg; 
      lock (_messages) 
      { 
       if (_messages.Count == 0) 
        return; 
       msg = _messages.Dequeue(); 
      } 

      _client.Send(msg) 
     } 
    } 
} 

के रूप में।

मैंने यह भी बदल दिया कि कक्षा ArgumentNullException फेंकता है और ExceptionfromMailAddress शून्य है। एक खाली Exception ज्यादा नहीं कहना है ..

अद्यतन

कोड भेजना अभी (और SmtpClient पुन: उपयोग) के लिए एक थ्रेड पूल धागा प्रयोग करते हैं।

Answer 2

आप Dispose, using या अपने वर्ग EmailSender के लिए डिस्पोजेबल पैटर्न को लागू करने से का उपयोग कर SmtpClient के निपटान के लिए की जरूरत है (जो और अधिक यहाँ क्योंकि आप निर्माता में EmailSender के जीवनकाल के लिए SmtpClient के जीवनकाल प्रयास कर रहे हैं उचित।)

कि इस अपवाद का समाधान हो सकता है।