Попытка сделать изображение перемещается случайным образом, но оно не движется

Question

Я пытаюсь сделать изображение перемещаться случайным образом, но он почему-то не работает. Изображение появляется на панели, но оно не перемещается. Для этого я использую классы буксировки: основной класс и подкласс. Это код для моего подкласса.

public Image() 
{ 

    randomPosition(); 

    try { 
    image = ImageIO.read(new File("image.png")); 

    } catch (IOException e) {} 


} 


public void paint(Graphics g) 
{ 
    g.drawImage(image, x, y, this); 
    Toolkit.getDefaultToolkit().sync(); 
    g.dispose();  
} 



public void move() 
{ 


    x += dx; 
    y += dy; 


    if (x >= 550) { 
     x = 550; 
     randomDirection(); 
     } 
     if (x <= 1) { 
     x = 1; 
     randomDirection(); 
     } 
     if (y >= 350) { 
     y = 350; 
     randomDirection(); 
     } 
     if (y <= 1) { 
     y = 1; 
     randomDirection(); 
     } 
} 

public void randomDirection() { 
    double speed = 2.0; 
    double direction = Math.random()*2*Math.PI; 
    dx = (int) (speed * Math.cos(direction)); 
    dy = (int) (speed * Math.sin(direction)); 
    } 


public void randomPosition() 
{ 
    x = LEFT_WALL + (int) (Math.random() * (RIGHT_WALL - LEFT_WALL)); 
    y = UP_WALL + (int) (Math.random() * (DOWN_WALL - UP_WALL)); 
} 

public void run() 
{ 

    long beforeTime, timeDiff, sleep; 
    beforeTime = System.currentTimeMillis(); 
    while (true) 
    { 
     move(); 
     repaint(); 

     timeDiff = System.currentTimeMillis() - beforeTime; 
     sleep = DELAY - timeDiff; 

     if (sleep > 2) 
     { 
      sleep = 1; 
     } 
     try 
     { 
      Thread.sleep(sleep); 
     } 
     catch (InterruptedException e) 
     { 
      System.out.println("interrupted"); 
     } 

     beforeTime = System.currentTimeMillis(); 
    } 
} 

и это мой главный класс, где я начинаю тему:

public void addImage(){ 
      Image I = new Image(); 
      x = panel.getGraphics(); 
      I.paint(x); 
      Thread thr=new Thread(I); 
      thr.start(); 
     } 

Answer 1

• paint() должен быть

Как это:

• Не звоните непосредственно paint()/paintComponent(). Вместо этого вызовите repaint(), это добавит его в очередь событий. Графический объект передается качели.

• Я не уверен, что g.dispose() необходим и может вызвать проблемы.

• Проверьте панель ваша картина в это размер правильно - быстрый способ отладки это panel.setBorder(BorderFactory.createLineBorder(Color.RED));
• Рассмотрите возможность использования javax.swing.Timer вместо ниток. Это не является причиной проблемы, но будет более чистым.