Arithmetic operation Result is not display in lable

Question

0.00/5 (No votes)

See more:

C#

protected void  RadioButton2_CheckedChanged(object sender, EventArgs e)
{
    int no1, no2;
    float ans;

    no1 = Convert.ToInt32("textbox2.Text");
    no2 = Convert.ToInt32("textbox3.Text");
    ans = no1 - no2;
    Label2.Text = ans.ToString();
}
    protected void RadioButton1_CheckedChanged(object sender, EventArgs e)
    {
        int no1, no2;
        float ans;

        no1 = Convert.ToInt32("textbox2.Text");
        no2 = Convert.ToInt32("textbox3.Text");
        ans = no1 + no2;
        Label2.Text = ans.ToString();
    }
    protected void RadioButton3_CheckedChanged(object sender, EventArgs e)
    {
        int no1, no2;
        float ans;

        no1 = Convert.ToInt32("textbox2.Text");
        no2 = Convert.ToInt32("textbox3.Text");


            ans = no1 * no2;
            Label2.Text = ans.ToString();

    }
    protected void RadioButton4_CheckedChanged(object sender, EventArgs e)
    {
        int no1, no2;
        float ans;

        no1 = Convert.ToInt16("textbox2.Text");
        no2 = Convert.ToInt16("textbox3.Text");
        ans = no1 /no2;
        Label2.Text =ans.ToString();
    }

Posted 13-Jun-13 2:50am

Patel Shweta

Add a Solution

Comments

Prasad_Kulkarni 13-Jun-13 8:55am

Use Convert.ToString() instead of ToString().
Convert.ToString() handles null, while ToString() doesn't.

..and debug your code line by line, check what input you're giving and what will be the expected output.

CHill60 13-Jun-13 8:59am

Nice explanation of why to use Convert in this instance

jkirkerx 13-Jun-13 15:15pm

are the values in the textbozes really integers?, or perhaps decimals

2 solutions

Add a Solution

Add your solution here

Treat my content as plain text, not as HTML

Preview 0

…

Existing Members

Sign in to your account

...or Join us

Download, Vote, Comment, Publish.

Your Email
Password
Forgot your password?

Your Email
This email is in use. Do you need your password?
Optional Password

I have read and agree to the Terms of Service and Privacy Policy
Please subscribe me to the CodeProject newsletters

When answering a question please:

Read the question carefully.
Understand that English isn't everyone's first language so be lenient of bad spelling and grammar.
If a question is poorly phrased then either ask for clarification, ignore it, or edit the question and fix the problem. Insults are not welcome.
Don't tell someone to read the manual. Chances are they have and don't get it. Provide an answer or move on to the next question.

Let's work to help developers, not make them feel stupid.

This content, along with any associated source code and files, is licensed under The Code Project Open License (CPOL)

Ron Beyer · Answer 1 · 2013-06-13T02:59:00

I don't know why, but you are trying to convert the text "textbox1.Text" to an integer, not the value in the textbox:

C#

protected void  RadioButton2_CheckedChanged(object sender, EventArgs e)
{
    int no1, no2;
    float ans;
 
    no1 = Convert.ToInt32(textbox2.Text);    //THIS IS WHERE THE PROBLEM WAS ON
    no2 = Convert.ToInt32(textbox3.Text);    //ALL THESE LINES WITH QUOTES
    ans = no1 - no2;
    Label2.Text = ans.ToString();
}
    protected void RadioButton1_CheckedChanged(object sender, EventArgs e)
    {
        int no1, no2;
        float ans;
 
        no1 = Convert.ToInt32(textbox2.Text);
        no2 = Convert.ToInt32(textbox3.Text);
        ans = no1 + no2;
        Label2.Text = ans.ToString();
    }
    protected void RadioButton3_CheckedChanged(object sender, EventArgs e)
    {
        int no1, no2;
        float ans;
 
        no1 = Convert.ToInt32(textbox2.Text);
        no2 = Convert.ToInt32(textbox3.Text);
 

            ans = no1 * no2;
            Label2.Text = ans.ToString();
 
    }
    protected void RadioButton4_CheckedChanged(object sender, EventArgs e)
    {
        int no1, no2;
        float ans;
 
        no1 = Convert.ToInt16(textbox2.Text);
        no2 = Convert.ToInt16(textbox3.Text);
        ans = no1 /no2;
        Label2.Text =ans.ToString();
    }

Ahmed Bensaid · Answer 2 · 2013-06-13T03:02:00

Solution 3

Suppress the quotes when you're doing conversions ;)

Replace this

C#

Convert.ToInt32("textbox2.Text");

With that

C#

Convert.ToInt32(textbox2.Text);

Posted 13-Jun-13 3:02am

Ahmed Bensaid

Comments

Patel Shweta 13-Jun-13 9:14am

still not display result in lable

Ahmed Bensaid 13-Jun-13 9:59am

You have to set AutoPostBack property to true for each of your radiobuttons ;)

Patel Shweta 13-Jun-13 13:04pm

Thank you .. its working :)

jkirkerx 13-Jun-13 15:16pm

how about marking this as solved