2D-Grafik Paint - Wie binde ich repaint ein?

[Kayuke]

Guten Tag, ich habe ein Menü und möchte dort jetzt ein Spiel einbinden wenn man auf Singleplayer klickt. Das ist die Klasse die derzeit erscheint wenn ich auf "Singleplayer" klicke:

package GruppeD;

import java.awt.*;
import java.awt.event.*;
import java.io.IOException;

import javax.imageio.ImageIO;
import javax.swing.*;


//Vorbereitetes Panel für den Aufbau der Spieloberfläche für Singleplayer-Modus
class SinglePlayerPanel extends JPanel implements ActionListener {
   private MainPanel mainPanel;
   private int bgX = 0;
   private int bgY = 0;
   Timer time;
   
   // Hintergrund
   Image backgroundImage;
   
   public SinglePlayerPanel(MainPanel mainPanel) {
      this.mainPanel = mainPanel;
          
      // Bilder laden
      loadImages();
      
     // Timer (Wenn die beiden Zeilen auskommentiert sind, funktionierts.)
     // time = new Timer(10,this);
     // time.start();
   }
   
   //Sämtliche Bilder für das Spiel laden
   private void loadImages() {
      try {
        backgroundImage = ImageIO.read(getClass().getResource("/GruppeD/images/Game/background.png"));
      }
      catch(IllegalArgumentException iae) {
         JOptionPane.showMessageDialog(this, "Eine Grafikdatei wurde nicht gefunden!", "IllegalArgumentException", JOptionPane.ERROR_MESSAGE);
         System.err.println(iae.toString()); //Ausgabe für den Entwickler auf der Konsole
      }
      catch(IOException ioe) {
         JOptionPane.showMessageDialog(this, "Fehler beim Einlesen einer Grafikdatei!", "IOException", JOptionPane.ERROR_MESSAGE);
         System.err.println(ioe.toString()); //Ausgabe für den Entwickler auf der Konsole
      }
   }  
   
   // Zeichnen
   @Override
   protected void paintComponent(Graphics g) {
       super.paintComponent(g);
       
       // Hintergrund zeichnen
       g.drawImage(backgroundImage, bgX, 0, this);
       
       // Hintergrund nachzeichnen (Sidescroll-Effekt)
       if (bgX < 0)
           g.drawImage(backgroundImage, getWidth()+bgX, 0, this);
       
       // Hintergrund verschieben
       if (bgX < -getWidth()) {
           bgX = 0;
       }
       else {
           bgX--;
       }  
       
       // Muss weg, ich weiß, aber wohin?
       repaint();
   }
   
   public void actionPerformed(ActionEvent e) {
      mainPanel.showMenuPanel();
   }
}

Nun die Frage, derzeit zeichnet er nur neu wenn ich in Paint am Ende Repaint schreibe, was halt Sinn ergibt aber eine Endlosschleife erzeugt was nicht gesund ist Ich habe versucht repaint in eine neue Methode zu schreiben, aber diese muss ja iwie immer wieder aufgerufen werden, wie krieg ich das genau hin?

 

[JohannisderKaeufer]

repaint() kann man aufrufen, wenn sich etwas ändert.

Ein Beispiel mit einem String.

class MyClass extends JPanel{
  private String value;

  public void paintComponent(Graphics g){
    super(g);
    ((Graphics2D)g).drawString(value);
  }

  public void setValue(String v){
    this.value = v;
    //Hier würde ich die logik zum Neuzeichnen unterbringen.
    validate();
    repaint();
  }

}

Das ganze kann auch über das Observer Pattern geschehen indem das JPanel Observer implementiert und Value ein Observable ist.

 

[Kayuke]

Hmm es ist so, ich habe vorher ein Helikopter Spiel direkt programmiert mit Paint und brauchte kein repaint aber da war ein Timer. Dort konnte ich den Delay einstellen und dann wurde z.B. alle 10 Millisekunden neugezeichnet.

Jetzt möchte ich dasselbe in dem Code haben wo ich bisher nur das Menü habe und möchte dort jetzt das Spiel komplett neu eincoden. Er soll also quasi alle 10 Millisekunden repaint machen, das muss doch so iwie machbar sein?

Weil später fliegen da auch Gegner etc. und ich will nicht bei jeder Aktion am Ende repaint schreiben.

 

[Marco13]

Da du ja vermutlich irgendeinen Game Loop hast, würde es sich anbieten, da das repaint reinzuschreiben

while (gameRunning)
{
    doLogic(); // Bewegt die Gegner etc...
    panel.repaint();
    Thread.sleep(someMS);
}

 

[Kayuke]

Klappt wieder mal garnix :/

