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
    6 апр 2016 в 17:23
  • 1
    @vitidev , ставить отдельную библиотеку в проэкт для того, что можно сделать своим 1м методом -- это как-то по-костыльному. Да я лучше вручную метод криво напишу. :) 6 апр 2016 в 18:06
  • 1
    @vitidev, можно и обычным Distinct, передав ему IComparer
    – Grundy
    11 апр 2016 в 13:28

4 ответа 4

7

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

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

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

4
  • Спасибо, то что нужно. Немного правок и заработало. Правки внес и сюда. 6 апр 2016 в 18:08
  • @Andrew, убрал ToList() - этим стоит пользоваться очень избирательно 6 апр 2016 в 18:17
  • @Artem_Flotsky ToList() там нужен т.к. результат селекта -- IEnumerable и без него результат не будет присвоен. Добавил назад. :) 6 апр 2016 в 18:18
  • @Andrew, с присвоением да, согласен. Не хотелось бы, чтобы те, кто придет сюда с таким же вопросом, восприняли, будто ToList() нужно использовать где только возможно =) 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 и делегировать непосредственно сравнение ему? 27 ноя 2016 в 11:56
  • @PavelMayorov, неа, по-моему этот код - компия из мсдн с замененным типом
    – Grundy
    27 ноя 2016 в 11:57
  • наследование кода - это плохо. Даже если код наследуется из msdn. 27 ноя 2016 в 11:59
  • @PavelMayorov, кстати, а как сюда можно StringComparer впхнуть? Создавать экземпляр и у него вызывать equals/gethashcode? Но тогда же только вызов x.Name == y.Name упростится? вернее заменится на вызов equals = на мой взгляд не сильно большая разница
    – Grundy
    27 ноя 2016 в 12:02
  • StringComparer реализует IEqualityComparer<string> 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
    27 ноя 2016 в 12:12
1

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

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

Ваш ответ

By clicking “Отправить ответ”, you agree to our terms of service and acknowledge that you have read and understand our privacy policy and code of conduct.

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