Java Graphics - Используя конструктор, дважды удалите старую графику

Question

У меня есть программа, которая отображает множество элементов одного класса на разных позициях. Для этого я использую конструктор класса много раз. Но проблема в том, что отображается только последний элемент, а не остальные.

Инициализация фрейма и где называются конструкторы:

private void initComponents() {

    setExtendedState(MAXIMIZED_BOTH);
    int width = (int) getContentPane().getBounds().getWidth();
    int height = (int) getContentPane().getBounds().getHeight();
    setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);

    add(new gleis(0, 0, 0, 0));
    add(new gleis(1, 0, 1, 0));
    add(new gleis(2, 0, 2, 0));

    this.setVisible(true);
}

Класс элемента с именем 'gleis':

public class gleis extends JPanel {

    int xposition;
    int yposition;
    int id;
    int status;

    public gleis(int xpos, int ypos, int id, int status){

        this.xposition = xpos * 11;
        this.yposition = ypos * 11 + 4;
        this.id = id;
        this.status = status;

        System.out.println(this.id);
    }

    public void paintComponent(Graphics g1) {
        super.paintComponent(g1);
        Graphics2D g = (Graphics2D) g1;
        g.setColor(Color.black);
        g.fillRect(this.xposition, this.yposition, 11, 4);

        if (this.status == 0)
            g.setColor(Color.gray);
        else if (this.status == 1)
            g.setColor(Color.red);
        else if (this.status == 2)
            g.setColor(Color.yellow);
        else 
            g.setColor(Color.gray);

        g.fillRect(this.xposition + 2, this.yposition + 1, 7, 2);


    }

}

Итак, как я могу показать все элементы, которые я хочу иметь?

Answer 1

Скорее всего, вы добавляете свой компонент в contentPane JFrame, компонент, который использует BorderLayout в качестве макета по умолчанию, и по этой причине последний введенный компонент покрывает все остальные, поскольку они по умолчанию добавляются в позицию BorderLayout.CENTER.

Исходя из вашего кода и моего предположения о вашем сценарии использования, я бы предложил изменить весь ваш подход:

• Используйте один единственный класс, который расширяет JPanel в качестве вашего чертежа JPanel, и переопределите его метод paintComponent.
• Добавьте этот JPanel к вашему JFrame.
• Измените имя вашего класса gleis на Gleis, чтобы он соответствовал конвенциям именования Java, и сделайте его логическим классом, а не классом компонента GUI, и поэтому он не должен переопределять paintComponent.
• Дайте Глейс public void draw(Graphics2D g2) метод.
• Добавьте столько объектов Gleis к вашему чертежу JPanel, а затем нарисуйте их в методе JPanel paintComponent, выполнив итерацию по массиву ArrayList Gleis, который содержит чертеж JPanel, и вызвав draw(...) на каждом экземпляре Gleis.

Например:

import java.awt.Color;
import java.awt.Graphics;
import java.awt.Graphics2D;
import java.util.ArrayList;
import java.util.List;

import javax.swing.*;

public class GleisMain {
    private static void createAndShowGui() {
        GleisPainter gleisPainter = new GleisPainter();

        gleisPainter.addGleis(new Gleis(10, 10, 0, 0));
        gleisPainter.addGleis(new Gleis(20, 20, 1, 1));
        gleisPainter.addGleis(new Gleis(30, 30, 2, 2));

        // I try to avoid having classes extend JFrame and instead use JFrames when needed
        JFrame frame = new JFrame("Gleis Example");
        frame.setDefaultCloseOperation(JFrame.DISPOSE_ON_CLOSE);
        frame.getContentPane().add(gleisPainter);
        frame.setExtendedState(JFrame.MAXIMIZED_BOTH);
        frame.setVisible(true);
    }

    public static void main(String[] args) {
        SwingUtilities.invokeLater(new Runnable() {
            public void run() {
                createAndShowGui();
            }
        });
    }
}

// a single class that extend JPanel and that draws all the Gleis objects
class GleisPainter extends JPanel {
    // an ArrayList to hold all the Gleis objects
    private List<Gleis> gleisList = new ArrayList<>();

    // to allow outside classes to add Gleis objects
    public void addGleis(Gleis gleis) {
        gleisList.add(gleis);
    }

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

        // loop through our gleis list and draw each instance
        Graphics2D g2 = (Graphics2D) g;
        for (Gleis gleis : gleisList) {
            gleis.draw(g2);  // by calling its draw(...) method
        }
    }
}

// don't have this extend JPanel
class Gleis {
    // fields should be private
    private int xposition;
    private int yposition;
    private int id;
    private int status;

    public Gleis(int xpos, int ypos, int id, int status) {    
        this.xposition = xpos * 11;
        this.yposition = ypos * 11 + 4;
        this.id = id;
        this.status = status;

        System.out.println(this.id);
    }

    // give this a public draw method where each instance can draw itself
    public void draw(Graphics2D g2) {
        g2.setColor(Color.black);

        // you'll want to avoid "magic" numbers like you're using here:
        g2.fillRect(this.xposition, this.yposition, 11, 4);

        if (this.status == 0)
            g2.setColor(Color.gray);
        else if (this.status == 1)
            g2.setColor(Color.red);
        else if (this.status == 2)
            g2.setColor(Color.yellow);
        else
            g2.setColor(Color.gray);

        // ditto about use of "magic" numbers
        g2.fillRect(this.xposition + 2, this.yposition + 1, 7, 2);
    }    
}