Question

我有一个传递到web服务的过滤器列表，我遍历集合并执行Linq查询，然后添加到产品列表中，但是当我执行GroupBy和Distinct()时不会删除重复项。我使用的是IEnumerable，因为当您使用Disinct时，它会将其转换为IEnumerable。如果您知道如何更好地构建它并使我的函数返回List<Product>类型，那将非常感谢。

这是我在C＃中的代码：

if (Tab == "All-Items")
{
    List<Product> temp = new List<Product>();
    List<Product> Products2 = new List<Product>();
    foreach (Filter filter in Filters)
    {                        
        List<Product> products = (from p in db.Products where p.Discontinued == false
                                  && p.DepartmentId == qDepartment.Id
                                  join f in db.Filters on p.Id equals f.ProductId
                                  join x in db.ProductImages on p.Id equals x.ProductId
                                  where x.Dimension == "180X180"
                                  && f.Name == filter.Name /*Filter*/
                                  select new Product
                                  {
                                      Id = p.Id,
                                      Title = p.Title,
                                      ShortDescription = p.ShortDescription,
                                      Brand = p.Brand,
                                      Model = p.Model,
                                      Image = x.Path,
                                      FriendlyUrl = p.FriendlyUrl,
                                      SellPrice = p.SellPrice,
                                      DiscountPercentage = p.DiscountPercentage,
                                      Votes = p.Votes,
                                      TotalRating = p.TotalRating
                                  }).ToList<Product>();

        foreach (Product p in products)
        {
            temp.Add(p);
        }                        

        IEnumerable temp2 = temp.GroupBy(x => x.Id).Distinct();
        IEnumerator e = temp.GetEnumerator();
        while (e.MoveNext()) {
            Product c = e.Current as Product;
            Products2.Add(c);
        }
    }
    pf.Products = Products2;// return type must be List<Product>                
}

Answer 1

您需要覆盖Equals 和GetHashCode ，以便按值比较Product。

Answer 2

您将GroupBy结果分配给temp2，但之后您永远不会使用它！

IEnumerable temp2 = temp.GroupBy(x => x.Id).Distinct();         
IEnumerator e = temp.GetEnumerator();   // <- why temp and not temp2???

但你根本没有正确使用GroupBy运算符。您需要分组，然后从每个组中选择一个项目，而不是选择不同的组。这些组已经不同。

当你按照ID进行分组时，你实际上会返回一个像这样的结构

-Key: 1
   -Product (Id = 1)
-Key: 2
   -Product (Id = 2)
   -Product (Id = 2)

您需要从每个组中检索项目。假设是重复，你不会关心你得到哪个项目，所以你可以从每个组中取出第一个项目。

var groups = temp.GroupBy(x => x.Id);
var distinctItems = groups.Select(g => g.First());

foreach (var item in distinctItems)
{
    // do stuff with item
    Products2.Add(item);
}

Answer 3

SLaks是对的，您必须覆盖Equals（）和GetHashCode（）方法才能完成此任务。

以下是我在我写的小程序中的Order对象中的做法。希望这有帮助！

    //overridden Equals() method from the Object class, determines equality based on order number
    public override bool Equals(object obj)
    {
        bool equal;
        if (this.GetType() != obj.GetType())
            equal = false;
        else
        {
            Order temp = (Order)obj;
            if(OrderNumber == temp.OrderNumber)
                equal = true;
            else
                equal = false;
        }
        return equal;
    }

    //overridden GetHashCode() method from Object class, sets the unquie identifier to OrderNumber
    public override int GetHashCode()
    {
        return OrderNumber;
    }

Distinct（）返回List＆lt;＆gt;返回重复项

3 个答案: