[OOC] Myriad Reality

[GalaxyRaider]

Time for Please Help Me Find Out What's Wrong With My Code Because It Is Killing Me 2!
If you don't mind it.
I'm mainly concerned with Shape and Rectangle right now. If I get those, I'll be fine. I get no errors when running this, but my values for the Area and Perimeter are always 0.0 no matter what I do.

Shape(dot)java

import java.util.Scanner;
public class Shape {
public static String name = "Unknown";
public static double Area = 0;
public static double Perimeter = 0;
public static void methodName() {
System.out.println("Name of Shape: " + name);
System.out.println("Area of Shape: " + Area);
System.out.println("Perimeter of Shape: " + Perimeter);
}
public static void main(String[] args) {
double length = 0;
double width = 0;
System.out.println("Greetings! Please enter 1 for Rectangle, 2 for Square, and 3 for Circle: ");
Scanner in = new Scanner(System.in);
int userInput = in.nextInt();
if(userInput == 1) {
System.out.println("Enter length: ");
Rectangle em = new Rectangle(length, width);
Rectangle dm = new Rectangle();
Rectangle.calculateArea();
}
else if(userInput == 2) {
Rectangle dm = new Rectangle();
Square dm1 = new Square(Area);
}
else if(userInput == 3) {
Circle dm = new Circle();
}
}
private static void getMethodName() {
// TODO Auto-generated method stub

}
}

Rectangle(dot)java

import java.util.Scanner;
public class Rectangle extends Shape {
Scanner in = new Scanner(System.in);
public static double length;
public static double width;
public Rectangle() {
name = "Rectangle";
length = 0;
width = 0;
System.out.println("Name of Shape: " + name);
}
public Rectangle(double length, double width) {
double tempLength = in.nextInt();
System.out.println("Enter width: ");
double tempWidth = in.nextInt();
length = tempLength;
width = tempWidth;
this.calculateArea();
this.calculatePerimeter();
}
public static void calculateArea() {
Area = length*width;
System.out.println("Area of Shape: " + Area);
}
public static void calculatePerimeter() {
Perimeter = (2*length) + (2*width);
System.out.println("Perimeter of Shape: " + Perimeter);
}

}

Circle(dot)java

public class Circle extends Shape {
public static double radius;
static double pi = 3.141592;
public Circle() {
name = "Circle";
radius = 0;
}
public Circle(double radius) {
radius = in.nextInt();
}
public static void calculateArea() {
Area = pi*radius*radius;
}
public static void calculatePerimeter() {
Perimeter = 2*pi*radius;
}
}

Square(dot)java

public class Square extends Rectangle {
public static double sideLength;
public Square(double sideLength) {
sideLength = in.nextInt();
length = sideLength;
width = sideLength;
}
}

[Legend]

Time for Please Help Me Find Out What's Wrong With My Code Because It Is Killing Me 2!
If you don't mind it.
I'm mainly concerned with Shape and Rectangle right now. If I get those, I'll be fine. I get no errors when running this, but my values for the Area and Perimeter are always 0.0 no matter what I do.

Shape(dot)java

import java.util.Scanner;
public class Shape {
public static String name = "Unknown";
public static double Area = 0;
public static double Perimeter = 0;
public static void methodName() {
System.out.println("Name of Shape: " + name);
System.out.println("Area of Shape: " + Area);
System.out.println("Perimeter of Shape: " + Perimeter);
}
public static void main(String[] args) {
double length = 0;
double width = 0;
System.out.println("Greetings! Please enter 1 for Rectangle, 2 for Square, and 3 for Circle: ");
Scanner in = new Scanner(System.in);
int userInput = in.nextInt();
if(userInput == 1) {
System.out.println("Enter length: ");
Rectangle em = new Rectangle(length, width);
Rectangle dm = new Rectangle();
Rectangle.calculateArea();
}
else if(userInput == 2) {
Rectangle dm = new Rectangle();
Square dm1 = new Square(Area);
}
else if(userInput == 3) {
Circle dm = new Circle();
}
}
private static void getMethodName() {
// TODO Auto-generated method stub

}
}

