zwei Objekte bewegen - der eine bewegt sich nicht flüssig?

[giogp]

Hallo

ich habe folgenden Code

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

public class Game extends JFrame implements KeyListener{

	JPanel root = new JPanel();
	JButton button1 = new JButton();
	JButton button2 = new JButton();
	int currx = 30;
   	int curry = 30;
   	
	int curr2x = 60;
   	int curr2y = 60;
	
	
	public Game(){
		super("CatchMeIfYouCan");
		
		root.setLayout(null);
	    root.add(button1);
	    root.add(button2);
	    
	    button1.setSize(30,30);
	    button1.setLocation(currx,curry);
	    button2.setSize(30,30);
	    button2.setLocation(curr2x,curr2y);
	    
	    button1.addKeyListener(this);
	    button2.addKeyListener(this);
	    button1.setFocusable(true);

	    
	    getContentPane().add (root);
		setDefaultCloseOperation(Game.EXIT_ON_CLOSE );
	}
	
	

	   public void keyPressed (KeyEvent f){
    	if (f.getKeyCode() == KeyEvent.VK_LEFT) {
    		currx -= 2;
    		button1.setLocation(currx,curry);	
        }
    	else if (f.getKeyCode() == KeyEvent.VK_RIGHT) {
    		currx += 2;
    		button1.setLocation(currx,curry);
    	}
    	else if (f.getKeyCode() == KeyEvent.VK_UP) {
    		curry -= 2;
    		button1.setLocation(currx,curry);
    	}
    	else if (f.getKeyCode() == KeyEvent.VK_DOWN) {
    		curry += 2;
    		button1.setLocation(currx,curry);
    	}
    	else if (f.getKeyCode() == KeyEvent.VK_A) {
    		curr2x -= 2;
    		button2.setLocation(curr2x, curr2y);
        }
    	else if (f.getKeyCode() == KeyEvent.VK_D) {
    		curr2x += 2;
    		button2.setLocation(curr2x, curr2y);
    	}
    	else if (f.getKeyCode() == KeyEvent.VK_W) {
    		curr2y -= 2;
    		button2.setLocation(curr2x, curr2y);
    	}
    	else if (f.getKeyCode() == KeyEvent.VK_S) {
    		curr2y += 2;
    		button2.setLocation(curr2x, curr2y);
    	}
  
    	
    	
    	
    	if(button1.getLocation().equals(button2.getLocation()))
		{
			System.out.println("gefangen!");
		}
	   }
	   
	
	
	
	   public void keyTyped (KeyEvent f) { } 
	   public void keyReleased (KeyEvent f) {} 
}

Nur jetzt habe ich folgendes Problem:
der button1 lässt sich auch mit gedrückter Pfeiltaste lenken,
der button2 lässt sich jedoch nicht mit gedrückter Taste lenken ..

Woran liegt das? 😱

 

[Marco13]

Auch wenn man davon ausgehen kann, dass das NUR erste Gehversuche und (etwas zielloses) "Rumspielen" sein soll: Der Ansatz ist nicht so günstig. Falls du Spiele programmieren willst, schau' dir ggf. mal sowas wie http://www.java-forum.org/spiele-multimedia-programmierung/54795-quaxli-2d-spiele-tutorial.html an.

Der eigentliche Problem ist hier vermutlich, dass immer nur der Button die KeyEvents bekommt, der den Focus hat (d.h. der, der zuletzt mit der Maus geklickt wurde). Abhilfen gäbe es zwar, aber da der Ansatz nicht sinnvoll ist, wäre es nicht zielführend, diese zu nennen.

 

[giogp]

Wenn button2.setFocusable(true); ändert sich auch nichts.

Und wenn ich den Button anklicke und dann das ganze versuche, wird ebenfalls nur der erste Button so behandelt ..

Aber auf jeden Fall danke für das Tutorial!

Ja, es soll nur eine Spielerei sein, da ich gerade erst auf Java gestoßen bin..

 

[Volvagia]

Kann es sein, dass du 2 Tasten (z. B. D und Right) gleichzeitig drückst? Ansonst funktioniert es.
Der KeyListener wird nur für die zuletzt physisch gedrückte Taste abgefeuert. Falls das gleichzeitig passieren soll musst du dir merken, welche Tasten gedrückt wurden.

 

[giogp]

Also gleichzeitig gehen die beiden Button (noch) nicht.

Aber bei mir NUR A oder S oder D oder W gedrückt halten bringt auch keinen Erfolg .. Der Button bewegt sich nur einmal.

Bei den Pfeiltasten ist das anders :s

 

[Kevin94]

Die Methode heist doch keyPressed. Wie oft drückst du die Taste, wenn du sie gedrückt hältst? Ein mal, also auch nur ein mal das Event. Der Gesunde Menschenverstand sollte sich wohl eher wundern, warum das bei den Pfeiltasten nicht so ist.
Für Buchstaben sollte die Methode keyTyped wohl das erwartete Verhalten liefern, ist aber auch nicht so optimal. Ideal wäre es wohl in keyPressed und keyReleased flags zu setzen und in einem extra Thread abhängig von den Flags die Buttons zu verschieben.

setFocusable(true)

auf einen Button anzuwenden, hat afaik überhaupt keinen Sinn, da Buttons von Haus aus fokusable sind, wenn dann müsste du auf den anderen

setFokusable(false)

aufrufen.

 

[Marco13]

Hab's gerade nochmal getestet (und die Frage ursprünglich wohl falsch verstanden 😳 , das Problem mit dem Focus stellt sich hier gar nicht) aber bei mir verhalten sich die beiden soweit ich das sehe gleich... :bahnhof:

