The part I'm on is this.

---

#

Write a testing class with a main() that constructs a RegularPolygon and calls each method. A sample run of the program for a polygon with 4 sides of length 10 would give:

number of sides = 4
length of side = 10.00
radius of circumscribed circle = 7.07
radius of inscribed circle = 5.00
vertex angle = 90.0
perimeter = 40.00
area = 100.00

---

I would like you note that some of the things you do are a little weird to me.  That asside, I have rewritten some of your code, I don't know if it compiles or runs correctly, since I do not have a java compiler on my computer right now.

Note:
If you are going to use a constant like 1.0, if you'd like to make it explicitly a double or a float, you can use F or D(I think).
such as,

Code:

float x = 1.0F + 1.2F;

Also, if you are using a class, like you are go ahead and name your data members whatever you want.

Code:

private double radius;

Then if you need to set the radius, do this.

Code:

public void setRadius(double radius) {   this.radius = radius; }

This gets you away from, 'myradius' or whatever.

I hope this helps you're problem.

-------
I've got spurs that jingle jangle jingle.
As I go riding merrily along.
And they sing, "Oh, ain't you glad you're single?"
And that song ain't so very far from wrong.

Thanks for the heads up. I sorta understand it, but I'm sure I'll grasp it better once we get more into it.


-------
"The most abundant element in the universe is
hydrogen, then second is stupidity."-Harlan Ellison
"What good fortune for our governments that the
people do not think."-Hitler

Fyi for those that don’t know

Code:

this.

Allows variables to evaluate them self’s, in other words (this var = the value of the argument var sent in)

-------
"Yes humor must be banned to prevent people getting offended!!" -osmoticdespair

---

The

Code:

this.

method seems unnecessary he doesn’t need that constructor to set anything because the radius will always be reset for each time the prog is run and there is no reason to change the initial value after the user gives input (or the program generates a test case) .

Fyi for those that don’t know

Code:

this.

Allows variables to evaluate them self’s, in other words (this var = the value of the argument var sent in)

---

The keyword

Code:

this

can be completely omitted from any Java code.  The thing is, it allows for a better way to name your member variables.

For example, you have a member variable

Code:

szFilename

You have a function which sets the file name.
What do you name the param. you pass in?

The this keyword allows you to name them the same name.  Of course, I've never tried to just say
x = x; instead of this.x = x, so I don't know if it can tell, but I doubt it.

I suppose it's up to you what you do.  I do this in c++ as well.

-------
I've got spurs that jingle jangle jingle.
As I go riding merrily along.
And they sing, "Oh, ain't you glad you're single?"
And that song ain't so very far from wrong.

---

Note:
If you are going to use a constant like 1.0, if you'd like to make it explicitly a double or a float, you can use F or D(I think).
such as,

Code:

float x = 1.0F + 1.2F;

---

This is garbage imo.. even if it does work it's unnecessary as a decimal is assumed as a double.

for example 10/3 = 3 (integer division)
However 10/3.0 = 3.3333  (treated as a double)

(Edited by Suke at 2:52 pm on Oct. 25, 2004)

Java supports long variable names for this reason.. you should probably try something more descriptive.

Code:

private double radiusCircumscribed; private double radiusInscribed;

I didn't look at what you are doing with the vars so there may be a better name, I just used what I could make from the description of it.

Also it's not good to have the var the same but just different capitolization.. that would lead to confusion.  It should be easy for someone else to understand your code and not have to be confused with such things as same name of variables with different capitol letters.

(Edited by Suke at 2:50 pm on Oct. 25, 2004)

---

Quote: from sakurag at 11:31 pm on Sep. 23, 2004

---

Note:
If you are going to use a constant like 1.0, if you'd like to make it explicitly a double or a float, you can use F or D(I think).
such as,

Code:

float x = 1.0F + 1.2F;

---

This is garbage imo.. even if it does work it's unnecessary as a decimal is assumed as a double.

for example 10/3 = 3 (integer division)
However 10/3.0 = 3.3333  (treated as a double)

(Edited by Suke at 2:52 pm on Oct. 25, 2004)

---

I disagree.  For instance, what if the data type you are using is a float.  Unless you explicitly say it is, you will have to type cast it later in order to get rid of warnings.

Code:

struct Point {  public float x;  public float y;  public float z; } ... ptMyPoint.x = 10/3.0; // downcast, may cause unexpected results ptMyPoint.x = 10.0F / 3.0F; // No casting necessary for(i = 0; i < 500000000000; i++) {   ptMyPoint.x = (float)(10/3.0); } // Explicit cast 500000000000 times, depending on compiler, EXTREMELY inefficient.

In my line of practice(though usually C), I try to be as precise as possible with my code.  I also try to avoid any uneccessary processor power, ie downcasting.  (I think upcasting is usually fine, but may eat cpu time).

Of course, we all program differently.  

-------
I've got spurs that jingle jangle jingle.
As I go riding merrily along.
And they sing, "Oh, ain't you glad you're single?"
And that song ain't so very far from wrong.

---

Also you need to probably choose better variable names for example

Code:

private double myR;            // radius of circumscribed circle private double myr;            // radius of inscribed circle

Java supports long variable names for this reason.. you should probably try something more descriptive.

Code:

private double radiusCircumscribed; private double radiusInscribed;

I didn't look at what you are doing with the vars so there may be a better name, I just used what I could make from the description of it.

Also it's not good to have the var the same but just different capitolization.. that would lead to confusion.  It should be easy for someone else to understand your code and not have to be confused with such things as same name of variables with different capitol letters.

(Edited by Suke at 2:50 pm on Oct. 25, 2004)

---

I agree with you here, this is me being lazy typing code into livewire.

I usually use Hungarian notation for my code.  For those of you who don't know, Hungarian notation precedes each variable name with an abbreviation for the type.

ie.
nWidth - Integer type
lpPoint - Long pointer type
sz       - string type, 0 terminated
hDC    - win32 HANDLE
etc, etc, etc.

Long names provides better readability + more typing ;]  Though.. I sometimes write cryptic code ;]

Code:

(*Rn) = 3.0F * ( (*(RPtn + (*(Ptn + 1)))) - (*(RPtn + (*(Ptn + (N - 1))))) );

-------
I've got spurs that jingle jangle jingle.
As I go riding merrily along.
And they sing, "Oh, ain't you glad you're single?"
And that song ain't so very far from wrong.