Графические петли глюков, как исправить?

Question

Я пытался сделать эту работу так, чтобы было 20 коробок с 3 различными размерами и 3 различными цветами, которые были выбраны случайным образом, но я не могу заставить их выходить в разное время, и они просто сливаются друг с другом ицвета сближаются и тому подобное, кто-нибудь знает, как это исправить? Вот что я получил до сих пор:

    import java.awt.*;
    import javax.swing.*;
public class testwork extends JPanel { //JPanel is a class

int l = 0;
private int x = 10; 
private int y = 500; 
private void move()
{
x++;
}
boolean red = false;
boolean blue = false;
boolean green = false;


@Override
public void paint(Graphics g) { //JPanel is a class defined in
super.paint(g);
Graphics2D g2d = (Graphics2D) g;

g2d.setRenderingHint(RenderingHints.KEY_ANTIALIASING,
RenderingHints.VALUE_ANTIALIAS_ON); 

Color rectColor = new Color(0, 66, 89);
g2d.setColor(rectColor);
//belt
g2d.setColor(Color.lightGray);
g2d.fillRect(0,450,1500,200);
g2d.fillRect(700,0,200,1000);
g2d.setColor(Color.orange);
for (int i = -10000; i<10000; i=i+50) {
    int m= i++;
    g2d.fillRect(m, 450, 25, 200);
}
g2d.setColor(Color.DARK_GRAY);
g2d.fillRect(700, 450, 200, 200);
//boxes
while (l<=20) {
if (Math.random() < 0.5) 
{g2d.setColor(Color.RED);;}
else if (Math.random() < 0.5) {g2d.setColor(Color.GREEN);}
else {g2d.setColor(Color.BLUE);}

if (Math.random() < 0.5) 
{g2d.fillRect(x,y,50,50);}
else if (Math.random() < 0.5) {g2d.fillRect(x,y,50,100);}
else {g2d.fillRect(x,y,100,50);}
l++;
}

}

public static void main(String[] args) throws InterruptedException {
JFrame frame = new JFrame("Frame"); //Add our JPanel to the frame
frame.add(new attempt());//instantiate a new object

frame.setSize(1500, 1000);

frame.setVisible(true);
frame.setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
testwork p = new testwork();
frame.add(p);
while (true)
{
p.move(); //Updates the coordinates
p.repaint();
Thread.sleep(10); //Pauses for a moment
}}

}

Answer 1

К сожалению, вы делаете несколько вещей неправильно.

1. Переопределите paintComponent, а не paint.
2. Не используйте Thread.sleep. Используйте Swing timer и ActionListener
3. Вы слишком много делаете в методе рисования. Вся обработка событий, включая вызовы repaint(), выполняется в потоке диспетчеризации событий (EDT). Таким образом, все ваши обновления выполняются внутри paintComponent, поэтому при выходе будут отображаться только последние нарисованные объекты. Обновите ваши координаты, структуру данных и все остальное, что нужно рисовать вне вашего метода рисования.

Поместите ваш boiler plate code в конструктор класса Testwork. Вот пример.

  public Testwork() {

      setPreferredSize(new Dimension(1000, 700));
      Timer timer = new Timer(0, this);
      frame.setVisible(true);
      frame.setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);

      frame.add(this);
      // sizes the frame and jpanel and organizes the components.
      frame.pack();
      // centers the window in the screen
      frame.setLocationRelativeTo(null);
      // sets the delay in milliseconds
      timer.setDelay(100);
      // starts the timer
      timer.start();
   }

Здесь будет ваш код actionListener.

     public void actionPerformed(ActionEvent ae) {
       // update any variables that need to be used int he
       // paint routine here. That means if you want to move something
       // update the coordinates here and then use them in the paint method.

     }

при запуске приложения сделайте это следующим образом

    public static void main(String[] args) {
      SwingUtilities.invokeLater(() -> new Testwork());
    }

Есть гораздо больше в живописи. Я рекомендую проверить Custom Painting в руководствах по Java. Здесь - еще один пример на этом (SO) сайте.