Как рисовать объекты из ArrayList в Java - PullRequest
Новая
       14

Как рисовать объекты из ArrayList в Java

0 голосов
/ 15 ноября 2018

Хорошо, поэтому я пытаюсь создать программу, которая рисует группу прямоугольников, которые перемещаются по экрану случайным образом. У меня есть класс Dot, где каждая точка содержит свои значения x и y, а в своем классе рисования я случайным образом изменяю значения x и y, а затем перекрашиваю (). То, что у меня есть сейчас, не загружает ничего, кроме пустого JFrame. Я подозреваю, что я рисую каждую точку неправильно. Вот мой код: 

import java.awt.EventQueue;
import java.awt.Graphics;
import java.awt.event.ActionEvent;
import java.awt.event.ActionListener;
import java.util.ArrayList;
import java.util.Random;

import javax.swing.AbstractAction;
import javax.swing.JFrame;
import javax.swing.JPanel;
import javax.swing.Timer;

public class Movement extends JFrame {
    public ArrayList<Dot> dots = new ArrayList<Dot>();
    Random rn = new Random();
    DrawPanel drawPanel = new DrawPanel();
    public Movement() {
        for(int i = 0; i < 100; i ++) {
            Dot dot = new Dot(5, 5);
            dots.add(dot);
        }
        ActionListener listener = new AbstractAction() {
            public void actionPerformed(ActionEvent e) {
                    for(int i = 0; i < dots.size();i++) {
                        dots.get(i).setX(dots.get(i).getX() + rn.nextInt(20)-10);
                        dots.get(i).setY(dots.get(i).getY() + rn.nextInt(20)-10);
                    }
                    drawPanel.repaint();

            }
        };
        Timer timer = new Timer(100, listener);
        timer.start();
        add(drawPanel);

        pack();
        setDefaultCloseOperation(EXIT_ON_CLOSE);
        setLocationRelativeTo(null);
        setVisible(true);
        setBounds(100, 100, 500, 500);
    }

    private class DrawPanel extends JPanel {

        protected void paintComponent(Graphics g) {
             for(int i = 0; i < dots.size(); i ++) {
                g.fillRect(dots.get(i).getX(), dots.get(i).getY(), 5, 5);;
                super.paintComponent(g);
             }


        }

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

Класс точек: 

public class Dot {
    private int x, y;
    public Dot(int x, int y) {
        this.x = x;
        this.y = y;

    }
    public int getX() {
        return x;
    }
    public void setX(int x) {
        this.x = x;
    }
    public int getY() {
        return y;
    }
    public void setY(int y) {
        this.y = y;
    }

}

Любая помощь приветствуется.

Ответы [ 2 ]

0 голосов
/ 15 ноября 2018

Вы, кажется, звоните super.paintComponent после того, как вы рисуете точки. Мало того, вы, кажется, вызываете его внутри цикла, вызывая его каждый раз, когда вы рисуете точку , после того, как вы ее рисуете.

Итак, вы продолжаете рисовать точку, позволяя super.paintComponent нарисовать прозрачную панель, затем нарисовать другую точку, затем отменить ее снова, нарисовать другую, отменить ... и так далее.

Позвоните super.paintComponent один раз и сделайте это, прежде чем вы сделаете свою обычную картину.

0 голосов
/ 15 ноября 2018

Если вы позвоните super.paintComponent(g); после того, как вы покрасите свои собственные компоненты, вы уничтожили свою собственную картину. Итак, 

    protected void paintComponent(Graphics g) {
        super.paintComponent(g);
        Graphics2D g2d = (Graphics2D)g;
         for(int i = 0; i < dots.size(); i ++) {
             g2d.fillRect(dots.get(i).x, dots.get(i).y, 5, 5);
         }
    }

Также 

// don't repeat type in constructor
// use built in point instead of custom class
public ArrayList<Point> dots = new ArrayList<>();

и 

    ActionListener listener = new AbstractAction() {
        public void actionPerformed(ActionEvent e) {
                for(int i = 0; i < dots.size();i++) {
                    dots.get(i).x = dots.get(i).x + rn.nextInt(20)-10;
                    dots.get(i).y = dots.get(i).y + rn.nextInt(20)-10;
                }
                drawPanel.repaint();
        }
    };

и это, вероятно, не имеет никакого значения, но 

    SwingUtilities.invokeLater(new Runnable() {
        @Override
        public void run() {
            Movement mf = new Movement();
        }
    });
...