Реализация универсального метода из интерфейса в Java

Question

У меня есть два интерфейса и один класс, реализующий их. В интерфейсе я показал один обычный метод и один универсальный метод. При реализации их в основном методе обычный метод показывает правильный результат, а общий - нет. Если возможно, не могли бы вы сказать мне, почему метод increment_twice() не возвращает правильный результат, пожалуйста? Вот мой код:

Rectangle.java:

public class Rectangle implements TwoDShape {

    private double width, height;

    public Rectangle (double width, double height) {
        this.width = width;
        this.height = height;
    }

    public double area() {
        return width * height;
    }

    public double perimeter() {
        return 2.0 * (width + height);
    }

    public void describe() {
        System.out.println("Rectangle[width=" + width + ", height=" + height + "]");
    }

    @Override
    public Rectangle increase_twice() {
        // TODO Auto-generated method stub
        return new Rectangle(2*this.width, 2*this.height);
    }


}

TwoDShape.java:

public interface TwoDShape extends GeometricShape {
    public double  area();


}

GeometricShape.java:

public interface GeometricShape<T extends GeometricShape<T>>
{
    public void describe();
    public T increase_twice();

}

И, наконец, протестируйте класс ArrayListExample.java

import java.util.ArrayList;


public class ArrayListExample {



     public static void describe_all( ArrayList<? extends GeometricShape> shapes )

     {
         for(int i=0;i<shapes.size();i++)
         {
            shapes.get(i).describe();

         } 
         System.out.println("Total number of shapes:"+ shapes.size());
     }


    private static ArrayList<Rectangle> increase_size(ArrayList<Rectangle> rects)
    {
          for(int i=0;i<rects.size();i++)
          {
            rects.get(i).increase_twice();

          }
        return rects; 

    }


     public static void main(String[] args) {


        System.out.println("The result of describe() method:"); 

        System.out.println();
        System.out.println("Example rectangles");
        ArrayList<Rectangle> rects = new ArrayList<Rectangle>();
        rects.add(new Rectangle(2.0, 3.0));
        rects.add(new Rectangle(5.0, 5.0));
        describe_all(rects);
        System.out.println();

        System.out.println("The result of increase_twice() method:"); 
        System.out.println();
        System.out.println("Example rectangles after increase:");
        ArrayList<Rectangle> double_rects =  increase_size(rects);
        describe_all(double_rects);

    }


}

Я ожидаю, что increment_twice вернет прямоугольники с Rectangle[width=4.0, height=6.0] Rectangle[width=10.0, height=10.0], но вместо этого вернет метод, описанный ниже: Rectangle[width=2.0, height=3.0] Прямоугольник [ширина =5,0, высота =5,0] Если возможно, подскажите, пожалуйста, где я ошибаюсь?

Answer 1

Я думаю, что проблема в функции incre ment_twice. Когда ты звонишь rects.get(i).increase_twice(); вы на самом деле создаете новый объект Rectangle с удвоенными значениями и возвращаете. Вы не обновляете текущий объект. Метод incre ment_twice должен быть:

@Override
    public void increase_twice() {
        // TODO Auto-generated method stub
        this.width *= 2;
        this.height *= 2;
    }

Таким образом, вы обновите значения ширины и высоты текущего объекта, которые вы получаете из ArrayList. Возможно, вам придется изменить определение incre ment_twice(), потому что я не думаю, что вам нужно возвращать новый объект Rectangle.

Дай мне знать, если это работает.

Answer 2

Rectangle.increase_twice() возвращает новый объект. Однако в ArrayListExample.increase_size() возвращаемый объект не установлен. Вы можете изменить этот метод следующим образом:

private static ArrayList<Rectangle> increase_size(ArrayList<Rectangle> rects) {
    for (int i = 0; i < rects.size(); i++) {
        rects.set(i, rects.get(i).increase_twice());
    }
    return rects;
}

Или вы можете изменить Rectangle.increase_twice() следующим образом:

@Override
    public void increase_twice() {
        // TODO Auto-generated method stub
        this.width = 2 * this.width;
        this.height = 2 * this.height;
    }

И объявление GeometricShape.increase_twice() для этого:

public void increase_twice();