Constructor

[roots]

When I am debugging this, the initial values of pi is no 3.14159 but some huge negative number as is the radius. Not sure what i am doing wrong here.

using namespace std;

Circle::Circle()
{
	radius = 0.0;
	pi = 3.14159;
}
Circle::Circle(double radius) 
{
	setRadius(radius);
}
void Circle::setRadius(double rad)
{
	radius = rad;
}
double Circle::getRadius()
{
	return radius;
}
double Circle::getArea()
{
	double area = pi * (radius*radius);
	return area;
}
double Circle::getDiameter()
{
	return radius + radius;
}
double Circle::getCircumference()
{
	return 2 * pi * radius;
}
// Demo program
int main()
{
   // To hold a radius
	
	
	double radius;
	cout << "Enter a radius" << endl;
   // Get the radius.
	cin >> radius;

   
   // Create a Circle object with the 
   // specified radius.
	Circle circularObject(radius);
	
	
   // Display the circle's data.
   cout << "Radius: " << circularObject.getRadius() <<endl;
   cout << "Area : " << circularObject.getArea() << endl;
   cout << "Diameter: " << circularObject.getDiameter()  <<endl;
   cout << "Circumference: " << circularObject.getCircumference() << endl;
   return 0;


#ifndef CIRCLE_H
#define CIRCLE_H
class Circle
{
private:
	// To hold a value for pi
	double pi;
	// To hold the radius
	double radius;


public:
	// The default constructor sets
	// radius to 0.0 and pi to 3.14159.
	Circle();
	Circle(double radius);
 
   // The overloaded constructor accepts
   // the radius as an arguemnt.
      
   // Mutator function for the radius
	void setRadius(double rad);
   // Accessor function for the radius
	double getRadius();

      
   // The getArea function returns the
   // circle's area.
	double getArea();


   // The getDiameter function returns the
   // circle's diameter.
	double getDiameter();


   // The getCircumference function returns
   // the circle's circumference.
	double getCircumference();
};
#endif

[SakurasouBusters]

Are you sure your code compiles?

You did not close your function main() with a closing-brace (})

[SakurasouBusters]

Circle::Circle(double radius) 
{
	setRadius(radius);
}

Should be :

Circle::Circle(double radius) 
{
	setRadius(radius);
	pi = 3.14159;
}

[roots]

I have an instruction that says the default constructor sets the radius to 0.0 and pi to 3.14159, if i have to set pi in the overlaoded constructor then what is the point of the dafault.
yea it compiles. must have left taht off

[SakurasouBusters]

I can't get what you mean.

[roots]

there is two constructors, one that takes an argument and one that dosent
the one that dosent is the default. it should set the two values in the definition to what they should be. However, these values are not what they should be when the program runs, but some crazy numbers instead. c c c

[SakurasouBusters]

So what is wrong with the result??

#include <iostream>
using namespace std;

class Circle
{
private:
	// To hold a value for pi
	double pi;
	// To hold the radius
	double radius;

public:
	// The default constructor sets
	// radius to 0.0 and pi to 3.14159.
	Circle();
	Circle(double radius);
 
   // The overloaded constructor accepts
   // the radius as an arguemnt.
      
   // Mutator function for the radius
	void setRadius(double rad);
   // Accessor function for the radius
	double getRadius();
      
   // The getArea function returns the
   // circle's area.
	double getArea();

   // The getDiameter function returns the
   // circle's diameter.
	double getDiameter();

   // The getCircumference function returns
   // the circle's circumference.
	double getCircumference();
};

Circle::Circle() : pi(3.14159)
{
	radius = 0.0;
}
Circle::Circle(double radius) : pi(3.14159)
{
	setRadius(radius);
}
void Circle::setRadius(double rad)
{
	radius = rad;
}
double Circle::getRadius()
{
	return radius;
}
double Circle::getArea()
{
	double area = pi * (radius*radius);
	return area;
}
double Circle::getDiameter()
{
	return radius + radius;
}
double Circle::getCircumference()
{
	return 2 * pi * radius;
}

// Demo program
int main()
{
   // To hold a radius
	
	
	double radius;
	cout << "Enter a radius : ";
   // Get the radius.
	cin >> radius;

   
   // Create a Circle object with the 
   // specified radius.
	Circle circularObject(radius);
	
	
   // Display the circle's data.
   cout << "Radius: " << circularObject.getRadius() <<endl;
   cout << "Area : " << circularObject.getArea() << endl;
   cout << "Diameter: " << circularObject.getDiameter()  <<endl;
   cout << "Circumference: " << circularObject.getCircumference() << endl;
   return 0;
}

Enter a radius : 25
Radius: 25
Area : 1963.49
Diameter: 50
Circumference: 157.08

[roots]

Ive never seen a value in that spot, where you have pi...hmm

[SakurasouBusters]

What exactly is your problem?

[roots]

Have you ran this through the debugger? if i initialize the default constructor the values are not what I want them to be.

[SakurasouBusters]

Why debugger?
The code is not supposed to produce any bugs.

[roots]

to check the values in the first constructor....

[SakurasouBusters]

Why? Do you even care if the final output is correct or not?

[roots]

yes, however i would like to know why cant i initialize these mother fuckin variables :)

[roots]

anyone? or am i just losing it

[SakurasouBusters]

It seems you do not know about member initializer list in C++.
http://en.cppreference.com/w/cpp/language/initializer_list

[vin]

The default constructor isn't called automatically.

[roots]

Even when I create a Circle circularObject; and then create the overloaded constructor, i cnat compile

[SakurasouBusters]

Did you modify something so your code could not even compile now?
Or you can post your compiler error logs here so that we can help.

[roots]

No. I have returned to this tonight. Gonna give it another try