แนวทางปฏิบัติที่ดีที่สุดสำหรับการใช้ 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อินสแตนซ์โดยเร็วที่สุดเสมอ ในกรณีของการโทรแบบ async จะอยู่ในการโทรกลับหลังจากส่งข้อความแล้ว

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 ที่รอ 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 ไปใช้กับอ็อบเจ็กต์มีจุดมุ่งหมายเพื่อสื่อสารความจริงที่ว่าคลาสที่มีปัญหามีทรัพยากรราคาแพงที่ควรได้รับการปลดปล่อยอย่างกำหนด อย่างไรก็ตามหากการสร้างทรัพยากรเหล่านั้นมีราคาแพงและคุณจำเป็นต้องสร้างวัตถุเหล่านี้จำนวนมากอาจเป็นการดีกว่า (ประสิทธิภาพที่ชาญฉลาด) ในการเก็บอินสแตนซ์หนึ่งรายการไว้ในหน่วยความจำและนำกลับมาใช้ใหม่ มีทางเดียวที่จะรู้ว่าสิ่งนั้นสร้างความแตกต่างหรือไม่: โปรไฟล์มัน

Re: การกำจัดและ Async: คุณไม่สามารถใช้usingอย่างชัดเจน โดยทั่วไปคุณจะทิ้งวัตถุในเหตุการณ์ SendCompleted แทน:

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

โอเคคำถามเก่าที่ฉันรู้ แต่ฉันสะดุดกับสิ่งนี้เมื่อฉันต้องการใช้สิ่งที่คล้ายกัน ฉันแค่อยากจะแชร์โค้ด

ฉันกำลังทำซ้ำหลาย ๆ SmtpClients เพื่อส่งจดหมายหลายฉบับแบบอะซิงโครนัส วิธีแก้ปัญหาของฉันคล้ายกับ TheCodeKing แต่ฉันกำลังกำจัดวัตถุเรียกกลับแทน ฉันยังส่ง MailMessage เป็น userToken เพื่อรับมันในเหตุการณ์ SendCompleted ดังนั้นฉันจึงสามารถเรียกใช้การกำจัดได้เช่นกัน แบบนี้:

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