0

这个java代码有问题。我想制作一个画家程序,但是每当我选择一个形状并绘制它时,之前绘制的所有形状都会与这个形状相同。这是代码。我知道问题出在 for 语句中,paintComponent但我可以用什么替换它?

class inner extends JPanel implements MouseListener,MouseMotionListener{
        private int oldx,oldy,newx,newy;
        private Point point1,point2;
        private Shape newRec,line1;
        Rectangle currRec;
        private Vector shape;
        private boolean status,status1;
        private int count=0;
        private Object line;
        inner(){
            point1=point2=new Point(0,0);
            shape = new Vector();
            addMouseListener(this);
            addMouseMotionListener(this);
        }

        public void mouseDragged(MouseEvent event) {

            point2=event.getPoint();
            newx = point2.x;
            newy = point2.y;
            if(Universal==7){
            line = new Object(oldx,oldy,newx,newy);
            status = true;
            repaint();
            }


            currRec = new Rectangle(Math.min(point1.x,point2.x),Math.min(point1.y,point2.y),Math.abs(point1.x-point2.x),Math.abs(point1.y-point2.y));   
            status = true;
            repaint();

        }

        @Override
        public void mouseMoved(MouseEvent arg0) {}
        public void mouseClicked(MouseEvent arg0) {}                
        public void mouseEntered(MouseEvent arg0) {}
        @Override
        public void mouseExited(MouseEvent arg0) {}


        public void mousePressed(MouseEvent event) {

            point1=event.getPoint();
            oldx=event.getX();
            oldy=event.getY();
        }

        @Override
        public void mouseReleased(MouseEvent event) {

            point2=event.getPoint();
            newx=event.getX();
            newy=event.getY();
            //count++;


        if(Universal==7){
                line1 = new Shape(point1.x,point1.y,point2.x,point2.y);
                shape.add(line1);
                //Graphics g = getGraphics();
                //g.setColor(colour);
                //g.drawLine(point1.x,point1.y,point2.x,point2.y);

                count++;
                repaint();
            }
        else if(Universal==1||Universal==2||Universal==3||Universal==4||Universal==5||Universal==6){
                newRec = new Shape(Math.min(point1.x,point2.x),Math.min(point1.y,point2.y),Math.abs(point1.x-point2.x),Math.abs(point1.y-point2.y));
                shape.add(newRec);
                count++;
                repaint();  
            }
        }

    ///// the problem is in here    
        public void paintComponent(Graphics g){
            super.paintComponent(g);
            Shape c;
            g.setColor(colour);
            for(int i=0;i<shape.size();i++){
                c = (Shape) shape.get(i);

            if(Universal==1){

                g.drawRect(c.x, c.y, c.w, c.h);
                if(status){
                    g.drawRect(currRec.x, currRec.y, currRec.width, currRec.height);
                }
            }

            if(Universal==2){
                g.fillRect(c.x, c.y, c.w, c.h);

                if(status){
                    g.fillRect(currRec.x, currRec.y, currRec.width, currRec.height);

                }
            }

            if(Universal==3){
                g.drawRoundRect(c.x, c.y, c.w, c.h, c.w/4, c.h/4);

                if(status){
                    g.drawRoundRect(currRec.x, currRec.y, currRec.width, currRec.height,currRec.width/4,currRec.height/4);

                }
            }

            if(Universal==4){
                g.fillRoundRect(c.x, c.y, c.w, c.h, c.w/4, c.h/4);

                if(status){
                    g.fillRoundRect(currRec.x, currRec.y, currRec.width, currRec.height,currRec.width/4,currRec.height/4);

                }
            }

                if(Universal==5){

                    g.drawOval(c.x, c.y, c.w, c.h);

                    if(status){
                        g.drawOval(currRec.x, currRec.y, currRec.width, currRec.height);
                    }
                }

                if(Universal==6){

                    g.fillOval(c.x, c.y, c.w, c.h);

                    if(status){
                        g.fillOval(currRec.x, currRec.y, currRec.width, currRec.height);
                    }
                }

                if(Universal==7){

                    g.drawLine(c.x, c.y, c.w, c.h);

                    if(status){
                        g.drawLine(line1.x, line1.y, line1.w,line1.h);
                    }
                }

                if(Universal==8){
                    shape.clear();
                }
            }
        }
4

2 回答 2

1

有关执行此操作的两种不同方法,请参见自定义绘画方法:

  1. 将形状添加到列表中,然后在每次调用paintComponent() 时重新绘制列表中的所有形状。

  2. 将形状绘制到 BufferedImage,然后在每次调用 paintComponent() 时重新绘制图像。

这两个示例都不完全符合您的要求,它仅向您展示了方法。

于 2013-05-07T22:49:42.580 回答
1

Universal只会在任何给定时间成为给定值。

油漆不是累积的,它们是破坏性的。

也就是说,每次paintComponent调用时,所有先前的内容都会被删除/擦除,并且您应该“重新绘制”这些内容。

不要依赖单个标志,您应该将Shapes 添加到某种类型并在调用List时重新绘制它们。paintComponent同样,您可以简单地将“类型”(int)添加到 aList并在每次重绘时处理该列表

查看在 Swing 中的 AWT 中的绘画,以了解绘画过程

于 2013-05-07T19:27:42.253 回答