Пишу тестовое сейчас на понимание ООП в Java и столкнулся с таким диссонансом:
public abstract class Shape {
protected String color;
protected double area;
Shape() {
color = null;
area = 0;
}
Shape (String color, double area) {
this.color = color;
this.area = area;
}
public String getColor() {
return color;
}
public abstract double getArea();
public void draw() {
System.out.println(this);
}
public String toString() {
return "Shape: " + this.getClass().getSimpleName() + " color: " + color;
}
}
class Square extends Shape {
private double side;
Square(double side) {
super(Colors.YELLOW.name(), side * side);
this.side = side;
}
public double getSide() {
return side;
}
@Override
public double getArea() {
return area;
}
}
Переменная area вроде как излишня. Её можно убрать, в конструкторы меньше инфы передавать и вычисления делать в геттере getArea()
.
Вот так:
public abstract class Shape {
protected String color;
Shape() {
color = null;
}
Shape (String color) {
this.color = color;
}
public String getColor() {
return color;
}
public abstract double getArea();
public void draw() {
System.out.println(this);
}
public String toString() {
return "Shape: " + this.getClass().getSimpleName() + " color: " + color;
}
}
class Square extends Shape {
private double side;
Square(double side) {
super(Colors.YELLOW.name());
this.side = side;
}
public double getSide() {
return side;
}
@Override
public double getArea() {
return side * side;
}
}
С одной стороны мне кажется, что это будет трудно читаемо и памяти потребуется куда больше (представим цикл, который с каждой итерацией будет не смотреть значение в переменной, а постоянно по новой его вычислять), а с другой, что так неправильно и появится куча лишних ненужных переменных, а конструктор будет перегружен их инициализацией.