1

Разбираюсь потихоньку в C#. Писала калькулятор winForm, появилась необходимость вынести всю логику в отдельный класс и вызывать его метод уже при нажатии на кнопку.
Код:

private void button1_Click(object sender, EventArgs e)
{
    double first, second;
    first = Convert.ToDouble(textBox1.Text);
    second = Convert.ToDouble(textBox2.Text);

    switch (comboBox1.Text)
    {
        case "+":
            textBox3.Text = Convert.ToString(first + second);
            break;

        case "-":
            textBox3.Text = Convert.ToString(first - second);
            break;

        case "*":
            textBox3.Text = Convert.ToString(first * second);
            break;
    }

}

Подскажите, как я могу это реализовать? Мне необходимо создать отдельный класс, к примеру calculate. Адаптировать код под него к примеру:

double first, second;
string rezult;
switch (rezult)
{
    case "+":
        Console.WriteLine(first + second);
        break;

    case "-":
        Console.WriteLine(first - second);
        break;
    
    case "*":
        Console.WriteLine(first * second);
        break;
}

Но думаю не совсем корректно во втором случае написан код, подскажите пожалуйста как будет правильнее и как я смогу вызвать метод из сторонего класса при нажатии на кнопку?

1 ответ 1

2

Класс в который инкапсулируется парсинг значений, парсинг операци и все вычисления:

public class CalcEngine
{ 
    public string Proceed(string action, string firstVar, string secondVar)
    {
        double first = Convert.ToDouble(firstVar);
        double second = Convert.ToDouble(secondVar);

        switch (action)
        {
            case "+":
                return Convert.ToString(first + second);

            case "-":
                return  Convert.ToString(first - second);

            case "*":
                return  Convert.ToString(first * second);
        }

        throw new Exception("Unknown action");
    }
}

Реакция на нажатие кнопки будет теперь выглядеть так:

private void button1_Click(object sender, EventArgs e)
{
    var calcEngine = new CalcEngine();
    textBox3.Text = calcEngine.Proceed(comboBox1.Text, textBox1.Text, textBox2.Text));
}
3
  • Спасибо! Но при нажатии на button1 не происходит ничего, даже ошибок никаких при отладке не выдает.
    – Лидия
    22 дек 2020 в 12:37
  • Я поначалу неправильно написал код: не присвоил результат вычисления текст боксу. Проверьте, пожалуйста, что вы скопировали код после изменений и последняя строчка в button1_Click выгляидт так: textBox3.Text = calcEngine.Proceed(comboBox1.Text, textBox1.Text, textBox2.Text));
    – motpfofs
    22 дек 2020 в 12:39
  • 1
    Спасибо большое, поняла в чем была проблема, буду изучать данную тему дальше.
    – Лидия
    22 дек 2020 в 12:42

Ваш ответ

By clicking “Отправить ответ”, you agree to our terms of service and acknowledge you have read our privacy policy.

Всё ещё ищете ответ? Посмотрите другие вопросы с метками или задайте свой вопрос.