Überprüfen, ob Bytes 0x00 sind

Question

Was ist am besten lesbar (und idiomatisch), um diese Methode zu schreiben?

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

Ich brauche eine Funktion, die die ersten drei Bytes einer Datei prüft, dass es nicht mit 00 00 00 beginnen wird.

Habe nicht viel Byte-Manipulation getan. Der obige Code scheint mir nicht korrekt zu sein, da t vom Typ Int32 abgeleitet wird.

Answer 1

t typ gefolgert ein Int32

Yup sein, da die Bediener | (wie die meisten Operatoren) nicht für byte definiert ist - der Bytes gefördert int Werte. (Siehe Abschnitt 7.11.1 der C# 4 Spezifikation für Details.)

Aber da Sie es nur mit 0 vergleichen wollen, ist das in Ordnung.

Persönlich würde ich schreibe es genauso:

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

Oder auch:

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

Beide scheinen klarer zu mir.

Answer 2

variable Array Längen zu antizipieren und null Bezug Ausnahmen vermeiden:

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

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

Non-Linq-Version:

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