Instructions d'utilisation imbriquées en C#

Question

Je travaille sur un projet. Je dois comparer le contenu de deux fichiers et voir s'ils correspondent exactement l'un à l'autre.

Avant un grand nombre de contrôles d'erreurs et de validations, mon premier projet est :

  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);
    }
  }

Il semble un peu étrange d'avoir imbriqué des using déclarations.

Y a-t-il une meilleure façon de procéder ?

Answer 1

La façon préférée de procéder est de ne mettre qu'une entretoise d'ouverture { après le dernier using comme ceci :

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

Answer 2

Si les objets sont de type même type vous pouvez faire ce qui suit

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

Answer 3

Lorsque le IDisposable sont du même type, vous pouvez procéder comme suit :

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

La page MSDN sur using contient de la documentation sur cette fonctionnalité du langage.

Vous pouvez effectuer les opérations suivantes, que le IDisposable sont du même type :

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

Answer 4

Si cela ne vous dérange pas de déclarer les variables de votre bloc using avant le bloc using, vous pouvez les déclarer toutes dans la même instruction using.

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

De cette façon, x et y sont juste des variables de type IDisposable que le bloc using peut utiliser et vous utilisez t et u dans votre code. J'ai juste pensé que je devais le mentionner.

Answer 5

Si vous voulez comparer les fichiers efficacement, n'utilisez pas du tout les StreamReaders, et alors les utilisations ne sont pas nécessaires - vous pouvez utiliser des lectures de flux de bas niveau pour extraire des tampons de données à comparer.

Vous pouvez également comparer des éléments comme la taille du fichier en premier lieu pour détecter rapidement les différents fichiers et vous éviter de devoir lire toutes les données.