Modo affidabile per convertire un file in un byte []

Ho trovato il seguente codice sul web:

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
}

È abbastanza affidabile da usare per convertire un file in byte [] in c # o c'è un modo migliore per farlo?

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

Questo dovrebbe fare il trucco. ReadAllBytes apre il file, ne legge il contenuto in un nuovo array di byte, quindi lo chiude. Ecco la pagina MSDN per quel metodo.

byte[] bytes = File.ReadAllBytes(filename) 

o ...

var bytes = File.ReadAllBytes(filename) 

Non ripetere ciò che tutti hanno già detto, ma tieni a portata di mano il seguente cheat sheet per le manipolazioni dei file:

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

Tutte queste risposte con .ReadAllBytes(). Un'altra domanda simile (non dico duplicata, poiché stavano cercando di rifattorizzare il loro codice) è stata posta su SO qui: Il modo migliore per leggere un file di grandi dimensioni in un array di byte in C #?

È stato fatto un commento su uno dei post riguardanti .ReadAllBytes():

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

Un approccio migliore, per me, sarebbe qualcosa del genere, con 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;
}

Ma sono solo io ...

Ovviamente, tutto questo presuppone che tu abbia la memoria per gestirlo byte[]una volta letto, e non ho inserito il File.Existscontrollo per assicurarti che il file sia lì prima di procedere, come avresti fatto prima di chiamare questo codice.

sembra abbastanza buono come versione generica. Puoi modificarlo per soddisfare le tue esigenze, se sono abbastanza specifiche.

verifica anche le eccezioni e le condizioni di errore, come il file non esiste o non può essere letto, ecc.

puoi anche fare quanto segue per risparmiare spazio:

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

Altri hanno notato che puoi usare il built-in File.ReadAllBytes. Il metodo integrato va bene, ma vale la pena notare che il codice che pubblichi sopra è fragile per due motivi:

1. Streamis IDisposable- dovresti inserire l' FileStream fs = new FileStream(filename, FileMode.Open,FileAccess.Read)inizializzazione in una clausola using per assicurarti che il file sia chiuso. In caso contrario, il flusso rimane aperto se si verifica un errore, il che significa che il file rimane bloccato e ciò può causare altri problemi in seguito.
2. fs.Readpotrebbe leggere meno byte di quelli richiesti. In generale, il .Readmetodo di Streamun'istanza leggerà almeno un byte, ma non necessariamente tutti i byte richiesti. Dovrai scrivere un ciclo che riprova a leggere finché tutti i byte non vengono letti. Questa pagina lo spiega in modo più dettagliato.