Anfängerfrage JPanel repaint()

[itsafiasco]

Schön guten Tag,

vorab muss ich sagen, dass ich im Bezug auf Zeichnen mit Java noch recht unerfahren bin. Doch für ein Schulprojekt bin ich grade dabei, mir das alles anzueignen. Dabei bin ich jedoch auf folgendes Problem gestoßen:

Erstmal der Code:

import javax.swing.*;
import java.awt.*;

public class PacMan extends JPanel{
  
  int cnr = 0;
  
  public PacMan() {
    
    super();
    this.animate();
    
  }  
  
  public void paintComponent(Graphics g) {
    
    super.paintComponent(g);
    g.setColor(Color.yellow);
    
    this.cnr++;
    
    g.fillArc(25+this.cnr,25,25,25,30,300);
    
  }  
  
  public void animate() {
    
    while(true) {
      super.repaint();
    }  
      
  }  
  
}

Wenn ich jetzt alles ausführen will, passiert gar nichts mehr. Kein Fenster öffnet sich, kein (Kompilier-)Fehler, nix. Lass ich das repainten raus, geht aber alles.
Vorher hatte ich es aber auch schonmal geschafft, was in ein Jframe zu zeichnen und zu repainten, also alles in einer Datei. Aber das fand ich nicht so optimal und gut.

Kann sich ein Experte das kurz angucken und mir n Tipp geben? :S

LG

 

[Gast2]

Mal ins blaue geraten:
Du rufst animate() auf dem EDT auf und blockierst ihn dann mit der Endlosschleife.

Außerdem frage ich mich welchen Zweck diese Methode haben soll? Die ruft nur unendlich oft repaint() auf. Was genau willst du da erreichen?

 

[itsafiasco]

Na ich möchte, dass letzten Endes sich der PacMan vorerst nur immer weiter nach links bewegt. Dazu erhöhe ich die X-Variable bei jedem Aufruf der Paint-Methode und das neuzeichnen passiert doch durch repaint oder? Zumindest hab ich das mal irgendwo gesehen und geklappt hatte es ja schonmal bei mir so

Und ich hatte den Code hier auch vereinfacht. Bei einer anderen Lösung, war schon ein KeyListener dran verknüpft, mit ich den Pacman starten und stoppen konnte.

 

[Gast2]

Achso, das habe ich jetzt erst gesehen. Das solltest du so NICHT machen. Du hast (wenig) Einfluss darauf, wann oder wie oft die paintComponent() Methode aufgerufen wird!

Sowas löst man in der Regel mit einer Gameloop. Die wird so gesteuert, dass bspw. 30 Loops pro Sekunde laufen (damit läuft dein Spiel auf allen Rechnern annähernd gleich schnell. Pro Durchlauf wird irgendwas berechnet (bspw. die Position deines PacMans verändert) und danach ein repaint() aufgerufen.
Ein Beispiel dafür findest du zum Beispiel im Tutorial von Quaxli: http://www.java-forum.org/spiele-multimedia-programmierung/54795-quaxli-2d-spiele-tutorial.html

Aktuell funktionierts bei dir nicht weil deine Schleife vermutlich im EDT läuft.

 

[itsafiasco]

Reicht es nicht, wenn ich noch ne Bedingung in die while-Schleife baue, worüber ich dann regel, dass Pacman mit nem Tastendruck gestartet wird und bei Kollision aufhört? Auf mehr muss ich doch gar nicht Einfluss haben?

Trotzdem danke für den Link, ich schau mir das zu Hause mal in Ruhe an. Aber passt das jetzt auch zu meinem Code oder basiert das auf was anderem? Ich würde mcih nämlich nicht so gerne noch in was anderes einarbeiten wollen.

Wenn ich das mit dem EDT richtig verstehe, müsste ich also n neuen Thread erstellen für das Neuzeichnen, damits funktioniert? Wenn ja, wäre das auch ne Möglichkeit?

 

[Gast2]

Du wirst deinen Code schon anpassen müssen. So wie du es jetzt geschrieben hast ists einfach murks.
Das Tutorial von Quaxli ist schonmal ne gute Anlaufstelle, da siehst du wie man sich so eine Gameloop zusammenbaut und auch auf Benutzereingaben reagiert.

 

[itsafiasco]

Ok gut, dann werd ich das mal durchgehen. Ein gewisses Niveau beim programmierten Code sollte man schon erreichen, da hast du recht Vielen Dank