ซ้อนกันโดยใช้คำสั่งใน C #

ฉันกำลังทำงานในโครงการ ฉันต้องเปรียบเทียบเนื้อหาของสองไฟล์และดูว่ามันตรงกันกันหรือไม่

ก่อนที่จะตรวจสอบข้อผิดพลาดและการตรวจสอบจำนวนมากร่างแรกของฉันคือ:

  DirectoryInfo di = new DirectoryInfo(Environment.CurrentDirectory + "\\TestArea\\");
  FileInfo[] files = di.GetFiles(filename + ".*");

  FileInfo outputFile = files.Where(f => f.Extension == ".out").Single<FileInfo>();
  FileInfo expectedFile = files.Where(f => f.Extension == ".exp").Single <FileInfo>();

  using (StreamReader outFile = new StreamReader(outputFile.OpenRead()))
  {
    using (StreamReader expFile = new StreamReader(expectedFile.OpenRead()))
    {
      while (!(outFile.EndOfStream || expFile.EndOfStream))
      {
        if (outFile.ReadLine() != expFile.ReadLine())
        {
          return false;
        }
      }
      return (outFile.EndOfStream && expFile.EndOfStream);
    }
  }

ดูเหมือนว่าแปลกเล็กน้อยที่จะมีusingคำสั่งซ้อนกัน

มีวิธีที่ดีกว่าในการทำเช่นนี้?

