ฉันควรจะเรียก Close () หรือ Dispose () สำหรับวัตถุ stream หรือไม่

การเรียนการสอนเช่นStream, StreamReader, StreamWriterฯลฯ ดำเนินการIDisposableอินเตอร์เฟซ นั่นหมายความว่าเราสามารถเรียกDispose()ใช้เมธอดบนวัตถุของคลาสเหล่านี้ พวกเขายังได้กำหนดวิธีการที่เรียกว่าpublic Close()ทีนี้ฉันจะโทรไปหาอะไรเมื่อฉันทำกับวัตถุแล้ว ถ้าฉันโทรหาทั้งคู่

รหัสปัจจุบันของฉันคือ:

using (Stream responseStream = response.GetResponseStream())
{
   using (StreamReader reader = new StreamReader(responseStream))
   {
      using (StreamWriter writer = new StreamWriter(filename))
      {
         int chunkSize = 1024;
         while (!reader.EndOfStream)
         {
            char[] buffer = new char[chunkSize];
            int count = reader.Read(buffer, 0, chunkSize);
            if (count != 0)
            {
               writer.Write(buffer, 0, count);
            }
         }
         writer.Close();
      }
      reader.Close();
   }
}

อย่างที่คุณเห็นฉันได้เขียนusing()โครงสร้างซึ่งเรียกDispose()วิธีการในแต่ละวัตถุโดยอัตโนมัติ แต่ฉันก็เรียกClose()วิธีการเช่นกัน ถูกต้องหรือไม่

กรุณาแนะนำฉันแนวทางปฏิบัติที่ดีที่สุดเมื่อใช้วัตถุกระแส :-)

ตัวอย่าง MSDN ไม่ได้ใช้using()โครงสร้างและClose()วิธีการโทร:

• วิธีการ: ดาวน์โหลดไฟล์ด้วย FTP

ดีไหม

การกระโดดเข้าไปใน Reflector.NET อย่างรวดเร็วแสดงให้เห็นว่าClose()วิธีการStreamWriterคือ:

public override void Close()
{
    this.Dispose(true);
    GC.SuppressFinalize(this);
}

และStreamReaderคือ:

public override void Close()
{
    this.Dispose(true);
}

การDispose(bool disposing)แทนที่ในStreamReaderคือ:

protected override void Dispose(bool disposing)
{
    try
    {
        if ((this.Closable && disposing) && (this.stream != null))
        {
            this.stream.Close();
        }
    }
    finally
    {
        if (this.Closable && (this.stream != null))
        {
            this.stream = null;
            /* deleted for brevity */
            base.Dispose(disposing);
        }
    }
}

StreamWriterเป็นวิธีการที่คล้ายกัน

ดังนั้นการอ่านรหัสเป็นที่ชัดเจนว่าคุณสามารถโทรClose()& Dispose()บนสตรีมได้บ่อยเท่าที่คุณต้องการและในลำดับใด ๆ มันจะไม่เปลี่ยนพฤติกรรม แต่อย่างใด

ดังนั้นมันจึงลงมาว่าจะสามารถอ่านได้มากกว่าหรือไม่ Dispose() , และClose() / หรือusing ( ... ) { ... }

ความชอบส่วนตัวของฉันคือusing ( ... ) { ... }ควรใช้เมื่อเป็นไปได้เสมอเพราะจะช่วยให้คุณ "ไม่วิ่งด้วยกรรไกร"

แต่ในขณะที่สิ่งนี้ช่วยแก้ไขความถูกต้อง แต่ก็ลดความสามารถในการอ่านได้ ใน C # เรามีวงเล็บปีกกาปิดมากมายดังนั้นเราจะทราบได้อย่างไรว่าอันที่จริงแล้วทำการปิดด้วยกระแส

ดังนั้นฉันคิดว่าเป็นการดีที่สุดที่จะทำสิ่งนี้:

using (var stream = ...)
{
    /* code */

    stream.Close();
}

มันไม่ส่งผลกระทบต่อพฤติกรรมของรหัส แต่มันช่วยให้อ่านง่าย

