C# class definition HELP

Question

This is a project question that i just cant seem to answer

using System;

namespace ConsoleApplication2
{
    internal class Equipment : IComparable
    {
        private readonly string type;
        private readonly int serialNo;
        private string colour;
        public decimal cost;

        public Equipment(string type, int serialNo)
        {
            this.type = type == null ? "" : type.Trim();
            this.serialNo = serialNo;
        }

        public string Key
        {
            get { return type + ":" + serialNo; }
        }

        int IComparable.CompareTo(object obj)
        {
            return 0;
        }
    }
}

(a) Override the appropriate method o ensure that different instances of the class that represent the same equipment item will be considered the same in the system.

(b) Override the appropriate method to enable instances of this class to be stored (and found) by key in a hash table

The key point is that the methods you should be looking for aren't found in this class directly, but rather in the class that this class inherits from. — Amy B, Aug 15 '11 at 12:35

score 3 · Answer 1 · answered Aug 15 '11 at 12:34

3

You should override the Equals and GetHashCode methods for this purpose.

answered Aug 15 '11 at 12:34

platon

5,310
1
22
24

score 1 · Answer 2 · edited May 23 '17 at 11:55

1

Override Equals() with an appropriate logic of comparision
Override GetHashCode(), see GetHashCode Guidelines in C#

You must start reading this before doing such a task Why is it important to override GetHashCode when Equals method is overriden in C#?

edited May 23 '17 at 11:55

Community

1
1

answered Aug 15 '11 at 12:34

sll

61,540
22
104
156

score 0 · Accepted Answer · edited May 23 '17 at 11:55

Writing GetHashCode manually is not that easy. Anyhow, that's code generated for this purpose by ReSharper. It's a complete solution. (It should be contained within your class definition of course). But what would you say, if you were asked - why and how it works? It might be embarassing.

So, apart from GetHashCode and Equals, which others have suggested you reading about, you might also look up http://msdn.microsoft.com/en-us/library/system.object.referenceequals.aspx as well as http://msdn.microsoft.com/en-us/library/a569z7k8(v=VS.100).aspx

As for the mystery behind 397 in GetHashCode, have a look at this question here on StackOverflow: Why is '397' used for ReSharper GetHashCode override?

        public bool Equals(Equipment other)
        {
            if (ReferenceEquals(null, other))
            {
                return false;
            }
            if (ReferenceEquals(this, other))
            {
                return true;
            }
            return Equals(other.colour, colour) && other.cost == cost && other.serialNo == serialNo && Equals(other.type, type);
        }

        public override bool Equals(object obj)
        {
            if (ReferenceEquals(null, obj))
            {
                return false;
            }
            if (ReferenceEquals(this, obj))
            {
                return true;
            }
            if (obj.GetType() != typeof (Equipment))
            {
                return false;
            }
            return Equals((Equipment) obj);
        }

// note: if "Override the appropriate method to enable instances of this class
// to be stored (and found) by key in a hash table" is supposed to mean that only type and
// serialNo should be taken into account (since they are used to generate
// the Key value) - just remove the lines with cost and colour
        public override int GetHashCode()
        {
            unchecked
            {
                int result = (colour != null ? colour.GetHashCode() : 0);
                result = (result*397) ^ cost.GetHashCode();
                result = (result*397) ^ serialNo;
                result = (result*397) ^ (type != null ? type.GetHashCode() : 0);
                return result;
            }
        }

C# class definition HELP

3 Answers3