Bilder auf Bilder zeichnen

[JUserToto]

Hallo zusammen,

ich studiere zur Zeit Informatik. Dort lernt man zwar relativ schnell all die technischen Feinheiten von Java, Swing wird allerdings etwas vernachlässigt. Ich möchte mich also selbst ein wenig schlauer auf dem Gebiet machen.
Deswegen habe ich mir eine Herausforderung gesetzt. Ich möchte eine kleine Adventure-Engine schreiben.
Nichts großartiges, man soll bloß Dinge nehmen und benutzen können.

Die Technik ist fertig und als Textadventure kann ich die Spiele ausgeben. Nun wollte ich das ganze mit Swing realisieren und werde langsam bekloppt.

Zur Programmstruktur:

Ich habe einen JFrame. Momentan ohne Layout. Er besteht aus:
- Einem JPanel. Hier wird die ganze Szenerie gemalt (Hintergrundbild + Objekte, die man benutzen oder nehmen kann).
- Einem JLabel. Es dient dazu, Gespräche darzustellen. Wenn ich also auf einen Schlüssel in der Szene mit Rechtsklick drücke steht dort für eine gewisse Zeit "Das ist ein Schlüssel" oder so etwas.

Das JPanel basiert nun auf einer Klasse Game, die alle Informationen über das Spiel sammelt. Zum Beispiel die aktuelle Szene. Eine Szene hat natürlich ein Hintergrundbild, dass auf den JPanel gezeichnet wird.

Ein größeres Problem ist es nun für mich, die einzelnen Szenerieobjekte darzustellen.
Ich wollte sie zunächst einfach über das Hintergrundbild malen. Das klappt aber nicht, weil diese Objekte ja anklickbar sein müssen. Also erzeuge ich für jedes Szenerieobjekt ein Objekt einer Klasse SceneryComponent mit der Größe des Bildes des Szenerieobjektes. Auf diese Komponente wird das Bild gezeichnet.
Das Bild wird nur bei der Erzeugung der Szene geladen und in eine Instanzvariable der SceneryComponent gespeichert, damit es beim neu zeichnen nicht jedes Mal neu geladen wird.

Nun funktioniert das ganze (und auch stabil). Wenn ich ein Objekt ansehe (also mit rechtsklick drauf klicke) startet nun der Thread für die Textausgabe. Dadurch wird das Bild neu gezeichnet, sodass es jedes mal, wenn ich auf ein Objekt klicke, flackert. Obwohl das Bild ja nicht neu geladen wird, sondern lediglich neu gezeichnet. Das neu-zeichnen ist ja in diesem Fall sogar überflüssig, weil am Bild ja nichts geändert wird.

Stimmt etwas grob an meiner Programmstruktur nicht oder muss ich nur genauer festlegen, wann Bilder neu gezeichnet werden sollen?

lg Toto

 

[Network]

Um solchen Problemen komplett aus dem Weg zu gehen - sodass man sich darüber überhaupt keine Gedanken mehr machen muss - ist die Doppelpufferung.

Bei kleineren Java Programmen funktioniert diese Methode sehr gut. Und es fallen viel Programmtext weg.

Deine Variante, nur ein paar Objekte neu zu laden, wird Exponentiell, bei größer werdenden Spielumgebung, unübersichtlicher und stellt damit auf lange Zeit keine Lösung oder Alternative dar.

 

[JUserToto]

Hallo,

erstmal danke für die Antwort. War schon kurz davor meinen Thread wieder zu löschen, weil ich dachte meine Beschreibung versteht eh niemand. Ist ja ohne Code immer nicht einfach.
Das mit dem DoubleBuffering hab ich nun öfters gelesen und es klingt für mich durchaus logisch.
Ich habe allerdings ein Problem mit der Implementierung...

Derzeit sieht die Struktur ja so aus:

JFrame (Hauptfenster) -> JPanel (Panel der das eigentliche Spiel zeigt) -> JComponents mit Bildern für die einzelnen Szenerieobjekte.

Ich habe also eben versucht eine Doppelpufferung ein zu programmieren...

ich schreibe also in paintComponent (vom JPanel):

    public void paintComponent(Graphics g) {
        if (buffImg == null || buffImg.getWidth() != getWidth()
                || buffImg.getHeight() != getHeight()) {
            buffImg = gfxConf.createCompatibleImage( getWidth(), getHeight() );
            paintImg(buffImg.createGraphics());
        }

        g.drawImage(buffImg,0, 0, this);
    }

