web-dev-qa-db-ja.com

クラスは抽象ではなく、抽象メソッドをオーバーライドしません

そのため、プログラミングクラスの抽象化に関する宿題に取り組んでおり、問題に陥りました。私の現在の目標は、抽象化を使用し、その後、長方形の建物や街灯の楕円形のライトのような単純な都市を長方形と楕円で描くことができるようにすることです。

コンパイル時に受け取るエラーは次のとおりです。MyTestApp.Rectangleは抽象的ではなく、MyTestApp.Shapeの抽象メソッドdrawEllipse（Java.awt.Graphics）をオーバーライドしません。このエラーは、「クラスRectangleがShape {を拡張する」という行に表示されます。

私の質問は、抽象化で何が間違っているのですか？しばらくの間、RectangleクラスとEllipseクラスのコンストラクタとdraw（）メソッドをいじり続けてきましたが、解決策を見つけることはできませんでした。

コードは次のとおりです。

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);
            }
        }
}

読んで助けてくれてありがとう！

javainheritanceabstract-classdraw
16
user063

RectangleとEllipseの両方のクラスは、両方の抽象メソッドをオーバーライドする必要があります。

これを回避するには、3つのオプションがあります。

  • 2つのメソッドを追加します
  • Shapeを拡張する各クラスを抽象化します

  • Shapeを拡張するクラスの機能を実行する単一のメソッドを用意し、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(/* ... */);

繰り返しますが、ほんの一例です。

6
Breeze

多態的な振る舞いを利用しようとする場合、外部のクラスに見えるメソッド（多態性を必要とする）が同じシグネチャを持つようにする必要があります。つまり、パラメータの名前、数、順序、およびパラメータタイプが同じである必要があります。

あなたの場合、一般的なdraw()メソッドを用意し、サブクラス（Rectangle、Ellipse）に依存してdraw()メソッドを実装する方が良いかもしれませんあなたが「drawEllipse」および「drawRectangle」と考えていたものとして。

2
Platinum Azure