Переопределение методов класса object

@kristinaya · Регистрация: 14.06.2014

Author24 — интернет-сервис помощи студентам

Привет!
Написала программу, но в коде есть ошибка. Не могу ее найти. Помогите мне, пожалуйста!

Условие:
Создать класс с закрытыми полями а и b, строковой переменной,
означающей операцию и свойством С. Свойство - значение выражения над
полями а и b (выражение и типы полей - см. в таблице 1). Поля
инициализировать при создании объекта. Выполнить переопределение метода
Equals(с одним и двумя параметрами) для сравнения объектов и метода
ToString()для вывода состояния объекта.
В программе должна выполняться проверка всех разработанных
элементов класса.

Тип полей а, b: int;
Выражения: --,++,/
Равенство объектов: По свойству С для всех операций;

C#

using System;
 
namespace ConsoleApplication1
{
 
    class MyClass
    {
 
        int a, b;
        string op;
 
        public int A
        {
            get
            {
                return a;
            }
            set
            {
                a = value;
            }
        }
        public int B
        {
            get
            {
                return b;
            }
            set
            {
                b = value;
            }
        }
        public string OP
        {
            get
            {
                return op;
            }
            set
            {
                op = value;
            }
        }
 
        public int C
        {
            get
            {
                switch (op)
                {
                    case "++": { a++; return a; }
 
                    case "--": { a--; return a; }
                    case "/":
                        return a / b;
                    default:
                        return 0;
                }
            }
        }
 
        public MyClass(int a, int b, string op)
        {
            this.a = a;
            this.b = b;
            this.op = op;
        }
 
        public override bool Equals(object obj)
        {
            if (obj == null || GetType() != obj.GetType())
                return false;
            MyClass temp = (MyClass)obj;
            return C == temp.C; // возвращаем результат сравнения 
        }
 
        new public bool Equals(object obj1, object obj2)
        {
            if ((obj1 == null || GetType() != obj1.GetType()) || (obj2 == null || GetType() != obj2.GetType()))
                return false;
            MyClass temp1 = (MyClass)obj1;
            MyClass temp2 = (MyClass)obj2;
            return temp1.C == temp2.C; // возвращаем результат сравнения
        }
 
        public override int GetHashCode()
        {
            return op.GetHashCode();
        }
 
        public override string ToString()
        {
            return string.Format("MyClass\t a = {0}\t b = {1}\t операция: {2}", a, b, op);
        }
    }
    class Program
    {
 
        public static void Main()
        {
            Console.WriteLine("Введите a1 и b1:");
            int a1 = int.Parse(Console.ReadLine());
            int b1 = int.Parse(Console.ReadLine());
            Console.WriteLine("Введите операцию ++, — или /");
            string op1 = Console.ReadLine();
            Console.WriteLine("Введите a2 и b2:");
            int a2 = int.Parse(Console.ReadLine());
            int b2 = int.Parse(Console.ReadLine());
            Console.WriteLine("Введите операцию ++, — или /");
            string op2 = Console.ReadLine();
            MyClass mc1 = new MyClass(a1, b1, op1);
            MyClass mc2 = new MyClass(a2, b2, op2);
            Console.WriteLine(mc1);
            Console.WriteLine(mc2);
            Console.WriteLine("Результат операции: {0} {1} {2} = {3}", mc1.A, mc1.OP, mc1.B, mc1.C);
            Console.WriteLine("Результат операции: {0} {1} {2} = {3}", mc2.A, mc2.OP, mc2.B, mc2.C);
 
            if (mc1 == mc2) Console.WriteLine(" Объекты равны");
            else Console.WriteLine(" Объекты не равны");
 
            if (mc1.Equals(mc2))
                Console.WriteLine("Объекты равны");
            else
                Console.WriteLine("Объекты не равны");
            if (Equals(mc1, mc2))
                Console.WriteLine("Объекты равны");
            else
                Console.WriteLine("Объекты не равны");
        }
    }
}

@Metall_Version · 14.06.2014, 18:34

в чем ошибка?
и зачем сравнивать свойство C ?

@kristinaya · 14.06.2014, 18:54 **[ТС]**

Сообщение от Metall_Version

в чем ошибка?

Операции ++ и -- выполняются неверно, при ++ вообще не понятно, как подсчет происходит, а при -- в ответе получается 0 всегда.

Сообщение от Metall_Version

и зачем сравнивать свойство C ?

В моем задании написано именно так, как я описала в первом своем сообщении. Понятия не имею, зачем это. Преподаватель требует.

@Metall_Version · 14.06.2014, 19:17

kristinaya, во первых в гетере нельзя изменять состояние объекта .. это противоречит логике. этот гетер в процессе вызывается много раз и значение "а" у вас каждый раз изменяется...

C#

            get
            {
                switch (op)
                {
                    case "++": return a + 1;
 
                    case "--": return a - 1;
                    case "/":
                        return a / b;
                    default:
                        return 0;
                }
            }

Добавлено через 1 минуту
во вторых операция -- работает
вы наверное не ту команду вводите

C#

1	Console.WriteLine("Введите операцию ++, - или /")

просите ввести один дефис "-" , а нужно два "--"

@Metall_Version 2151 / 1288 / 516 Регистрация: 04.03.2014 Сообщений: 4,092
	14.06.2014, 18:34	2
	в чем ошибка? и зачем сравнивать свойство C ? 0

@kristinaya 0 / 0 / 0 Регистрация: 14.06.2014 Сообщений: 4
	14.06.2014, 18:54 [ТС]	3
	Сообщение от Metall_Version в чем ошибка? Операции ++ и -- выполняются неверно, при ++ вообще не понятно, как подсчет происходит, а при -- в ответе получается 0 всегда. Сообщение от Metall_Version и зачем сравнивать свойство C ? В моем задании написано именно так, как я описала в первом своем сообщении. Понятия не имею, зачем это. Преподаватель требует. 0

	14.06.2014, 19:17