Question

我有一个列表，其中包含我在gui上绘制的线条。我希望能够选择一行并通过单击该行删除它。我可以选择和删除，但问题是重绘方法擦除所有绘制的行，当我这样做。当我开始绘制一条新线时，线条减去先前删除的线条重新出现。

我觉得这可能是因为迭代器，但我不完全确定。任何人都可以帮助解决此问题吗？或者我做错了吗？

这是我的代码：

import java.awt.Color;
import java.awt.Dimension;
import java.awt.Graphics;
import java.awt.Graphics2D;
import java.awt.Point;
import java.awt.RenderingHints;
import java.awt.Shape;
import java.awt.event.MouseAdapter;
import java.awt.event.MouseEvent;
import java.awt.event.MouseMotionAdapter;
import java.awt.geom.Line2D;
import java.util.ArrayList;
import java.util.Iterator;
import java.util.List;

import javax.swing.JFrame;
import javax.swing.JPanel;

public class LinesDemo extends JPanel {

private List<Line2D> linesList = new ArrayList<>();
private List<Line2D> lineCollection = new ArrayList<>();
private Line2D line = null;
Point pointStart = null;
Point pointEnd   = null;
// Width and height of rectangular region around mouse
// pointer to use for hit detection on lines
private static final int HIT_BOX_SIZE = 4;

/**
 * Create the panel.
 */
public LinesDemo() {

    addMouseListener(new MouseAdapter() {
        @Override
        public void mousePressed(MouseEvent e) {
            pointStart = e.getPoint();
            line = new Line2D.Double(e.getPoint(), e.getPoint());
            linesList.add(line);
        }

        @Override
        public void mouseReleased(MouseEvent e) {
            pointStart = null;
        }

        @Override
        public void mouseClicked(MouseEvent e) {
            int x = e.getX();
            int y = e.getY();
            getClickedLine(x, y);
        }
    });

    addMouseMotionListener(new MouseMotionAdapter() {
        @Override
        public void mouseMoved(MouseEvent e) {
            pointEnd = e.getPoint();
        }

        @Override
        public void mouseDragged(MouseEvent e) {
            pointEnd = e.getPoint();
            line.setLine(pointStart, pointEnd);
            lineCollection.add(line);
            repaint();
        }
    });
}

private Shape getClickedLine(int x, int y) {

    int boxX = x - HIT_BOX_SIZE / 2;
    int boxY = y - HIT_BOX_SIZE / 2;

    int width = HIT_BOX_SIZE;
    int height = HIT_BOX_SIZE;

    for(Line2D selectedLine:lineCollection) {
        if (selectedLine.intersects(boxX, boxY, width, height)) {
            System.out.println("intersects!");
            removeLine(selectedLine);
            return selectedLine;
        }
    }
    return null;
}

private void removeLine(Line2D line) {
    Iterator<Line2D> it = lineCollection.iterator();
    while(it.hasNext()) {
        Line2D selectedLine = it.next();
        if(selectedLine.equals(line)) {
            it.remove();
            repaint();
        }
    }
}

@Override
public void paintComponent(Graphics g){
    super.paintComponent(g);
    Graphics2D g2d = (Graphics2D) g;
    if(pointStart != null) {
        g2d.setPaint(Color.RED);
        g2d.setRenderingHint(RenderingHints.KEY_ANTIALIASING, RenderingHints.VALUE_ANTIALIAS_ON);
        for(Shape content : lineCollection){
            g2d.draw(content);
        }
        g2d.dispose();

    }
}

@Override
public Dimension getPreferredSize() {
    return new Dimension(300, 300);
}

public static void main(String[] args) {
    JFrame frame = new JFrame("Moving and Scaling");
    LinesDemo m = new LinesDemo();
    frame.add(m);
    frame.setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
    frame.setSize(300, 300);
    frame.setLocationRelativeTo(null);
    frame.setVisible(true);
}
}

Answer 1

线条消失的原因是您在paintComponent()中明确禁止绘制：