Rectangle(dot)java

import java.util.Scanner;
public class Rectangle extends Shape {
Scanner in = new Scanner(System.in);
public static double length;
public static double width;
public Rectangle() {
name = "Rectangle";
length = 0;
width = 0;
System.out.println("Name of Shape: " + name);
}
public Rectangle(double length, double width) {
double tempLength = in.nextInt();
System.out.println("Enter width: ");
double tempWidth = in.nextInt();
length = tempLength;
width = tempWidth;
this.calculateArea();
this.calculatePerimeter();
}
public static void calculateArea() {
Area = length*width;
System.out.println("Area of Shape: " + Area);
}
public static void calculatePerimeter() {
Perimeter = (2*length) + (2*width);
System.out.println("Perimeter of Shape: " + Perimeter);
}

}

Circle(dot)java

public class Circle extends Shape {
public static double radius;
static double pi = 3.141592;
public Circle() {
name = "Circle";
radius = 0;
}
public Circle(double radius) {
radius = in.nextInt();
}
public static void calculateArea() {
Area = pi*radius*radius;
}
public static void calculatePerimeter() {
Perimeter = 2*pi*radius;
}
}

Square(dot)java

public class Square extends Rectangle {
public static double sideLength;
public Square(double sideLength) {
sideLength = in.nextInt();
length = sideLength;
width = sideLength;
}
}

@GalaxyRaider

I'll get you answers as soon as I'm out of class.

[Heroic]

<Snipped quote by GalaxyRaider>

I'll get you answers as soon as I'm out of class.

@Legend

It'll be much appreciated.

[Meta]

<Snipped quote by Legend>

It'll be much appreciated.

@Heroic

Unless you need them right away, of course.

[GalaxyRaider]

<Snipped quote by Heroic>

Unless you need them right away, of course.

@Meta

Nah. It's due in 2 and a half hours, but once I see where the mistake is, I should be able to fix it in a snap.

[Legend]

Time for Please Help Me Find Out What's Wrong With My Code Because It Is Killing Me 2!
If you don't mind it.
I'm mainly concerned with Shape and Rectangle right now. If I get those, I'll be fine. I get no errors when running this, but my values for the Area and Perimeter are always 0.0 no matter what I do.

Shape(dot)java

import java.util.Scanner;
public class Shape {
public static String name = "Unknown";
public static double Area = 0;
public static double Perimeter = 0;
public static void methodName() {
System.out.println("Name of Shape: " + name);
System.out.println("Area of Shape: " + Area);
System.out.println("Perimeter of Shape: " + Perimeter);
}
public static void main(String[] args) {
double length = 0;
double width = 0;
System.out.println("Greetings! Please enter 1 for Rectangle, 2 for Square, and 3 for Circle: ");
Scanner in = new Scanner(System.in);
int userInput = in.nextInt();
if(userInput == 1) {
System.out.println("Enter length: ");
Rectangle em = new Rectangle(length, width);
Rectangle dm = new Rectangle();
Rectangle.calculateArea();
}
else if(userInput == 2) {
Rectangle dm = new Rectangle();
Square dm1 = new Square(Area);
}
else if(userInput == 3) {
Circle dm = new Circle();
}
}
private static void getMethodName() {
// TODO Auto-generated method stub

}
}

Rectangle(dot)java

import java.util.Scanner;
public class Rectangle extends Shape {
Scanner in = new Scanner(System.in);
public static double length;
public static double width;
public Rectangle() {
name = "Rectangle";
length = 0;
width = 0;
System.out.println("Name of Shape: " + name);
}
public Rectangle(double length, double width) {
double tempLength = in.nextInt();
System.out.println("Enter width: ");
double tempWidth = in.nextInt();
length = tempLength;
width = tempWidth;
this.calculateArea();
this.calculatePerimeter();
}
public static void calculateArea() {
Area = length*width;
System.out.println("Area of Shape: " + Area);
}
public static void calculatePerimeter() {
Perimeter = (2*length) + (2*width);
System.out.println("Perimeter of Shape: " + Perimeter);
}

}

Circle(dot)java

public class Circle extends Shape {
public static double radius;
static double pi = 3.141592;
public Circle() {
name = "Circle";
radius = 0;
}
public Circle(double radius) {
radius = in.nextInt();
}
public static void calculateArea() {
Area = pi*radius*radius;
}
public static void calculatePerimeter() {
Perimeter = 2*pi*radius;
}
}

Square(dot)java

public class Square extends Rectangle {
public static double sideLength;
public Square(double sideLength) {
sideLength = in.nextInt();
length = sideLength;
width = sideLength;
}
}

@GalaxyRaider

Did you not import or initialize the scanner in Circle or Square?

[DarkwolfX37]

Time for Please Help Me Find Out What's Wrong With My Code Because It Is Killing Me 2!
If you don't mind it.
I'm mainly concerned with Shape and Rectangle right now. If I get those, I'll be fine. I get no errors when running this, but my values for the Area and Perimeter are always 0.0 no matter what I do.

Shape(dot)java

import java.util.Scanner;
public class Shape {
public static String name = "Unknown";
public static double Area = 0;
public static double Perimeter = 0;
public static void methodName() {
System.out.println("Name of Shape: " + name);
System.out.println("Area of Shape: " + Area);
System.out.println("Perimeter of Shape: " + Perimeter);
}
public static void main(String[] args) {
double length = 0;
double width = 0;
System.out.println("Greetings! Please enter 1 for Rectangle, 2 for Square, and 3 for Circle: ");
Scanner in = new Scanner(System.in);
int userInput = in.nextInt();
if(userInput == 1) {
System.out.println("Enter length: ");
Rectangle em = new Rectangle(length, width);
Rectangle dm = new Rectangle();
Rectangle.calculateArea();
}
else if(userInput == 2) {
Rectangle dm = new Rectangle();
Square dm1 = new Square(Area);
}
else if(userInput == 3) {
Circle dm = new Circle();
}
}
private static void getMethodName() {
// TODO Auto-generated method stub

}
}

Rectangle(dot)java

import java.util.Scanner;
public class Rectangle extends Shape {
Scanner in = new Scanner(System.in);
public static double length;
public static double width;
public Rectangle() {
name = "Rectangle";
length = 0;
width = 0;
System.out.println("Name of Shape: " + name);
}
public Rectangle(double length, double width) {
double tempLength = in.nextInt();
System.out.println("Enter width: ");
double tempWidth = in.nextInt();
length = tempLength;
width = tempWidth;
this.calculateArea();
this.calculatePerimeter();
}
public static void calculateArea() {
Area = length*width;
System.out.println("Area of Shape: " + Area);
}
public static void calculatePerimeter() {
Perimeter = (2*length) + (2*width);
System.out.println("Perimeter of Shape: " + Perimeter);
}

}

Circle(dot)java

public class Circle extends Shape {
public static double radius;
static double pi = 3.141592;
public Circle() {
name = "Circle";
radius = 0;
}
public Circle(double radius) {
radius = in.nextInt();
}
public static void calculateArea() {
Area = pi*radius*radius;
}
public static void calculatePerimeter() {
Perimeter = 2*pi*radius;
}
}

Square(dot)java

public class Square extends Rectangle {
public static double sideLength;
public Square(double sideLength) {
sideLength = in.nextInt();
length = sideLength;
width = sideLength;
}
}

@GalaxyRaider

Well for one, you might be better off getting the length and width in the same place.
Secondly, this might be a different thing with java than I learned, but personally I'd put "width * 2" and "length * 2" instead of "2*width" and "2*length". Not really a problem per say unless java has a hard time with the way you put it.
Third, you're defining length and width as 0. Instead of "templength" and "tempwidth" I'd suggest creating new variables and pulling those instead of trying to apply them back to length and width.
Fourth, it could be an issue of order. Having Rectangle(double length, double width) after defining them as zero could be the problem. Again, I suggest new variables entirely than ones you've already used.
But since I'm not much for Java I bet none of this is applicable.

