6

我有通过HTTPWebRequest. 在我将其解析NewtonSoft.Deserialize为自定义类型（具有公共字符串属性的简单类）后，我想使用这些数据来操作LINQ- 更具体地说，我想对数据进行分组。

我的问题是，如果我按单个字符串属性分组，分组工作正常

from x in myList
group x by x.myStr into grp
select grp;

因为我想按更多列分组，所以我返回一个自定义类型

new MyType { a = ..., b = ... }

然而，该小组没有工作。我认为原因一定是编译器不知道如何比较这些对象-所以如果实现了这种类型IEqualityComparer<MyType>，它将解决它。

但是不，它仍然没有相应地分组，它创建了几个具有完全相同字符串值的键。

我分组的自定义类型类似于

public class MyType
{
    public string a;
    public string b;
    public string c;
}

关于我缺少什么的任何想法？

这是上述场景的一个具体示例：

//The type that models the data returned from the web service
public class MyClass
{
    public string a { get; set; }

    public string b { get; set; }

    public string c { get; set; }

    public DateTime d { get; set; }

    public DateTime e { get; set; }
}

// the type by which I want to group my data
public class MyGroup : IEquatable<MyGroup>, IEqualityComparer<MyGroup>
{
    public string f1 { get; set; }

    public DateTime d1 { get; set; }

    public DateTime d2 { get; set; }

    public bool Equals(MyGroup other)
    {
        return string.Compare(this.f1, other.f1) == 0;
    }

    public bool Equals(MyGroup x, MyGroup y)
    {
        return string.Compare(x.f1, y.f1) == 0;
    }

    public int GetHashCode(MyGroup obj)
    {
        return obj.GetHashCode();
    }
}    
    List<MyClass> l = new List<MyClass>();
    l.Add(new MyClass { a = "aaa", b = "bbb", c = "ccc", d = DateTime.ParseExact("20081405", "yyyyddMM", Thread.CurrentThread.CurrentCulture), e = DateTime.ParseExact("20140101", "yyyyddMM", Thread.CurrentThread.CurrentCulture) });
    l.Add(new MyClass { a = "aaaa", b = "bbb", c = "ccc", d = DateTime.ParseExact("20090105", "yyyyddMM", Thread.CurrentThread.CurrentCulture), e = DateTime.ParseExact("20140201", "yyyyddMM", Thread.CurrentThread.CurrentCulture) });
    l.Add(new MyClass { a = "aa", b = "bbbb", c = "cccc", d = DateTime.ParseExact("20081405", "yyyyddMM", Thread.CurrentThread.CurrentCulture), e = DateTime.ParseExact("20140201", "yyyyddMM", Thread.CurrentThread.CurrentCulture) });
    l.Add(new MyClass { a = "aaa", b = "bbbbb", c = "ccc", d = DateTime.ParseExact("20121111", "yyyyddMM", Thread.CurrentThread.CurrentCulture), e = DateTime.ParseExact("20140101", "yyyyddMM", Thread.CurrentThread.CurrentCulture) });
    l.Add(new MyClass { a = "aaaaa", b = "bbb", c = "ccc", d = DateTime.ParseExact("20081405", "yyyyddMM", Thread.CurrentThread.CurrentCulture), e = DateTime.ParseExact("20140101", "yyyyddMM", Thread.CurrentThread.CurrentCulture) });
    l.Add(new MyClass { a = "aaaa", b = "bbbbb", c = "ccc", d = DateTime.ParseExact("20121111", "yyyyddMM", Thread.CurrentThread.CurrentCulture), e = DateTime.ParseExact("20140101", "yyyyddMM", Thread.CurrentThread.CurrentCulture) });
    l.Add(new MyClass { a = "aaaa", b = "bbbb", c = "cccccc", d = DateTime.ParseExact("20081405", "yyyyddMM", Thread.CurrentThread.CurrentCulture), e = DateTime.ParseExact("20140201", "yyyyddMM", Thread.CurrentThread.CurrentCulture) });
    l.Add(new MyClass { a = "aaaaa", b = "bbb", c = "cccc", d = DateTime.ParseExact("20090105", "yyyyddMM", Thread.CurrentThread.CurrentCulture), e = DateTime.ParseExact("20140301", "yyyyddMM", Thread.CurrentThread.CurrentCulture) });
    l.Add(new MyClass { a = "aaa", b = "bbb", c = "cccc", d = DateTime.ParseExact("20081405", "yyyyddMM", Thread.CurrentThread.CurrentCulture), e = DateTime.ParseExact("20140201", "yyyyddMM", Thread.CurrentThread.CurrentCulture) });

    //The following does not really group
    //IEnumerable<IGrouping<MyGroup, MyClass>> r = from x in l
    IEnumerable<IGrouping<string, MyClass>> r = from x in l
                                                //group x by new MyGroup { f1 = x.a /*, d1 = x.d, d2 = x.e*/ } into grp
                                                orderby x.a
                                                group x by x.a into grp
                                                select grp;

    //foreach (IGrouping<MyGroup, MyClass> g in r)
    foreach (IGrouping<string, MyClass> g in r)
    {
        //Console.WriteLine(g.Key.f1);

        Console.WriteLine(g.Key);
    }