if (pointStart != null) {

pointStart已设置为null中的mouseReleased，因此不会绘制任何行。删除该检查，您的代码应该按预期工作。

作为旁注，您应该在event dispatch thread。

中创建GUI

Answer 2

UPD2：

这是您的代码已修复。还有许多其他问题，例如：每次拖动鼠标时都会在列表中添加一个点等。它还远未完美，但已有足够的变化供您分析。

import java.awt.Color;
import java.awt.Dimension;
import java.awt.Graphics;
import java.awt.Graphics2D;
import java.awt.Point;
import java.awt.RenderingHints;
import java.awt.Shape;
import java.awt.event.MouseAdapter;
import java.awt.event.MouseEvent;
import java.awt.event.MouseMotionAdapter;
import java.awt.geom.Line2D;
import java.util.ArrayList;
import java.util.Iterator;
import java.util.List;

import javax.swing.JFrame;
import javax.swing.JPanel;

public class LinesDemo extends JPanel {

private List<Line2D> linesList = new ArrayList<>();
private Line2D line = new Line2D.Double();
Point pointStart = null;
Point pointEnd   = null;
// Width and height of rectangular region around mouse
// pointer to use for hit detection on lines
private static final int HIT_BOX_SIZE = 4;

/**
 * Create the panel.
 */
public LinesDemo() {

    addMouseListener(new MouseAdapter() {
        @Override
        public void mousePressed(MouseEvent e) {
            pointStart = e.getPoint();
            pointEnd = e.getPoint();
            line = new Line2D.Double(pointStart, pointEnd);
            linesList.add(line);
        }

        @Override
        public void mouseReleased(MouseEvent e) {
            line.setLine(pointStart, pointEnd);
            pointStart = null;
        }

        @Override
        public void mouseClicked(MouseEvent e) {
            int x = e.getX();
            int y = e.getY();
            getClickedLine(x, y);
        }
    });

    addMouseMotionListener(new MouseMotionAdapter() {
        @Override
        public void mouseMoved(MouseEvent e) {
            pointEnd = e.getPoint();
        }

        @Override
        public void mouseDragged(MouseEvent e) {
            pointEnd = e.getPoint();
            line.setLine(pointStart, pointEnd);
            repaint();
        }
    });
}

private Shape getClickedLine(int x, int y) {

    int boxX = x - HIT_BOX_SIZE / 2;
    int boxY = y - HIT_BOX_SIZE / 2;

    int width = HIT_BOX_SIZE;
    int height = HIT_BOX_SIZE;

    for(Line2D selectedLine:linesList) {
        if (selectedLine.intersects(boxX, boxY, width, height)) {
            System.out.println("intersects!");
            removeLine(selectedLine);
            return selectedLine;
        }
    }
    return null;
}

private void removeLine(Line2D line) {
    Iterator<Line2D> it = linesList.iterator();
    while(it.hasNext()) {
        Line2D selectedLine = it.next();
        if(selectedLine.equals(line)) {
            it.remove();
            repaint();
        }
    }
}

@Override
public void paintComponent(Graphics g){
    super.paintComponent(g);
    Graphics2D g2d = (Graphics2D) g;
    g2d.setPaint(Color.RED);
    g2d.setRenderingHint(RenderingHints.KEY_ANTIALIASING, RenderingHints.VALUE_ANTIALIAS_ON);
    for(Shape content : linesList){
        g2d.draw(content);
    }
    g2d.dispose();
}

@Override
public Dimension getPreferredSize() {
    return new Dimension(300, 300);
}

public static void main(String[] args) {
    JFrame frame = new JFrame("Moving and Scaling");
    LinesDemo m = new LinesDemo();
    frame.add(m);
    frame.setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
    frame.setSize(300, 300);
    frame.setLocationRelativeTo(null);
    frame.setVisible(true);
}
}

原件：我无法运行您的程序，但您可以尝试替换

while(it.hasNext()) {
    Line2D selectedLine = it.next();
    if(selectedLine.equals(line)) {
        it.remove();
        repaint();
    }
}

与

boolean repaintNeeded = false;
while(it.hasNext()) {
    Line2D selectedLine = it.next();
    if(selectedLine.equals(line)) {
        it.remove();
        repaintNeeded = true;
    }
}
if (repaintNeeded) {
  repaint();
}

UPD1：我能够运行你的代码。实际上有许多事情需要改进。特别是对于你的问题，这两件事会导致你的画面没有画出来：

    @Override
    public void mouseReleased(MouseEvent e) {
        pointStart = null;
    }

和

if(pointStart != null) {

因此，每次释放鼠标时，您的代码都不会绘制任何内容。

删除java图形中的绘制线条

2 个答案: