Trying to get values for the perimeter and center of a rectangle.

[kenny gill]

My assignment wants me to get and return values for a rectangle.

I'm having difficulty getting the perimeter and center values to show up.

I'm also thinking that my code does not look that great for getting values. so if you have any critiques I would be happy to hear them

import java.awt.*;
import java.applet.Applet;


public class ReturnDemo extends Applet {
	
	int height = 60;
	
	int width = 42;
	
	int answer1 = areaRectangle(42, 60);
	int answer2;
	int answer3;
	
	public void paint(Graphics g){
		g.drawRect(40, 80, 42, 60); 
		g.drawString("area of rectangle is" +answer1, 20, 20);
		g.drawString("perimeter of rectangle is" +answer2, 20,40);
		g.drawString("rectangle center is" +answer3, 20, 60);
		
	}
 private int areaRectangle(int width, int height){
	 int area = width * height;
	 return area;
 }
 private int perimeterRectangle(int width, int height){
	 int perimeter = 2*width + 2*height;
	 return perimeter;
 }
 private int centerX(){
	 int centerX = width/2;
	 return centerX;
	
 }
 private int centerY(){
	 int centerY = height/2;
	 return centerY;
 }
}

here are the instructions just in case:

"Have the class define methods to calculate and return values for:
the area of a rectangle: areaRectangle()
the perimeter of the rectangle: perimeterRectangle()
the X value of the center of the rectangle: centerX()
the Y value of the center of the rectangle: centerY()


Be sure to have an applet that has a paint method that first draws a rectangle using


Be careful to change the y values in your drawString calls so they do not overwrite each other and so that the first one is low enough to not be in the way of the drawn rectangle.

Hint: the drawString for the center of the rectangle will probably use a lot of String concatenation to get the point brackets and comma in ( , ) "

[Winston Gutkowski]

kenny gill wrote:I'm also thinking that my code does not look that great for getting values. so if you have any critiques I would be happy to hear them...

OK:
First: I would expect to see two classes: A Rectangle, and an Applet (RectangleApplet?) that uses a Rectangle. The added advantage to that is that you don't have to add 'Rectangle' to the names of all your methods.
A good rule to remember: A Rectangle is NOT (necessarily) an Applet; so it doesn't make sense to tie the two together.

Second: You haven't assigned any values to your 'answer2' and 'answer3' fields, so the likelihood is that you're getting 0 (and can you think of better names? It's part of the programmer's art).

Third: Your call to Graphics.drawRect() uses values copied from the values for 'width' and 'height'. How do you think you might do it better?

Winston