Thread: Circle Program Issues

My assignment is to have a program that creates a structure called Circle that has radius, diameter, area, and circumference. The program also needs 4 functions under the structure. Circle getCircleData(); void calculateCircleData(Circle &); void displayCircleData(Circle); and bool tryAgain();. Right now my code has a "do" statement in between the main and the first function, because it has to be able to run the functions again, with ANOTHER circle, while the bool statement returns true. I know that won't work because the "do" is not inside any function. But I don't know where to put it. I think my program has other problems too, but I'm not sure what they are. Anyways, if you could help me out, it would be greatly appreciated.

Code:

#include <iostream>
#include <iomanip>
using namespace std;

struct Circle
{
	double radius; 
	double diameter; 
	double area; 
	double circumference; 

	Circle( )
	{
		radius = 0; 
		diameter = 0; 
		area = 0; 
		circumference = 0; 

	}

};

Circle getCircleData();
void calculateCircleData(Circle &);
void displayCircleData(Circle); 
bool tryAgain(); 

int main()
{

	cout << "Welcome to our Circle Calculator" << endl;
	cout << "--------------------------------" << endl;


return 0; 
}
do
{

Circle getCircleData ( )
{
	Circle c1;  

	cout << "Enter the radius of the circle (number greater than 0)" << endl; 
	cin >> c1.radius; 
	
	return c1; 
}

void calculateCircleData (Circle &value)
{
	Circle c1; 
	c1.diameter = (c1.radius * 2); 
	c1.area = (c1.radius * c1.radius * 3.14159);
	c1.circumference = (2 * c1.radius * 3.14159);

}


void displayCircleData (Circle)
{
	Circle c1; 

	cout << "The circle data is:" << endl; 
	cout << "Radius: " << c1.radius << fixed << setprecision(2) << endl;
	cout << "Diameter: " << c1.diameter << fixed << setprecision(2) << endl; 
	cout << "Area: " << c1.area << fixed << setprecision(2) << endl; 
	cout << "Circumference" << c1.circumference << fixed << setprecision(2) << endl; 
}
}
while (tryAgain());

bool tryAgain ()
{
	char again;

	cout << "Do you want to try again? (y/n)" << endl; 
	cin >> again; 

	if ((again == 'y')||(again == 'Y'))
	{
		return 1; 
	}
	else 
	{
		return 0; 
	}


}

You would put the function calls (not the body of the functions themselves!) where you want them to happen. Main is usually a good place to start.

Ohhh duh!
okay. here's what I have now, and it's running, but all the values are being displayed as 0 still. What am I doing wrong?

Code:

#include <iostream>
#include <iomanip>
using namespace std;

struct Circle
{
	double radius; 
	double diameter; 
	double area; 
	double circumference; 

	Circle( )
	{
		radius = 0; 
		diameter = 0; 
		area = 0; 
		circumference = 0; 

	}

};

Circle getCircleData();
void calculateCircleData(Circle &);
void displayCircleData(Circle); 
bool tryAgain(); 

int main()
{
	Circle c1; 

	cout << "Welcome to our Circle Calculator" << endl;
	cout << "--------------------------------" << endl;
do
{
	getCircleData( ); 
	calculateCircleData (c1);
	displayCircleData (c1);
	tryAgain( );
}
while (tryAgain());



return 0; 
}

Circle getCircleData ( )
{
	Circle c1;  

	cout << "Enter the radius of the circle (number greater than 0)" << endl; 
	cin >> c1.radius; 
	
	return c1; 
}

void calculateCircleData (Circle &value)
{
	Circle c1; 
	c1.diameter = (c1.radius * 2); 
	c1.area = (c1.radius * c1.radius * 3.14159);
	c1.circumference = (2 * c1.radius * 3.14159);

}


void displayCircleData (Circle)
{
	Circle c1; 

	cout << "The circle data is:" << endl; 
	cout << "Radius: " << c1.radius << fixed << setprecision(2) << endl;
	cout << "Diameter: " << c1.diameter << fixed << setprecision(2) << endl; 
	cout << "Area: " << c1.area << fixed << setprecision(2) << endl; 
	cout << "Circumference" << c1.circumference << fixed << setprecision(2) << endl; 
}


bool tryAgain ()
{
	char again;

	cout << "Do you want to try again? (y/n)" << endl; 
	cin >> again; 

	if ((again == 'y')||(again == 'Y'))
	{
		return 1; 
	}
	else 
	{
		return 0; 
	}


}

You've initialized all of the circle's values to 0 and are using that in your calculations.

Well I just initialized them all as 0 in my default thing in my structure. For some reasons the calculations in my functions aren't working.

Right, look at your getCircleData( ) function. You don't keep the value entered, so the circle is still initialized to 0.

Oh.. I thought by putting the .radius after c1, it put the new inputted value into radius? How do I make the value that the user put in the new radius value?

Yes, correct, but that function has a local circle object c1 that is not the same as the one you declared in main. In main, you want to store the returned value of getCircleData() in c1.

like,
c1 = getCircleData; ?
Do I need to do that with the rest of the functions too?

No, but the other 2 circle functions are incorrect as well. You should study a little bit more about passing by reference and passing by value, as well as getting a little more understanding of your function arguments.

You pass data to your functions but then declare a new object inside the function and use that object instead of the data you passed, which is what you really want to use. E.g., in calculateCircleData(), you should be doing calculations on 'value' not 'c1'. 'value' is the same object as the 'c1' object you declared in main.