Today's exercise in stupidity...

[w00t]

#include <iostream.h>

class Car
{
   public:
      int itsAge;
      int getAge();
      int itsCost;
      int itsRank;
      int itsModel();
      void start();
};
  	int Car::getAge()
{
		return itsAge;
}
	int Car::itsModel()
{
	cout <<"1990 Legacy";
}

	void Car::start()
{
     cout <<"RRR..Brumbrumbrummrumbrumbroooombroom...";
}



int main()
{
     Car Subaru;
     Subaru.itsAge = 11;
     Subaru.itsCost = 2900;
     Subaru.itsRank = 2;
     cout <<"My car is a" <<Subaru.itsModel() <<"/n";
     cout <<"it cost me" <<Subaru.itsCost <<"/n";
     cout <<"It's ranked #" <<Subaru.itsRank <<"/n";
     cout <<"Man, I love this car./n";

     return 0;
};

Error: line 20:
function should return a value.

[redhead]

int Car::itsModel()
{
cout <<"1990 Legacy";
}

The int type indicats it should return a value. Either set it to return 0; or set it to be a void type function.

[kenshi]

You have a bit of a mess there. First of all, like redhead said, you have a function of int type that returns nothing. You either need to make it return something or declare it void. Second, the function is going to be called from the cout line, and the cout line within the function will be performed, which is desired, but the return value of the function will also be printed, which is not. I think the easiest way to change it up is to make Car::itsModel() return a char pointer that points to a string holding the model name. Do something like:

char* Car::itsModel()
{
  char[] model = "1990 Legacy";
  return model;
}

[sdeming]

If you are going to return a local variable then you must declare it static or it can be destroyed before you ever actually use it, like this:

char *Car::itsModel()
{
  static char[] model = "2002 Camarro SS +35";
  return model;
}

Even better would to be to throw out char* altogether and start using std::string, it's immensly more powerful and much much safer.

[kenshi]

Good point.