4

3 回答 3

6

我认为原因一定是编译器不知道如何比较这些对象-所以如果实现了这种类型IEqualityComparer<MyType>，它将解决它。

实际上，要在 Linq 函数中使用自定义“相等”检查，您需要实现IEquatable<T>. IEquatable<T>用于将一个对象的实例与另一个相同类型的对象进行比较 - 而IEqualityProvider<T>旨在由外部类实现以比较两个任意Ts（和/或具有确定“相等”的多种方法）。

请注意，您还应该实现Object.Equalsand Object.GetHashCode-IEquatable<T>只允许您以类型安全的方式进行比较。

为什么需要覆盖Object'sEquals和GetHashCode？

确保用于比较两个对象的任何方法（Object.Equals(object)、静态方法Object.Equals(object, object等）都是一致的。并且任何时候你覆盖Equals，你也应该覆盖GetHashCode以确保对象可以正确地存储在一个基于哈希的集合中，比如 a Dictionaryor HashSet。

这是什么意思 IEquitable 仅以类型安全的方式进行比较？

使用 时IEquatable<T>，您要比较的对象保证是 a T（或 的子类型T），而使用 时Object.Equals，您不知道另一个对象的类型，必须先检查它的类型。

例如：

// IEquatable<T>.Equals()
public bool Equals(MyGroup other)
{
    return string.Compare(this.f1, other.f1) == 0;
}

相对

// Object.Equals()
public bool Equals(object other)
{
    // need to check the type of the passed in object
    MyGroup grp = other as MyGroup;

    // other is not a MyGroup
    if(grp == null return false);        

    return string.Compare(this.f1, grp.f1) == 0;

    // you could also use
    //    return this.Equals(grp);
    // as a shortcut to reuse the same "equality" logic
}
于 2014-07-15T14:27:37.650 回答
3

关于我缺少什么的任何想法？

就像是：

public class MyType : IEquatable<MyType>
{
  public string a;
  public string b;
  public string c;

  public bool Equals(MyType other)
  {
    if (other == null)
      return false;

    if (GetType() != other.GetType()) // can be omitted if you mark the CLASS as sealed
      return false;

    return a == other.a && b == other.b && c == other.c;
  }

  public override bool Equals(object obj)
  {
    return Equals(obj as MyType);
  }

  public override int GetHashCode()
  {
    int hash = 0;
    if (a != null)
      hash ^= a.GetHashCode();
    if (b != null)
      hash ^= b.GetHashCode();
    if (c != null)
      hash ^= c.GetHashCode();
    return hash;
  }
}

补充：请注意，上面是可变的，如果其中一个字段,和被重新分配MyType，则哈希码会发生变化。如果在实例被保存在等时发生重新分配，那将是有问题的。abcDictionary<MyType, whatever>HashSet<MyType>

或者，您可以按照 DavidG 的回答中的建议“分组”匿名类型，或“分组依据” Tuple.Create(.. , .. , ..)。

于 2014-07-15T14:43:29.937 回答
1

使用匿名类型进行分组。各个值本身似乎是字符串（即已经内置比较器的简单类型），因此除了返回之外您不需要自定义类型。这样就不用担心了IEqualityComparer。

from x in myList
group x by new { x.myStr, x.otherStr, x.AnotherStr } into grp
select new MyTpe 
{
    a = grp.Key.myStr, 
    b = grp.Key.otherStr, 
    c = grp.Key.AnotherStr
};
于 2014-07-15T14:28:42.367 回答