Реализация GetHashCode и Equals в словаре C #

Question

Я пришел на этот сайт в поисках сравнения объектов в словаре и узнал, что переопределение GetHashCode и Equals является обязательным условием для сравнения объектов в C #.Вот фрагмент кода, который я пытался решить, используя метод итерации FOREACH.Но мой Босс говорит, что нужно делать то же самое без использования какой-либо итерации (возможно, с помощью метода containskey или containsvalue) из-за проблем с производительностью.Любая помощь приветствуется.

  public class employee
    {
        public string empname { get; set; }
        public string location { get; set; }
        public double kinid { get; set; }
        public double managerKin { get; set; }
        public override bool Equals(object obj)
        {
            return base.Equals(obj);
        }
        public override int GetHashCode()
        {
            return base.GetHashCode();
        }
    }

    public class manager
    {
        public string managername { get; set; }
        public double kinid { get; set; }

        public override int GetHashCode() 
        { 
          return 17 * managername.GetHashCode() + kinid.GetHashCode();
        }
    }
    public class program
    {
        public static void Main()
        {
            employee emp = new employee();
            employee emp2 = new employee();
            manager mng = new manager();
            manager mng2 = new manager();

            emp.empname = "Deepak";
            emp.location = "Pune";
            emp.kinid = 36885;
            emp.managerKin = 007;


            emp2.empname = "Astha";
            emp2.location = "Pune";
            emp2.kinid = 30000;
            emp2.managerKin = 007;

            mng.kinid = 007;
            mng.managername = "Gaurav";
            mng2.kinid = 001;
            mng2.managername = "Surya";

            Dictionary<employee, manager> relations = new Dictionary<employee, manager>();
            relations.Add(emp, mng);
            relations.Add(emp2, mng2);

            Console.ForegroundColor = ConsoleColor.Yellow;
            Console.WriteLine("The Manager details are :");
            foreach (var element in relations)
            Console.WriteLine(" \n KINID : {0} \n  Manager'sName :                    {1}",element.Value.kinid, element.Value.managername);
            Console.WriteLine("Enter the details of the manager..");
            Console.ForegroundColor = ConsoleColor.Gray;
            Console.Write("\nManager's Kin : ");
            double mkin = Convert.ToDouble(Console.ReadLine());

            Console.Write("Manager's Name : ");
            string mname = Console.ReadLine();

            manager mng1 = new manager();
            mng1.kinid = mkin;
            mng1.managername = mname;
            int hashvalue = 17 * mname.GetHashCode() + mkin.GetHashCode();



            #region BY USING FOREACH LOOP
            int i = 0;
            foreach (var element in relations)
            {
                if (element.Value.GetHashCode() == hashvalue)
                {
                    i += 1;
                    if (i == 1)
                    {
                        Console.WriteLine("The Following employees report to the Manager : {0}", mname);

                    }
                    Console.WriteLine(element.Key.empname + " " + element.Key.kinid + " " + element.Key.location + " " + element.Key.managerKin);

                }
            }
            if (i == 0)
            {
                Console.WriteLine("sorry the manager's details you entered \"{0}\" \"{1}\" does not exist in our database..", mng1.managername, mng1.kinid);

            }
            #endregion

            Console.ReadLine();
        }

    }

Answer 1

Для поиска объекта в словаре с помощью ключевого слова ContainsKey или ContainsValue компилятор использует две неявные функции, а именно: GetHashCode () и Equals (). Поэтому, когда у нас есть объект для сравнения, нам нужно переопределить оба эти метода !!

Вот код

#region USING DICTIONARY TO STORE CLASS OBJECTS (check employee existence and print manager's name)
public class employee
{
    public string empname { get; set; }
    public string location { get; set; }
    public double kinid { get; set; }
    public double managerKin { get; set; }

    //public override bool Equals(object obj) // ANY OF THE TWO EQUALS METHOD WORKS.
    //{
    //    employee otheremployee;
    //    otheremployee = (employee)obj;
    //    return (otheremployee.kinid == this.kinid && otheremployee.location == this.location && otheremployee.empname == this.empname && otheremployee.managerKin == this.managerKin);

    //}
    public override bool Equals(object obj)   //When Running this entire code, put a break-point on both the Equals() and GetHashCode() methods, and see the execution flow.
    {
        employee otheremployee;
        otheremployee = (employee)obj;
        return (obj.GetHashCode() == otheremployee.GetHashCode());

    }
    public override int GetHashCode()    //When Running this entire code, put a break-point on both the Equals() and GetHashCode() methods, and see the execution flow.
    {
        //int temp = base.GetHashCode(); // DONT USE THIS
        //return base.GetHashCode();
        int temp = empname.GetHashCode() + location.GetHashCode() + kinid.GetHashCode() + managerKin.GetHashCode();
        return temp;
    }
}

public class manager
{
    public string managername { get; set; }
    public double kinid { get; set; }



