c# - 可以改进 IEqualityComparer 的这种实现吗？

Question

我没有看到这段代码有任何问题，但感觉就像我错过了一些东西。也许可以减少行数。或者甚至有一个错误需要修复？我愿意接受任何建议。

public class NameComparer : IEqualityComparer<FileInfo>
{
    public bool Equals (FileInfo x, FileInfo y)
    {
        if (x == null) {
            return y == null;
        }

        if (y == null) {
            return false;   
        }

        return x.Name.Equals (y.Name);
    }

    public int GetHashCode (FileInfo obj)
    {
        return obj.Name.GetHashCode ();
    }
}

Answer 1

如果 FileInfo 的相等运算符返回 true，则应首先返回 true。此外，指定要进行的字符串比较的类型。大概你想忽略大小写，因为这些是文件名。

public class NameComparer : IEqualityComparer<FileInfo>
{
   public bool Equals(FileInfo x, FileInfo y)
   {
      if (x == y)
      {
         return true;
      }

      if (x == null || y == null)
      {
         return false;
      }

      return string.Equals(x.FullName, y.FullName, StringComparison.OrdinalIgnoreCase);
   }

   public int GetHashCode (FileInfo obj)
   {
      return StringComparer.OrdinalIgnoreCase.GetHashCode(obj.FullName);
   }
}

Answer 2

Name仅当您总是比较同一目录的文件时，仅比较才会起作用。我建议FullName改为比较。

您可以通过为和FileSystemInfo的基类实现相等比较器，轻松地将其范围扩展到目录。FileInfoDirectoryInfo

public sealed class FullNameComparer : IEqualityComparer<FileSystemInfo>
{
    public bool Equals(FileSystemInfo x, FileSystemInfo y)
    {
        if (x == y)
        {
            return true;
        }

        if (x == null || y == null)
        {
            return false;
        }

        return String.Equals(x.FullName.TrimEnd('\\'), y.FullName.TrimEnd('\\'), StringComparison.OrdinalIgnoreCase);
    }

    public int GetHashCode(FileSystemInfo obj)
    {
        return obj.FullName.GetHashCode();
    }
}