Jak włączyć pole List <string> w GroupBy ()

Aug 19 2020

Mam listę obiektów Diff Diff wygląda tak

public class Diff
{
    ChangeAction Action // ChangeAction is an Enum
    string Type
    DateTime StartDateTime
    DateTime EndDateTime
    int rateId
    string rateDescription
    List<string> properties
    DayOfWeek day
    List<DaysOfWeek> DaysOfWeek
    DayOfWeek DayOfWeek

}

Moje zapytanie LINQ nie robi tego, co myślę, że zrobi. Przekazuję diff.propertieselement GroupBy(), który jest listą i chcę, aby był grupowany, gdy wszystkie wartości ciągów na liście są zgodne

var results = diffs
    .GroupBy(diff => new { diff.properties, diff.Action, diff.Type, diff.StartDateTime, 
                           diff.EndDateTime, diff.rateId, diff.rateDescription}) 
    .Select(group => new Diff(
        group.Key.Action,
        group.Key.ScheduleType,
        group.Key.StartDateTime,
        group.Key.EndDateTime,
        group.Key.rateId,
        group.Key.rateDescription,
        group.Key.properties,
        group
            .Select(ts => ts.DayOfWeek)
            .Distinct()
            .OrderBy(dow => dow)
            .ToList()))
    .ToList();

Jedyna różnica między resultsi diffspolega na tym, że liczba pojedyncza, DayOfWeekktóra była wcześniej przechowywana w, diffsjest teraz umieszczana w polu liczby mnogiej DaysOfWeek(ale tylko 1 pozycja na liście). Obecnie ta sama liczba elementów w obu resultsi diffs.

To, co chciałbym zobaczyć na liście wyników, to:

  1. Krótsza lista, która konsoliduje się na podstawie dopasowywania wszystkich grup (w tym wartości listy diff.properties)
  2. Oznaczałoby to również więcej niż 1 pozycję na liście DaysOfWeek.

Moje pytanie brzmi:

Jak mogę zmienić powyższe zapytanie LINQ, aby zobaczyć, co chcę zobaczyć results?

Odpowiedzi

4 TravisJ Aug 19 2020 at 06:02

Grupowanie, którego używasz z typem anonimowym, zawiera znak, List<string>który powoduje, że otrzymujesz niezgrupowany zestaw 1-1.

Ty też musisz

  • użyj niestandardowej klasy do grupowania i przeciążania GetHashCodei Equalsjak pokazano w tym pytaniu: Używanie LINQ GroupBy do grupowania według obiektów referencyjnych zamiast obiektów wartości

- LUB -

  • skomponuj ciąg z wartości lub ich podzbioru, wybierając ( diff.properties, diff.Action, diff.Type, diff.startdatetime, diff.enddatetime, diff.rateId, diff.rateDescription), który będzie służył jako unikalny klucz do grupowania z
2 RufusL Aug 19 2020 at 07:16

Niektóre z twoich GroupBywłaściwości to typy referencyjne, a domyślną funkcją porównującą dla tych typów jest porównanie odniesień, więc żadna z nich nigdy nie będzie pasować. Aby temu zaradzić, możemy napisać własne EqualityComparerdla Diffklasy, abyśmy mogli je porównać na swój własny sposób:

public class DiffEqualityComparer : IEqualityComparer<Diff>
{
    public bool Equals(Diff first, Diff second)
    {
        if (first == null || second == null) return ReferenceEquals(first, second);

        if (first.Properties == null && second.Properties != null) return false;
        if (first.Properties != null && second.Properties == null) return false;
        if (first.Properties != null && second.Properties != null &&
            !first.Properties.OrderBy(p => p)
                .SequenceEqual(second.Properties.OrderBy(p => p)))
            return false;
        if (!first.Action.Equals(second.Action)) return false;
        if (!string.Equals(first.Type, second.Type)) return false;
        if (!first.Start.Equals(second.Start)) return false;
        if (!first.End.Equals(second.End)) return false;
        if (!first.RateId.Equals(second.RateId)) return false;
        if (!string.Equals(first.RateDescription, second.RateDescription)) return false;

        return true;
    }

    public int GetHashCode(Diff obj)
    {
        var hash = obj.Properties?.Aggregate(0,
            (accumulator, current) => accumulator * 17 + current.GetHashCode()) ?? 0;
        hash = hash * 17 + obj.Action.GetHashCode();
        hash = hash * 17 + obj.Type?.GetHashCode() ?? 0;
        hash = hash * 17 + obj.Start.GetHashCode();
        hash = hash * 17 + obj.End.GetHashCode();
        hash = hash * 17 + obj.RateId.GetHashCode();
        hash = hash * 17 + obj.RateDescription?.GetHashCode() ?? 0;

        return hash;
    }
}

I wreszcie możemy użyć tego niestandardowego modułu porównującego w naszej GroupBymetodzie:

var results = diffs
    .GroupBy(diff => new DiffEqualityComparer())
    .Select( // rest of code omitted 
1 surprised_ferret Aug 19 2020 at 06:48

Rozwiązałem to!

Przeczytanie innego pytania i komentarzy + odpowiedzi w tym pytaniu pomogło mi to zrozumieć!

public class DiffComparer : IEqualityComparer<Diff>
    {
        public bool Equals(Diff x, Diff y)
        {
            return x.Action == y.Action &&
                x.Type == y.Type &&
                x.StartDateTime == y.StartDateTime &&
                x.EndDateTime == y.EndDateTime &&
                x.rateId== y.rateId &&
                x.rateDescription == y.rateDescription &&
                x.properties.SequenceEqual(y.properties);
        }

        public int GetHashCode(Diff x)
        {
            int hash = 17;

            hash = hash * 23 + x.Action.GetHashCode();
            hash = hash * 23 + x.Type.GetHashCode();
            hash = hash * 23 + x.StartDateTime .GetHashCode();
            hash = hash * 23 + x.EndDateTime.GetHashCode();
            hash = hash * 23 + x.rateId.GetHashCode();
            hash = hash * 23 + x.rateDescription.GetHashCode();

            foreach (string prop in x.properties)
            {
                hash = hash * 31 + prop.GetHashCode();
            }

            return hash;
        }
    }

Dokonałem tej edycji w LINQ:


var results = diffs
    .GroupBy(diff => diff, new DiffComparer()) 
    .Select(group => new Diff(
        group.Key.Action,
        group.Key.ScheduleType,
        group.Key.StartDateTime,
        group.Key.EndDateTime,
        group.Key.rateId,
        group.Key.rateDescription,
        group.Key.properties,
        group
            .Select(ts => ts.DayOfWeek)
            .Distinct()
            .OrderBy(dow => dow)
            .ToList()))
    .ToList();