Как при делении на 0, за место бесконечности вывести фразу в textbox?

Question

[neitoo]

Кнопка '='

private void button17_Click(object sender, EventArgs e)
        {
            Double Value2 = Convert.ToDouble(textBox1.Text);
            Double Result = Value2;
            switch (Oper)
            {
                case 1:
                    Result = Value + Value2;
                    break;
                case 2:
                    Result = Value - Value2;
                    break;
                case 3:
                    Result = Value * Value2;
                    break;
                case 4:
                    if (Value2 == 0)
                    {
                        textBox1.Text = "На 0 делить нельзя!";
                    }
                    else
                    {
                        Result = Value / Value2;
                    }
                    break;
            }
            textBox1.Text = Math.Round(Result,4).ToString();
            Value = 0;
            Oper = 0;

        }

Кнопка '/'

private void button4_Click(object sender, EventArgs e)
        {
            button17_Click(sender, e);
            Value = Convert.ToDouble(textBox1.Text);
            Oper = 4;
            textBox1.Text = "0";
        }

В 4 кейсе если if else убрать, то выведет бесконечность, а если с if else, то просто остается 0 в textBox

Comments

[edward_freedom]

button17_Click(sender, e);

не логичнее ли сделать функцию и вызывать ее?

switch (Oper)

в чем преимущество в выборе свича в данном случае, а не иф?
Переписывай код с нуля и по нормальному

[neitoo]

edward_freedom, так проще кажется, с учетом того что первая программа

[edward_freedom]

neitoo, можешь уточнить в чем именно простота?

if(Oper == 1) {

}
else if(Oper == 2) {

}

 switch (Oper)
            {
                case 1:
                    Result = Value + Value2;
                    break;
                case 2:
                    Result = Value - Value2;
                    break;

[edward_freedom]

neitoo, https://qna.habr.com/answer?answer_id=1652237#answ...

[neitoo]

edward_freedom, ладно, так реально проще...но суть вопроса не в этом, разве с моим кодом невозможно бесконечность заменить на фразу?

[edward_freedom]

neitoo,

Double Value2 = Convert.ToDouble(textBox1.Text);
Double Result = Value2;
if (Value2 == 0 && Oper == 4)
{
	textBox1.Text = "На 0 делить нельзя!";
}
switch (Oper)
{

[edward_freedom]

neitoo, в твоем коде это должно и так работать, у тебя условие выполняется?

[neitoo]

edward_freedom, всё точно так же работает, тупо 0 в строке остается за место бесконечности

[neitoo]

Double Value2 = Convert.ToDouble(textBox1.Text);
Double Result = Value2;
if (Value2 == 0 && Oper == 4)
{
  textBox1.Text = "На 0 делить нельзя!";
}
switch (Oper)
{

А так не видит условие и бесконечность выводит

[alexalexes]

Либо сравнивать с вещественным нулем:
if (Value2 == 0.0 && Oper == 4)
А вообще, нужно иметь ввиду, что вещественные числа в вычислительной технике не такие просты на вид, как кажется. Особенно если с ними произвести матем. операции (легко можно получить 2.99999999997 вместо 3.0), а не работать сразу после ввода.
Так что, нужно вводить некую малую величину и сравнивать два вещественных числа на предмет, не стало ли меньше этого малого числа по модулю. Если меньше малого, то считаем, что эти числа равны. Только таким способом проверяется равенство вещественных чисел.

Double epsilon = 0.000000001;
if (Oper == 4
   && Math.Abs(0.0 - Value2) < epsilon // сравнили разницу 0.0 и Value2 в абсолютном выражении с epsilon
   )
{
  textBox1.Text = "На 0 делить нельзя!";
}

[neitoo]

alexalexes, такими способами тоже не работает, наверное проблема в остальном коде :(

[neitoo]

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

Double Value2 = Convert.ToDouble(textBox1.Text);
            Double Result;
            switch (Oper)
            {
                case 1:
                    Result = Value + Value2;
                    textBox1.Text = Math.Round(Result, 2).ToString();
                    break;
                case 2:
                    Result = Value - Value2;
                    textBox1.Text = Math.Round(Result, 2).ToString();
                    break;
                case 3:
                    Result = Value * Value2;
                    textBox1.Text = Math.Round(Result, 2).ToString();
                    break;
                case 4:
                    if (Value2 != 0)
                    {
                        Result = Value / Value2;
                        textBox1.Text = Math.Round(Result, 2).ToString();
                    }
                    else
                    {
                        textBox1.Text = "Делить на 0 нельзя!";
                    }
                    break;
            }
            Value = 0;
            Oper = 0;

[edward_freedom]

neitoo, neitoo, https://qna.habr.com/answer?answer_id=1652237#answ...

[neitoo]

edward_freedom, может просто как-то textBox1.Text = Math.Round(Result, 2).ToString(); в метод отправить, а иначе мне всё придется переписывать...хотя это изначально и предлагалось