区别于不使用LINQ to Objects

Question

class Program
{
    static void Main(string[] args)
    {
        List<Book> books = new List<Book> 
        {
            new Book
            {
                Name="C# in Depth",
                Authors = new List<Author>
                {
                    new Author 
                    {
                        FirstName = "Jon", LastName="Skeet"
                    },
                     new Author 
                    {
                        FirstName = "Jon", LastName="Skeet"
                    },                       
                }
            },
            new Book
            {
                Name="LINQ in Action",
                Authors = new List<Author>
                {
                    new Author 
                    {
                        FirstName = "Fabrice", LastName="Marguerie"
                    },
                     new Author 
                    {
                        FirstName = "Steve", LastName="Eichert"
                    },
                     new Author 
                    {
                        FirstName = "Jim", LastName="Wooley"
                    },
                }
            },
        };


        var temp = books.SelectMany(book => book.Authors).Distinct();
        foreach (var author in temp)
        {
            Console.WriteLine(author.FirstName + " " + author.LastName);
        }

        Console.Read();
    }

}
public class Book
{
    public string Name { get; set; }
    public List<Author> Authors { get; set; }
}
public class Author
{
    public string FirstName { get; set; }
    public string LastName { get; set; }
    public override bool Equals(object obj)
    {
        return true;
        //if (obj.GetType() != typeof(Author)) return false;
        //else return ((Author)obj).FirstName == this.FirstName && ((Author)obj).FirstName == this.LastName;
    }

}

这基于"LINQ in Action"中的示例.代码清单4.16

这两次打印Jon Skeet.为什么？我甚至尝试在Author类中重写Equals方法.仍然不同似乎没有用.我错过了什么？

编辑:我也添加了==和!=运算符重载.仍然没有帮助.

 public static bool operator ==(Author a, Author b)
    {
        return true;
    }
    public static bool operator !=(Author a, Author b)
    {
        return false;
    }

Answer 1

LINQ Distinct在自定义对象方面并不聪明.

它所做的就是查看你的列表并看到它有两个不同的对象(它们并不关心它们对于成员字段具有相同的值).

一个解决办法是实现IEquatable接口如图所示这里.

如果你像这样修改你的Author类它应该工作.

public class Author : IEquatable<Author>
{
    public string FirstName { get; set; }
    public string LastName { get; set; }

    public bool Equals(Author other)
    {
        if (FirstName == other.FirstName && LastName == other.LastName)
            return true;

        return false;
    }

    public override int GetHashCode()
    {
        int hashFirstName = FirstName == null ? 0 : FirstName.GetHashCode();
        int hashLastName = LastName == null ? 0 : LastName.GetHashCode();

        return hashFirstName ^ hashLastName;
    }
}

尝试使用DotNetFiddle

Answer 2

该Distinct()方法检查引用类型的引用相等性.这意味着它正在寻找重复的相同对象,而不是包含相同值的不同对象.

有一个带有IEqualityComparer的重载,因此您可以指定不同的逻辑来确定给定的对象是否等于另一个.

如果希望Author通常表现得像普通对象(即只有引用相等),但为了通过名称值检查相等性,请使用IEqualityComparer.如果您总是希望根据名称值比较Author对象,则覆盖GetHashCode和Equals,或实现IEquatable.

IEqualityComparer界面上的两个成员是Equals和GetHashCode.确定两个Author对象是否相等的逻辑似乎是First和Last名称字符串是否相同.

public class AuthorEquals : IEqualityComparer<Author>
{
    public bool Equals(Author left, Author right)
    {
        if((object)left == null && (object)right == null)
        {
            return true;
        }
        if((object)left == null || (object)right == null)
        {
            return false;
        }
        return left.FirstName == right.FirstName && left.LastName == right.LastName;
    }

    public int GetHashCode(Author author)
    {
        return (author.FirstName + author.LastName).GetHashCode();
    }
}

Answer 3

而不实施另一种解决方案IEquatable,Equals并且GetHashCode是使用LINQs GroupBy方法和选择来自IGrouping的第一项.

var temp = books.SelectMany(book => book.Authors)
                .GroupBy (y => y.FirstName + y.LastName )
                .Select (y => y.First ());

foreach (var author in temp){
  Console.WriteLine(author.FirstName + " " + author.LastName);
}

Answer 4

还有一种方法可以从用户定义的数据类型列表中获取不同的值:

YourList.GroupBy(i => i.Id).Select(i => i.FirstOrDefault()).ToList();

当然,它会提供不同的数据集

Answer 5

Distinct()对枚举中的对象执行默认的相等比较.如果你还没有覆盖Equals()和GetHashCode(),然后它使用的默认实现object,这比较引用.

简单的解决办法是增加一个正确的实施Equals()和GetHashCode()向参与您比较(即图书和作者)对象图的所有类.

该IEqualityComparer接口是方便,可让您实现Equals()并GetHashCode()在一个单独的类,当你不能够访问你需要比较,或者如果您使用比较不同的方法类的内部结构.

Answer 6

你已经覆盖了Equals(),但要确保你也覆盖了GetHashCode()

Answer 7

以上答案都是错误的!MSDN上声明的区别是返回默认的Equator,如上所述,Default属性检查类型T是否实现System.IEquatable接口,如果是,则返回使用该实现的EqualityComparer.否则,它返回一个EqualityComparer,它使用由T提供的Object.Equals和Object.GetHashCode的覆盖.

这意味着只要你超越Equals就可以了.

您的代码无效的原因是您检查firstname == lastname.

请参阅https://msdn.microsoft.com/library/bb348436(v=vs.100).aspx和https://msdn.microsoft.com/en-us/library/ms224763(v=vs.100).aspx

Answer 8

代替

var temp = books.SelectMany(book => book.Authors).Distinct();

做

var temp = books.SelectMany(book => book.Authors).DistinctBy(f => f.Property);