33

我正在检查两个字节数组的相等性，我需要一些帮助，因为即使数组应该相等，我也返回 false。

在我的调试中，我可以看到 a1 和 b1 都相等，但它并没有进入 while 循环来增加 i。

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

这总是返回 false: (a1[i]==b1[i])。

4

5 回答 5

60

您需要在某处添加返回值。这应该有效：

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

或者，您可以IStructuralEquatable从 .NET 4 使用：

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

如果性能是一个问题，我建议重写您的代码以使用Binary该类，该类专门针对这种用例进行了优化：

public bool Equality(Binary a1, Binary b1)
{
    return a1.Equals(b1);
}

我机器上的快速基准测试提供了以下统计信息：

Method                   Min         Max         Avg
binary equal:          0.868       3.076       0.933    (best)
for loop:              2.636      10.004       3.065
sequence equal:        8.940      30.124      10.258
structure equal:     155.644     381.052     170.693

下载此 LINQPad 文件以自行运行基准测试。

于 2013-08-27T18:33:37.330 回答
42

要检查相等性，您可以写：

var areEqual =  a1.SequenceEqual(b1);
于 2013-08-27T18:32:20.517 回答
6

我建议进行一些短路以使事情变得更简单，并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;
}
于 2013-08-27T19:01:13.020 回答
1

这应该有效：

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;        
}
于 2013-08-27T18:31:04.140 回答
0

您应该添加一些返回语句：

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;
}
于 2013-08-27T18:31:16.623 回答