    public override int GetHashCode()
    {
        return base.GetHashCode();
    }
    public override bool Equals(object obj)
    {
        return base.Equals(obj);
    }
}
public class program
{
    public static void Main()
    {
        employee emp = new employee();
        employee emp2 = new employee();
        manager mng = new manager();
        manager mng2 = new manager();

        emp.empname = "Deepak";
        emp.location = "Pune";
        emp.kinid = 36885;
        emp.managerKin = 007;


        emp2.empname = "Astha";
        emp2.location = "Pune";
        emp2.kinid = 30000;
        emp2.managerKin = 001;

        mng.kinid = 007;
        mng.managername = "Gaurav";
        mng2.kinid = 001;
        mng2.managername = "Surya";

        Dictionary<employee, manager> relations = new Dictionary<employee, manager>();
        relations.Add(emp, mng); // put a BreakPoint here and see the execution flow
        relations.Add(emp2, mng2);// put a BreakPoint here and see the execution flow

        Console.ForegroundColor = ConsoleColor.Yellow;
        Console.WriteLine("The Employee details are :");
        foreach (var element in relations)
            Console.WriteLine(" \n Employee Name : {0} \n Location : {1} \n Employee KinId : {2} \n Manager's KinId : {3} ",
                element.Key.empname, element.Key.location, element.Key.kinid, element.Key.managerKin);

        Console.WriteLine("Enter the details of the Employee..");
        Console.ForegroundColor = ConsoleColor.Gray;
        Console.Write("\nEmployee Name : "); string ename = Console.ReadLine();
        Console.Write("Location : "); string elocn = Console.ReadLine();
        Console.Write("Employee KinId : "); double ekinid = Convert.ToDouble(Console.ReadLine());
        Console.Write("Manager's ID : "); double emngr = Convert.ToDouble(Console.ReadLine());
        employee emp1 = new employee();
        emp1.empname = ename;
        emp1.location = elocn;
        emp1.kinid = ekinid;
        emp1.managerKin = emngr;


        int i = 0; // This variable acts as a indicator to find whether the Employee Key exists or not.
        if (relations.ContainsKey(emp1)) //Put a break point here and see the execution flow.
        {
            Console.WriteLine("the Employee : {0} exists..", emp1.empname);
            Console.WriteLine("the Employee reports to the following manager : {0} \n and the Manager's KinId is {1}.", (relations[emp1]).managername, relations[emp1].kinid);
            i = 1;
            Console.ReadLine();
        }

        if (i == 0)
        {
            Console.WriteLine("the details of the employee named {0} does not exist !!", emp1.empname);
            Console.ReadLine();
        }

#endregion

Answer 2

Dictionary.ContainsKey(employee) здесь не поможет, потому что сотрудник является «неизвестным» значением, а Contains не поможет, потому что он принимает KeyValuePair<employee,manager> и ... еще раз ... ни один сотрудник не известен. ContainsValue(manager) не поможет, потому что не возвращает никаких ключей и , потому что это не ключ, это операция O(n), а не O(1), как ContainsKey!

Со структурой current единственный способ - использовать некоторую форму циклов, хотя я бы написал это так:

// Key is Employee, Value is Manager
// This is O(n)
var theEmployees = relations
  .Where(rel => rel.Value.Equals(theManager))
  .Select(rel => rel.Key);

Это будет работать только после того, как manager дана правильная реализация Equals. Обратите внимание, что хеш-код не используется вообще . ( Поскольку объекты, которые отличаются , могут совместно использовать тот же самый хеш-код, просто сравнение хеш-кода не является заменой для Equals, или ==, или CompareTo! - в зависимости от того, какой из них подходит.)

Если таких запросов будет много , то исходная структура может быть "инвертирована".

// Build a reverse lookup-up
var employeesForManager = relations
  .GroupBy(rel => rel.Value)            // group on Manager
  .ToDictionary(g => g.Key, g => g);    // Key is the group's Manager

// This is O(1), but only valid AFTER employeesForManager is [re-]generated
var theEmployees = employeesForManager[theManager]

Это будет работать, только если manager имеет допустимую реализацию Equals и GetHashCode. (GetHashCode требуется, потому что manager объекты используются ключом к новому словарю.)

Что касается того, что "лучше" - хорошо, это зависит. Например, глупо создавать обратный поиск, чтобы использовать его только один раз. Нет проблем с производительностью, пока нет проблем с производительностью: напишите чистый код и профиль.

Удачного кодирования.

Answer 3

Чтобы иметь возможность сравнивать 2 экземпляра на равенство, вы должны переопределить метод Equals, и это также хорошая практика для реализации IEquatable<T>.Когда вы переопределяете Equals, вы также должны переопределить GetHashcode (это используется, когда вы помещаете свои экземпляры в словарь для вычисления сегмента).

Вам не следует использовать GetHashcode для сравнения двух экземпляровобъект для равенства;вместо этого вы должны использовать Equals (или EqualityComparer, который также будет использовать метод Equals).

Если вы хорошо реализовали GetHashCode и Equals, тогда вы сможете определить, является ли словарьсодержит конкретный экземпляр, выполнив:

var myDictionary<int, Manager> = new Dictionary<int,Manager>();

myDictionary.ContainsKey (someKey)

или

var mySet = new HashSet<Manager>();
mySet.Contains(someManagerObject);

Answer 4

Для поиска элемента в словаре вы можете использовать методы ContainsKey, ContainsValue или просто написать запрос LINQ

var dict = (from pair in relations
where pair.Value.Equals(mng1)
select pair).ToDictionary<employee,manager>();

Answer 5

Я полагаю, у вас есть ошибка в вашем окончательном ответе.

Строка

return (obj.GetHashCode () == otheremployee.GetHashCode ());

Возможно, должно быть

return (this.GetHashCode() == otheremployee.GetHashCode ());

Таким образом, вы сравниваете хеш-код для этого объекта и другого объекта.Как написано в вашем ответе, вы, кажется, сравниваете другой объект с самим собой.