Niezawodny sposób konwersji pliku na bajt []

Znalazłem następujący kod w sieci:

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
}

Czy jest wystarczająco wiarygodne, aby przekonwertować plik na bajt [] w języku C #, czy jest lepszy sposób, aby to zrobić?

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

To powinno załatwić sprawę. ReadAllBytes otwiera plik, wczytuje jego zawartość do nowej tablicy bajtów, a następnie zamyka go. Oto strona MSDN dla tej metody.

byte[] bytes = File.ReadAllBytes(filename) 

lub ...

var bytes = File.ReadAllBytes(filename) 

Nie powtarzać tego, co wszyscy już powiedzieli, ale trzymaj pod ręką następującą ściągawkę do manipulacji plikami:

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

Wszystkie te odpowiedzi z .ReadAllBytes(). Kolejne, podobne (nie powiem duplikat, ponieważ próbowali refaktoryzować swój kod) zostało zadane na SO tutaj: Najlepszy sposób na odczytanie dużego pliku do tablicy bajtów w C #?

Do jednego z postów dodano komentarz dotyczący .ReadAllBytes():

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

Dla mnie lepszym podejściem byłoby coś takiego 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;
}

Ale to tylko ja ...

Oczywiście, wszystko to zakłada, że ​​masz pamięć do obsługi byte[]po wczytaniu, a nie File.Existssprawdziłem, czy plik istnieje przed kontynuowaniem, tak jak zrobiłbyś to przed wywołaniem tego kodu.

wygląda wystarczająco dobrze jako wersja ogólna. Możesz go zmodyfikować, aby spełniał swoje potrzeby, jeśli są wystarczająco szczegółowe.

testuj również pod kątem wyjątków i warunków błędów, takich jak plik nie istnieje lub nie można go odczytać itp.

możesz także wykonać następujące czynności, aby zaoszczędzić miejsce:

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

Inni zauważyli, że możesz użyć wbudowanego File.ReadAllBytes. Wbudowana metoda jest w porządku, ale warto zauważyć, że kod, który opublikujesz powyżej, jest delikatny z dwóch powodów:

1. Streamis IDisposable- powinieneś umieścić FileStream fs = new FileStream(filename, FileMode.Open,FileAccess.Read)inicjalizację w klauzuli using, aby upewnić się, że plik jest zamknięty. Niezastosowanie się do tego może oznaczać, że strumień pozostanie otwarty, jeśli wystąpi awaria, co będzie oznaczać, że plik pozostanie zablokowany - a to może spowodować później inne problemy.
2. fs.Readmoże czytać mniej bajtów niż żądasz. Ogólnie .Readmetoda Streaminstancji odczyta co najmniej jeden bajt, ale niekoniecznie wszystkie żądane bajty. Będziesz musiał napisać pętlę, która ponowi próbę odczytu do momentu odczytania wszystkich bajtów. Ta strona wyjaśnia to bardziej szczegółowo.