Legend

Myriad Reality

<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.

Myriad Reality

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.

Myriad Reality

<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.

Myriad Reality

<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.

Myriad Reality

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.

Myriad Reality

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?

Myriad Reality

<Snipped quote by Legend>

I won't do much but get in the way.

@GalaxyRaider

Only with that attitude!

Myriad Reality

<Snipped quote by Legend>

Oh, no. That's the level of shenanigans in there. My level is meh.

@GalaxyRaider

Meh is still pretty good.

Myriad Reality

<Snipped quote by Legend>

Reeeaaaaally high.

@GalaxyRaider

If your level is high, what do you have to worry about?

Myriad Reality

<Snipped quote by Legend>

I don't think so. Sounds from here like it's not something on my level.

@GalaxyRaider

What level is that?