วิธีที่แนะนำในการทำเช่นนี้คือการใส่วงเล็บปีกกาเปิด{หลังจากusingคำสั่งสุดท้ายเช่นนี้:

using (StreamReader outFile = new StreamReader(outputFile.OpenRead()))
using (StreamReader expFile = new StreamReader(expectedFile.OpenRead())) 
{
    ///...
}

หากวัตถุประเภทเดียวกันคุณสามารถทำสิ่งต่อไปนี้

using (StreamReader outFile = new StreamReader(outputFile.OpenRead()), 
                    expFile = new StreamReader(expectedFile.OpenRead()))
{
    // ...
}

เมื่อIDisposables เป็นประเภทเดียวกันคุณสามารถทำสิ่งต่อไปนี้:

 using (StreamReader outFile = new StreamReader(outputFile.OpenRead()), 
     expFile = new StreamReader(expectedFile.OpenRead()) {
     // ...
 }

หน้า MSDN ในusingมีเอกสารเกี่ยวกับคุณสมบัติภาษานี้

คุณสามารถทำสิ่งต่อไปนี้ได้หรือไม่ว่าIDisposables นั้นเป็นประเภทเดียวกัน:

using (StreamReader outFile = new StreamReader(outputFile.OpenRead()))
using (StreamWriter anotherFile = new StreamReader(anotherFile.OpenRead()))
{ 
     // ...
}

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

    Test t; 
    Blah u;
    using (IDisposable x = (t = new Test()), y = (u = new Blah())) {
        // whatever...
    }

ด้วยวิธีนี้ x และ y เป็นเพียงตัวยึดตำแหน่งตัวแปรชนิด IDisposable สำหรับการใช้บล็อกที่จะใช้และคุณใช้ t และ u ในรหัสของคุณ แค่คิดว่าฉันจะพูดถึง

หากคุณต้องการเปรียบเทียบไฟล์อย่างมีประสิทธิภาพอย่าใช้ StreamReaders เลยจากนั้นไม่จำเป็นต้องใช้งาน - คุณสามารถใช้การอ่านระดับต่ำในการดึงข้อมูลบัฟเฟอร์เพื่อเปรียบเทียบ

คุณยังสามารถเปรียบเทียบสิ่งต่าง ๆ เช่นขนาดไฟล์ก่อนเพื่อตรวจจับไฟล์ต่าง ๆ อย่างรวดเร็วเพื่อช่วยให้คุณไม่ต้องอ่านข้อมูลทั้งหมดด้วย

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

Connection c = new ...; 
Transaction t = new ...;

using (new DisposableCollection(c, t))
{
   ...
}

คอนสตรัคเตอร์สำหรับ DisposableCollection เป็นอาร์เรย์ params ในกรณีนี้เพื่อให้คุณสามารถป้อนได้มากเท่าที่คุณต้องการ

คุณสามารถพูดได้ว่า:

using (StreamReader outFile = new StreamReader(outputFile.OpenRead()))
using (StreamReader expFile = new StreamReader(expectedFile.OpenRead()))
{
   ...
}

แต่บางคนอาจพบว่าอ่านยาก BTW เพื่อเพิ่มประสิทธิภาพให้กับปัญหาของคุณทำไมคุณไม่ตรวจสอบว่าขนาดไฟล์เป็นขนาดเดียวกันก่อนที่จะไปทีละบรรทัด?

คุณสามารถละทิ้งวงเล็บเหลี่ยมได้เลยยกเว้นการใช้งานภายใน:

using (StreamReader outFile = new StreamReader(outputFile.OpenRead()))
using (StreamReader expFile = new StreamReader(expectedFile.OpenRead()))
{
  while (!(outFile.EndOfStream || expFile.EndOfStream))
  {
    if (outFile.ReadLine() != expFile.ReadLine())
    {
      return false;
    }
  }
}

ฉันคิดว่าสิ่งนี้สะอาดกว่าการใช้งานประเภทเดียวกันหลายอย่างในแบบเดียวกันตามที่คนอื่น ๆ แนะนำ แต่ฉันมั่นใจว่าหลายคนจะคิดว่านี่เป็นเรื่องที่สับสน

คุณสามารถจัดกลุ่มวัตถุที่ใช้แล้วทิ้งหลายรายการในหนึ่งเดียวโดยใช้คำสั่งด้วยเครื่องหมายจุลภาค:

using (StreamReader outFile = new StreamReader(outputFile.OpenRead()), 
       expFile = new StreamReader(expectedFile.OpenRead()))
{

}

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

แก้ไข: ช้าเกินไปในการพิมพ์เพื่อแสดงตัวอย่างรหัสรวม +1 ให้กับคนอื่น ๆ

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

using (StreamReader outFile = new StreamReader(outputFile.OpenRead()))
  using (StreamReader expFile = new StreamReader(expectedFile.OpenRead()))
    while (!(outFile.EndOfStream || expFile.EndOfStream))  
       if (outFile.ReadLine() != expFile.ReadLine())    
          return false;  

ตั้งแต่C # 8.0คุณสามารถใช้การประกาศได้

using var outFile = new StreamReader(outputFile.OpenRead());
using var expFile = new StreamReader(expectedFile.OpenRead());
while (!(outFile.EndOfStream || expFile.EndOfStream))
{
    if (outFile.ReadLine() != expFile.ReadLine())
    {
         return false;
    }
}
return (outFile.EndOfStream && expFile.EndOfStream);

สิ่งนี้จะกำจัดตัวแปรที่ใช้ในตอนท้ายของขอบเขตของตัวแปรนั่นคือในตอนท้ายของวิธี

สิ่งเหล่านี้เกิดขึ้นเมื่อฉันรหัสเช่นกัน คุณสามารถย้ายคำสั่งที่สองโดยใช้คำสั่งไปยังฟังก์ชั่นอื่นได้หรือไม่?

คุณยังถามด้วยว่ามีวิธีที่ดีกว่าในการเปรียบเทียบกับไฟล์หรือไม่? ฉันชอบการคำนวณ CRC หรือ MD5 สำหรับทั้งสองไฟล์และเปรียบเทียบมัน

ตัวอย่างเช่นคุณสามารถใช้วิธีการต่อไปนี้:

public static class ByteArrayExtender
    {
        static ushort[] CRC16_TABLE =  { 
                      0X0000, 0XC0C1, 0XC181, 0X0140, 0XC301, 0X03C0, 0X0280, 0XC241, 
                      0XC601, 0X06C0, 0X0780, 0XC741, 0X0500, 0XC5C1, 0XC481, 0X0440, 
                      0XCC01, 0X0CC0, 0X0D80, 0XCD41, 0X0F00, 0XCFC1, 0XCE81, 0X0E40, 
                      0X0A00, 0XCAC1, 0XCB81, 0X0B40, 0XC901, 0X09C0, 0X0880, 0XC841, 
                      0XD801, 0X18C0, 0X1980, 0XD941, 0X1B00, 0XDBC1, 0XDA81, 0X1A40, 
                      0X1E00, 0XDEC1, 0XDF81, 0X1F40, 0XDD01, 0X1DC0, 0X1C80, 0XDC41, 
                      0X1400, 0XD4C1, 0XD581, 0X1540, 0XD701, 0X17C0, 0X1680, 0XD641, 
                      0XD201, 0X12C0, 0X1380, 0XD341, 0X1100, 0XD1C1, 0XD081, 0X1040, 
                      0XF001, 0X30C0, 0X3180, 0XF141, 0X3300, 0XF3C1, 0XF281, 0X3240, 
                      0X3600, 0XF6C1, 0XF781, 0X3740, 0XF501, 0X35C0, 0X3480, 0XF441, 
                      0X3C00, 0XFCC1, 0XFD81, 0X3D40, 0XFF01, 0X3FC0, 0X3E80, 0XFE41, 
                      0XFA01, 0X3AC0, 0X3B80, 0XFB41, 0X3900, 0XF9C1, 0XF881, 0X3840, 
                      0X2800, 0XE8C1, 0XE981, 0X2940, 0XEB01, 0X2BC0, 0X2A80, 0XEA41, 
                      0XEE01, 0X2EC0, 0X2F80, 0XEF41, 0X2D00, 0XEDC1, 0XEC81, 0X2C40, 
                      0XE401, 0X24C0, 0X2580, 0XE541, 0X2700, 0XE7C1, 0XE681, 0X2640, 
                      0X2200, 0XE2C1, 0XE381, 0X2340, 0XE101, 0X21C0, 0X2080, 0XE041, 
                      0XA001, 0X60C0, 0X6180, 0XA141, 0X6300, 0XA3C1, 0XA281, 0X6240, 
                      0X6600, 0XA6C1, 0XA781, 0X6740, 0XA501, 0X65C0, 0X6480, 0XA441, 
                      0X6C00, 0XACC1, 0XAD81, 0X6D40, 0XAF01, 0X6FC0, 0X6E80, 0XAE41, 
                      0XAA01, 0X6AC0, 0X6B80, 0XAB41, 0X6900, 0XA9C1, 0XA881, 0X6840, 
                      0X7800, 0XB8C1, 0XB981, 0X7940, 0XBB01, 0X7BC0, 0X7A80, 0XBA41, 
                      0XBE01, 0X7EC0, 0X7F80, 0XBF41, 0X7D00, 0XBDC1, 0XBC81, 0X7C40, 
                      0XB401, 0X74C0, 0X7580, 0XB541, 0X7700, 0XB7C1, 0XB681, 0X7640, 
                      0X7200, 0XB2C1, 0XB381, 0X7340, 0XB101, 0X71C0, 0X7080, 0XB041, 
                      0X5000, 0X90C1, 0X9181, 0X5140, 0X9301, 0X53C0, 0X5280, 0X9241, 
                      0X9601, 0X56C0, 0X5780, 0X9741, 0X5500, 0X95C1, 0X9481, 0X5440, 
                      0X9C01, 0X5CC0, 0X5D80, 0X9D41, 0X5F00, 0X9FC1, 0X9E81, 0X5E40, 
                      0X5A00, 0X9AC1, 0X9B81, 0X5B40, 0X9901, 0X59C0, 0X5880, 0X9841, 
                      0X8801, 0X48C0, 0X4980, 0X8941, 0X4B00, 0X8BC1, 0X8A81, 0X4A40, 
                      0X4E00, 0X8EC1, 0X8F81, 0X4F40, 0X8D01, 0X4DC0, 0X4C80, 0X8C41, 
                      0X4400, 0X84C1, 0X8581, 0X4540, 0X8701, 0X47C0, 0X4680, 0X8641, 
                      0X8201, 0X42C0, 0X4380, 0X8341, 0X4100, 0X81C1, 0X8081, 0X4040 };


        public static ushort CalculateCRC16(this byte[] source)
        {
            ushort crc = 0;

            for (int i = 0; i < source.Length; i++)
            {
                crc = (ushort)((crc >> 8) ^ CRC16_TABLE[(crc ^ (ushort)source[i]) & 0xFF]);
            }

            return crc;
        }

เมื่อคุณทำเสร็จแล้วมันค่อนข้างง่ายที่จะเปรียบเทียบไฟล์:

public bool filesAreEqual(string outFile, string expFile)
{
    var outFileBytes = File.ReadAllBytes(outFile);
    var expFileBytes = File.ReadAllBytes(expFile);

    return (outFileBytes.CalculateCRC16() == expFileBytes.CalculateCRC16());
}

คุณสามารถใช้คลาส System.Security.Cryptography.MD5 ในตัว แต่แฮชที่คำนวณได้เป็นไบต์ [] ดังนั้นคุณยังคงต้องเปรียบเทียบทั้งสองอาร์เรย์

นอกจากนี้หากคุณทราบเส้นทางแล้วไม่มีจุดใดที่จะสแกนไดเรกทอรี

ฉันขอแนะนำบางสิ่งเช่นนี้แทน:

string directory = Path.Combine(Environment.CurrentDirectory, @"TestArea\");

using (StreamReader outFile = File.OpenText(directory + filename + ".out"))
using (StreamReader expFile = File.OpenText(directory + filename + ".exp"))) 
{
    //...

Path.Combine จะเพิ่มโฟลเดอร์หรือชื่อไฟล์ลงในพา ธ และตรวจสอบให้แน่ใจว่ามีแบ็กสแลชหนึ่งอันระหว่างพา ธ และชื่อ

File.OpenTextจะเปิดไฟล์และสร้างได้StreamReaderในครั้งเดียว

โดยการนำหน้าสตริงด้วย @ คุณสามารถหลีกเลี่ยงการใช้แบ็กสแลชทุกครั้ง (เช่น@"a\b\c")

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

using(var uow = new UnitOfWorkType1(), uow2 = new UnitOfWorkType2()){}.

ถ้าใครรู้ว่าทำไมสิ่งนี้ ไม่ถูกต้องโปรดแจ้งให้เราทราบ

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

ถูกใช้แล้ว

using (StreamReader outFile = new StreamReader(outputFile.OpenRead()))
  {
    using (StreamReader expFile = new StreamReader(expectedFile.OpenRead()))
    {
      while (!(outFile.EndOfStream || expFile.EndOfStream))
      {
        if (outFile.ReadLine() != expFile.ReadLine())
        return false;
      }
    }
  }

ตัวเลือกที่ 1

using (StreamReader outFile = new StreamReader(outputFile.OpenRead()))
    using (StreamReader expFile = new StreamReader(expectedFile.OpenRead()))
    {
      while (!(outFile.EndOfStream || expFile.EndOfStream))
      {
        if (outFile.ReadLine() != expFile.ReadLine())
        return false;
      }
    }
  }

ตัวเลือก 2

using (StreamReader outFile = new StreamReader(outputFile.OpenRead()),
                    expFile = new StreamReader(expectedFile.OpenRead()))
   {
      while (!(outFile.EndOfStream || expFile.EndOfStream))
       {
         if (outFile.ReadLine() != expFile.ReadLine())
         return false;
       }
    }