CODEWITHSUNDEEP
c#math
jugo
jugo

Reputation: 17

Simple Calculator Application in C#

I have two quick question about this simple calculator application I am trying to build in C#. (not homework by the way) I am trying to get the MessageBox.Show message to show in the multiply and add sections of my code, but they don't seem to be displaying even if I enter a negative value. The application just seems to do the math anyways. Also, this may be a dumb one, how do I get rid of the label5 text that appears in the application with out deleting it in the properties window?

Any help will be greatly appreciated, thanks!

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

namespace AddMultiply
{
     public partial class AddMultiply : Form
{
    public AddMultiply()
    {
        InitializeComponent();
    }

    private void label1_Click(object sender, EventArgs e)
    {

    }

    private void txtFirstValue_TextChanged(object sender, EventArgs e)
    {

    }

    private void btnAdd_Click(object sender, EventArgs e)
    {
        double firstValue;
        double secondValue;
        double answer;

        while (double.TryParse(txtFirstValue.Text, out firstValue) == false)
        {
            MessageBox.Show("The value(s) entered must be > 0");
        }
        while(double.TryParse(txtSecondValue.Text, out secondValue) == false)
        {
            MessageBox.Show("The value(s) entered must be > 0");
        }


            answer = firstValue + secondValue; 
            lblAnswer.Text = answer.ToString();

    }

    private void btnMultiply_Click(object sender, EventArgs e)
    {
        double firstValue;
        double secondValue;
        double answer;

        while (double.TryParse(txtFirstValue.Text, out firstValue) == false)
        {
            MessageBox.Show("The value(s) entered must be > 0");
        }
        while (double.TryParse(txtSecondValue.Text, out secondValue) == false)
        {
            MessageBox.Show("The value(s) entered must be > 0");
        }


            answer = firstValue * secondValue; 
            lblAnswer.Text = answer.ToString();

    }

    private void lblAnswer_Click(object sender, EventArgs e)
    {
        lblAnswer.Text = ""; //tries to get rid of "label5" text in application, but fails to do so
    }
}

}

Upvotes: 0

Views: 504

Answers (2)

Santhosh Rajoo
Santhosh Rajoo

Reputation: 79

You can try the following code to show the MessageBox then the value is less than 0:

if (n1 < 0 || n2 < 0)
                MessageBox.Show("Value less than ZERO ", "Value less than ZERO",MessageBoxButtons.OK , MessageBoxIcon.Exclamation);

As for getting rid of the label you can try :

label5.Visible = false;

Upvotes: 1

israel altar
israel altar

Reputation: 1794

1)you should change the while to "if":

private void btnAdd_Click(object sender, EventArgs e) { double firstValue; double secondValue; double answer;

    if (double.TryParse(txtFirstValue.Text, out firstValue) == false)
    {
        MessageBox.Show("The value(s) entered must be > 0");
    }
    if(double.TryParse(txtSecondValue.Text, out secondValue) == false)
    {
        MessageBox.Show("The value(s) entered must be > 0");
    }


        answer = firstValue + secondValue; 
        lblAnswer.Text = answer.ToString();

}

2) where is Ladel5 ? It doesn't seem to be exist...

Upvotes: 1

Related Questions