[Munkybones]

ok i have been racking my brain trying to figure this out..and for some reason i cant quite get it to display. I am writing a method to make a calculation and return a value inside an if statement to a label i already have set up for the display text.
This is the method i have set up to return the value of farenheit converted to celsius

private double converttocelsius(double farenheit)
        {
            
             
            
            double convcelsius = farenheit * 1.8 + 32;
            return convcelsius;
        }

and this is what i have come up with thus far trying to come up with the solution. But im getting an error about converting from double to string and for invalid arguements

private void conversionbtn_Click(object sender, EventArgs e)
        {
            string celsius;
            string farenheit;
            double convcelsius;
            double convfarenheit;
            celsius = celsiusinput.Text;
            farenheit = farenheitinput.Text;
            
            
            if (celsius != "")
                MessageBox.Show(converttofarenheit);
                //conversionlbl.Text = "celsius";
            else if (farenheit != "")
                MessageBox.Show(converttocelsius(convcelsius,2));
                //conversionlbl.Text = "converttofarenheit";
            else
                conversionlbl.Text = "Must enter number in only one box!";

        }

Im new to C# this is my first class..and i had trouble when i was first starting vb too...so im hoping that it will be something simple that im overlooking or forgetting about. Any help is appreciated..thanks in advance

[Skaggles]

IF you'd like to turn the return value into a string to display with a message box, try adding .ToString() at the end of your call: converttocelsius(200.0).ToString()

[Momerath]

Just want to point out that your converttocelcius is wrong (it will return the wrong values)

[poncho4all]

I dont think this is right

MessageBox.Show(converttocelsius(convcelsius,2));

your function, besides the said above, does not receives 2 parameters only one, and since it receive a double as value you should be sending option 1:

MessageBox.Show(converttocelsius(double.Parse(sfarenheit)).ToString());

option 2:

dfarenheit = double.Parse(sfarenheit);
MessageBox.Show(converttocelsius(dfarenheit).ToString());

Where sfarenheit is string, and dfarenheit is double.

This post has been edited by poncho4all: 08 May 2010 - 01:47 PM

[PsychoCoder]

First, it's true your formula is incorrect. It should look like this (for F to C)

double convcelsius = (farenheit - 32.0) / 1.8;

So your conversion method should look like this

private double converttocelsius(double farenheit)
{
    double convcelsius = (farenheit - 32.0) / 1.8;
    return convcelsius;
}

But as mentioned I would change it so it returns a string like this

private string converttocelsius(double farenheit)
{
    return string.Format("Converted: {0}", (farenheit - 32.0) / 1.8);
}

Then to use it (made some slight modifications to your code, hope you don't mind )

private void conversionbtn_Click(object sender, EventArgs e)
{
    double c, f;

    if (Double.TryParse(celsiusinput.Text, out c))
    {
        MessageBox.Show(converttofarenheit);
    }
    else if (Double.TryParse(farenheitinput.Text, out f))
    {
        MessageBox.Show(converttocelsius(f));
    }
    else
    {
        conversionlbl.Text = "Must enter number in only one box!";
    }
}

Hope that helps

[Munkybones]

K first i did fix the algorithm for the conversion methods. and have tried a couple of calls with message box

MessageBox.Show(converttocelsius(convcelsius).ToString);

i could not get this to work gives me an error about converting method to string.
but i was just using the message box's to try and help me see if any part of it is working. I actually need to figure out how to call to a label i just dont understand the syntax of how to word it so i can call the method i have set up with return value. Is there a special way that you are supposed to set up the return..or call the return to a label?
still have it to pass the value of input temperature to the method for the conversion then return convcelsius/convfarenheit but how do you call convcelsius or convfarenheit

[PsychoCoder]

You obviously skipped completely over my post?

[eclipsed4utoo]

ToString() is a method, therefore you need the opening and closing parenthesis after it.

MessageBox.Show(converttocelsius(convcelsius).ToString());

[sithius92]

If you are trying to output the result to a label, you could try something like this:

conversionlbl.Text = converttocelsius(32);

As PsychoCoder had stated previously, you could apply formatting to your value when it is returned or apply the formatting when you set the text for your label.

conversionlbl.Text = string.Format("{0:0}", converttocelsius(32));

This post has been edited by sithius92: 10 May 2010 - 02:30 PM

[Munkybones]

Psychocoder, no i didnt mean for it to seem like that. I had tried some stuff with how you had it built up but there were things that i didnt understand like the outc within the if statements. i figured that was something specific with messagebox's. but on that note thank you for showing me a better way of writing the methods that i had set up...cut's down only 1 line of code but good to know it can be set up inline setting variable and declaring it within the same line.

And on the issue of formatting the line

conversionlbl.Text = string.Format("{0:0}", converttocelsius(32));

i have learned only vb and am confused as to the way the last part of this strign is set up. I thought that anything that you put within "" will come out as text to wherever you have it set to display.
My other question in this line is the (32) what is this value for? my book is unclear my thinking is its the value your calling from that method. and if that is the case why 32 in this situation when the value will be changing on a regular?

[Momerath]

Answering your questions backwards:

The 32 is the parameter being sent to the method convertocelcius().

As for the "" part, String.Format() (in this case) has 2 parameters. The first is a string used to define how you want the output to look, the second is the data to be formated.

[Munkybones]

ok with the parameter that is what i had thought it was, now that is a set number ...32 is going to come through and display...but how would i got about using my inputtextbox as my variable?

Thanks for explaining the format set up too