ไม่คุณไม่ควรเรียกวิธีการเหล่านั้นด้วยตนเอง ในตอนท้ายของusingบล็อกDispose()วิธีการจะถูกเรียกโดยอัตโนมัติซึ่งจะดูแลทรัพยากรที่ไม่มีการจัดการฟรี (อย่างน้อยสำหรับคลาส. NET BCL มาตรฐานเช่นลำธารผู้อ่าน / นักเขียน, ... ) ดังนั้นคุณสามารถเขียนรหัสของคุณเช่นนี้:

using (Stream responseStream = response.GetResponseStream())
    using (StreamReader reader = new StreamReader(responseStream))
        using (StreamWriter writer = new StreamWriter(filename))
        {
            int chunkSize = 1024;
            while (!reader.EndOfStream)
            {
                 char[] buffer = new char[chunkSize];
                 int count = reader.Read(buffer, 0, chunkSize);
                 if (count != 0)
                 {
                     writer.Write(buffer, 0, count);
                 }
            }
         }

วิธีการโทรClose()Dispose()

เอกสารบอกว่าทั้งสองวิธีนี้เทียบเท่า:

StreamReader.Close : การใช้งานของการปิดการโทรนี้ทิ้งวิธีการส่งผ่านค่าที่แท้จริง

StreamWriter.Close : การใช้งานของการปิดการโทรนี้วิธีการกำจัดผ่านค่าที่แท้จริง

Stream.Close : เมธอดนี้เรียกใช้ Dispose ระบุจริงเพื่อปล่อยทรัพยากรทั้งหมด

ดังนั้นทั้งสองสิ่งนี้ใช้ได้อย่างเท่าเทียมกัน:

/* Option 1, implicitly calling Dispose */
using (StreamWriter writer = new StreamWriter(filename)) { 
   // do something
} 

/* Option 2, explicitly calling Close */
StreamWriter writer = new StreamWriter(filename)
try {
    // do something
}
finally {
    writer.Close();
}

โดยส่วนตัวแล้วฉันจะติดกับตัวเลือกแรกเพราะมันมี "เสียง" น้อยกว่า

ในหลาย ๆ คลาสที่รองรับทั้งสองClose()และDispose()วิธีการการโทรทั้งสองจะเทียบเท่ากัน อย่างไรก็ตามในบางคลาสมีความเป็นไปได้ที่จะเปิดวัตถุที่ถูกปิดอีกครั้ง บางคลาสดังกล่าวอาจทำให้ทรัพยากรบางอย่างมีชีวิตอยู่หลังจากการปิดเพื่ออนุญาตให้เปิดใหม่; อื่น ๆ อาจไม่เก็บทรัพยากรใด ๆ ไว้Close()แต่อาจตั้งค่าสถานะDispose()เป็นห้ามการเปิดอีกครั้งอย่างชัดเจน

สัญญาที่IDisposable.Disposeกำหนดไว้อย่างชัดเจนว่าการเรียกมันบนวัตถุที่จะไม่ถูกใช้อีกจะไม่เป็นอันตรายที่สุดดังนั้นฉันขอแนะนำให้โทรอย่างใดอย่างหนึ่งIDisposable.Disposeหรือวิธีการที่เรียกว่าDispose()บนIDisposableวัตถุทุกอย่างไม่ว่าใครก็โทรมาClose()ด้วย

นี่เป็นคำถามเก่า แต่ตอนนี้คุณสามารถเขียนโดยใช้คำสั่งโดยไม่จำเป็นต้องบล็อกแต่ละรายการ พวกเขาจะถูกกำจัดในลำดับย้อนกลับเมื่อบล็อกที่มีเสร็จแล้ว

using var responseStream = response.GetResponseStream();
using var reader = new StreamReader(responseStream);
using var writer = new StreamWriter(filename);

int chunkSize = 1024;
while (!reader.EndOfStream)
{
    char[] buffer = new char[chunkSize];
    int count = reader.Read(buffer, 0, chunkSize);
    if (count != 0)
    {
        writer.Write(buffer, 0, count);
    }
}

https://docs.microsoft.com/en-us/dotnet/csharp/language-reference/proposals/csharp-8.0/using