1

我正在为学校制作一个Java小程序，其功能是随机选择六个数字作为三个点的坐标并将它们连接成一个三角形。它只应该绘制一个三角形并找到“边长”。但是，当我把它放在我的网站上时，它会多次重绘自己。

我制作了另一个更简单的小程序，它只选择 4 个随机数作为坐标来绘制一条线。同样的问题。

重绘问题似乎发生在用户移动屏幕时，例如当我滚动或在 Eclipse 中调整小程序查看器的大小时。我的源代码发布在这里。

我很感激任何帮助！谢谢！

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

@SuppressWarnings("serial")
public class LineApplet extends JApplet {

    /**
     * Create the applet.
     */

    static int width;
    int height;


    public void init() {

          width = getSize().width;
          height = getSize().height;

       }

    public static int[] randomLine() {
        int[] pointArray = new int[4];
        int x;
        for (int i = 0; i < 4; i++) {
            x = ((int)(Math.random()*(width/10-2)))*20+10;
            pointArray[i] = x;
        }
        return pointArray;
    }

    public void paint(Graphics g) {
        g.setColor(Color.blue);
        int[] coords = new int[4];
        coords = randomLine();
        g.drawLine(coords[0], coords[1], coords[2], coords[3]);
        g.drawString(coords[0]/10 + ", " + coords[1]/10, coords[0], coords[1]);
        g.drawString(coords[2]/10 + ", " + coords[3]/10, coords[2], coords[3]);
        int midpointx = (coords[0] + coords[2])/2;
        int midpointy = (coords[1] + coords[3])/2;
        g.drawString(midpointx/10 + ", " + midpointy/10, midpointx, midpointy);
    }
}
4

3 回答 3

2

正如 Reimues 所指出的，每次重新绘制小程序时，您都会重新生成坐标。

您的方法的另一个问题paint实际上是您不清楚图形上下文的先前状态（这将由 完成paint，但是当您覆盖它时您没有尊重它的功能）。

你有两个选择。

  1. 称呼super.paint(g)
  2. 打电话super.paint(g)和打电话Graphics#clearRect(int, int, int, int)或Graphics#fillRect(int, int, int, int)

您还应该（很少）需要覆盖paint顶级容器的方法。一个原因是它们没有双缓冲，另一个是油漆链复杂且容易断裂......

你最好使用一个JPanel（或这样的）并覆盖该paintComponent方法......

更新

我更新了您的代码以演示这些问题。

public class TestBadApplet extends JApplet {

    public void init() {
    }

    @Override
    public void start() {

        final LinePane linePane = new LinePane();

        setLayout(new BorderLayout());
        JButton update = new JButton("Update");
        add(linePane);
        add(update, BorderLayout.SOUTH);

        update.addActionListener(new ActionListener() {
            @Override
            public void actionPerformed(ActionEvent e) {
                linePane.regenerate();
            }
        });

    }

    protected class LinePane extends JPanel {

        private int[] coords = new int[4];

        public void regenerate() {
            coords = randomLine();
            repaint();
        }

        public int[] randomLine() {
            int[] pointArray = new int[4];
            int x;
            for (int i = 0; i < 4; i++) {
                x = ((int) (Math.random() * (Math.min(getWidth(), getHeight()) / 10 - 2))) * 20 + 10;
                pointArray[i] = x;
            }
            return pointArray;
        }

        @Override
        protected void paintComponent(Graphics g) {
            super.paintComponent(g);
            g.setColor(Color.blue);
            g.drawLine(coords[0], coords[1], coords[2], coords[3]);
            g.drawString(coords[0] / 10 + ", " + coords[1] / 10, coords[0], coords[1]);
            g.drawString(coords[2] / 10 + ", " + coords[3] / 10, coords[2], coords[3]);
            int midpointx = (coords[0] + coords[2]) / 2;
            int midpointy = (coords[1] + coords[3]) / 2;
            g.drawString(midpointx / 10 + ", " + midpointy / 10, midpointx, midpointy);
        }
    }
}

和super.paintComponent

好小程序

没有super.paintComponent

坏小程序

于 2012-10-31T22:25:29.477 回答
2

paint()每次调用时，您都在计算新坐标。

paint()每次调整小程序窗口大小或重新获得焦点时调用。

要修复，您可以制作

int[] coords = new int[4];

类成员变量并移动

coords = randomLine();

到您的init()方法，该方法只会在初始化时调用一次。

附录：

  • super.paint(g);覆盖时总是调用paint()。

  • 对于使用 Swing 进行自定义绘制，首选方法是扩展JComponent组件，利用paintComponent.

有关更多信息，请参阅：执行自定义绘画。

于 2012-10-31T22:15:15.460 回答
0

问题似乎在于paint()每次组件需要重新绘制时都会调用它。

paint()方法应该以不产生副作用的方式编写，并且它们不应该改变小程序的内部状态。paint()必须仅限于这样做：绘画。

于 2012-10-31T22:15:52.360 回答