Der Beschreibung nach ... könnte es damit zusammenhängen, dass bei den Pfeiltasten (wenn man sie gedrückt hält) NUR KeyPressed-Events kommen, aber bei den Buchstabentasten abwechselnd KeyPressed und KeyTyped (letztere eben für die vielen AAAAAAA's, die man damit eigentlich eintippt 😉 ). (Ist das vielleicht sogar vom Betriebssystem abhängig? Hast du Win, Linux oder Mac?)

Aber nochmal: Das so zu machen macht nicht viel Sinn.

 

[giogp]

Habe Mac 😉

Könnte es aber auch auf Win oder Linux gerade testen, wenn das einen Unterschied macht 🙂

 

[Kevin94]

Das dürfte OS abhängig sein. Wenn es Hardware-abhängig wäre, dann könnte man in Windoof nicht einstellen wie schnell bei gedrückter Taste die Buchstaben nochmal getippt werden.

Und ich würde lieber das ganze sauberer Implentieren als für solche Probleme Workarounds zu finden.

 

[Volvagia]

Bei mir (XP) wird nach der Tastenwiederholungsrate gefeuert. Aber du wirst sowieso nicht um einen Thread herumkommen, du willst die Blöcke ja auch so lange verschieben wie die Taste gedrückt ist und nicht immer, wenn über einen Tastendruck informiert wird, oder? ^^


[EDIT]Hier ein Beispiel, wie so etwas aussehen könnte. Ich wollte mich sowieso vor meinen eigenen Code und seinen unaussprechbaren Klassennamen drücken. Sonst räume ich mein Zimmer auf, aber das geht auch. 😀 (Wenigstens geb ichs zu)[/EDIT]

package org.javaforum;

import java.awt.EventQueue;
import java.awt.event.KeyEvent;
import java.awt.event.KeyListener;
import java.util.Collections;
import java.util.HashSet;
import java.util.Set;

import javax.swing.JButton;
import javax.swing.JComponent;
import javax.swing.JFrame;

public class Game extends JFrame implements KeyListener {
	private static final long CONTROL_TIMEOUT_MS = 10;
	
	private Set<Integer> pressedKeys;
	    
    public Game(){
        super("CatchMeIfYouCan");
        
        pressedKeys = Collections.synchronizedSet(new HashSet<Integer>());
        
        setLayout(null);
        
        JButton button1 = new JButton();
        button1.setSize(30,30);
        button1.setLocation(30,30);
        button1.setFocusable(false);
        add(button1);
        
        JButton button2 = new JButton();
        button2.setSize(30,30);
        button2.setLocation(60, 60);
        button2.setFocusable(false);
        add(button2);
        
        new Thread(new ControllRun(KeyEvent.VK_UP, KeyEvent.VK_RIGHT, KeyEvent.VK_DOWN, KeyEvent.VK_LEFT, button1)).start();
        new Thread(new ControllRun(KeyEvent.VK_W, KeyEvent.VK_A, KeyEvent.VK_S, KeyEvent.VK_D, button2)).start();
        
        addKeyListener(this);
        
        setDefaultCloseOperation(Game.EXIT_ON_CLOSE );
    }

    private class ControllRun implements Runnable {
    	private int upKey;
    	private int rightKey;
    	private int downKey;
    	private int leftKey;
		private JComponent component;
    	
    	public ControllRun(int upKey, int rightKey, int downKey, int leftKey, JComponent component) {
			this.upKey = upKey;
			this.rightKey = rightKey;
			this.downKey = downKey;
			this.leftKey = leftKey;
			this.component = component;
		}
    	public void run() {
    		do {
    			if(isKeyPressed(upKey)) {
        			invokeCompMove(0, -2);
        		}
    			if(isKeyPressed(rightKey)) {
        			invokeCompMove(2, 0);
        		}
    			if(isKeyPressed(downKey)) {
        			invokeCompMove(0, 2);
        		}
    			if(isKeyPressed(leftKey)) {
        			invokeCompMove(-2, 0);
        		}
    			
    			try {
					Thread.sleep(CONTROL_TIMEOUT_MS);
				} catch (InterruptedException e) {
					e.printStackTrace();
				}
    		} while(true);
    	}
    	public void invokeCompMove(final int modX, final int modY) {
    		EventQueue.invokeLater(new Runnable() {
				public void run() {
					component.setLocation(component.getX() + modX, component.getY() + modY);					
				}
			});
    	}
    }
    
    private boolean isKeyPressed(int i) {
    	return(pressedKeys.contains(i));
    }

	private void setKeyPressed(int keyCode, boolean b) {
    	if(b) {
    		pressedKeys.add(keyCode);
    	} else {
    		pressedKeys.remove(keyCode);
    	}
	}
    
    public void keyPressed (KeyEvent f){
       setKeyPressed(f.getKeyCode(), true);
    }
    
	public void keyTyped (KeyEvent f) { } 
    public void keyReleased (KeyEvent f) {
    	setKeyPressed(f.getKeyCode(), false);
    } 

    public static void main(String[] args) {
    	EventQueue.invokeLater(new Runnable() {
    		public void run() {
    			Game game = new Game();
    			game.setSize(500, 500);
    			game.setVisible(true);
    		}
    	});
    }
}

Mir persönlich gefallen die Tastenabfragen noch nicht, die kannst du ja selbst etwas OO abstrakter machen. 😀

 

[giogp]

So ist es 😀

Threads muss ich mir noch einmal anschauen ..