3

например

List<Student> StudentsGroup = new List<Student>();

StudentsGroup.add(new Student(){Name="Vasia", Surname="Andrienko"});
StudentsGroup.add(new Student(){Name="Vasia", Surname="Potapenko"});
StudentsGroup.add(new Student(){Name="Fedia", Surname="Matsenko"});
StudentsGroup.add(new Student(){Name="Kolia", Surname="Mischenknko"});
StudentsGroup.add(new Student(){Name="Vasilisa", Surname="Vynnychenko"});
StudentsGroup.add(new Student(){Name="Vasilisa", Surname="Onoprienko"});

после очистки по Name должно выдать только следующие данные

{Name="Vasia", Surname="Andrienko"}//иль второго студента
{Name="Fedia", Surname="Matsenko"}
{Name="Kolia", Surname="Mischenknko"}
{Name="Vasilisa", Surname="Vynnychenko"}//иль второго студента
3
  • 2
    поставить MoreLinq и вызвать DistinctBy
    – vitidev
    Commented 6 апр. 2016 в 17:23
  • 1
    @vitidev , ставить отдельную библиотеку в проэкт для того, что можно сделать своим 1м методом -- это как-то по-костыльному. Да я лучше вручную метод криво напишу. :) Commented 6 апр. 2016 в 18:06
  • 1
    @vitidev, можно и обычным Distinct, передав ему IComparer
    – Grundy
    Commented 11 апр. 2016 в 13:28

4 ответа 4

7

Без использования сторонних библиотек

StudentsGroup = StudentsGroup.GroupBy(x => x.Name).Select(x => x.First()).ToList();

Взято с англоязычного SO

4
  • Спасибо, то что нужно. Немного правок и заработало. Правки внес и сюда. Commented 6 апр. 2016 в 18:08
  • @Andrew, убрал ToList() - этим стоит пользоваться очень избирательно Commented 6 апр. 2016 в 18:17
  • @Artem_Flotsky ToList() там нужен т.к. результат селекта -- IEnumerable и без него результат не будет присвоен. Добавил назад. :) Commented 6 апр. 2016 в 18:18
  • @Andrew, с присвоением да, согласен. Не хотелось бы, чтобы те, кто придет сюда с таким же вопросом, восприняли, будто ToList() нужно использовать где только возможно =) Commented 6 апр. 2016 в 18:32
3

Еще один вариант, воспользоваться перегрузкой Distinct, принимающей вторым параметром IEqualityComparer<T>

Например:

class StudentComparer : IEqualityComparer<Student>
{
    // Students are equal if their names are equal.
    public bool Equals(Student x, Student y)
    {
        //Check whether the compared objects reference the same data.
        if (Object.ReferenceEquals(x, y)) return true;

        //Check whether any of the compared objects is null.
        if (Object.ReferenceEquals(x, null) || Object.ReferenceEquals(y, null))
            return false;

        //Check whether the Student's properties are equal.
        return x.Name == y.Name;
    }

    // If Equals() returns true for a pair of objects 
    // then GetHashCode() must return the same value for these objects.

    public int GetHashCode(Student student)
    {
        //Check whether the object is null
        if (Object.ReferenceEquals(student, null)) return 0;

        //Get hash code for the Name field if it is not null.
        return student.Name == null ? 0 : student.Name.GetHashCode();
    }
}

Пример использования:

IEnumerable<Student> noduplicates = StudentsGroup.Distinct(new StudentComparer());
6
  • А не проще было бы взять StringComparer или EqualityComparer<string>.Default и делегировать непосредственно сравнение ему? Commented 27 нояб. 2016 в 11:56
  • @PavelMayorov, неа, по-моему этот код - компия из мсдн с замененным типом
    – Grundy
    Commented 27 нояб. 2016 в 11:57
  • наследование кода - это плохо. Даже если код наследуется из msdn. Commented 27 нояб. 2016 в 11:59
  • @PavelMayorov, кстати, а как сюда можно StringComparer впхнуть? Создавать экземпляр и у него вызывать equals/gethashcode? Но тогда же только вызов x.Name == y.Name упростится? вернее заменится на вызов equals = на мой взгляд не сильно большая разница
    – Grundy
    Commented 27 нояб. 2016 в 12:02
  • StringComparer реализует IEqualityComparer<string> Commented 27 нояб. 2016 в 12:03
2

Можно воспользоваться перегрузкой метода Distinct, принимающей IEqualityComparer<T>

Вместо полной реализации IEqualityComparer<Student> можно делегировать все стандартной реализации, оставив себе только обращение к свойству Name:

class StudentByNameComparer : IEqualityComparer<Student> {
  private readonly IEqualityComparer<string> cmp;

  public StudentByNameComparer(IEqualityComparer<string> cmp) {
    this.cmp = cmp;
  }

  public static StudentByNameComparer Default { get; } 
      = new StudentByNameComparer(EqualityComparer<string>.Default);
      // Также тут можно использовать StringComparer

  public bool Equals(Student x, Student y)
      => cmp.Equals(x?.Name, y?.Name);

  public int GetHashCode(Student x)
      => cmp.GetHashCode(x?.Name);
}

Пример использования:

IEnumerable<Student> noduplicates = StudentsGroup.Distinct(StudentByNameComparer.Default);
1
  • никак не привыкну к новым возможностям:)
    – Grundy
    Commented 27 нояб. 2016 в 12:12
1

Если коллекцию формируете вы, и свойство для проверки всегда одно, используйте HashSet для этих целей, и дубликаты просто не добавятся. Не забудьте, что надо будет переопределить GetHashCode для составления хэша только по изменяющемуся свойству

class Student 
{
... /// существующий код
     public override int GetHashCode()
     {
          unchecked 
          {
             int hash = 13; // любое простое число
              return 13* Name.GetHashCode();
          }
     }
}
4
  • Ну и зачем его умножать на 13?
    – Qwertiy
    Commented 11 апр. 2016 в 15:07
  • Это уже другой вопрос, выходящий за рамки текущего. Вкратце, при хэшировании используется далеко не только умножение, но и деление наряду с логическими операциями. Простые номера дают меньше коллизий, вот и всего. Commented 11 апр. 2016 в 20:33
  • 2
    Это намёк, что в данном случае этого делать не надо ;) Потому что тут нет вычисления хеша на основе нескольких хешей, а у единственного поля, чей хеш возвращается, обо всём уже позаботились. Умножение его на какое-либо число ничего не улучшает.
    – Qwertiy
    Commented 11 апр. 2016 в 20:34
  • А, ну так вы бы не начинали из-за печки) Да, конечно. Но просвещение же) Commented 12 апр. 2016 в 9:59

Ваш ответ

Нажимая «Отправить ответ», вы соглашаетесь с условиями пользования и подтверждаете, что прочитали политику конфиденциальности.

Всё ещё ищете ответ? Посмотрите другие вопросы с метками или задайте свой вопрос.