Калькулятор C#

[[TrusT]]

Начали изучение шарпа с написание простенького калькулятора.
Дома как бы все доделала - отлично работает, считает правильно, но два но:
- Когда устанавливаю запятую, она ставится сначала перед всеми числами, потом только на место
- Минус ставится всегда в конец, и не меняет свое положение, даже если вводить новые числа.
вывод числа в textBox. Прошу помощи.

Скриншот

@темы: Вопрос, C#

Comments

[Ri]

У вас ошибки в коде.

[[TrusT]]

Ri, понимаете, в чем проблема. я тоже так решила и кинулась построчно проверять код.
Так вот - строка выглядит нормальной, то есть textBox1.Text это например -3,0, а в строке в приложении написано 3,0-.

А если ошибки - это касается texBox-а, или функций, с ним связанными? Я попробовала вывести в текстбокс -8 просто, в одной из функций, мне выведено в ответ было 8-.

[Ri]

Может, код покажете всё-таки?

[[TrusT]]

Ri, да-да, тормознула слегонца) сейчас
Form1.cs

using System;
using System.Collections.Generic;
using System.ComponentModel;
using System.Data;
using System.Drawing;
using System.Linq;
using System.Text;
using System.Windows.Forms;

namespace MyCalc
{
    public partial class Form1 : Form
    {
        public Form1()
        {
            InitializeComponent();
        }
        private bool reset_numbers = false;//очищение
        private double a = 0.0;//сохраненная переменная
        private bool spot = false;//запятая
        private char operation = '=';//знак операции
       
        private void Numbers_Click(object sender, EventArgs e)
        {
            if (res.Text == "0" && ((Button)sender).Text.ToString() != "0") reset_numbers = true;
            if (res.Text == "0" && ((Button)sender).Text.ToString() == "0") reset_numbers = true;
            if (res.Text == "0" && ((Button)sender).Text.ToString() == ",") reset_numbers = false;
            if (reset_numbers) { res.ResetText(); reset_numbers = false; spot = false; }
            if (((Button)sender).Text.ToString() == "," && spot != true)
            {
                spot = true;
                res.Text += ((Button)sender).Text.ToString();
            }
            if (((Button)sender).Text.ToString() != ",") res.Text += ((Button)sender).Text.ToString();
        }

        private void BackSpace_Click(object sender, EventArgs e)
        {
            res.Text = "0";
            spot = false;
            a = 0.0;
        }

        private void Choice(double b, char what)
        {
            switch (what)
            {
                case '+':
                    a += b;
                    break;
                case '-':
                    a -= b;
                    break;
                case '*':
                    a *= b;
                    break;
                case '/':
                    a /= b;
                    break;
                case '^':
                    a = Math.Pow(a, b);
                    break;
                case 's':
                    a = Math.Sqrt(b);
                    break;
            }
        }

        private void operation_Click(object sender, EventArgs e)
        {
            char op = operation;
            operation = ((Button)sender).Text.ToString()[0];
            if (operation == 's') Choice(double.Parse(res.Text.ToString()), operation);
            else if (op == '=' || op == 's')  a = double.Parse(res.Text.ToString());
            else
            {
                double b = double.Parse(res.Text.ToString());
                Choice(b, op);
            }
            reset_numbers = true;
            res.Text = a.ToString();
        }

        private void plusminus_Click(object sender, EventArgs e)
        {
            double f = double.Parse(res.Text.ToString());
            f *= -1.0;
            res.Text = f.ToString();
        }
    }
}

Кстати, в поле у меня выравнивание справа, а вот сделала выравнивание слева - и все вроде адекватно...

[[TrusT]]

вопрос решен, не такое выравнивание поставила == было Right to left, а нужно TextAlign ==
Вот так всегда.
Ri, спасибо! Если бы не начала с вами разговаривать, так бы и не поняла)

[Ri]

Хм, у вас такой запутанный код, что я даже не знаю, что посоветовать. Дам лучше пару общих замечаний.

Надо отделять логику от представления. Текущие значения и состояние калькулятора нужно хранить в переменных, а не в поле ввода. То есть res.Text.ToString() у вас нигде не должно быть. Нужно только устанавливать текст в поле на основе значений переменных. Попробуйте переделать код так, чтобы это выполнялось. Уверен, проблем станет меньше.

Ну, и нельзя делать переменные с именем вроде "а", имена должны быть говорящие.

[[TrusT]]

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