1

我正在尝试制作一个线描程序。当我尝试重新绘制所有线条（在创建新线条之后）时，只有最后一条被绘制出来。问题可能出在重新绘制中。

有人可以看到我做错了什么吗？

代码在这里：

public class Kimp extends JFrame {

private ArrayList<Point[]> pointsArray = new ArrayList<>();
private Point points[] = new Point[10000];
private int pointCounter = 0;

public Kimp () {

     panel paintArea = new panel();
     add(paintArea, BorderLayout.CENTER);
}

private class panel  extends JPanel {

    public panel () {
        HandlerClass handler = new HandlerClass();
        this.addMouseListener(handler);
        this.addMouseMotionListener(handler);
    }

    @Override
     void paintComponent(Graphics g) {
        super.paintComponent(g);
        try {


            for (Point[] p : pointsArray) {
                for(int i = 0; i < p.length; i++) {                         
                    if (p[i].x == 0) {
                       continue;
                    } else {
                        if (p[i + 1].x == 0) {
                            g.setColor(Color.BLUE);
                            g.drawLine(p[i].x, p[i].y, p[i].x, p[i].y);
                        } else {
                            g.setColor(Color.BLUE);
                            g.drawLine(p[i].x, p[i].y, p[i + 1].x, p[i + 1].y);
                        }
                    } 
                } 
            }
            points = preFill(points);
        } catch (NullPointerException e) {

        }                  
    }
}

private class HandlerClass implements MouseListener , MouseMotionListener {

    @Override
    public void mouseDragged(MouseEvent e) {
        points[pointCounter++] = e.getPoint();
    }

    @Override
    public void mousePressed(MouseEvent e) {
        points[pointCounter] = e.getPoint();
    }

    @Override
    public void mouseMoved(MouseEvent e) {
    }

    @Override
    public void mouseClicked(MouseEvent e) {
    }

    @Override
    public void mouseEntered(MouseEvent e) {
    }

    @Override
    public void mouseExited(MouseEvent e) {
    }

}

public static Point[] preFill (Point[] points) {
    for (int i = 0; i < points.length; i++) {
        points[i] = new Point(-999,-999);
    }
    return points;
}

}

4

3 回答 3

1

我迅速重写了您的代码以尽可能简化它。您可能能够更好地理解其背后的概念。

import java.awt.Graphics;
import java.awt.Point;
import java.awt.Color;
import java.awt.event.*;

import java.util.*;
import javax.swing.*;

public class Kimp {
    public static void main(String[] args) {
        JFrame frame = new JFrame("Kimp!");
        frame.setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
        frame.setSize(800, 600);
        frame.setLocationRelativeTo(null);
        frame.add(new CanvasPanel());
        frame.setVisible(true);
    }
}

class CanvasPanel extends JPanel {
    private final List<List<Point>> lines = new LinkedList<List<Point>>();
    private List<Point> points = new LinkedList<Point>();

    public CanvasPanel() {
        addMouseListener(mouseAdapter);
        addMouseMotionListener(mouseAdapter);
    }

    @Override
    public void paintComponent(Graphics g) {
        g.setColor(Color.WHITE);
        g.fillRect(0, 0, getWidth(), getHeight());

        for (List<Point> line : lines)
            drawLine(line, g);
        drawLine(points, g);
    }

    private void drawLine(List<Point> points, Graphics g) {
        if (points.size() < 2) return;

        Point p1 = points.get(0);

        for (int i=1, n=points.size(); i<n; i++) {
            Point p2 = points.get(i);

            g.setColor(Color.BLUE);
            g.drawLine(p1.x, p1.y, p2.x, p2.y);

            p1 = p2;
        }
    }

    private MouseAdapter mouseAdapter = new MouseAdapter() {
        @Override
        public void mousePressed(MouseEvent e) {
            points.add(e.getPoint());
            repaint();
        }

        @Override
        public void mouseDragged(MouseEvent e) {
            points.add(e.getPoint());
            repaint();
        }

        @Override
        public void mouseReleased(MouseEvent e) {
            if (points.size() > 1) {
                lines.add(points);
                points = new LinkedList<Point>();
            }
        }
    };
}
于 2013-10-29T18:56:47.210 回答
1 
private ArrayList<Point[]> pointsArray = new ArrayList<>();

这是 Point[] 的列表。这里只private Point points[] = new Point[10000];分配一个。这意味着每次添加points到pointArray中时，都会添加 的相同实例points。

当您更改 的元素时points，所有现有Point[]的 inpointArray都会更新。由于所有元素都指的是相同的points.

绘制新线时分配 new Point[]。

于 2013-10-29T18:46:09.483 回答
1

每次绘制它时都会重置点数组（在paint() 中调用preFillPoints()）。您的 pointsArray （看在上帝的份上应该称为 pointsList ）完全没有意义，每次释放鼠标时都添加相同的 points[] 数组。

代码没有意义。您只需要列表或数组。不是都。声明一个 List<Point>（不是 Point[]），每次鼠标移动时添加一个新的 Point。

于 2013-10-29T18:50:47.613 回答