KeyListener

[Guest]

Hallo,

der folgende Code sammelt leider alle Tastatureingaben in einer Warteschlange, und arbeitet alle Eingaben ab, wobei für JEDE Eingabe die KOMPLETTE zeitaufwendige Schleife ausgeführt wird!!!

Ziel: Wenn eine Tastatureingabe gemacht wird, dann soll die Schleife für die vorherige Tastatureingabe NICHT mehr zuende abgearbeitet werden. Die Schleife soll dann nur für die aktuelle Tastatureingabe neu ausgeführt werden.

public class GUI extends JFrame {
	static GUI fenster;
	...
	
	public static void main(String args[]) {	
		fenster = new GUI(); 		
		...
	}
	
	class KeyListener extends KeyAdapter {
		public void keyPressed(KeyEvent e) {		 
			if( e.getKeyCode() == KeyEvent.VK_LEFT ) 
			{
				left=true;
				fenster.repaint(); //Problem: die zeitaufwendige Schleife unten um Code wird komplett abgearbeitet, 
				//erst danach werden die nächsten Keyinputs abgearbeitet!
			}
			...
		}
	}	
	
	class BildLeinwand extends JPanel {
		public void paintComponent(Graphics g) {
			...
			for (int x=0; x<9999999; x++) { //Hier eine zeitaufwendige Schleife! }
		}
	}
	
	GUI() {
		...
		addKeyListener(new KeyListener()); 
	}
}

 

[The_S]

Dein Code hilft da leider nicht viel weiter. Da kann man nur empfehlen, dass alles, was nicht zwingend in paintComponent erstellt werden muss, ausgelagert werden sollte (z. B. in Klassenvariablen falls möglich und dann in paint auf diese Variablen zugreifen und nicht jedesmal neu generieren).

 

[Guest]

Da kann man nur empfehlen, dass alles, was nicht zwingend in paintComponent erstellt werden muss, ausgelagert werden sollte

Meine zeitaufwändige Schleife in der paintComponent ist zu 99% damit beschäftigt unzählige Bilder von HDD zu lesen und zu zeichnen. D.h. man kann die paintComponent nicht wirklich schlänker/zeiteffektiver machen.

Ich Frage mich also nach wie vor was zu tun ist um mein Problem -siehe mein ersten Post- zu lösen?

 

[SlaterB]

> D.h. man kann die paintComponent nicht wirklich schlänker/zeiteffektiver machen.

doch, indem man diese 99% herausnimmt

solange paintComponent arbeitet, kannst du auf neue Tastatureingaben nicht reagieren,
diese Tatsache ist unumstößlich in Java,

in deinem Listener könntest du weniger oft repaint() aufrufen,
löst aber nur einen Teil deiner Probleme,
denn wenn paintComponent erst einmal losläuft..,

naja, einfach zu reparieren, wenn du nicht willst, dann eben nicht

 

[Guest]

doch, indem man diese 99% herausnimmt
solange paintComponent arbeitet, kannst du auf neue Tastatureingaben nicht reagieren, diese Tatsache ist unumstößlich in Java

Aha, danke.

in deinem Listener könntest du weniger oft repaint() aufrufen

Hmm, das repaint stößt doch aber die Ausführung des Codes in der paintComponent an. Und es ist mein Ziel das bei jeder Keyeingabe der komplette Code der paintComponent gezeichnet wird.

naja, einfach zu reparieren, wenn du nicht willst, dann eben nicht

Hab den Code von PaintComponent in eine zeichnen-Methode ausgelagert. Und was mach in nun?
Denn ich hab weiterhin das Problem welches ich in meinem ersten Post beschrieben habe.

public class GUI extends JFrame { 
	static GUI fenster; 
	... 
	
	public static void main(String args[]) {    
		fenster = new GUI();        
		... 
	} 
	
	class KeyListener extends KeyAdapter { 
		public void keyPressed(KeyEvent e) {       
			if( e.getKeyCode() == KeyEvent.VK_LEFT ) 
			{ 
				left=true; 
				zeichnen();//Problem: die zeitaufwendige Schleife unten um Code wird komplett abgearbeitet, 
				//erst danach werden die nächsten Keyinputs abgearbeitet! 
			} 
			... 
		} 
	}
	
