Надійний спосіб перетворення файлу в байт []

Я знайшов такий код в Інтернеті:

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[] 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 #?

Був зроблений коментар до одного з дописів щодо .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. Вбудований метод чудовий, але варто зазначити, що код, який ви розміщуєте вище, є неміцним з двох причин:

1. Streamє IDisposable- вам слід розмістити FileStream fs = new FileStream(filename, FileMode.Open,FileAccess.Read)ініціалізацію у фразі using, щоб переконатися, що файл закритий. Якщо цього не зробити, це може означати, що потік залишається відкритим, якщо сталася помилка, що означатиме, що файл залишається заблокованим - і це може спричинити інші проблеми пізніше.
2. fs.Readможе прочитати менше байтів, ніж ви вимагаєте. Як правило, .Readметод Streamекземпляра буде читати принаймні один байт, але не обов’язково всі байти, про які ви просите. Вам потрібно буде написати цикл, який повторюватиме читання, поки не будуть прочитані всі байти. Ця сторінка пояснює це більш докладно.