Need help with a class please

Learning classes, and trying to make it all work. This code compiles OK, but when I execute, and say use the number 100 as the Farenheit temperature input, I always get -17.7778 as the Celcius equivalent, even when I change the Farenheit temp input. What is my problem here?

code:_____________________________________________ __

#include <iostream.h>

class Thermometer
{
public:
Thermometer(); //constructor
void setThermometer(double); //function to set temp
void getFarenheit(double); //function to get temp in F
void getCelcius(double); //function to get temp in C
private:
double FarenTemp; //data members in F
};

//Temperature constructor to initialize the data members to zero

Thermometer::Thermometer() {FarenTemp = 0;}

//Function to set temperature using Farenheit.

void Thermometer::setThermometer(double temp)
{
temp = FarenTemp;
}

//Function to get temperature in degrees F from the user

void Thermometer::getFarenheit(double t)
{
cout<<"Enter a temperature in degrees Farenheit: ";
cin>>t;

}

//Funtion to get temperature in degrees C from the user

void Thermometer::getCelcius(double)
{
cout<<"The temperature in Celcius: "<<(((FarenTemp - 32 ) * 5 ) / 9 );


}


int main()
{
Thermometer x; //instantiates object x of class Thermometer

x.getFarenheit(1);

x.getCelcius(1);

cout<<endl;

return 0;
}

end code:___________________________________________

Click Here and adjust your post accordingly.

edit: your code is pretty weird. you can't assign parameters values.

your conversion formula is wrong
Farenheit to celsius 5.0/9.0*(Fahren_temp-32.0)
Celsius to farenheit 9.0/5.0*(Celsius_temp + 32.0)

Fahrentemp and celsius temp represent your variable name

Yes, Alpha, I know I can't post parameters as values, but ...... LOL..... I am only a beginner, and if I put nothing in the () then I get an error, and if I put (double) as I think I should, I get errors......... Got any suggestions on how to fix this problem???? You know, us beginners just try to get the code to work, any way that we can. I am going to be starting intermediate programming soon, so hopefully I won't be so laughable with my code. :)

To Gamer4life....thanks for the input on the conversion. I will try that and see how it works out.

I'm somewhat of a beginner also. what i meant though is not what is inside the parentheses.

for example:

Code:

---

void Thermometer::setThermometer(double temp)
{
temp = FarenTemp;
}

---

you cannot do this, it should be changed to:

Code:

---

void Thermometer::setThermometer(double temp)
{
FarenTemp = temp;
}

---

in a class, constructors and set modifier functions are supposed to set private data to something.

also, you cannot do things such as:

Code:

---

void Thermometer::getFarenheit(double t)
{
cout<<"Enter a temperature in degrees Farenheit: ";
cin>>t;
}

---

you have to do things such as

Code:

---

std::cin >> FarenTemp;

---

when you cin to t, this doesn't do anything.

here, try this:

Code:

---

#include <iostream>
using std::cout;
using std::cin;
using std::endl;

class Thermometer
{
        public:
        Thermometer(); //constructor
                //void setThermometer(double); //function to set temp
                void getFarenheit(); //function to get temp in F
                void getCelcius(); //function to get temp in C
        private:
                double FarenTemp; //data members in F
};
//Temperature constructor to initialize the data members to zero
Thermometer::Thermometer() {FarenTemp = 0;}
//Function to set temperature using Farenheit.

/*void Thermometer::setThermometer(double temp)
{
        FarenTemp = temp;
}*/
//Function to get temperature in degrees F from the user

void Thermometer::getFarenheit()
{
        cout<<"Enter a temperature in degrees Farenheit: ";
        cin>>FarenTemp;
}
//Funtion to get temperature in degrees C from the user
void Thermometer::getCelcius()
{
        cout<<"The temperature in Celcius: "<<(((FarenTemp - 32 ) * 5 ) / 9 );
}
int main()
{
        Thermometer x; //instantiates object x of class Thermometer
        x.getFarenheit();
        x.getCelcius();
        cout<<endl;
return 0;
}

---

This runs and works, at least with 32F to 0C and 212F to 100C.

edit: your formula in getCelcius works, it does not need to be changed. what was making it produce weird answers was the cin >> t line, this doesn't do anything, and once the function is over, it goes away. If you cin >> FarenTemp, it will store this in private data, and when you use that private data, it has what the temp was. Otherwise, FarenTemp contains garbage.

also, when you assign FarenTemp, you should assign FarenTemp the value of the parameter, and not assign the parameter the value of FarenTemp. That function wasn't needed, so I commented it out. But if it is a homework, and you are supposed to have the function, you have to think through your code more.

If I offended you with my previous post, it was not intended and I am sorry for that. I am here to help people learn, and to learn myself.

I hope this helps some, if you have any more questions, post up.

you should be settting the farenheight temp in SET temp not get temp...once you set it THEN get it...and yes, you calc are offf..
the ones provided are correct.
gets should never modify data..a good way to ensure this is to make your 'gets' const
---M

oh, good catch. i didn't even look at the formula, it worked with 3 tests, so i assumed it was right. change it to what gamer said, and it should be good.

to clarify a bit on what mouse said for the get fxns:

Code:

---

void Thermometer::getCelcius() const
{
cout<<"The temperature in Celcius: "<<(((FarenTemp - 32 ) * 5 ) / 9 );
}

---

notice the const after the fxn; this would be an accessor.

also, as mouse said, get fxns should not be modifiers; the code in your getFarenheit fxn should just return FarenTemp. That code should be placed in setThermometer. Actually, ideally, you shouldn't have any cout or cin statements in your class functions.

No offense taken! I would never learn anything if I got offended by my own mistakes.

Thanks both Alpha and Mouse for your input so far. I will try to fix all this up. It isn't homework, just a project I came up with to make sure I understand how to do classes, before I go to the homework. Looks like it was a good idea to make up some work before I actually hit the homework stuff, huh!

Thanks so much. If I run into more questions, I will be sure to ask you.

:)