[Legend]

Your static methods can't access the members of a class; they can only access static variables. You need to make your methods public but not static so they can be used by instances of the class. Also, the keyword this is not required, but feel free to use it if you want.

[Legend]

<Snipped quote by GalaxyRaider>

Well for one, you might be better off getting the length and width in the same place.
Secondly, this might be a different thing with java than I learned, but personally I'd put "width * 2" and "length * 2" instead of "2*width" and "2*length". Not really a problem per say unless java has a hard time with the way you put it.
Third, you're defining length and width as 0. Instead of "templength" and "tempwidth" I'd suggest creating new variables and pulling those instead of trying to apply them back to length and width.
Fourth, it could be an issue of order. Having Rectangle(double length, double width) after defining them as zero could be the problem. Again, I suggest new variables entirely than ones you've already used.
But since I'm not much for Java I bet none of this is applicable.

@DarkwolfX37

It's because static methods. They have to be instance methods because otherwise, the class looks at static variables length and width. I mean, you could technically pass this into the static method as a parameter, but that's super convoluted. Otherwise, the static method has absolutely no way to see instances of your class.

[DarkwolfX37]

<Snipped quote by DarkwolfX37>

It's because static methods. They have to be instance methods because otherwise, the class looks at static variables length and width. I mean, you could technically pass this into the static method as a parameter, but that's super convoluted. Otherwise, the static method has absolutely no way to see instances of your class.

@Legend

Ah, that does make more sense. I figured making new variables would work better, but I totally missed that they were static in the first place.
Geez I always feel like I'm back in class when you come up with a much simpler solution. Bad memories of making a long bit of code for something there was a function in the language for. :/

[Legend]

<Snipped quote by Legend>

Ah, that does make more sense. I figured making new variables would work better, but I totally missed that they were static in the first place.
Geez I always feel like I'm back in class when you come up with a much simpler solution. Bad memories of making a long bit of code for something there was a function in the language for. :/

@DarkwolfX37

Yeah, but also analyzing other peoples' code is difficult. The problem is easy to miss when you aren't familiar with the internals.

[DarkwolfX37]

<Snipped quote by DarkwolfX37>

Yeah, but also analyzing other peoples' code is difficult. The problem is easy to miss when you aren't familiar with the internals.

@Legend

True, but I prided myself on helping the people next to me when they had trouble so it hurts double that I can't with this.

[GalaxyRaider]

<Snipped quote by GalaxyRaider>

Did you not import or initialize the scanner in Circle or Square?

@Legend

Your static methods can't access the members of a class; they can only access static variables. You need to make your methods public but not static so they can be used by instances of the class. Also, the keyword this is not required, but feel free to use it if you want.

@Legend

Not yet. I'm trying to get Rectangle to work so that I can base my code in Square and Circle off it.

Also, I now have edited my code to where none of them are static anymore except for the main method, but results haven't changed.

[GalaxyRaider]

<Snipped quote by GalaxyRaider>

Well for one, you might be better off getting the length and width in the same place.
Secondly, this might be a different thing with java than I learned, but personally I'd put "width * 2" and "length * 2" instead of "2*width" and "2*length". Not really a problem per say unless java has a hard time with the way you put it.
Third, you're defining length and width as 0. Instead of "templength" and "tempwidth" I'd suggest creating new variables and pulling those instead of trying to apply them back to length and width.
Fourth, it could be an issue of order. Having Rectangle(double length, double width) after defining them as zero could be the problem. Again, I suggest new variables entirely than ones you've already used.
But since I'm not much for Java I bet none of this is applicable.

@DarkwolfX37

Well, whether it's right or not, some of this might be more efficient or appealing to coding etiquette so thank you.

[Legend]

Okay Sven, since that solution kind of opened up a can of worms, I went ahead and repaired all static issues and implemented Perimeter and Area methods in circle. This has been tested with all three shapes and produces correct results.

Shape

