3

辞書でオブジェクトの比較を検索してこのサイトにアクセスしましたが、C#でオブジェクトの比較を行うには、GetHashCodeとEqualsをオーバーライドする必要があることがわかりました。これは、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();
        }

    }
4

5 に答える 5

3

コンパイラは、ContainsKeyまたはContainsValueキーワードを使用してディクショナリ内のオブジェクトを検索するために、GetHashCode()とEquals()という2つの暗黙的な関数を使用します。したがって、比較対象のオブジェクトがある場合は、これらの両方のメソッドをオーバーライドする必要があります。

これがコードです

#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
于 2011-12-23T15:21:05.433 に答える
1

Dictionary.ContainsKey(employee)従業員は「不明」な値であるため、ここでは役に立ちContainsませんKeyValuePair<employee,manager>。また、...もう一度...従業員が不明であるため、役に立ちません。キーをContainsValue(manager)返さないため、またキーではないため、操作であり、類似ではないため、役に立ちません。O(n)O(1)ContainsKey

現在の構造では、次のように記述しますが、唯一の方法は何らかの形式のループを使用することです。

// 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にのみ機能します。manager(オブジェクトは新しいディクショナリのキーとして使用されるため、GetHashCodeが必要です。)

どちらが「より良い」かについては-まあ、それは状況次第です。たとえば、逆引き参照を1回だけ使用するように作成するのはばかげています。パフォーマンスの問題が発生するまで、パフォーマンスの問題はありません。クリーンなコードとプロファイルを記述します。

ハッピーコーディング。

于 2011-12-23T07:26:48.383 に答える
1

辞書内の要素を検索するには、ContainsKey、ContainsValueメソッドを使用するか、LINQクエリを記述します。

var dict = (from pair in relations
where pair.Value.Equals(mng1)
select pair).ToDictionary<employee,manager>();
于 2011-12-23T07:15:48.643 に答える
1

2つのインスタンスが等しいかどうかを比較できるようにするには、Equalsメソッドをオーバーライドする必要があります。また、を実装することもお勧めしますIEquatable<T>。Equalsをオーバーライドするときは、GetHashcodeもオーバーライドする必要があります(これは、インスタンスを辞書に入れてバケットを計算するときに使用されます)。

GetHashcodeオブジェクトの2つのインスタンスが等しいかどうかを比較するために、自分自身を使用しないでください。代わりにEquals(またはEqualityComparer、Equalsメソッドも使用する)を使用する必要があります。

GetHashCodeとEqualsを適切に実装している場合は、次の手順を実行することで、ディクショナリに特定のインスタンスが含まれているかどうかを判断できます。

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

myDictionary.ContainsKey (someKey)

また

var mySet = new HashSet<Manager>();
mySet.Contains(someManagerObject);
于 2011-12-23T07:36:31.567 に答える
0

最終的な回答にバグがあると思います。

この線

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

おそらく

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

このようにして、このオブジェクトと他のオブジェクトのハッシュコードを比較します。応答に書かれているように、他のオブジェクトをそれ自体と比較しているように見えます。

于 2013-05-09T17:30:37.447 に答える