und zeichne in der Methode paintImg(Graphics g) das Hintergrundbild (mit g.drawImage).
Nun müsste ich die Bilder der Szenerieobjekte malen. Das sind ja aber erstmal keine Bilder, sondern Komponenten, auf denen Bilder gemalt werden...
Ich kann es also nicht einfach so machen:

  public void paintImg(Graphics g) {
        // Hintergrundbild
        g.drawImage(sceneImage, 0, 0, this);
        
       // Einzelne Szenerieobjekte werden erzeugt.
        if (sceneryImages != null && !sceneryImages.isEmpty()) {

            Iterator szenerieIterator = sceneryImages.keySet().iterator();

            Position currentPos;
            SceneryComponent2D currentPanel;

            while (szenerieIterator.hasNext()) {
                currentPanel = (SceneryComponent2D) szenerieIterator.next();
                currentPanel.setOpaque(false);
                currentPos = (Position) sceneryImages.get(currentPanel);
                this.add(currentPanel);
                currentPanel.setBounds(currentPos.x, currentPos.y
                        ,currentPanel.gibImage().getWidth(null)
                        ,currentPanel.gibImage().getHeight(null));
                currentPanel.getGraphics().drawImage(currentPanel.gibImage(), 0, 0, this);

            }
        }
    }

Zur Erklärung: sceneryImages enthält alle Szenerieobjekt-Komponenten und deren Positionen in der Szene. Es ist eine HashMap. Der Szenerieobjekt-Komponente ist also eine Position zugeordnet...
Kann ich das so überhaupt mit Doppelpufferung lösen?

 

[JUserToto]

Hallo nochmal,

mittlerweile habe ich es hinbekommen.
Mein Fehler war, dass ich die Bilder der Szenerieobjekte auf die Szenerieobjekt-Komponenten gezeichnet habe.
Jetzt habe ich die Bilder direkt über das Hintergrundbild gemalt (mit Doppelpufferung) und darüber eine Unsichtbare Komponente gelegt, damit sie auch anklickbar sind.
Jetzt blinkt garnichts mehr und ich bin voll zufrieden.
Was ich etwas schade find ist, dass manchmal der MouseListener beim ersten Klick nicht reagiert. Kann man da etwas gegen tun

 

[Marco13]

In Swing braucht man Double Buffering eigentlich nicht selbst zu machen, das ist da schon eingebaut...

 

[TX]

Wenn es so geht, dann ist gut, obwohl mir diese Szenerieobjekte jetzt nichts sagen.
Hättest aber vielleicht auch einfach einen Mouselistener verwenden können, der alle Elemente durchgeht und checkt, ob und welches angeklickt wurde.

z.B. so, (ist nur ein grober Entwurf)

public class Schluessel{

	private Rectangle clickableField;
	
	public setClickableField(int x, int y, int w, int h){
		clickableField = new Rectangle(x, y, w, h);
	}
	
	public boolean isClicked(int x, int y){
		return clickableField.contains(x, y);
	}

}

public class MouseL extends MouseAdapter{
	
	private Schluessel schluessel;
	
	public void mouseReleased(MouseEvent e) {
		if(schluessel.isClicked(e.getX(), e.getY());
		
	}
	
}

 

[JUserToto]

Mit Szenerieobjekte meine ich einfach alle Objekte in einer Adventure Szene, die man ansehen oder nehmen oder benutzen kann. Weiß nicht, wie ich sie sonst nennen soll.

Im Moment funktioniert alles ohne Flackern und ohne Probleme. Deswegen: Never change a running System
Dein Vorschlag hilft mir vielleicht noch bei einem anderen Problem.
Da ich nun alles implementiert hab, was ich ich geplant hatte, möchte ich mich nun der Spielfigur widmen...
Überlege noch, ob ich lieber ein Pfadsystem benutzen soll, oder Dijkstra o.Ä. implementieren soll...
Sollte ich mich wirklich für die Wegfindung entscheiden, muss ich ja einen bestimmten Bereich des Bildes freigeben... dabei könnte mir dein Vorschlag helfen.

Der Thread hier ist aber erstmal erledigt. Danke nochmal

 

[Network]

Hallo nochmal,

mittlerweile habe ich es hinbekommen.
Mein Fehler war, dass ich die Bilder der Szenerieobjekte auf die Szenerieobjekt-Komponenten gezeichnet habe.
Jetzt habe ich die Bilder direkt über das Hintergrundbild gemalt (mit Doppelpufferung) und darüber eine Unsichtbare Komponente gelegt, damit sie auch anklickbar sind.

Schön

In Swing braucht man Double Buffering eigentlich nicht selbst zu machen, das ist da schon eingebaut...

Seit wann denn das?
Also ich meine, die Frageperson hat es implementiert und es funktioniert nun besser denn vorher. Also ist es scheins doch nicht bereits eingebaut. Versteh mich nicht falsch. Mich interessiert gerade nur die Aussage daran. Man lernt nie aus

 

[Marco13]

Painting in AWT and Swing