दो बाइट एरे के लिए समानता की जांच

Question

मैं दो बाइट एरे की समानता की जांच कर रहा हूं, और मुझे कुछ मदद चाहिए क्योंकि सरणी बराबर होती है, भले ही मैं झूठ लौटाता हूं।

मेरे डीबग के भीतर मैं देख सकता था कि ए 1 और बी 1 दोनों बराबर हैं, लेकिन यह लूप को बढ़ाने के लिए अंदर नहीं जा रहा है।

public bool Equality(byte[] a1, byte[] b1) 
{ 
    int i; 
    bool bEqual; 
    if (a1.Length == b1.Length) 
    { 
     i = 0; 
     while ((i < a1.Length) && (a1[i]==b1[i])) 
     { 
      i++; 
     } 

     if (i == a1.Length) 
     { 
      bEqual = true; 
     } 
    } 
    return bEqual; 
} 

यह हमेशा झूठा लौटाता है: (a1[i]==b1[i])।

Answer 1

आपको कहीं भी वापसी मूल्य जोड़ने की आवश्यकता है। यह काम करना चाहिए:

public bool Equality(byte[] a1, byte[] b1) 
{ 
    int i; 
    if (a1.Length == b1.Length) 
    { 
     i = 0; 
     while (i < a1.Length && (a1[i]==b1[i])) //Earlier it was a1[i]!=b1[i] 
     { 
      i++; 
     } 
     if (i == a1.Length) 
     { 
      return true; 
     } 
    } 

    return false; 
} 

लेकिन यह बहुत सरल है:

return a1.SequenceEqual(b1); 

वैकल्पिक रूप से, आप नेट 4 से IStructuralEquatable इस्तेमाल कर सकते हैं:

return ((IStructuralEquatable)a1).Equals(b1, StructuralComparisons.StructuralEqualityComparer) 

Answer 2

यह काम करना चाहिए:

public bool Equality(byte[] a1, byte[] b1) 
{ 
    if(a1 == null || b1 == null) 
     return false; 
    int length = a1.Length; 
    if(b1.Length != length) 
     return false; 
    while(length >0) { 
     length--; 
     if(a1[length] != b1[length]) 
      return false;   
    } 
    return true;   
} 

Answer 3

आप कुछ बदले बयान जोड़ना चाहिए:

public bool Equality(byte[] a1, byte[] b1) 
{ 
    int i = 0; 
    if (a1.Length == b1.Length) 
    { 
     while ((i < a1.Length) && (a1[i]==b1[i])) 
     { 
      i++; 
     } 
    } 
    return i == a1.Length; 
} 

या, बेहतर अभी तक

public bool Equality(byte[] a1, byte[] b1) 
{ 
    if(a1.Length != b1.Length) 
    { 
     return false; 
    } 

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

Answer 4

समानता की जाँच करने के लिए आप बस लिख सकते हैं:

var areEqual = a1.SequenceEqual(b1); 

Answer 5

मैं कुछ शॉर्ट सर्किट की सलाह देते हैं चीजों को थोड़ा सा सरल बनाने के लिए, और object.ReferenceEquals का उपयोग शॉर्ट-सर्किट में मामलों के लिए करते हैं जब सरणी एक ही संदर्भ होते हैं (a1 = b1):

public bool Equality(byte[] a1, byte[] b1) 
{ 
    // If not same length, done 
    if (a1.Length != b1.Length) 
    { 
     return false; 
    } 

    // If they are the same object, done 
    if (object.ReferenceEquals(a1,b1)) 
    { 
     return true; 
    } 

    // Loop all values and compare 
    for (int i = 0; i < a1.Length; i++) 
    { 
     if (a1[i] != b1[i]) 
     { 
      return false; 
     } 
    } 

    // If we got here, equal 
    return true; 
}