Mein erstes Programm flackert

[Dauzo]

Hallo liebe Forumleser,

ich habe ein Problem mit meinem ersten Programm, und zwar flackert es wenn ich es ausführe.

Quellcode:

package pingpong;

import java.awt.*;
import java.util.logging.Level;
import java.util.logging.Logger;
import javax.swing.*;

/**
 *
 * @author pmeier
 */
public class PingPong extends JFrame implements Runnable {
    
    private Ball ball = new Ball(100, 150, this);
    
    public PingPong(String title) {
        super(title);
    }
    @Override
    public void paint(Graphics gr) {
        super.paint(gr);
        // Malen
        
        //Hintergrund
        gr.drawImage(Toolkit.getDefaultToolkit().getImage("images/background.png"), 0, 0, this);
        
        //Ball
        ball.paint(gr);
    }
    
    @Override
    public void run() {
        while(true) {
            //Code
            
            //Ball
            ball.moveBall();
            
            //Code
            repaint();
            try {
                Thread.sleep(20);
            } catch (InterruptedException ex) {
                Logger.getLogger(PingPong.class.getName()).log(Level.SEVERE, null, ex);
            }
        }
    }

    public static void main(String[] args) {
        PingPong ping = new PingPong("Mein super cooles Spiel!");
        new Thread(ping).start();
        ping.setSize(800, 500);
        ping.setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
        ping.setVisible(true);
    }
}

package pingpong;

import java.awt.Component;
import java.awt.*;

/**
 *
 * @author pmeier
 */
public class Ball {
    
    private int x, y;
    private int velocityX, velocityY;
    private Component comp;
    
    public Ball(int x, int y, Component comp) {
        this.x = x;
        this.y = y;
        this.comp = comp;
        this.velocityX = 5;
        this.velocityY = 5;
    }
    
    public int getX() {
        return x;
    }
    
    public int getY() {
        return y;
    }
    
    public void moveBall() {
        if(x + 50 > 800) {
            velocityX = -5;
        }else if(x < 0) {
            velocityX = 5;
        }
        
        if(y + 50 > 500) {
            velocityY = -5;
        }else if(y < 0) {
            velocityY = 5;
        }
        
        x += velocityX;
        y += velocityY;
    }
    
    public void paint(Graphics gr) {
        gr.drawImage(Toolkit.getDefaultToolkit().getImage("images/ball.png"), x, y, comp);
    }
}

Und noch eine zusätzliche Frage: Wie kann ich das Java Programm von Cmd ausführen? (Befehle)

 

[Crian]

Lade nicht jedes mal das Bild neu in Ball#paint().

 

[Dauzo]

Sondern? Ich bin kein Java-Experte ich wüsste jetzt nicht wie ich den Quellcode umändern soll. Ich habe das ganze von einem Tutorial und bei ihm flackert das Bild nicht. Es wurde auch nicht erwähnt das es flackern könnte.

 

[Gast2]

Also:
1) Bei Swingkomponenten überschreibt man in der Regel paintComponent(), und nicht paint()
2) In den Zeichenvorgang vom JFrame sollte man sich besser nicht einmischen. PingPong sollte besser von JPanel ableiten dass du dann als conentPane eines JFrames setzt.
3) Wie schon erwähnt solltest du nicht bei jedem neuzeichnen das Bild laden, das dauert. Lade dir das nur einmal, bspw. im Konstruktor und speicher dir das als Instanzvariable zwischen.
4) Verwende statt Toolkit#getImage besser ImageIO.

 

[HimBromBeere]

Und noch eine zusätzliche Frage: Wie kann ich das Java Programm von Cmd ausführen? (Befehle)

1. navigier dich in das Zielverzeichnis
2. Dort:

java <Name deines Programmes ohne Dateiendung>

eingeben

 

[KingOfExceptions]

Sondern? Ich bin kein Java-Experte ich wüsste jetzt nicht wie ich den Quellcode umändern soll. Ich habe das ganze von einem Tutorial und bei ihm flackert das Bild nicht. Es wurde auch nicht erwähnt das es flackern könnte.

Ansonsten, auf das noch nicht geantwortet wurde:

1. Ein Bild läd man am besten so:

BufferedImage img;
String dateiName;
try
{
     img = ImageIO.read(getClass().getResourceAsStream(dateiName));
}
catch (IOException e)
{
     JOptionPane.showMessageDialog(null, dateiName + " wurde nicht gefunden!");
}

Also muss das vorher aufgerufen werden und dann muss in deiner paintComponent(Graphics g) das hier stehen:

public void paintComponent(Graphics gr) 
{
        super.paintComponent(gr);
        // Malen
        
        //Hintergrund
        gr.drawImage(img, 0, 0, this);
        
        //Ball
        ball.paintComponent(gr);
    }

genauso sollte in deiner paint methode in Ball nicht das hier stehen:

public void paint(Graphics gr) {
        gr.drawImage(Toolkit.getDefaultToolkit().getImage("images/ball.png"), x, y, comp);
    }

sonder wohl eher das hier:

public void paintComponent(Graphics gr) {
        gr.drawImage(img, x, y, comp);
    }

wobei

img

in dieser klasse natürlich eine separate variable sein muss und so wie oben initialisiert werden muss.

Und zum Aufruf einer Java Datei:

1. mit

cd\

in den richtigen ordner wechseln.
2. mit

javac (DATEINAME.java)

kompilieren
3. mit

java (DATEINAME)

ausführen

Ansonsten siehe Bild im Anhang.