Класс не является абстрактным и не отменяет абстрактного метода

Поэтому я работал над домашней работой по абстракции для своего класса программирования и попал в проблему. Цель для меня прямо сейчас - иметь возможность использовать абстракцию, а затем иметь возможность рисовать с прямоугольниками и овалами простой город, как прямоугольное здание или овальный свет на светлом столбе.

Ошибка, которую я получаю при компиляции: MyTestApp.Rectangle не является абстрактным и не переопределяет абстрактный метод drawEllipse (java.awt.Graphics) в MyTestApp.Shape. Эта ошибка появляется в строке "class Rectangle extends Shape {" прямо под классом Shape.

Мой вопрос: что я делаю неправильно с моей абстракцией? Я уже давно разбираюсь в конструкторах и методах draw() в классах Rectangle и Ellipse, и до сих пор не удастся найти решение.

Код ниже:

import java.awt.*;
import javax.swing.*;

public class MyTestApp extends JPanel {
    Rectangle rect;
    Ellipse oval;
    public static void main(String [] args) {
        MyTestApp myTestApp = new MyTestApp ();
        myTestApp.test();
    }

    public MyTestApp () { //creates the jframe
        JFrame frame = new JFrame("MyClass Driver");
        setBackground(new Color(200, 250, 200));
        setPreferredSize(new Dimension(500, 400));
        frame.add(this);
        frame.pack();
        frame.setVisible(true);
    }

    public void delay(int msecs) {
        try {
            Thread.sleep(msecs);
        } catch (InterruptedException e) {
        }
    }

    public void paint(Graphics g) {//paints the rectangle and ellipse
        super.paint(g);
        if (rect != null)
            rect.drawRectangle(g);
        if (oval != null)
            oval.drawEllipse(g);
    }

    public void test() {//gives the x/y position, width/height, and fill/outline color for the rectangle and oval
        delay(1000);
        rect = new Rectangle(20, 30, 23, 75, Color.GREEN, Color.BLUE);
        oval = new Ellipse(10, 10, 10 , 34, Color.RED, Color.MAGENTA);
        repaint();
    }

    public abstract class Shape{//abstract class Shape that sets the x/y, width/height, and colors for the shapes
        private int x, y, width, height;
        private Color fillColor;
        private Color outlineColor;
        public Shape(int x, int y, int width, int height, Color fillColor, Color outlineColor) {
            setXY(x, y);
            setSize(width, height);
            setFillColor(fillColor);
            setOutlineColor(outlineColor);  
        }

        public boolean setXY(int x, int y) {
            this.x = x;
            this.y = y;
            return true;
        }

        public void setSize(int width, int height) {
            if (width > 0)
                this.width = width;
            if (height > 0)
                this.height = height;
        }

        public boolean setFillColor(Color fillColor){
            if (fillColor == null) return false;
            this.fillColor = fillColor; 
            return true;
        }

        public boolean setOutlineColor(Color outlineColor){
            if (outlineColor == null) return false;
            this.outlineColor = outlineColor; 
            return true;
        }

        public Color getFillColor() {
            return fillColor;
        } 

        public Color getOutlineColor() {
            return outlineColor;
        } 

        public abstract void drawRectangle(Graphics g);//do i need two?
        public abstract void drawEllipse(Graphics g);//do i need both?
    }
    class Rectangle extends Shape{//!!!!!!!!!! where the error shows
        public Rectangle(int x, int y, int width, int height, Color fillColor, Color outlineColor) {
            super(x, y, width, height, fillColor, outlineColor);
        }

        public void drawRectangle(Graphics g){//draws the retangle
            g.setColor(fillColor);
            g.fillRect(x, y, width, height);
            g.setColor(outlineColor);
            g.drawRect(x, y, width, height);
        }
    }
    class Ellipse extends Shape{
        public Ellipse(int x, int y, int width, int height, Color fillColor, Color outlineColor) {
            super(x, y, width, height, fillColor, outlineColor);
        }

        public void drawEllipse(Graphics g){//draws the ellipse
            g.setColor(fillColor);
            g.fillOval(x, y, width, height);
            g.setColor(outlineColor);
                g.drawOval(x, y, width, height);
            }
        }
}

Спасибо за чтение и помощь!

Ответ 1

Оба класса Rectangle и Ellipse должны переопределять оба абстрактных метода.

Чтобы обойти это, у вас есть 3 варианта:

Добавьте два метода
Сделайте каждый класс, который расширяет абстрактную форму
Имейте один метод, который выполняет функцию классов, которые будут расширять форму, и переопределяет этот метод в Rectangle и Ellipse, например:
```
abstract class Shape {
    // ...
    void draw(Graphics g);
}
```

А также

    class Rectangle extends Shape {
        void draw(Graphics g) {
            // ...
        }
    }

в заключение

    class Ellipse extends Shape {
        void draw(Graphics g) {
            // ...
        }
    }

И вы можете переключаться между ними, например:

    Shape shape = new Ellipse();
    shape.draw(/* ... */);

    shape = new Rectangle();
    shape.draw(/* ... */);

Опять же, просто пример.

Ответ 2

Если вы пытаетесь воспользоваться полиморфным поведением, вам необходимо убедиться, что методы, видимые для внешних классов (которые требуют полиморфизма), имеют одну и ту же подпись. Это означает, что они должны иметь одинаковое имя, номер и порядок параметров, а также типы параметров.

В вашем случае вам может быть лучше иметь общий метод draw() и полагаться на подклассы (Rectangle, Ellipse), чтобы реализовать метод draw() как то, о чем вы думали как "drawEllipse" и "drawRectangle".