Meine Prognose: Er führt paint erst nachher aus und kommt sofort in die While Schleife mit repaint. Außerdem kommts mir so vor als ob der gesamte Code von der "SinglePlayerPanel.java" schon bei Programmstart ausgeführt wird und nicht erst wenn ich auf SinglePlayer im Menü klicke. Dies glaube ich deshalb weil wenn ich bei der Methode loadImages in dem SinglePlayerPanel eine falsche Url angebe so kommt sofort eine Fehlermeldung und nicht erst wenn ich auf Singleplayer klicke.

Würde mich sehr freuen um Hilfe, weiß nicht was ich noch versuchen soll.

Also hier der Code:

package GruppeD;

import java.awt.*;
import java.awt.event.*;
import java.io.IOException;

import javax.imageio.ImageIO;
import javax.swing.*;


//Vorbereitetes Panel für den Aufbau der Spieloberfläche für Singleplayer-Modus
class SinglePlayerPanel extends JPanel implements ActionListener {
   private MainPanel mainPanel;
   private int bgX = 0;
   private int bgY = 0;
   Timer time;
   boolean running;
   
   // Hintergrund
   Image backgroundImage;
   
   public SinglePlayerPanel(MainPanel mainPanel) {
      this.mainPanel = mainPanel;
          
      // Bilder laden
      loadImages();
      
      // Game running
      running = true;
      game();

   }
   
   //Sämtliche Bilder für das Spiel laden
   private void loadImages() {
      try {
        backgroundImage = ImageIO.read(getClass().getResource("/GruppeD/images/Game/background.png"));
      }
      catch(IllegalArgumentException iae) {
         JOptionPane.showMessageDialog(this, "Eine Grafikdatei wurde nicht gefunden!", "IllegalArgumentException", JOptionPane.ERROR_MESSAGE);
         System.err.println(iae.toString()); //Ausgabe für den Entwickler auf der Konsole
      }
      catch(IOException ioe) {
         JOptionPane.showMessageDialog(this, "Fehler beim Einlesen einer Grafikdatei!", "IOException", JOptionPane.ERROR_MESSAGE);
         System.err.println(ioe.toString()); //Ausgabe für den Entwickler auf der Konsole
      }
   }  
   
   
   private void game(){
	   while (running)
	   {
		   // Neuzeichnen
		   repaint();
		   
		   // Sleep
		   try {
			Thread.sleep(50);
		} catch (InterruptedException e) {
			// TODO Auto-generated catch block
			e.printStackTrace();
		}
	   }
   }
   // Zeichnen
   @Override
   protected void paintComponent(Graphics g) {
       super.paintComponent(g);
       
       // Hintergrund zeichnen
       g.drawImage(backgroundImage, bgX, 0, this);
       
       // Hintergrund nachzeichnen (Sidescroll-Effekt)
       if (bgX < 0)
           g.drawImage(backgroundImage, getWidth()+bgX, 0, this);
       
       // Hintergrund verschieben
       if (bgX < -getWidth()) {
           bgX = 0;
       }
       else {
           bgX--;
       }  
       
   }
   
   public void actionPerformed(ActionEvent e) {
      mainPanel.showMenuPanel();
   }
}

 

[xehpuk]

[JAVA=22] public SinglePlayerPanel(MainPanel mainPanel) {
this.mainPanel = mainPanel;

// Bilder laden
loadImages();

// Game running
running = true;
game(); // YIKES!!!

}[/code]
Dadurch landet man beim Aufruf von

new SinglePlayerPanel(...)

in der Schleife. Das hat da nichts zu suchen. Die Methode muss nach der Erstellung des Panels von einem separaten Thread (bloß nicht dem EDT) aufgerufen werden.
Wobei die Methode gar nicht zum Panel gehören sollte, genauso wenig wie die Information, dass das Spiel gerade läuft.

 

[vanny]

Ich halte es generell für eine schlechte Idee, nur den timer als Steurung für die Framerate zu nutzen.
Besser ist es, wie schon mal angeschnitten, jede "sichtbare" aktualisierung neu zu zeichnen.
Was, wenn deine Daten nach 10 millis nur halb fertig berechnet wurden oder du wesentlich mehr FPS rausholen könntest?

 

[Kayuke]

Hmm wäre sehr dankbar für noch weitere Hilfe:

Da es ein Sidescroller ist und der Hintergrund sich quasi durchgehend neuzeichnet, wollte ich nicht nach jeder Aktion ein repaint, da es durch den Hintergrund sowieso ganze zeit Ausgeführt wird.

Kann nochmal jemand versuchen mir es zu erklären ?

Es steht im obigen Post: die Methode Repaint soll nach dem erstellen des Panels in einem separaten Thread aufgerufen werden, wie genau stelle ich dies an? Ich kriege es nicht hin 2 Threads zu erstellen und es aufrufen zu lassen. Genau wie ich nicht verstehe warum die Methode "repaint" nicht zum Panel gehören soll, ich überschreibe ja schließlich auch da die paint?

 

[javaman13]

Lass einfach in deinem Gameloop zuerst die Aktualistion durchführen und dann das ganze durch paint() zeichnen. Funktioniert toll!