Verificando byte sono 0x00

Question

Qual è il più leggibile (e idiomatica) per scrivere questo metodo?

private bool BytesAreValid(byte[] bytes) { 
    var t = (bytes[0] | bytes[1] | bytes[2]); 
    return t != 0; 
} 

Ho bisogno di una funzione che mette alla prova i primi tre byte di un file che non è cominciare 00 00 00.

Non ho eseguito molta manipolazione dei byte. Il codice sopra non sembra corretto per me, dal momento che t è dedotto dal tipo Int32.

Answer 1

t è di tipo-dedotta per essere un Yup Int32

, perché il | operatore (come la maggior parte degli operatori) non è definito per byte - i byte sono promossi al int valori. (Vedere la sezione 7.11.1 della specifica C# 4 per i dettagli.)

Ma dato che si desidera solo confrontarlo con 0, va bene comunque.

Personalmente avevo appena lo scrivo come:

return bytes[0] != 0 && bytes[1] != 0 && bytes[2] != 0; 

O anche:

return (bytes[0] != 0) && (bytes[1] != 0) && (bytes[2] != 0); 

Entrambi questi sembrano più chiaro per me.

Answer 2

di anticipare le lunghezze degli array di variabili e evitare Indicazione eccezioni riferimento null:

private bool BytesAreValid(byte[] bytes) 
{ 
    if (bytes == null) return false; 

    return !Array.Exists(bytes, x => x == 0); 
} 

non Linq versione:

private bool BytesAreValid(byte[] bytes) 
{ 
    if (bytes == null) return false; 

    for (int i = 0; i < bytes.Length; i++) 
    { 
     if (bytes[i] == 0) return false; 
    } 
    return true; 
} 

Answer 3

private bool BytesAreValid(byte[] bytes) { 
    return !bytes.Take(3).SequenceEqual(new byte[] { 0, 0, 0 }); 
}