ファイルをbyte []に​​変換する信頼できる方法

私はウェブ上で次のコードを見つけました：

private byte [] StreamFile(string filename)
{
   FileStream fs = new FileStream(filename, FileMode.Open,FileAccess.Read);

   // Create a byte array of file stream length
   byte[] ImageData = new byte[fs.Length];

   //Read block of bytes from stream into the byte array
   fs.Read(ImageData,0,System.Convert.ToInt32(fs.Length));

   //Close the File Stream
   fs.Close();
   return ImageData; //return the byte data
}

ファイルをc＃でbyte []に​​変換するために使用するのに十分な信頼性がありますか、それともこれを行うためのより良い方法がありますか？

byte[] bytes = System.IO.File.ReadAllBytes(filename);

それでうまくいくはずです。ReadAllBytesはファイルを開き、その内容を新しいバイト配列に読み込んでから閉じます。これがそのメソッドのMSDNページです。

byte[] bytes = File.ReadAllBytes(filename) 

または..。

var bytes = File.ReadAllBytes(filename) 

誰もがすでに言ったことを繰り返すのではなく、ファイル操作のために次のチートシートを手元に置いておきます。

1. System.IO.File.ReadAllBytes(filename);
2. File.Exists(filename)
3. Path.Combine(folderName, resOfThePath);
4. Path.GetFullPath(path); // converts a relative path to absolute one
5. Path.GetExtension(path);

これらすべての答えは.ReadAllBytes()。別の同様の（コードをリファクタリングしようとしていたので、重複とは言いません）質問がSOでここで尋ねられました： C＃で大きなファイルをバイト配列に読み込む最良の方法は？

以下に関する投稿の1つにコメントがありました.ReadAllBytes()。

File.ReadAllBytes throws OutOfMemoryException with big files (tested with 630 MB file 
and it failed) – juanjo.arana Mar 13 '13 at 1:31

私にとってより良いアプローチは、次のようなものですBinaryReader。

public static byte[] FileToByteArray(string fileName)
{
    byte[] fileData = null;

    using (FileStream fs = File.OpenRead(fileName)) 
    { 
        var binaryReader = new BinaryReader(fs); 
        fileData = binaryReader.ReadBytes((int)fs.Length); 
    }
    return fileData;
}

しかし、それは私だけです...

もちろん、これはすべて、byte[]一度読み込まれFile.Existsたファイルを処理するためのメモリがあることを前提としています。このコードを呼び出す前に行うように、先に進む前にファイルがそこにあることを確認するためのチェックを入れませんでした。

ジェネリック版としては十分に良さそうです。十分に具体的であれば、ニーズに合わせて変更できます。

また、ファイルが存在しない、または読み取れないなどの例外やエラー状態をテストします。

スペースを節約するために、次のこともできます。

 byte[] bytes = System.IO.File.ReadAllBytes(filename);

他の人はあなたがビルトインを使うことができると述べましたFile.ReadAllBytes。組み込みの方法は問題ありませんが、上記のコードは2つの理由で壊れやすいことに注意してください。

1. StreamですIDisposable-あなたは配置する必要がありFileStream fs = new FileStream(filename, FileMode.Open,FileAccess.Read)、ファイルが閉じていることを確認するために使用した句で初期化します。これを怠ると、障害が発生した場合にストリームが開いたままになる可能性があります。これは、ファイルがロックされたままになることを意味し、後で他の問題を引き起こす可能性があります。
2. fs.Read要求よりも少ないバイトを読み取る可能性があります。一般に、インスタンスの.ReadメソッドはStream少なくとも1バイトを読み取りますが、必ずしもすべてのバイトを読み取る必要はありません。すべてのバイトが読み取られるまで読み取りを再試行するループを作成する必要があります。 このページでは、これについて詳しく説明しています。