	void zeichnen () {
		Graphics g = getGraphics(); 
		for (int x=0; x<9999; x++) { //Hier eine zeitaufwendige Schleife! }         
		}
		
		class BildLeinwand extends JPanel { 
			public void paintComponent(Graphics g) { 
				//hier kein Code mehr. Code nun in der Methode "zeichnen"			}
			public Dimension getPreferredSize() { 
				...
				return new Dimension(...);
			}
		} 
		
		GUI() { 
			... 
			addKeyListener(new KeyListener()); 
		} 
	}

 

[The_S]

Du könntest z. B. deine Bilder als Klassenvariable laden, dann musst du sie schon nicht direkt in paintComponent laden. Du kannst repaint auch Parameter übergeben - dann wird nur ein bestimmter Bereich neugezeichnet.

Anschließend sehen wir weiter .

 

[Guest]

Du könntest z. B. deine Bilder als Klassenvariable laden

Sorry, was heisst als Klassenvariable laden? Wo müsste ich deinen Vorschlag in meinen Code reincoden, und wie sieht der Code dazu aus?

Ich lade meine Bilder übrigens so:

bild = ImageIO.read(new File(...));
g.drawImage(bild, ...,...,...,...,this);

Aber wieder zum anderen Thema:
In meinen Code meiner vorherigen Post hab ich die Zeile:
System.out.println("Tastatur wurde betätigt");
hinzugefügt. Was ich aber gar nicht verstehe:
Meine zeichnen-Methode wird zuerst leider KOMPLETT ausgeführt, erst danach spuckt er mir die System.out.println-Meldung aus!? Er müsste mir doch eigentlich die System-out Meldung sofort beim Tastaturdrücken anzeigen, oder?!

if( e.getKeyCode() == KeyEvent.VK_LEFT ) 
         { 
            left=true; 
            System.out.println("Tastatur wurde betätigt");
            zeichnen();//Problem: die zeitaufwendige Schleife unten um Code wird komplett abgearbeitet, 
            //erst danach werden die nächsten Keyinputs abgearbeitet! 
         }

 

[SlaterB]

rufe niemals in irgendeinem deiner Programm getGraphics(); auf,
das ist ein weiterer sehr guter allgemeiner Tipp

bleibe bei paintComponent();

---------

was genau ist eigentlich auf dem Bildschirm vor/ während des Ladens der 9999 Bilder zu sehen?

du brauchst einen Thread,
mit einer Hauptoperation
managePaint() {
1. evtl. Vorzeichen, maximal 100ms
2. Bilder laden, dauert lange, nach jedem Bild prüfen, ob abgebrochen werden soll
3. Bilder zeichen, maximal 100ms
}

nach einem Button-Klick wird die Operation gestartet, wenn sie noch nicht läuft

ansonsten wird ein Feld 'sarte von vorne' markiert
und die Operation managePaint() prüft das ja nach jeden Bild,
malt evtl. irgendwas in diesem Sonderfall,
und fängt wieder von vorne an

-----------


> Und es ist mein Ziel das bei jeder Keyeingabe der komplette Code der paintComponent gezeichnet wird.

nein, dein Ziel ist es, den kompletten Code auch abbrechen zu können,
das zählt höher, daher muss du dich anpassen

 

[SlaterB]

Meine zeichnen-Methode wird zuerst leider KOMPLETT ausgeführt, erst danach spuckt er mir die System.out.println-Meldung aus!? Er müsste mir doch eigentlich die System-out Meldung sofort beim Tastaturdrücken anzeigen, oder?!

ein Grund mehr, die GUI nicht allzulange zu blockieren,
System.out ist gepuffert und wird nur von Zeit zu Zeit weitergeleitet, wenn ich mich recht erinnere
(Vermutung)

 

[Guest]

rufe niemals in irgendeinem deiner Programm getGraphics(); auf, das ist ein weiterer sehr guter allgemeiner Tipp.
bleibe bei paintComponent();

Jetzt bin ich aber froh -dank einer deiner ersten Posts- es doch nicht mit paintComponent realisiert zu haben :bae:
Denn mit PaintComponent blitzte das fertig berechnete Bild bei mir nach der langen Berechnung erst am Ende nur kurz auf, und verschwindet dann sofort, kann aber auch gut daran liegen dass ichs nicht so richtig gecodet hab.

du brauchst einen Thread,
managePaint() {
2. Bilder laden, dauert lange, nach jedem Bild prüfen, ob abgebrochen werden soll
3. Bilder zeichen, maximal 100ms
}

Danke, jetzt funtzt es! (Ich habs jetzt doch nicht wie empfohlen mit paintComponent realisiert, aber hauptsache es funtzt )

 

[The_S]

Es funzt => Noch Fragen/Probleme?