Como incorporar o campo List <string> em GroupBy ()

Aug 19 2020

Eu tenho uma lista de objetos Diff. Diff se parece com isto

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

}

Minha consulta LINQ não faz o que eu acho que fará. Eu estou passando diff.propertiesna GroupBy(), que é uma lista e eu quero que grupo quando todos os valores de cadeia em uma partida lista

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();

A única diferença entre resultse diffsé que o singular DayOfWeekque foi armazenado anteriormente diffsagora é colocado no DaysOfWeekcampo de plural (mas apenas 1 item na lista). Atualmente, o mesmo número de itens em resultse diffs.

O que eu gostaria de ver na lista de resultados são:

  1. Uma lista mais curta que consolida com base na correspondência de todos os agrupamentos (incluindo valores de lista de diff.properties)
  2. Isso também significaria mais de 1 item na lista DaysOfWeek.

Minha pergunta é:

Como posso alterar minha consulta do LINQ acima para ver o que desejo ver results?

Respostas

4 TravisJ Aug 19 2020 at 06:02

O agrupamento que você está usando com o tipo anônimo contém um List<string>que está fazendo com que você obtenha um conjunto desagrupado 1-1.

Você precisa também

  • use uma classe personalizada para o agrupamento e sobrecarga GetHashCodee Equalsconforme mostrado nesta pergunta: Usando LINQ GroupBy para agrupar por objetos de referência em vez de objetos de valor

- OU -

  • compor uma string a partir dos valores, ou um subconjunto deles, sendo selecionado ( diff.properties, diff.Action, diff.Type, diff.startdatetime, diff.enddatetime, diff.rateId, diff.rateDescription) que servirá como uma chave única para o grupo
2 RufusL Aug 19 2020 at 07:16

Algumas de suas GroupBypropriedades são tipos de referência, e o comparador padrão para esses tipos é uma comparação de referência, portanto, nenhuma delas irá corresponder. Para superar isso, podemos escrever nossos próprios EqualityComparerpara a Diffclasse para que possamos compará-los à nossa maneira:

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;
    }
}

E, finalmente, podemos usar este comparador personalizado em nosso GroupBymétodo:

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

Eu resolvi!

Ler outra pergunta e os comentários + respostas nesta pergunta me ajudaram a descobrir!

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;
        }
    }

E fiz esta edição no 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();