Класс и метод

[bufalo]

Разработать собственный класс, содержащий метод для вычисления стоимости покупки с учетом скидки. Скидка в 3% предоставляется, если сумма покупки больше 5000 руб., в 5% - если сумма больше 10000 руб. Результат оформить в виде:
Введите сумму покупки: = …
Вам предоставляется скидка = …% Сумма с учетом скидки составит: … руб.
заранее благодарен за помощь.

Вот мои нароботки этой программы, препод сказал, что она сделана неправильно.
Единственное, что он сказал, что скидка должна отниматься, а не прибавляться.

using System;
using System.Collections.Generic;
using System.Linq;
using System.Text;
 
namespace ConsoleApplication1
{
    class Program
    {
        static void metod (int x,double rez)
        {
            if (x > 5000 && x<10000) { rez =x-(x * 0.03); Console.WriteLine("Вам предоставляется скидка в 3%"); Console.WriteLine("Цена покупки со скидкой равна: " + "руб"+ rez); Console.ReadKey(); } 
            else if (x > 10000) { rez =x-(x * 0.05); Console.WriteLine("Вам предоставляется скидка в 5%"); Console.WriteLine("Цена со скидкой 5% равна: "+ rez); Console.ReadKey(); }
            else if (x != 5000) { Console.WriteLine("У вас нет скидки, сумма Вашей покупки равна:"+x); }
        }
 
        static void Main(string[] args)
        {
            int x;
            double rez=0;
            Console.WriteLine("Введите цену покупки: ");
            x = Convert.ToInt16(Console.ReadLine());
            metod(x,rez);
            Console.ReadKey();
        }
    }
}

[Lasur]

Неужели оно компилируется? Тут, как минимум, одна ошибка, не дающая скомпилировать проект, и одна, приводящая к неправильному ответу.
Вы ее запускали?
Вру, прога работает, просто не привык к такому коду...
Если не учитывать оформление, вроде все правильно.

Я все же выпишу замечания по коду:
1)Зачем все операторы писать в одну строку?
2)Зачем конвертить в Int16, если тип x - Int32?

x = Convert.ToInt16(Console.ReadLine());

3)Вместо

rez =x-(x * 0.03)

проще

rez =x * 0.97

4)Дерево выбора можно упростить:

if (x>10000)
{
    //Если цена больше 10000
    return;
}
if (x>5000)
{
    //Если цена между 5000 и 10000
    return;
}
//Если цена меньше или равна 5000

5)И каков смысл передавать res в метод, если можно инициализировать внутри метода?

[bufalo]

Lasur, если это все добавить и тем самым исправить программу, она будет соответствовать условию или нет?

[Lasur]

Она возвращала правильный результат и до этого.
Единственное мое предположение, почему "препод" мог сказать, что она сделана неправильно, так как увидел +rez в Console.WriteLine().

[bufalo]

ясно, спасибо.

[KobolD]

Тольк я не вижу чтобы вы собственный клас разработали.
Подразумевается что вы должны создать экземпляр класса и уже в нем вести обработку. Попробуй так.

using System;
using System.Collections.Generic;
using System.Linq;
using System.Text;

namespace ConsoleApplication1
{
    class Program
    {
        static void Main(string[] args)
        {
            Console.WriteLine("Введите цену покупки: ");
            double x = Convert.ToDouble(Console.ReadLine());
            MyClass NewItem = new MyClass(x);//Создаешь новый экземпляр класса
            NewItem.Print();//Вывод на экран информации о стоимости и скидке
            Console.ReadKey();
        }
    }

    //Вот тут ты объявляешь новый класс
    class MyClass 
    {
        //Поля класса
        private double Sum;
        private int Sk;

        //Конструктор класса
        public MyClass(double Price)
        {
            if (Price > 10000)
                Sk = 5;
            if (Price > 5000)
                Sk = 3;
            Sum = Price*(1-Sk/100);
            //Print();//Вывод на экран информации о стоимости и скидке
        }

        //Инкапсуляция
        public double Price { get { return Sum; } }
        public int Skidka { get { return Sk; } }

        // Методы класса
        public void Print()
        {
            if (Sk == 0)
                Console.WriteLine("У вас нет скидки, сумма Вашей покупки равна:" + Sum.ToString());
            else
                Console.WriteLine("Вам предоставляется скидка в {0}%. Цена с учетом скидки равна: {1}", Sk,Sum);
        }
    }
}

[bufalo]

всем спасибо!