import java.util.Scanner;
public class Shape {
public static String name = "Unknown";
public static double Area = 0;
public static double Perimeter = 0;
public static void methodName() {
System.out.println("Name of Shape: " + name);
System.out.println("Area of Shape: " + Area);
System.out.println("Perimeter of Shape: " + Perimeter);
}
public static void main(String[] args) {
double length = 0;
double width = 0;
System.out.println("Greetings! Please enter 1 for Rectangle, 2 for Square, and 3 for Circle: ");
Scanner in = new Scanner(System.in);
int userInput = in.nextInt();
if(userInput == 1) {
System.out.println("Enter length: ");
Rectangle em = new Rectangle(length, width);
// Rectangle dm = new Rectangle();
// em.calculateArea();
}
else if(userInput == 2) {
// Rectangle dm = new Rectangle();
Square dm1 = new Square(Area);
}
else if(userInput == 3) {
Circle dm = new Circle();
}
}
private static void getMethodName() {
// TODO Auto-generated method stub

}
}

Rectangle

import java.util.Scanner;
public class Rectangle extends Shape {
Scanner in = new Scanner(System.in);
double length;

double width;
public Rectangle() {
name = "Rectangle";
length = 0;
width = 0;
System.out.println("Name of Shape: " + name);
}
public Rectangle(double length, double width) {
double tempLength = in.nextInt();
System.out.println("Enter width: ");
double tempWidth = in.nextInt();
length = tempLength;
width = tempWidth;
this.calculateArea(length, width);
this.calculatePerimeter(length, width);
}
public void calculateArea(double length, double width) {
Area = length*width;
System.out.println("Area of Shape: " + Area);
}
public void calculatePerimeter(double length, double width) {
Perimeter = (2*length) + (2*width);
System.out.println("Perimeter of Shape: " + Perimeter);
}

}

Square

import java.util.Scanner;
public class Square extends Rectangle {
Scanner in = new Scanner(System.in);
public static double sideLength;
public Square(double sideLength) {
System.out.print("Enter Length: ");
sideLength = in.nextInt();
length = sideLength;
width = sideLength;
calculatePerimeter(length, width);
calculateArea(length, width);
}
}

Circle

import java.util.Scanner;
public class Circle extends Shape {
Scanner in = new Scanner(System.in);
public static double radius;
static double pi = 3.141592;
public Circle() {
name = "Circle";
radius = 0;

System.out.print("Enter Radius: ");
radius = in.nextInt();

calculatePerimeter(radius);
calculateArea(radius);

}
public Circle(double radius) {
radius = in.nextInt();
}
public void calculateArea(double radius) {
Area = pi*radius*radius;
System.out.println("Area of Shape: " + Area);
}
public void calculatePerimeter(double radius) {
Perimeter = 2*pi*radius;
System.out.println("Perimeter of Shape: " + Perimeter);
}
}

The only downside is that I don't know what coding practices your class is supposed to follow, but I tried modeling it after what you already had. There are some constructors/methods that do nothing, but those were there before. I tried leaving everything as intact as possible, unless you want me to go in and pull out some of the extraneous stuff.

[Legend]

<Snipped quote by Legend>

True, but I prided myself on helping the people next to me when they had trouble so it hurts double that I can't with this.

@DarkwolfX37

At least it doesn't hurt int.



But you don't know Java, so it's extra not a big deal.

[GalaxyRaider]

@Legend@DarkwolfX37

Thanks for the help. I got done just in time.

[Legend]

@Legend@DarkwolfX37

Thanks for the help. I got done just in time.

@GalaxyRaider

Yeah, no problem. What did you have to do beyond area/perimeter?

[GalaxyRaider]

<Snipped quote by GalaxyRaider>

Yeah, no problem. What did you have to do beyond area/perimeter?

@Legend

Not much. But I did have to make some changes once I got the code to work so that it fit the specifications of the assignment.

[Legend]

<Snipped quote by Legend>

Not much. But I did have to make some changes once I got the code to work so that it fit the specifications of the assignment.

@GalaxyRaider

Coolio. Classes and objects are really hard.