Car class

[thesagaof]

I'm having a few problems with this code. I couldn't get it to work in time for class the other day and it has been bothering me.

When I compile this it tells me that current_speed (assuming they're referring to the one in int main() function) is being used without being initialized.

It then asks me if I want to continue or break. I select continue and it spits out this number always: -858993460

And it goes into the Y/N loop, which works fine. Where did I go wrong?

Specifically:

//PHILLIP RODGERS
//CS226; GORADIA


#include <iostream>
#include <iomanip>
#include <string>

using namespace std;

// CAR CLASS DEFINED
class Car
{
private:
	int current_speed;
	int year;
	string make;
	string model;
public:
	Car(int, int, string, string);
	int accelerate();
	int brake();
};

//MAIN FUNCTION
int main()
{

	//OBJECT CREATION
	Car turboCar(0, 1994, "Toyota", "RX-S");

	//STUFF FOR MENU & CAR OPERATIONS!
	int current_speed;
	int operationChoice;
	char continueChoice;

	//MENU CODE
	do
	{
		
		cout << "Hello, welcome to car number one.\n";
		cout << "What would you like to do with your brand new car?\n";
		cout << "Press 1 to accelerate.\n Press 2 to brake.\n Choose now: ";
		cin >> operationChoice;

		if(operationChoice = 1)
			turboCar.accelerate();
		if(operationChoice = 2)
			turboCar.brake();

		cout << current_speed;

		cout << "\n\nWould you still like to use the car?\n If so, press Y. If not, press N.\n Choice: ";
		cin >> continueChoice;


	}while((continueChoice == 'y') || (continueChoice == 'Y'));



	
	return 0;
}

//CAR FUNCTION DEFINITIONS
Car::Car(int cs, int y, string m, string mo)
{
	current_speed = cs;
	year = y;
	make = m;
	model = mo;
}

int Car::accelerate()
{ return current_speed + 5; }

int Car::brake()
{ return current_speed - 5; }

[thesagaof]

Can anyone help me with this?

[Stewbond]

You have current_speed defined in main, and you have current_speed defined in your class. These are two different variables and they don't communicate. This is a problem of scope.

To solve this create an accessor and eliminate current_speed from your main function

// CAR CLASS DEFINED
class Car
{
private:
	int current_speed;
	int year;
	string make;
	string model;
public:
	Car(int, int, string, string);
	int accelerate();
	int brake();
        int getSpeed() { return current_speed; } // Added this:
};

Then in your main:

int current_speed;
...
cout << current_speed;
cout << turboCar.getSpeed();

[thesagaof]

Oh, Duh. Cause I have have function that increases/decreases the speed, but I don't have a function that actually gets and displays it. Thanks!