keyListener - Genau 1 Aktion soll bei 1 Tastenausdruck ausgeführt werden

[Jack159]

Hallo,

Sobald der Benutzer z.b. die Taste "F" drückt, soll (nur!) ein einziges mal die Meldung "F wurde gedrückt" ausgegeben werden.
Mein Problem ist aber, dass wenn man auf 1x "F" tippt, dutzende male die Meldung "F wurde gedrückt" ausgegeben wird, da ich genau genommen ja nicht 1x tippe, sondern die Taste ja einen gewissen Zeitraum aktiviert ist.

Wie krieg ich es aber hin, dass pro Tastendruck nur auch jeweils 1x etwas geschieht?

Wenn ich es mit keyTyped() probiere, passiert überhaupt nichts...
Momentan läufts mit keypressed() (boolean x=true) und keyreleased() (boolean x= false) und eben einer if(x) dann "F wurde gedrückt"...

 

[Noctarius]

Geht KeyDown nicht?

 

[HimBromBeere]

Schreib mal bissl Code, wo du dein x prüfst.

EDIT:

public void keyPressed(KeyEvent e) {
	 switch (e.getKeyCode()) {
	 	case KeyEvent.VK_F:

 

[Michael...]

Momentan läufts mit keypressed() (boolean x=true) und keyreleased() (boolean x= false) und eben einer if(x) dann "F wurde gedrückt"...

Warum mit dem boolean? Läuft die Ausgabe in einer Schleife in einem anderen Thread?
Warum nicht einfach in der keyPressed auf F prüfen und die Ausgabe machen? Die Methode wird nämlich nur einmal aufgerufen, wenn die Taste gedrückt wurde.

 

[Jack159]

Warum mit dem boolean? Läuft die Ausgabe in einer Schleife in einem anderen Thread?
Warum nicht einfach in der keyPressed auf F prüfen und die Ausgabe machen? Die Methode wird nämlich nur einmal aufgerufen, wenn die Taste gedrückt wurde.

Nein, es läuft alles im selben Thread.
Also ohne boolean klappt es, wenn man wirklich nur 1x kurz die Taste antippt (Besser auf jeden Fall schonmal als vorher)!

Nur wenn man die Taste F etwas länger gedrückt hält, wird trozdem dutzende male die Methode ausgeführt.
Gibts eine Möglichkeit, dass die Methode wirklich nur 1x ausgeführt wird (auch wenn ich die Taste gedrückt halte) und erst wieder erneut ausgeführt wird, wenn die Taste losgelassen wurde, und anschließend wieder erneut gedrückt wird?

Hintergrund es ganzen:
Ich will ein Space Invanders programmieren. Wenn man z.b. auf F (für schießen) drückt/gedrückt hält, dann soll nur genau 1 Schuss kommen.


Hier der Code:



Hier steht der relevante Code drinnen (Ab Zeile 59 kommen die keylistener):

package A;

import java.awt.Color;
import java.awt.Graphics;
import java.awt.Graphics2D;
import java.awt.event.ActionEvent;
import java.awt.event.ActionListener;
import java.awt.event.KeyEvent;
import java.awt.event.KeyListener;
import java.awt.geom.Ellipse2D;


import javax.swing.JFrame;
import javax.swing.JPanel;
import javax.swing.Timer;

public class StartPanel extends JPanel implements KeyListener, ActionListener {
	
	
	Timer t = new Timer(5, this);
	
	public int x = 200;
	public int y = 200;
	public int dx = 0;
	public int dy = 0;
    boolean links = false;
    boolean rechts = false;
    boolean oben = false;
    boolean unten = false;
   
 
	
	public StartPanel(JFrame frame1) {
		
		
		frame1.add(this);
		setFocusable(true);
		addKeyListener(this);
		
		t.start();
		
		
		
	}
	
	
	public void paintComponent(Graphics g) {
	     super.paintComponent(g);            
	     Graphics2D g2 = (Graphics2D) g;
	     Ellipse2D circle = new Ellipse2D.Double(x, y, 40, 40);
	     g2.fill(circle);
	     
	  
	     
	     t.start();
	   }


	public void keyPressed(KeyEvent e) {
		
		int code = e.getKeyCode();
		
		if(code==KeyEvent.VK_UP) {
			oben = true;
			
		} else if(code==KeyEvent.VK_DOWN) {
			unten = true;
			System.out.println("UNTEN");
			
		} else if(code==KeyEvent.VK_RIGHT) {
			rechts = true;
			
			
		} else if(code==KeyEvent.VK_LEFT) {
			links = true;
			
		}
		

		
	      if(code==70) {
	    	  System.out.println("F wurde gedrückt");
	      }
		
	}

	
	public void keyReleased(KeyEvent e) {
		
		int code = e.getKeyCode();
		
		if(code==KeyEvent.VK_UP) {
			oben = false;
			
		} else if(code==KeyEvent.VK_DOWN) {
			unten = false;
			
			
		} else if(code==KeyEvent.VK_RIGHT) {
			rechts = false;
			
			
		} else if(code==KeyEvent.VK_LEFT) {
			links = false;
			
		}
		
		

	}

	
	public void keyTyped(KeyEvent e) {
		int code = e.getKeyCode();
	

	}


	@Override
	public void actionPerformed(ActionEvent e) {

		if(oben) {
            dy = -1;
            dx = 0;
            
        }
    
        if(unten) {
        	dy = 1;
        	dx = 0;
        }
        
        if (links) {
        	dy = 0;
        	dx = -1;
        }
        
        if (rechts) {
        	dy = 0;
        	dx = 1;
        }
        

        
   
        
        x +=dx;
        y +=dy;
        repaint();
        dx = 0;
        dy = 0;
	
	}

}

package A;

import javax.swing.JFrame;

public class StartFrame extends JFrame {
	
	
	
	public StartFrame(int x, int y, String titel) {
		
		this.setSize(x, y);
		this.setVisible(true);
		this.setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
		this.setTitle(titel);
		this.setLocation(500, 200);
		
	}

}

package A;

import java.awt.event.ActionEvent;
import java.awt.event.ActionListener;
import java.awt.event.KeyEvent;
import java.awt.event.KeyListener;



public class Start {
	
	
	
	public static void main(String[] args) {
		
		StartFrame frame = new StartFrame(1000, 600, "Start");
		StartPanel panel = new StartPanel(frame);
		
		
		
		
	}

	
	
	

}

 

[HimBromBeere]

Warum leitest du dein keyPressed nicht einfach an keyReleased weiter? So wie du´s jetzt hast, wird ein un der selbe Code zumindest schonmal zweimal ausgeführt, einmal in keyPressed, einmal in keyReleased

EDIT: Oh, hab´s gefunden... hat sich erldigt.... einfach ignorieren

EDIT: Kann es sein, dass es am Timer liegt? Denn der wird ja 5ms lang ausgeführt und prüft in dieser Zeit sicherlich öfter als nur einmal, ob oben wahr ist...

EDIT: Du kannst übrigens sinnvollerweise deine Richtungsangaben auf wenigstens zwei reduzieren: vertikal -1, 0 oder +1 und horizontal -1, 0 oder 1

 

[Jack159]

Am Timer liegts nicht. Selbst wenn ich 1 Sekunde beim Timer Einstelle, kommt zig mal die Meldung "F wurde gedrückt" wenn ich F gedrückt halte, der Kreis jedoch bewegt sich nur jede 1 Sekunde weiter.

 

[HimBromBeere]

Hab ich das richtig interpretiert, dass du jede Bewegung 5ms lang durchführen möchtest?

 

[Jack159]

Nein das hat damit nichts zu tun.

Starte das Programm mal ruhig und teste es aus.

Ohne den Timer würde die Bewegung des Balles ziemlich komisch/unflüssig/hackelig wirken.

 

[SSAT]

Keypressed() nur einmal aufzurufen wird wohl nicht gehen. Unter Windows hat man beim Gedrückthalten einer Taste immer den folgenden Ablauf:

keyPressed()
keyPressed()
...
keyReleased()

Es wird immer ständig keypressed aufgerufen, solange bis man die Taste loslässt. Unter Linux ist es noch schlimmer. Da hat man folgenden Ablauf:

keyPressed()
keyReleased()
keyPressed()
keyReleased()
...

Lösungen für das Problem gibt es auch. Google mal nach TimedKeyListener. Es gibt noch andere Methoden, die ich allerdings nicht mehr kenne -> Google nach "keypressed + Linux" oder so etwas ähnliches.

 

[Noctarius]

Gibt es in Swing echt kein KeyDown Event? Oo Armes Swing...

 

[Michael...]

Sorry meine gestrige Aussage war falsch. (zur Verteidigung ;-) :nutzte eigentlich keine KeyListener)

Für solche Zwecke verwende ich Key Bindings. Hier mal ein Bsp. wie man das Problem mit Hilfe eines Flags lösen könnten. (Hätte eigentlich erwartet, das keyTyped(KeyEvent), diesen Zweck erfüllt)

Die statische boolean Variable ist nur zur Demo, im Produktivumfeld bitte eine Instanzvariable daraus machen:

	static boolean toBeReleased = false;

	public static void main(final String[] args) {
		JFrame frame = new JFrame();
		frame.setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
		frame.setBounds(0, 0, 300, 300);
		JPanel panel = new JPanel();
		frame.add(panel);
		frame.setVisible(true);

		InputMap inputMap = panel.getInputMap(JComponent.WHEN_IN_FOCUSED_WINDOW);
		inputMap.put(KeyStroke.getKeyStroke(KeyEvent.VK_F, 0), "fPressed");
		inputMap.put(KeyStroke.getKeyStroke(KeyEvent.VK_F, 0, true), "fReleased");
		panel.getActionMap().put("fPressed", new AbstractAction() {
			public void actionPerformed(ActionEvent e) {
				if (!toBeReleased) {
					System.out.println("F");
					toBeReleased = true;
				}
			}
		});
		panel.getActionMap().put("fReleased", new AbstractAction() {
			public void actionPerformed(ActionEvent e) {
				toBeReleased = false;
			}
		});
	}