Надежный способ конвертировать файл в байтовый []

Я нашел в сети следующий код:

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
}

Достаточно ли он надежен для преобразования файла в byte [] в 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. Streamis IDisposable- вы должны поместить FileStream fs = new FileStream(filename, FileMode.Open,FileAccess.Read)инициализацию в предложение using, чтобы файл был закрыт. Невыполнение этого может означать, что поток останется открытым в случае сбоя, что будет означать, что файл останется заблокированным - и это может вызвать другие проблемы позже.
2. fs.Readможет прочитать меньше байтов, чем вы запрашиваете. В общем, .Readметод Streamэкземпляра будет читать по крайней мере один байт, но не обязательно все байты, которые вы запрашиваете. Вам нужно будет написать цикл, который повторяет попытку чтения, пока не будут прочитаны все байты. Эта страница объясняет это более подробно.